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/112/20230801/v1 #9320

Merged
merged 7 commits into from Aug 2, 2023
Merged

Conversation

victorjulien
Copy link
Member

victorjulien and others added 7 commits August 1, 2023 19:43
Normalize the timestamps that are too far in the past to epoch.

Bug: OISF#6240.
Data on RST packets is not invalid, but also shouldn't be used
in reassembly.

RFC 1122:

  4.2.2.12  RST Segment: RFC-793 Section 3.4

    A TCP SHOULD allow a received RST segment to include data.

    DISCUSSION
        It has been suggested that a RST segment could contain
        ASCII text that encoded and explained the cause of the
        RST.  No standard has yet been established for such
        data.

RST data will be presented to the detection engine per packet,
but will not be part of stream reassembly.

Bug: OISF#6244.
Signed-off-by: jason taylor <jtfas90@gmail.com>
@codecov
Copy link

codecov bot commented Aug 1, 2023

Codecov Report

Merging #9320 (89f1837) into master (26130d9) will decrease coverage by 0.02%.
The diff coverage is 86.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9320      +/-   ##
==========================================
- Coverage   82.41%   82.40%   -0.02%     
==========================================
  Files         968      968              
  Lines      274055   274064       +9     
==========================================
- Hits       225874   225838      -36     
- Misses      48181    48226      +45     
Flag Coverage Δ
fuzzcorpus 64.65% <86.66%> (-0.04%) ⬇️
suricata-verify 60.87% <80.00%> (+<0.01%) ⬆️
unittests 62.91% <26.66%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 15478

@victorjulien victorjulien merged commit 89f1837 into OISF:master Aug 2, 2023
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants