Project

General

Profile

Actions

Bug #6656

closed

detect/requires: assertion failed !(ret == -4)

Added by Philippe Antoine 10 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Target version:
Affected Versions:
Effort:
Difficulty:
Label:

Description

Found by oss-fuzz
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=65320

Reproducer is drop ip :: 0 <> :: 0 requireS:feature;sid:1;


Subtasks 1 (0 open1 closed)

Bug #6700: detect/requires: assertion failed !(ret == -4) (7.0.x backport)ClosedJason IshActions

Related issues 1 (0 open1 closed)

Related to Suricata - Feature #5972: rules: "requires" keyword representing the minimum version of suricata to support the ruleClosedJason IshActions
Actions #1

Updated by Philippe Antoine 10 months ago

The bug looks like
alert ip any any -> any any (requireS:feature geoip;sid:1;) triggers with an uppercase S in requireS
alert ip any any -> any any (requires:feature geoip;sid:1;) does not trigger with all lowercase requires

Actions #2

Updated by Philippe Antoine 10 months ago

SigTableGet uses strcasecmp and so should definition of requires_only instead of using strcmp

Actions #3

Updated by Philippe Antoine 10 months ago

  • Status changed from New to In Review
Actions #4

Updated by Philippe Antoine 10 months ago

  • Status changed from In Review to Closed
  • Assignee changed from Jason Ish to Philippe Antoine
Actions #5

Updated by Jason Ish 9 months ago

  • Label Needs backport to 7.0 added
Actions #6

Updated by Jason Ish 9 months ago

  • Related to Feature #5972: rules: "requires" keyword representing the minimum version of suricata to support the rule added
Actions #7

Updated by Jason Ish 9 months ago

  • Status changed from Closed to Resolved
Actions #8

Updated by OISF Ticketbot 9 months ago

  • Subtask #6700 added
Actions #9

Updated by OISF Ticketbot 9 months ago

  • Label deleted (Needs backport to 7.0)
Actions #10

Updated by Philippe Antoine 9 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF