On 4/5/2020 1:41 PM, Anthony Scarpino wrote:
On 4/4/20 6:11 PM, Xuelei Fan wrote:
Hi,
Could I have the following update reviewed?
http://cr.openjdk.java.net/~xuelei/8215711/webrev.00/
In the current TLS implementation, if one of "supported_groups"
extension and "key_share" extension is not present in the ClientHello
handshake message, the internal_error alter will be used. Per the
spec (RFC 8846), the alert should be "missing_extension" alert.
The fuzzing test passed with this update. No new regression test
(noreg-external).
Thanks,
Xuelei
The change looks fine. It looks like you implemented what is in section
9.2, and it looks like the absent methods will be called from the
consumeOnLoad() as all those extensions can be in a ClientHello msg.
Yes.
Thanks for the review!
Xuelei
