Skip to content

Decode IPv6 Jumbograms#1196

Merged
p-l- merged 1 commit intosecdev:masterfrom
guedou:ipv6_jumbo
Mar 1, 2018
Merged

Decode IPv6 Jumbograms#1196
p-l- merged 1 commit intosecdev:masterfrom
guedou:ipv6_jumbo

Conversation

@guedou
Copy link
Copy Markdown
Member

@guedou guedou commented Feb 28, 2018

This PR is a bit tricky as some parsing must be in IPv6.extract_padding() to retrieve the payload length from the payload itself.

@guedou
Copy link
Copy Markdown
Member Author

guedou commented Feb 28, 2018

Ready to be merged. The failing AppVeyor is related to nmap.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 28, 2018

Codecov Report

Merging #1196 into master will increase coverage by 0.05%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##           master    #1196      +/-   ##
==========================================
+ Coverage   84.04%   84.09%   +0.05%     
==========================================
  Files         159      159              
  Lines       38211    38228      +17     
==========================================
+ Hits        32114    32148      +34     
+ Misses       6097     6080      -17
Impacted Files Coverage Δ
scapy/layers/inet6.py 85.78% <90%> (+0.02%) ⬆️
scapy/fields.py 90.7% <0%> (+0.17%) ⬆️
scapy/config.py 85.39% <0%> (+0.27%) ⬆️
scapy/asn1/ber.py 82.28% <0%> (+0.28%) ⬆️
scapy/layers/tls/basefields.py 80.27% <0%> (+0.68%) ⬆️
scapy/layers/tls/record.py 92.08% <0%> (+0.87%) ⬆️
scapy/layers/tls/record_sslv2.py 88.82% <0%> (+1.67%) ⬆️
scapy/layers/tls/keyexchange.py 84.76% <0%> (+1.71%) ⬆️

@p-l- p-l- merged commit 254ab6d into secdev:master Mar 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants