Skip to content

DRAFT: check cocci improvement catches missing NULL check - #15508

Closed
catenacyber wants to merge 1 commit into
OISF:mainfrom
catenacyber:kenifor/fix-redis-stream-format-alloc-cocci-test
Closed

DRAFT: check cocci improvement catches missing NULL check#15508
catenacyber wants to merge 1 commit into
OISF:mainfrom
catenacyber:kenifor/fix-redis-stream-format-alloc-cocci-test

Conversation

@catenacyber

Copy link
Copy Markdown
Contributor

#15423 second commit without the first one adding the NULL check

…unction args

The @istested rule's '... when != x' only excluded reassignments of x,
not statements that passed x as a function argument. Add
'when != callee(..., x, ...)' so that any use of the allocated pointer
as a function argument before a NULL check is no longer treated as
tested and is correctly flagged.
@codecov

codecov Bot commented Jun 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.78%. Comparing base (736f506) to head (0af5174).
⚠️ Report is 15 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15508      +/-   ##
==========================================
+ Coverage   82.75%   82.78%   +0.02%     
==========================================
  Files         998      999       +1     
  Lines      272550   272666     +116     
==========================================
+ Hits       225562   225716     +154     
+ Misses      46988    46950      -38     
Flag Coverage Δ
fuzzcorpus 61.43% <ø> (+0.02%) ⬆️
livemode 18.33% <ø> (+0.05%) ⬆️
netns 22.72% <ø> (-0.08%) ⬇️
pcap 44.93% <ø> (-0.06%) ⬇️
suricata-verify 66.40% <ø> (-0.08%) ⬇️
unittests 58.40% <ø> (-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.

@catenacyber

Copy link
Copy Markdown
Contributor Author

Cocci is green but should not be

@catenacyber catenacyber closed this Jun 2, 2026
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.

1 participant