Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

next/598/20241010/v1 #11929

Merged
merged 5 commits into from
Oct 10, 2024
Merged

Conversation

catenacyber and others added 5 commits October 10, 2024 09:07
Ticket: 7309

Do not use a constant expression in a condition
Issue: 7314

Fixup macro usages to eliminate compiler warnings.
Currently cbindgen from system packages is broken, for now use the
cbindgen artifact we build.
Ticket: 7315

Completes commit 4a7567b

Allows keyword template.buffer to work properly when template
protocol is enabled
Copy link

codecov bot commented Oct 10, 2024

Codecov Report

Attention: Patch coverage is 60.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 82.74%. Comparing base (d5dd549) to head (58556b7).
Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11929      +/-   ##
==========================================
+ Coverage   82.70%   82.74%   +0.03%     
==========================================
  Files         912      912              
  Lines      249102   249102              
==========================================
+ Hits       206018   206117      +99     
+ Misses      43084    42985      -99     
Flag Coverage Δ
fuzzcorpus 60.72% <100.00%> (+0.04%) ⬆️
livemode 18.85% <33.33%> (+0.13%) ⬆️
pcap 44.11% <66.66%> (+<0.01%) ⬆️
suricata-verify 62.18% <100.00%> (+0.03%) ⬆️
unittests 58.99% <40.00%> (+<0.01%) ⬆️

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

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 23077

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