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

af-packet: Error on invalid peer device setting #9296

Closed
wants to merge 1 commit into from

Conversation

inashivb
Copy link
Member

If the copy-iface was set to the same as the af-packet interface, the peer won't be set. This leads to segfault. Catch it and return properly.

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

If the copy-iface was set to the same as the af-packet interface, the
peer won't be set. This leads to segfault. Catch it and return properly.

Bug 5870
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #9296 (64f2743) into master (ff8bbda) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9296      +/-   ##
==========================================
- Coverage   82.42%   82.41%   -0.01%     
==========================================
  Files         968      968              
  Lines      274011   274014       +3     
==========================================
- Hits       225853   225835      -18     
- Misses      48158    48179      +21     
Flag Coverage Δ
fuzzcorpus 64.64% <0.00%> (+<0.01%) ⬆️
suricata-verify 60.81% <0.00%> (-0.04%) ⬇️
unittests 62.92% <0.00%> (-0.01%) ⬇️

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

@inashivb inashivb marked this pull request as ready for review July 27, 2023 13:01
@suricata-qa
Copy link

Information:

ERROR: QA failed on SURI_TLPR1_alerts_cmp.

field baseline test %
SURI_TLPR1_stats_chk
.uptime 1267 1352 106.71%
.tcp.active_sessions 0 2879 -
.flow.active 0 107183 -
.flow.get_used 0 107183 -
.flow.get_used_eval 0 11866171 -
.flow.get_used_eval_reject 0 6535 -
.flow.get_used_eval_busy 0 2597 -
.flow.get_used_failed 0 1636132 -
.flow.emerg_mode_entered 0 2 -
.flow.emerg_mode_over 0 1 -
.app_layer.error.tls.parser 23942 29751 124.26%

Pipeline 15403

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