Project

General

Profile

Actions

Optimization #8579

open
PM PM

no error/warning on missing closing parentheses in a rule

Optimization #8579: no error/warning on missing closing parentheses in a rule

Added by Peter Manev about 2 months ago. Updated 2 days ago.

Status:
Feedback
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:
Needs Suricata-Verify test

Description

Notice the missing parentheses at the end of the rule - no error/complain in testing mode on main and 8.0.4

$ cat parentheses.rules 
alert tcp $HOME_NET any -> $HOME_NET any (msg:"testing parentheses"; flow:to_server, established; content:"tralala"; sid:123; rev:1;

$ /opt/suritest-main/bin/suricata  -S parentheses.rules -l logs -T
Notice: suricata: This is Suricata version 9.0.0-dev (367ca7f43 2026-05-16) running in SYSTEM mode [LogVersion:suricata.c:1241]
Notice: suricata: Configuration provided was successfully loaded. Exiting. [SuricataInit:suricata.c:3163]
$ 

Subtasks 1 (0 open1 closed)

Bug #8589: no error on missing closing parentheses in a rule (8.0.x backport)RejectedOISF DevActions

JF Updated by Juliana Fajardini Reichow about 2 months ago Actions #1

  • Status changed from New to Feedback
  • Target version changed from TBD to 9.0.0-beta1

Could you create an SV test for this, please? :)

JF Updated by Juliana Fajardini Reichow about 2 months ago Actions #2

  • Assignee set to OISF Dev

JF Updated by Juliana Fajardini Reichow about 2 months ago Actions #3

  • Label Needs backport to 8.0 added

OT Updated by OISF Ticketbot about 2 months ago Actions #4

  • Subtask #8589 added

OT Updated by OISF Ticketbot about 2 months ago Actions #5

  • Label deleted (Needs backport to 8.0)

PA Updated by Philippe Antoine 23 days ago Actions #6

  • Label Needs Suricata-Verify test added

PA Updated by Philippe Antoine 23 days ago Actions #7

  • Tracker changed from Bug to Optimization
  • Subject changed from no error on missing closing parentheses in a rule to no error/warning on missing closing parentheses in a rule
  • Affected Versions deleted (8.0.4, git main)

LS Updated by Lukas Sismis 2 days ago Actions #8

  • Assignee changed from OISF Dev to Peter Manev
Actions

Also available in: PDF Atom