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

Backports 5.0.x V1 Batched backports of 6 issues, 1 undisclosed. #4718

Closed
wants to merge 16 commits into from

Conversation

victorjulien and others added 16 commits March 22, 2020 11:39
When a TCP DNS flow would start with a GAP on the TS side, the successful
protocol detection on the TC side would trigger 'opposing side' reassembly
and app-layer processing. In this case the stream flags would indicate the
wrong direction and the wrong parser would be called.

(cherry picked from commit efee458)
Suricata generates an error on unrecognised ERF types.
Suricata should ignore ERF 'Provenance' records with ERF_TYPE_META.

(cherry picked from commit 47082dd)
This commit places restrictions on the length of the file name specified
in attachments (`name=` or `filename=`) to `NAME_MAX`. Names exceeding
these limits will be truncated and processing will continue with the
truncated name.

(cherry picked from commit d0d20bd)
(cherry picked from commit 9a33b5d)
(cherry picked from commit 6d94b09)
This commit replaces usages of pcre_get_substring with
pcre_copy_substring to avoid leaking memory on error conditions.

(cherry picked from commit 6c35039)
This commit replaces usages of pcre_get_substring with
pcre_copy_substring to avoid leaking memory on error conditions.

(cherry picked from commit 9fe51a8)
This commit replaces usages of pcre_get_substring with
pcre_copy_substring to avoid leaking memory on error conditions.

(cherry picked from commit c2071e1)
Corrects misplaced backticks preventing proper formatting of `mpm-algo`
section.

(cherry picked from commit 8c132c0)
(cherry picked from commit 427ec4e)
(cherry picked from commit ed6c976)
(cherry picked from commit aec4e9a)
(cherry picked from commit e00de3d)
For the backport, ERSPAN TypeI decode is

1. Disabled by default
2. Configurable: `decoder.erspan_typeI.enabled`

(cherry picked from commit ae6beed)
Switch to isspace() as well.

(cherry picked from commit 52970d8)
@jlucovsky jlucovsky requested review from norg and a team as code owners March 22, 2020 17:46
@jlucovsky jlucovsky changed the title Backports 5.x/1 Backports 5.0.x V1 Batched backports of 6 issues, 1 undisclosed. Mar 22, 2020
@jlucovsky jlucovsky added the backport Backport to released Suricata versions label Mar 22, 2020
@victorjulien
Copy link
Member

Not sure I agree with using 'undisclosed' for 'missing ticket' :)

@jlucovsky
Copy link
Contributor Author

jlucovsky commented Mar 24, 2020

Continued in #4725

@jlucovsky jlucovsky closed this Mar 24, 2020
@jlucovsky jlucovsky deleted the backports-5.x/1 branch June 9, 2020 12:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backport to released Suricata versions
4 participants