Skip to content

nfs: optionally parse nfs3 read attr-follows - #16095

Closed
catenacyber wants to merge 1 commit into
OISF:mainfrom
catenacyber:nfs-v3-read-attrfollows-8750-v1
Closed

nfs: optionally parse nfs3 read attr-follows#16095
catenacyber wants to merge 1 commit into
OISF:mainfrom
catenacyber:nfs-v3-read-attrfollows-8750-v1

Conversation

@catenacyber

Copy link
Copy Markdown
Contributor

Link to ticket: https://redmine.openinfosecfoundation.org/issues/8750

Describe changes:

  • nfs: optionally parse nfs3 read attr-follows

SV_BRANCH=OISF/suricata-verify#3309

Was already staged in gitlab, making it public since it is low severity

@victorjulien victorjulien added this to the 9.0 milestone Aug 24, 2026
@suricata-qa

Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline = 33270

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.03%. Comparing base (acd9d3a) to head (e83ac55).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #16095      +/-   ##
==========================================
- Coverage   83.05%   83.03%   -0.03%     
==========================================
  Files        1004     1004              
  Lines      277359   277360       +1     
==========================================
- Hits       230370   230292      -78     
- Misses      46989    47068      +79     
Flag Coverage Δ
fuzzcorpus 61.51% <100.00%> (-0.03%) ⬇️
livemode 18.42% <0.00%> (-0.17%) ⬇️
netns 22.84% <0.00%> (-0.07%) ⬇️
pcap 45.43% <100.00%> (+0.01%) ⬆️
suricata-verify 67.11% <100.00%> (-0.04%) ⬇️
unittests 58.52% <100.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.

@victorjulien

Copy link
Copy Markdown
Member

Merged in #16099, thanks!

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.

3 participants