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

Regarding warning Unknown crypto suite from ClientMasterKey #4248

Open
russdm opened this issue Feb 2, 2024 · 4 comments
Open

Regarding warning Unknown crypto suite from ClientMasterKey #4248

russdm opened this issue Feb 2, 2024 · 4 comments

Comments

@russdm
Copy link

russdm commented Feb 2, 2024

Brief description

Segmented packets are read in and misinterpreted as SSLv2 handshakes.
Packets are often segmented TLS application Data packets when compared to Wireshark.

Scapy version

2.5.0

Python version

3.12.1

Operating system

Windows 11

Additional environment information

Develop a class in the SSLv2 handshake that recognizes and ignores segmented packets.

How to reproduce

sniff TLS packets and segmented TLS packets get misinterpreted as SSLv2_handshakes.

Actual result

No response

Expected result

No response

Related resources

No response

@gpotter2
Copy link
Member

gpotter2 commented Feb 2, 2024

Please provide a pcap and the code you're using to sniff.

@russdm
Copy link
Author

russdm commented Feb 2, 2024

See the following I've raised the warning for UnknownCipherSuite as an error, and have been logging them in my packet captures. In comparison to the packet captures in Wireshark. Application Data is being read as SSLv2ClientMasterkey.
See the zip file containing code I'm using to capture packets with warning raised as error and logged.
Tests for unknown cipher suite.zip

@russdm
Copy link
Author

russdm commented Feb 2, 2024

Just a note the first two captures in the log file are in the pcap. To save longevity of analysis. Only meant to do two to save having to analyze a bunch of packets.

@russdm
Copy link
Author

russdm commented Feb 4, 2024

Information regarding the SSLv2 header if your interested.
SSLv2 header.txt

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

No branches or pull requests

2 participants