Project

General

Profile

Actions

Documentation #8261

closed
AD AD

doc/userguide: fix within-distance pointer graphics in payload-keywords doc

Documentation #8261: doc/userguide: fix within-distance pointer graphics in payload-keywords doc

Added by Andreas Dolp 2 months ago. Updated 2 months ago.

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

Description

Hi Suricata team,

according to this forum thread [1], the graphic 'within_distance2.png' in the userguide docs contain an error.
If the 'within' payload keyword is combined with the 'distance' keyword, 'within' does not refer only to the end of the previous match, but also includes the distance-pointer. In the graphic named this is stated otherwise, which can lead to confusion.

The graphics should be fixed by placing the within-pointer between g-h instead of f-g.

Fixed this in a new graphic by some GIMP arts.

I did a first PR [2] requesting to open a redmine ticket, because this should also be backported to 7.x and 8.x.
After getting the ticket number, I will do a second PR with the fix referencing the ticket number.

Thanks and best regards!
Andreas

PS: Special thanks to one of our team members Annika C. for initially spotting this!

[1] https://forum.suricata.io/t/is-within-affected-by-distance/1688
[2] https://github.com/OISF/suricata/pull/14744


Subtasks 2 (0 open2 closed)

Documentation #8263: doc/userguide: fix within-distance pointer graphics in payload-keywords doc (8.0.x backport)ClosedPhilippe AntoineActions
Documentation #8264: doc/userguide: fix within-distance pointer graphics in payload-keywords doc (7.0.x backport)ClosedOISF DevActions

SB Updated by Shivani Bhardwaj 2 months ago Actions #2

  • Assignee set to Andreas Dolp
  • Target version changed from TBD to 9.0.0-beta1

OT Updated by OISF Ticketbot 2 months ago Actions #3

  • Subtask #8263 added

OT Updated by OISF Ticketbot 2 months ago Actions #4

  • Label deleted (Needs backport to 8.0)

OT Updated by OISF Ticketbot 2 months ago Actions #5

  • Subtask #8264 added

OT Updated by OISF Ticketbot 2 months ago Actions #6

  • Label deleted (Needs backport to 7.0)

SB Updated by Shivani Bhardwaj 2 months ago Actions #7

  • Status changed from In Progress to Resolved

Closed by: https://github.com/OISF/suricata/pull/14748
Thank you, Andreas and Annika! :)

PA Updated by Philippe Antoine 2 months ago Actions #8

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom