Skip to content

eve: threadinit/deinit callbacks are optional for filetypes#12069

Closed
jasonish wants to merge 1 commit intoOISF:masterfrom
jasonish:eve-filetype-7359/v1
Closed

eve: threadinit/deinit callbacks are optional for filetypes#12069
jasonish wants to merge 1 commit intoOISF:masterfrom
jasonish:eve-filetype-7359/v1

Conversation

@jasonish
Copy link
Member

Only call ThreadInit and ThreadDeinit for custom eve filetypes if they
exist. They are not required by all filetypes.

Ticket: https://redmine.openinfosecfoundation.org/issues/7359

Only call ThreadInit and ThreadDeinit for custom eve filetypes if they
exist. They are not required by all filetypes.

Ticket: OISF#7359
@codecov
Copy link

codecov bot commented Oct 30, 2024

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 83.41%. Comparing base (3a7eef8) to head (3d70708).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #12069      +/-   ##
==========================================
- Coverage   83.42%   83.41%   -0.02%     
==========================================
  Files         910      910              
  Lines      257642   257643       +1     
==========================================
- Hits       214949   214915      -34     
- Misses      42693    42728      +35     
Flag Coverage Δ
fuzzcorpus 61.60% <0.00%> (-0.05%) ⬇️
livemode 19.41% <20.00%> (-0.01%) ⬇️
pcap 44.50% <20.00%> (+0.02%) ⬆️
suricata-verify 62.76% <20.00%> (+<0.01%) ⬆️
unittests 59.37% <0.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 23224

@victorjulien victorjulien added this to the 8.0 milestone Oct 31, 2024
@victorjulien
Copy link
Member

Merged in #12074, thanks!

@jasonish jasonish deleted the eve-filetype-7359/v1 branch October 1, 2025 14:36
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