Skip to content

next/1468/80x/20260826/v1 - #16123

Merged
victorjulien merged 13 commits into
OISF:main-8.0.xfrom
victorjulien:next/1468/80x/20260826/v1
Aug 27, 2026
Merged

next/1468/80x/20260826/v1#16123
victorjulien merged 13 commits into
OISF:main-8.0.xfrom
victorjulien:next/1468/80x/20260826/v1

Conversation

victorjulien and others added 13 commits August 26, 2026 19:51
Mostly meant to help unittests and fuzz targets alloc the storage
correctly.

(cherry picked from commit ad1459d)
Seed value will have to be used by the caller's hash function.

(cherry picked from commit b742d58)
Use for IPv6 in the main hash.

Update IPv4 thread cache to also use a seed. For this the thread
initialization is done at thread init instead on the first packet. The
thread ctx is no longer stored in a thread_local, but instead uses the
thread storage API to store in the ThreadVars. This way it will stay
active during rule reloads as well.

Ticket: OISF#8617.
(cherry picked from commit c27868a)
Can get called multiple times in the multi-tenancy case.

(cherry picked from commit cefee13)
Use new allocator to properly set up thread storage.

(cherry picked from commit 9e82ff8)
Mapping an app-layer progress state to its generic request-/response-
hook alias was hardcoded in multiple places.

This commit adds a wrapper to unify it to a single function.
It returns the config-form (hyphenated) name, or NULL for an
intermediate state.

Ticket: 8770
(cherry picked from commit 1aa0259)
Every hook has a built-in default policy, but expressing anything other
than the built-in meant naming each hook explicitly.
This commit adds a `default-policy` setting that covers all hooks below it.
For any hook the most specific setting present wins.

Ticket: 8770
(cherry picked from commit 0aaa80d)
Validate the resolved scope against the class of hook it is being applied
to and fail at startup if it does not fit.

Ticket: 8770
(cherry picked from commit 092ae27)
Ticket: 8678

In FilePruneFile() `window * 3` was computed in uint32_t arithmetic.
The guard exists to ensure `file->size > window`, so on wrap around
`file->size - window` underflows and content_inspected is set to a
bogus value, breaking file inspection.

Make window uint64_t so the multiplication cannot wrap.

(cherry picked from commit f3910d5)
Ticket: 8678

With an inspect window of 0xAAAAAAAB the `window * 3` guard wrapped to 1
in uint32_t arithmetic, so it passed for a 64 byte file and
content_inspected was set to `file->size - window`, an underflow.

(cherry picked from commit 8e551f2)
@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.38583% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.73%. Comparing base (098022a) to head (3b04b76).
⚠️ Report is 13 commits behind head on main-8.0.x.

Additional details and impacted files
@@              Coverage Diff               @@
##           main-8.0.x   #16123      +/-   ##
==============================================
+ Coverage       81.71%   81.73%   +0.02%     
==============================================
  Files            1016     1017       +1     
  Lines          277201   277278      +77     
==============================================
+ Hits           226509   226628     +119     
+ Misses          50692    50650      -42     
Flag Coverage Δ
fuzzcorpus 63.86% <19.74%> (-0.03%) ⬇️
livemode 18.65% <16.80%> (+<0.01%) ⬆️
netns 20.16% <44.84%> (+0.01%) ⬆️
pcap 44.50% <33.19%> (-0.01%) ⬇️
suricata-verify 65.24% <71.82%> (+<0.01%) ⬆️
unittests 58.56% <66.45%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@suricata-qa

Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline = 33396

@inashivb inashivb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 🚀

@victorjulien
victorjulien merged commit 3b04b76 into OISF:main-8.0.x Aug 27, 2026
61 checks passed
@victorjulien
victorjulien deleted the next/1468/80x/20260826/v1 branch August 27, 2026 07: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.

4 participants