Add missing protocol check in the sig matching process. This prevents FP's such as the one reported in bug #209.

remotes/origin/master-1.0.x
Victor Julien 16 years ago
parent 4c94a27b71
commit 689d05b10b

@ -841,6 +841,11 @@ int SigMatchSignatures(ThreadVars *th_v, DetectEngineCtx *de_ctx, DetectEngineTh
s = det_ctx->match_array[idx];
SCLogDebug("inspecting signature id %"PRIu32"", s->id);
if (DetectProtoContainsProto(&s->proto, p->proto) == 0) {
SCLogDebug("proto didn't match");
goto next;
}
/* check the source & dst port in the sig */
if (p->proto == IPPROTO_TCP || p->proto == IPPROTO_UDP) {
if (!(s->flags & SIG_FLAG_DP_ANY)) {

Loading…
Cancel
Save