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

Fix (Ether() / ARP()).show() #1774

Merged
merged 1 commit into from
Jan 9, 2019
Merged

Fix (Ether() / ARP()).show() #1774

merged 1 commit into from
Jan 9, 2019

Conversation

p-l-
Copy link
Member

@p-l- p-l- commented Jan 9, 2019

Bug introduced in 435918e

Test added.

Bug introduced in 435918e

Test added.
@p-l- p-l- force-pushed the fix-eth-arp-show branch from f03ef06 to d139af4 Compare January 9, 2019 12:01
@codecov-io
Copy link

codecov-io commented Jan 9, 2019

Codecov Report

Merging #1774 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #1774      +/-   ##
=========================================
+ Coverage   85.57%   85.6%   +0.03%     
=========================================
  Files         181     181              
  Lines       42206   42209       +3     
=========================================
+ Hits        36117   36133      +16     
+ Misses       6089    6076      -13
Impacted Files Coverage Δ
scapy/layers/l2.py 83.66% <100%> (+0.37%) ⬆️
scapy/layers/ntp.py 91.77% <0%> (+0.26%) ⬆️
scapy/asn1/ber.py 82.38% <0%> (+0.28%) ⬆️
scapy/layers/tls/basefields.py 80.53% <0%> (+0.67%) ⬆️
scapy/layers/tls/handshake_sslv2.py 92.77% <0%> (+0.76%) ⬆️
scapy/layers/tls/record.py 92.1% <0%> (+0.87%) ⬆️
scapy/arch/bpf/core.py 82.1% <0%> (+1.05%) ⬆️
scapy/layers/tls/record_sslv2.py 88.7% <0%> (+1.69%) ⬆️

@p-l- p-l- added the bug label Jan 9, 2019
@guedou guedou merged commit 4b37729 into secdev:master Jan 9, 2019
@p-l- p-l- deleted the fix-eth-arp-show branch January 9, 2019 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants