Skip to content

Detect count 5044 v4.3 - #15356

Closed
catenacyber wants to merge 5 commits into
OISF:mainfrom
catenacyber:detect-count-5044-v4.3
Closed

Detect count 5044 v4.3#15356
catenacyber wants to merge 5 commits into
OISF:mainfrom
catenacyber:detect-count-5044-v4.3

Conversation

@catenacyber

Copy link
Copy Markdown
Contributor

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

Describe changes:

  • adds other modes to multi-buffers like all, all_or_absent, matches`, and precise indexing

SV_BRANCH=OISF/suricata-verify#3062

Draft : I would like some approval before doing all the individual modifications for each multi-buffer

#15302 with needed rebase

TODOs :

  • add support for all multi-buf keywords
  • add count support

Also rename parse_uint_count as parse_multi_count

This allows to prepare multi-buffers using this code
As for multi-integers, multi-buffers can now have the following
arguments
- nb
- precise index
- all

Ticket: 5044
- or_absent can also be any_or_absent
- nb can also be matches
- precise index can have prefix index
@codecov

codecov Bot commented May 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 79.26267% with 45 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.64%. Comparing base (c48bb1b) to head (19a6027).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15356      +/-   ##
==========================================
- Coverage   82.65%   82.64%   -0.02%     
==========================================
  Files         996      997       +1     
  Lines      271111   271299     +188     
==========================================
+ Hits       224089   224204     +115     
- Misses      47022    47095      +73     
Flag Coverage Δ
fuzzcorpus 60.98% <25.48%> (-0.04%) ⬇️
livemode 18.37% <6.73%> (-0.01%) ⬇️
netns 22.60% <6.73%> (-0.04%) ⬇️
pcap 45.19% <12.50%> (-0.05%) ⬇️
suricata-verify 66.38% <78.36%> (-0.03%) ⬇️
unittests 58.53% <22.58%> (-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

ERROR:

ERROR: QA failed on SURI_TLPW2_autofp_suri_time.

Pipeline = 31346

@victorjulien victorjulien self-assigned this May 21, 2026
@catenacyber

Copy link
Copy Markdown
Contributor Author

Status : I would like some approval before doing all the individual modifications for each multi-buffer

This PR incorporated the changes discussed during our in-meeting person

@catenacyber

Copy link
Copy Markdown
Contributor Author

Rebased in #16087

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