Project

General

Profile

Actions

Bug #4821

closed
JL JL

xbits: no error on invalid 'expire' values

Bug #4821: xbits: no error on invalid 'expire' values

Added by Jeff Lucovsky over 4 years ago. Updated almost 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

alert http any any -> any any (msg:"TEST - No Error")"; flow:established,to_server; http.method; content:"GET"; xbits:set,ET.2020_8260.1,track ip_src,expire 10,noalert; sid:1;)
alert http any any -> any any (msg:"TEST - Error")"; flow:established,to_server; http.method; content:"GET"; xbits:set,ET.2020_8260.1,noalert,track ip_src,expire 10; sid:2;)
alert http any any -> any any (msg:"TEST - No Error")"; flow:established,to_server; http.method; content:"GET"; xbits:set,ET.2020_8260.1,track ip_src,expire 10,asdf; sid:3;)

only sid 2 produces an error, despite that all 3 sids should be considered "invalid"

Error Produced by sid:2

[1539] 27/10/2021 -- 21:58:51 - (detect-xbits.c:208) <Error> (DetectXbitParse) -- [ERRCODE: SC_ERR_PCRE_MATCH(2)] - "set,ET.2020_8260.1,noalert,track ip_src,expire 10" is not a valid setting for xbits.

I'd also add that the documentation is a bit vague on the proper use of the noalert keyword in relation to xbits. It currently reads

To not alert, use noalert;

I suggest adding a bit of context which indicates it should be a standalone keyword in the rule and not an "option" to the xbits keyword.


Related issues 1 (0 open1 closed)

Copied from Suricata - Bug #4786: xbits: no error on invalid 'expire' valuesClosedVictor JulienActions

JL Updated by Jeff Lucovsky over 4 years ago Actions #1

  • Copied from Bug #4786: xbits: no error on invalid 'expire' values added

JL Updated by Jeff Lucovsky over 4 years ago Actions #2

  • Target version changed from 5.0.8 to 5.0.9

SB Updated by Shivani Bhardwaj about 4 years ago Actions #3

  • Assignee changed from Jeff Lucovsky to Shivani Bhardwaj

SB Updated by Shivani Bhardwaj about 4 years ago Actions #4

  • Assignee changed from Shivani Bhardwaj to Jeff Lucovsky

The fix will have to be backported from 6.0.x and not master

VJ Updated by Victor Julien almost 4 years ago Actions #5

  • Priority changed from High to Normal
  • Target version changed from 5.0.9 to 5.0.10

VJ Updated by Victor Julien almost 4 years ago Actions #6

  • Status changed from Assigned to Rejected

5.0.x is close to EOL, so not fixing this minor issue.

Actions

Also available in: PDF Atom