Skip to content

8.0.x: backports for issues: 8300, 9106 - v1 - #16468

Closed
jasonish wants to merge 2 commits into
OISF:main-8.0.xfrom
jasonish:8.0.x-issue-8300-9106/v1
Closed

jasonish wants to merge 2 commits into
OISF:main-8.0.xfrom
jasonish:8.0.x-issue-8300-9106/v1

Conversation

@jasonish

@jasonish jasonish commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

SV_BRANCH=OISF/suricata-verify#3463

Automated backports to main-8.0.x for 8.0.8 from merged staging PRs
#14961 and #16459.

Dataset inputs must now be regular files. This also rejects previously
accepted /dev/null inputs.

Both commits built with the normal development sanitizer configuration. The
dataset SV regression passes after failing the bounded baseline check. The
OpenBSD-specific pledge race was not reproduced on this Linux test host.
Full Suricata-Verify: 2375 passed, 0 failed, 607 skipped.

[This backport PR was generated by a new automation in test.]

victorjulien and others added 2 commits October 19, 2026 11:16
Avoids a race condition with pcap file directory mode.

Ticket: OISF#8300.
(cherry picked from commit 56514f3)
Such as /dev/zero

Ticket: 9106
(cherry picked from commit 46aa14f)

@suricata-review-bot suricata-review-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

AI-generated review.

Verdict: approve

No high- or medium-severity findings.

Automated at commit 712830938018 using Claude Code with claude-opus-5-5/xhigh. Run: 0b26f5bf4c1be502.

@codecov

codecov Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.67%. Comparing base (6ac6a4e) to head (7128309).

Additional details and impacted files
@@              Coverage Diff               @@
##           main-8.0.x   #16468      +/-   ##
==============================================
- Coverage       81.68%   81.67%   -0.01%     
==============================================
  Files            1020     1020              
  Lines          281718   281727       +9     
==============================================
- Hits           230116   230100      -16     
- Misses          51602    51627      +25     
Flag Coverage Δ
fuzzcorpus 63.59% <84.61%> (-0.02%) ⬇️
livemode 18.45% <7.69%> (-0.08%) ⬇️
netns 19.95% <6.66%> (+<0.01%) ⬆️
pcap 44.30% <7.69%> (-0.07%) ⬇️
suricata-verify 65.12% <100.00%> (-0.02%) ⬇️
unittests 58.67% <0.00%> (-0.01%) ⬇️

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

WARNING:

field baseline test %
SURI_TLPR1_stats_chk
.app_layer.flow.ftp-data 688 665 96.66%

Pipeline = 34981

@victorjulien victorjulien added this to the 8.0 milestone Oct 19, 2026

@catenacyber catenacyber left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good backport for me

I am not sure if Victor wants the openbsd pledge now or later...

@victorjulien

Copy link
Copy Markdown
Member

Merged in #16474, thanks!

@jasonish
jasonish deleted the 8.0.x-issue-8300-9106/v1 branch October 20, 2026 21:40
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