Skip to content

next/1339/80x/20260510/v1#15359

Merged
victorjulien merged 1 commit into
OISF:main-8.0.xfrom
victorjulien:next/1339/80x/20260510/v1
May 11, 2026
Merged

next/1339/80x/20260510/v1#15359
victorjulien merged 1 commit into
OISF:main-8.0.xfrom
victorjulien:next/1339/80x/20260510/v1

Conversation

@victorjulien
Copy link
Copy Markdown
Member

When we got a bind without the first fragment flag, and did not
set any_frag in the signature, the signature always matched,
whatever the uuid value

Ticket: 8457
(cherry picked from commit e388888)
@victorjulien victorjulien requested a review from jasonish as a code owner May 11, 2026 02:44
@suricata-qa
Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline = 31345

Copy link
Copy Markdown
Contributor

@jufajardini jufajardini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checks pass.
Consistent with original, approved PR.

If Shivani's comment on the continue statement without a semi-colon (line 98) isn't an issue, this looks good, imho.

@victorjulien victorjulien merged commit ad9d979 into OISF:main-8.0.x May 11, 2026
118 checks passed
@victorjulien victorjulien deleted the next/1339/80x/20260510/v1 branch May 11, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants