Skip to content

Commit 1095668

Browse files
committed
Bug #18384260: MULTIPLE SECURITY ISSUES IN CERTIFICATE VALIDATION
the 5.5 version of the fix. Added a call to X509_verify_cert_error_string() into the client certificate verification code.
1 parent 832bebc commit 1095668

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

sql-common/client.c

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1909,6 +1909,12 @@ static int ssl_verify_server_cert(Vio *vio, const char* server_hostname, const c
19091909
DBUG_RETURN(1);
19101910
}
19111911

1912+
if (X509_V_OK != SSL_get_verify_result(ssl))
1913+
{
1914+
*errptr= "Failed to verify the server certificate";
1915+
X509_free(server_cert);
1916+
DBUG_RETURN(1);
1917+
}
19121918
/*
19131919
We already know that the certificate exchanged was valid; the SSL library
19141920
handled that. Now we need to verify that the contents of the certificate

0 commit comments

Comments
 (0)