Skip to content

Commit

Permalink
Bug 1335069 - Don't leak public key when falling back to SHA-1 signat…
Browse files Browse the repository at this point in the history
…ures for client certs r=bustage
  • Loading branch information
Tim Taubert committed Jan 31, 2017
1 parent 8fa4317 commit 0e83c0f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/ssl/ssl3con.c
Expand Up @@ -6503,7 +6503,9 @@ ssl_PickClientSignatureScheme(sslSocket *ss, const SSLSignatureScheme *schemes,
if (!isTLS13 && numSchemes == 0) {
/* If the server didn't provide any signature algorithms
* then let's assume they support SHA-1. */
return ssl_PickFallbackSignatureScheme(ss, pubKey);
rv = ssl_PickFallbackSignatureScheme(ss, pubKey);
SECKEY_DestroyPublicKey(pubKey);
return rv;
}

PORT_Assert(schemes && numSchemes > 0);
Expand Down

0 comments on commit 0e83c0f

Please sign in to comment.