Snort mailing list archives

Re: invalid use of byte_test on sid:16603


From: Will Metcalf <william.metcalf () gmail com>
Date: Thu, 24 Jun 2010 14:30:50 -0500

Haha... Well I guess thanks for silently fixing this ;-).... Maybe you
should update the snort manual to reflect this eh?

Regards,

Will

case '<': idx->operator = BT_LESS_THAN;
break;

case '>': idx->operator = BT_GREATER_THAN;
break;

Is now...

            case '>': idx->operator = BT_GREATER_THAN;
                      cptr++;
                      if (*cptr == '=')
                          idx->operator = BT_GREATER_THAN_EQUAL;
                      else
                          cptr--;
                      break;

           case '<': idx->operator = BT_LESS_THAN;
                      cptr++;
                      if (*cptr == '=')
                          idx->operator = BT_LESS_THAN_EQUAL;
                      else
                          cptr--;
                      break;

Regards,

Will

On Thu, Jun 24, 2010 at 11:21 AM, Will Metcalf
<william.metcalf () gmail com> wrote:
Unless something has changed I think the use of byte_test in sid:16603
is invalid as snort only parses the first char in ">=", actually
making the test byte_test:4,>,97612894,0,relative,little;. Also I have
the registered feed so if this is fixed already please disregard.

So I think you probably want to modify...

byte_test:4,>=,97612894,0,relative,little;

to be..

byte_test:4,>,97612893,0,relative,little;

Regards,

Will


------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate 
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the 
lucky parental unit.  See the prize list and enter to win: 
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
Snort-users mailing list
Snort-users () lists sourceforge net
Go to this URL to change user options or unsubscribe:
https://lists.sourceforge.net/lists/listinfo/snort-users
Snort-users list archive:
http://www.geocrawler.com/redir-sf.php3?list=snort-users


Current thread: