Ignore the status_request extension in a resumption handshake
authorMatt Caswell <matt@openssl.org>
Fri, 6 Apr 2018 13:33:07 +0000 (14:33 +0100)
committerMatt Caswell <matt@openssl.org>
Tue, 17 Apr 2018 15:47:37 +0000 (16:47 +0100)
We cannot provide a certificate status on a resumption so we should
ignore this extension in that case.

Fixes #1662

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5898)

ssl/t1_lib.c

index 75c2f4154dfe6496d68497b0aa481699d2957b4a..179802ca598a27358966c9e4a0c24f6405612bdc 100644 (file)
@@ -2408,8 +2408,7 @@ static int ssl_scan_clienthello_tlsext(SSL *s, unsigned char **p,
                 goto err;
             if (!tls1_save_sigalgs(s, data, dsize))
                 goto err;
-        } else if (type == TLSEXT_TYPE_status_request) {
-
+        } else if (type == TLSEXT_TYPE_status_request && !s->hit) {
             if (size < 5)
                 goto err;