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

Static ClientHello message could/should have the Random number randomized? #316

Open
jphughes opened this issue Jun 17, 2021 · 1 comment

Comments

@jphughes
Copy link

The TLS Security proof requires that the ClientHello Random actually be random. Without this being random there are problems with cut-paste and replay attacks. I assume the security of the traffic is not your concern so this may not be an issue you want to fix. This also applies to zgrab but since that is deprecated for this repo I have opened the issue here.

@mzpqnxow
Copy link
Contributor

mzpqnxow commented Jun 21, 2021

Hey @jphughes I'm not able to reproduce (though I admit I didn't try very hard)

I only tested with the http module, using:

echo '142.250.141.99,www.google.com' | zgrab2 http -p 443 --use-https

I ran that twice and observed that the ClientHello random bytes were different in both cases. I did the same with a few sites using SSLv3, TLSv1.0 and TLSv1.1 and saw the same

Can I ask if you are seeing this:

  1. In a specific module?
  2. With a specific SSL/TLS protocol version and/or a specific cipher-suite ?

I don't speak for the zmap project, but I am interested in ensuring these values are random, mainly to prevent signature-based blocks on network devices that may fire on a fixed ClientRandom value. The problem for me there is that it would impact the statistics that are gathered

Thanks

BTW- if you want to do any testing on your own, you can force the SSL/TLS protocol version using --min-version and --max-version. For SSLv3, --min-version=0x300 --max-version=0x300, for TLSv1.0 --min-version=0x301 --max-version=0x301, and so on...

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