]> granicus.if.org Git - curl/commitdiff
openssl: output the correct cipher list on TLS 1.3 error
authorDaniel Stenberg <daniel@haxx.se>
Fri, 26 Oct 2018 11:34:37 +0000 (13:34 +0200)
committerDaniel Stenberg <daniel@haxx.se>
Sat, 27 Oct 2018 08:46:38 +0000 (10:46 +0200)
When failing to set the 1.3 cipher suite, the wrong string pointer would
be used in the error message. Most often saying "(nil)".

Reported-by: Ricky-Tigg on github
Fixes #3178
Closes #3180

lib/vtls/openssl.c

index b067053121fcb0b14fbbf238ff4665374cb21555..4c5e8c19c05210629a9c22c30baa2041354a387a 100644 (file)
@@ -2466,7 +2466,7 @@ static CURLcode ossl_connect_step1(struct connectdata *conn, int sockindex)
     char *ciphers13 = SSL_CONN_CONFIG(cipher_list13);
     if(ciphers13) {
       if(!SSL_CTX_set_ciphersuites(BACKEND->ctx, ciphers13)) {
-        failf(data, "failed setting TLS 1.3 cipher suite: %s", ciphers);
+        failf(data, "failed setting TLS 1.3 cipher suite: %s", ciphers13);
         return CURLE_SSL_CIPHER;
       }
       infof(data, "TLS 1.3 cipher selection: %s\n", ciphers13);