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

flow/icmpv4: fix vlan.use-for-tracking backport6 #7913

Conversation

catenacyber
Copy link
Contributor

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

Describe changes:

  • flow/icmpv4: fix vlan.use-for-tracking

clean cherry-pick

For ICMPv4 error messages the vlan ids were always considered,
even if the 'vlan.use-for-tracking' option was disabled.

Ticket: OISF#5330
(cherry picked from commit ce2775d)
@catenacyber catenacyber requested a review from a team as a code owner September 23, 2022 09:40
That is it should be less than 1 000 000.
Have the same for fuzz targets where the bug came from.

https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=44177
(cherry picked from commit 8ecf7e4)
@catenacyber
Copy link
Contributor Author

Pushing a missing backport to get CIFuzz happier

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 9430

@suricata-qa
Copy link

ERROR:

ERROR: QA failed on SURI_TLPR1_alerts_cmp.

Pipeline 9449

This was referenced Sep 26, 2022
@victorjulien
Copy link
Member

Merged in #7936, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants