Actions
Bug #3855
closedmqtt: coverity static analysis issues
Affected Versions:
Effort:
Difficulty:
Label:
Description
*** CID 1465604: Null pointer dereferences (REVERSE_INULL) /src/detect-mqtt-flags.c: 192 in DetectMQTTFlagsParse() 186 flagv = strtok_r(NULL, ",", &xsaveptr); 187 } 188 189 return de; 190 191 error: >>> CID 1465604: Null pointer dereferences (REVERSE_INULL) >>> Null-checking "de" suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 192 if (de != NULL) 193 SCFree(de); 194 return NULL; 195 } 196 197 /** ** CID 1465603: Uninitialized variables (UNINIT) /src/detect-engine-uint.c: 376 in DetectU8Parse() ________________________________________________________________________________________________________ *** CID 1465603: Uninitialized variables (UNINIT) /src/detect-engine-uint.c: 376 in DetectU8Parse() 370 } 371 } 372 u8d = SCCalloc(1, sizeof (DetectU8Data)); 373 if (unlikely(u8d == NULL)) 374 return NULL; 375 u8d->arg1 = u8da.arg1; >>> CID 1465603: Uninitialized variables (UNINIT) >>> Using uninitialized value "u8da.arg2". 376 u8d->arg2 = u8da.arg2; 377 u8d->mode = u8da.mode; 378 379 return u8d; ** CID 1465602: Null pointer dereferences (REVERSE_INULL) /src/detect-mqtt-connack-sessionpresent.c: 157 in DetectMQTTConnackSessionPresentParse() ________________________________________________________________________________________________________ *** CID 1465602: Null pointer dereferences (REVERSE_INULL) /src/detect-mqtt-connack-sessionpresent.c: 157 in DetectMQTTConnackSessionPresentParse() 151 goto error; 152 } 153 154 return de; 155 156 error: >>> CID 1465602: Null pointer dereferences (REVERSE_INULL) >>> Null-checking "de" suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 157 if (de != NULL) 158 SCFree(de); 159 return NULL; 160 } 161 162 /** ** CID 1465601: Null pointer dereferences (REVERSE_INULL) /src/detect-mqtt-connect-flags.c: 212 in DetectMQTTConnectFlagsParse() ________________________________________________________________________________________________________ *** CID 1465601: Null pointer dereferences (REVERSE_INULL) /src/detect-mqtt-connect-flags.c: 212 in DetectMQTTConnectFlagsParse() 206 flagv = strtok_r(NULL, ",", &xsaveptr); 207 } 208 209 return de; 210 211 error: >>> CID 1465601: Null pointer dereferences (REVERSE_INULL) >>> Null-checking "de" suggests that it may be null, but it has already been dereferenced on all paths leading to the check. 212 if (de != NULL) 213 SCFree(de); 214 return NULL; 215 } 216 217 /**
Updated by Sascha Steinbiss about 4 years ago
Updated by Victor Julien about 4 years ago
- Status changed from Assigned to Closed
Actions