]> Cypherpunks repositories - gostls13.git/commit
crypto/tls: don't check whether an ec point is on a curve twice
authorAndreas Auernhammer <aead@mail.de>
Fri, 26 May 2017 09:33:49 +0000 (11:33 +0200)
committerAdam Langley <agl@golang.org>
Tue, 15 Aug 2017 18:44:20 +0000 (18:44 +0000)
commitd38d357c787f38eadd511b7dadda4fe3b1d7391b
treef3a16b4aa17e1d2dd7ed4294ef5d8df18e80053a
parent0a1a65c9ce2a55a41ccce8e681faabb719c827db
crypto/tls: don't check whether an ec point is on a curve twice

The processClientKeyExchange and processServerKeyExchange functions unmarshal an
encoded EC point and explicitly check whether the point is on the curve. The explicit
check can be omitted because elliptic.Unmarshal fails if the point is not on the curve
and the returned error would always be the same.

Fixes #20496

Change-Id: I5231a655eace79acee2737dd036a0c255ed42dbb
Reviewed-on: https://go-review.googlesource.com/44311
Reviewed-by: Adam Langley <agl@golang.org>
Reviewed-by: Avelino <t@avelino.xxx>
Run-TryBot: Adam Langley <agl@golang.org>
src/crypto/tls/key_agreement.go