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

Next/20220325/v1 #7172

Merged
merged 8 commits into from
Mar 27, 2022
Merged

Next/20220325/v1 #7172

merged 8 commits into from
Mar 27, 2022

Conversation

victorjulien
Copy link
Member

Staging:

Additionally, removes dead code from the streaming buffer API to simplify error handling.

Fixes #7171, fixing formatting
Replaces #7155

TheKharleeci and others added 8 commits March 21, 2022 15:32
These tests are reimplemented in Suricata-Verify

Task: 4911
Ticket: 4972

Completes commit c3a2206

DETECT_CONFIG is added as DETECT_SM_LIST_POSTMATCH and not
as DETECT_SM_LIST_MATCH as other keywords handled in SignatureCreateMask
Add debug validation checks for "impossible" conditions.
Fix parsing of content like "|aa b cc|" which was parsed as "|aa bc|"
without error or warning. This will now fail out, requiring all hex
values to be 2 chars.

Ticket OISF#5201
Even if the code seems unreachable for now
Coverity ID: 1502953

As we check just on the next line my_pool against NULL, we should
not dereference it, even for debug validation
Ticket: 4948

When parsing the response for starttls
@victorjulien victorjulien requested a review from a team as a code owner March 25, 2022 07:03
@victorjulien victorjulien mentioned this pull request Mar 25, 2022
@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 6655

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.

5 participants