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

Batch backports to master-5.0.x #6384

Closed
wants to merge 8 commits into from

Conversation

jlucovsky
Copy link
Contributor

This PR contains backports for the issues listed below

Link to redmine ticket:

Describe changes:

#suricata-verify-pr:
#suricata-verify-repo:
#suricata-verify-branch:
#suricata-update-pr:
#suricata-update-repo:
#suricata-update-branch:
#libhtp-pr:
#libhtp-repo:
#libhtp-branch:

inashivb and others added 8 commits September 17, 2021 09:20
As is done already in C
cf commit ea09361

(cherry picked from commit 5ec9688)
Ticket: OISF#4558
Avoids intra structure overflow

(cherry picked from commit 126a7dc)
Ticket: OISF#4558
So as to avoid intra-structure overflow

(cherry picked from commit 44bd316)
When tracking data track the size of the blocks so that in case
of gaps we can still know how much data we hold.

(cherry picked from commit be1baa8)
For size limit checks consider only available data at the stream start
and before any GAPS.

The old check would consider too much data if there were temporary gaps,
like when a data packet was in-window but (far) ahead of the expected
segment.

(cherry picked from commit 7a114e5)
To take TCP window into account
And to actually bail out if we received too much data
where the limit is configured by stream.reassembly.depth

(cherry picked from commit f77b027)
@jlucovsky jlucovsky requested a review from a team as a code owner September 17, 2021 13:31
@victorjulien
Copy link
Member

We probably need a gitlab-ci update to match master/master-6.0.x as well?

@jlucovsky
Copy link
Contributor Author

We probably need a gitlab-ci update to match master/master-6.0.x as well?

Which commit(s) are these?

@catenacyber
Copy link
Contributor

The ones seen in git log master -- .github I guess

@jlucovsky
Copy link
Contributor Author

Continued in #6386

@jlucovsky jlucovsky closed this Sep 17, 2021
@jlucovsky jlucovsky deleted the batch-5.0.x/1 branch September 28, 2021 12:40
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.

4 participants