diff options
| author | Tim Hudson <tjh@cryptsoft.com> | 2014-05-05 10:53:39 +1000 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2014-05-08 23:19:19 +0100 |
| commit | 9e456a853749de84a7efff1fd4324134ff98187e (patch) | |
| tree | c56cfc7281d4ee2b4650aaabd35a131af7e50fcc /apps | |
| parent | f179e2b89928459dfbb6b56466b2a53169f8e83f (diff) | |
| download | openssl-new-9e456a853749de84a7efff1fd4324134ff98187e.tar.gz | |
coverity 966576 - close socket in error path
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/s_socket.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/apps/s_socket.c b/apps/s_socket.c index d49837a405..94eb40f3ff 100644 --- a/apps/s_socket.c +++ b/apps/s_socket.c @@ -450,6 +450,7 @@ redoit: if ((*host=(char *)OPENSSL_malloc(strlen(h1->h_name)+1)) == NULL) { perror("OPENSSL_malloc"); + closesocket(ret); return(0); } BUF_strlcpy(*host,h1->h_name,strlen(h1->h_name)+1); @@ -458,11 +459,13 @@ redoit: if (h2 == NULL) { BIO_printf(bio_err,"gethostbyname failure\n"); + closesocket(ret); return(0); } if (h2->h_addrtype != AF_INET) { BIO_printf(bio_err,"gethostbyname addr is not AF_INET\n"); + closesocket(ret); return(0); } } |