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/20220208/v2 #6956

Merged
merged 6 commits into from
Feb 9, 2022
Merged

Next/20220208/v2 #6956

merged 6 commits into from
Feb 9, 2022

Conversation

victorjulien
Copy link
Member

@victorjulien victorjulien commented Feb 8, 2022

victorjulien and others added 6 commits February 8, 2022 09:55
UDP frames point to the UDP packet payloads.

The frames are removed after each packet.

Ticket: OISF#4983.
Frames:
- sip.pdu
- sip.request_line
- sip.response_line
- sip.request_headers
- sip.response_headers
- sip.request_body
- sip.response_body

The `sip.pdu` frame is always created, the rest only if the record
parser succeeded.

Ticket: OISF#5036.
These tests are reimplemented as Suricata-Verify

Task: 4911
@victorjulien victorjulien requested review from jasonish and a team as code owners February 8, 2022 21:19
@codecov
Copy link

codecov bot commented Feb 8, 2022

Codecov Report

Merging #6956 (786cf41) into master (97ef60c) will decrease coverage by 0.04%.
The diff coverage is 52.56%.

@@            Coverage Diff             @@
##           master    #6956      +/-   ##
==========================================
- Coverage   77.74%   77.70%   -0.05%     
==========================================
  Files         628      628              
  Lines      185648   185653       +5     
==========================================
- Hits       144332   144255      -77     
- Misses      41316    41398      +82     
Flag Coverage Δ
fuzzcorpus 58.31% <27.27%> (+0.01%) ⬆️
suricata-verify 54.35% <49.19%> (-0.08%) ⬇️
unittests 63.03% <17.55%> (-0.06%) ⬇️

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 6161

@victorjulien victorjulien merged commit 786cf41 into OISF:master Feb 9, 2022
@victorjulien victorjulien deleted the next/20220208/v2 branch February 24, 2022 10:20
glongo added a commit to glongo/suricata that referenced this pull request Apr 13, 2024
PDU frames are now created prior to input parsing to ensure that their
creation is independent of the parsing outcome.

Additionally, a minor style change has been applied by rustfmt.

Ticket OISF#6956
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants