diff options
author | Daniel Stenberg <daniel@haxx.se> | 2015-03-21 16:50:51 +0100 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2015-03-21 16:53:43 +0100 |
commit | ed429b72d7dee4f63f1e1abc10b52e7b0739da82 (patch) | |
tree | d566e0224945cbb38754ae4309825140ca30c9b8 /lib | |
parent | fea13a17d849e6db94c553811423aa98017a2c77 (diff) |
gtls: add check of return code
Coverity CID 1291167 pointed out that 'rc' was received but never used when
gnutls_credentials_set() was used. Added return code check now.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/vtls/gtls.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/lib/vtls/gtls.c b/lib/vtls/gtls.c index 82c09bc80..3ad0f82ff 100644 --- a/lib/vtls/gtls.c +++ b/lib/vtls/gtls.c @@ -671,13 +671,21 @@ gtls_connect_step1(struct connectdata *conn, if(data->set.ssl.authtype == CURL_TLSAUTH_SRP) { rc = gnutls_credentials_set(session, GNUTLS_CRD_SRP, conn->ssl[sockindex].srp_client_cred); - if(rc != GNUTLS_E_SUCCESS) + if(rc != GNUTLS_E_SUCCESS) { failf(data, "gnutls_credentials_set() failed: %s", gnutls_strerror(rc)); + return CURLE_SSL_CONNECT_ERROR; + } } else #endif + { rc = gnutls_credentials_set(session, GNUTLS_CRD_CERTIFICATE, conn->ssl[sockindex].cred); + if(rc != GNUTLS_E_SUCCESS) { + failf(data, "gnutls_credentials_set() failed: %s", gnutls_strerror(rc)); + return CURLE_SSL_CONNECT_ERROR; + } + } /* set the connection handle (file descriptor for the socket) */ gnutls_transport_set_ptr(session, |