Hi,
Please find the latest report on new defect(s) introduced to Pike-master found with Coverity Scan.
1 new defect(s) introduced to Pike-master found with Coverity Scan.
New defect(s) Reported-by: Coverity Scan
Showing 1 of 1 defect(s)
** CID 1375989: (CHECKED_RETURN)
/home/covscan/pike/Pike-v8.1-snapshot/src/pike_types.c: 3872 in low_pike_types_le()
/home/covscan/pike/Pike-v8.1-snapshot/src/pike_types.c: 3895 in low_pike_types_le()
/home/covscan/pike/Pike-v8.1-snapshot/src/pike_types.c: 4090 in low_pike_types_le()
________________________________________________________________________________________________________
*** CID 1375989: (CHECKED_RETURN)
/home/covscan/pike/Pike-v8.1-snapshot/src/pike_types.c: 3872 in low_pike_types_le()
3866 goto recurse;
3867 }
3868 if ((a->flags | b->flags) & PT_FLAG_ASSIGN) {
3869 /* Note that we need to recurse, since there are T_ASSIGN nodes
3870 * in the cdr that need to be executed.
3871 */
>>> CID 1375989: (CHECKED_RETURN)
>>> Calling "low_pike_types_le" without checking return value (as is done elsewhere 28 out of 35 times).
3872 low_pike_types_le(a->cdr, b, array_cnt, flags);
3873 }
3874 return 1;
3875
3876 case T_OR:
3877 /* OK, if both of the parts are a subset,
/home/covscan/pike/Pike-v8.1-snapshot/src/pike_types.c: 3895 in low_pike_types_le()
3889 } else {
3890 ret = low_pike_types_le(a->car, b, array_cnt, flags);
3891 #ifdef TYPE_GROUPING
3892 if (!ret == !(flags & LE_A_GROUPED)) {
3893 if ((a->flags | b->flags) & PT_FLAG_ASSIGN) {
3894 /* Note: Needed for side effects... */
>>> CID 1375989: (CHECKED_RETURN)
>>> Calling "low_pike_types_le" without checking return value (as is done elsewhere 28 out of 35 times).
3895 low_pike_types_le(a->cdr, b, array_cnt, flags);
3896 }
3897 return ret;
3898 }
3899 #else
3900 if (!ret) return 0;
/home/covscan/pike/Pike-v8.1-snapshot/src/pike_types.c: 4090 in low_pike_types_le()
4084 */
4085 ret = low_pike_types_le(a, b->car, array_cnt, flags);
4086 #ifdef TYPE_GROUPING
4087 if (!ret != !(flags & LE_B_GROUPED)) {
4088 if ((a->flags | b->flags) & PT_FLAG_ASSIGN) {
4089 /* Note: Needed for side effects... */
>>> CID 1375989: (CHECKED_RETURN)
>>> Calling "low_pike_types_le" without checking return value (as is done elsewhere 28 out of 35 times).
4090 low_pike_types_le(a, b->cdr, array_cnt, flags);
4091 }
4092 return ret;
4093 }
4094 #else
4095 if (ret) return ret;
________________________________________________________________________________________________________
To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…
To manage Coverity Scan email notifications for "pike-automation(a)lists.lysator.liu.se", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…