Bug #12
closedNegated pcre treated as a normal match
Description
given a packet with the following payload of:
AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy AllWorkAndNoPlayMakesVictorADullBoy
the following rules should not fire but they do. I have attached a patch with a failing unit tests.
alert tcp any any -> any any (msg:"all work and no play"; content:"AllWork"; pcre:"!/AndNoPlay/i"; sid: 1);
alert tcp any any -> any any (msg:"all work and no play"; content:"AllWork"; pcre:!"/AndNoPlay/i"; sid: 2);
Files
Updated by Will Metcalf about 15 years ago
rules should have been...
alert tcp any any -> any any (msg:"all work and no play"; content:"AllWork"; pcre:"!/AndNoPlay/i"; sid:1;)
alert tcp any any -> any any (msg:"all work and no play"; content:"AllWork"; pcre:!"/AndNoPlay/i"; sid:2;)
Updated by Victor Julien about 15 years ago
- Estimated time changed from 0.50 h to 2.50 h
Updated by Victor Julien almost 15 years ago
- Assignee changed from OISF Dev to Victor Julien
Updated by Victor Julien almost 15 years ago
- Status changed from New to Closed
Fixed by current master.