scapy icon indicating copy to clipboard operation
scapy copied to clipboard

Fix payload_len calculation for `DataPacket`.

Open Splinter1984 opened this issue 1 year ago • 1 comments

This PR fix pl_len calculation issue for DataPacket of RTPS protocol. Take into account the size of inlineQoS during payload length calculation. Closes #4594

Splinter1984 avatar Oct 01 '24 08:10 Splinter1984

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 80.87%. Comparing base (ef72e1d) to head (1bb70c5). :warning: Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4545      +/-   ##
==========================================
+ Coverage   80.48%   80.87%   +0.38%     
==========================================
  Files         368      368              
  Lines       90255    90256       +1     
==========================================
+ Hits        72645    72992     +347     
+ Misses      17610    17264     -346     
Files with missing lines Coverage Δ
scapy/contrib/rtps/common_types.py 95.31% <100.00%> (+0.03%) :arrow_up:
scapy/contrib/rtps/rtps.py 100.00% <100.00%> (+1.17%) :arrow_up:

... and 23 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Oct 01 '24 09:10 codecov[bot]