Coverity 1038138 fix

Clean up parsing code to suppress Coverity:
Dereference before null check (REVERSE_INULL)

Proper checking was already done.
pull/552/head
Victor Julien 12 years ago
parent 27ea4232fe
commit 3cf3b485f2

@ -152,11 +152,8 @@ static DetectThresholdData *DetectThresholdParse (char *rawstr)
if(strstr(threshold_opt,"track"))
track_found++;
}
if (copy_str != NULL) {
SCFree(copy_str);
copy_str = NULL;
}
SCFree(copy_str);
copy_str = NULL;
if(count_found != 1 || second_found != 1 || type_found != 1 || track_found != 1)
goto error;
@ -226,8 +223,6 @@ error:
}
if (de != NULL)
SCFree(de);
if (copy_str != NULL)
SCFree(copy_str);
return NULL;
}

Loading…
Cancel
Save