diff options
author | Todd Short <tshort@akamai.com> | 2018-03-02 09:36:28 -0500 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2018-03-05 13:16:33 +0000 |
commit | aa2ed504ea9d55be6e75a9647d874709dc71823b (patch) | |
tree | d0868fee8e49b57d7f38d26e1bae1540eaf17a64 /ssl | |
parent | 5de683d2c6ae2f8c30b97326e72ff19f41228b74 (diff) |
If not sending key_share (no TLSv1.3), return appropriately.
This fixes an issue raised in PR #4964 by kaduk.
Reviewed-by: Ben Kaduk <kaduk@mit.edu>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5491)
Diffstat (limited to 'ssl')
-rw-r--r-- | ssl/statem/extensions_clnt.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ssl/statem/extensions_clnt.c b/ssl/statem/extensions_clnt.c index d569f3b84e..fa6c65b266 100644 --- a/ssl/statem/extensions_clnt.c +++ b/ssl/statem/extensions_clnt.c @@ -705,9 +705,10 @@ EXT_RETURN tls_construct_ctos_key_share(SSL *s, WPACKET *pkt, ERR_R_INTERNAL_ERROR); return EXT_RETURN_FAIL; } -#endif - return EXT_RETURN_SENT; +#else + return EXT_RETURN_NOT_SENT; +#endif } EXT_RETURN tls_construct_ctos_cookie(SSL *s, WPACKET *pkt, unsigned int context, |