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

tests/lua: Test for issue 7466 #2209

Closed
wants to merge 1 commit into from
Closed

Conversation

jlucovsky
Copy link
Contributor

This test was contributed by the reporter of issue 7466. Slight modifications not affecting the structure nor results were made.

This test exercises the problem reported in the issue and a memory leak will be reported on Suricata binaries without the fix (ASAN-only).

Non-ASAN builds will not report the issue (regardless of whether the Suricata fix is included).

Ticket

If your pull request is related to a Suricata ticket, please provide
the full URL to the ticket here so this pull request can monitor
changes to the ticket status:

Redmine ticket: https://redmine.openinfosecfoundation.org/issues/7466

This test was contributed by the reporter of issue 7466. Slight
modifications not affecting the structure nor results were made.

This test exercises the problem reported in the issue and a memory leak
will be reported on Suricata binaries without the fix (ASAN-only).

Non-ASAN builds will not report the issue (regardless of whether the
Suricata fix is included).
@jlucovsky jlucovsky added the requires suricata fix This PR requires an issue in Suricata to be fixed first label Dec 29, 2024
@victorjulien
Copy link
Member

Merged in #2221, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires suricata fix This PR requires an issue in Suricata to be fixed first
Development

Successfully merging this pull request may close these issues.

2 participants