From: Matt Caswell <matt@openssl.org>
Date: Thu, 19 Apr 2018 09:38:57 +0000 (+0100)
Subject: Fix the alert sent if no shared sig algs
X-Git-Tag: OpenSSL_1_1_0i~174
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f55e2fa7b9d4692b31ad7ad72e71915e853de384;p=openssl

Fix the alert sent if no shared sig algs

We were sending illegal parameter. This isn't correct. The parameters are
legal, we just don't have an overlap. A more appropriate alert is
handshake failure.

Fixes #2919

Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/6010)
---

diff --git a/ssl/t1_lib.c b/ssl/t1_lib.c
index 5ba7377fe3..6f4078e88e 100644
--- a/ssl/t1_lib.c
+++ b/ssl/t1_lib.c
@@ -2800,7 +2800,7 @@ int tls1_set_server_sigalgs(SSL *s)
         if (!s->cert->shared_sigalgs) {
             SSLerr(SSL_F_TLS1_SET_SERVER_SIGALGS,
                    SSL_R_NO_SHARED_SIGNATURE_ALGORITHMS);
-            al = SSL_AD_ILLEGAL_PARAMETER;
+            al = SSL_AD_HANDSHAKE_FAILURE;
             goto err;
         }
     } else {