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:14:52 +0100 |
commit | b6e69d284b79097d0d9e39996cbe59eae6bb36e2 (patch) | |
tree | ef35571107580c8d35b61476285c75dc49d23498 /apps | |
parent | 8e94fadd0b79491714401d89b338db27211b9819 (diff) |
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 1841607ef5..65003fb6f0 100644 --- a/apps/s_socket.c +++ b/apps/s_socket.c @@ -452,6 +452,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); @@ -460,11 +461,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); } } |