diff options
author | Andy Polyakov <appro@openssl.org> | 2018-04-17 21:30:22 +0200 |
---|---|---|
committer | Andy Polyakov <appro@openssl.org> | 2018-04-18 20:08:05 +0200 |
commit | b3199e54d65f25c5ef6e54e129e228831a1fb363 (patch) | |
tree | 52ec4d3ef7dc97a321445ad68b3e2bf70ea7e209 | |
parent | f3d3b362553d95047d93608edd15d20aa6e6e0fd (diff) |
apps/s_socket.c: fix memory sanitizer problem in ACCEPT printout.
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/5994)
-rw-r--r-- | apps/s_socket.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/apps/s_socket.c b/apps/s_socket.c index d21bfc60c1..69f0f4f05e 100644 --- a/apps/s_socket.c +++ b/apps/s_socket.c @@ -211,7 +211,7 @@ int do_server(int *accept_sock, const char *host, const char *port, int i; BIO_ADDRINFO *res = NULL; const BIO_ADDRINFO *next; - int sock_family, sock_type, sock_protocol; + int sock_family, sock_type, sock_protocol, sock_port; const BIO_ADDR *sock_address; int sock_options = BIO_SOCK_REUSEADDR; int ret = 0; @@ -280,10 +280,12 @@ int do_server(int *accept_sock, const char *host, const char *port, } #endif + sock_port = BIO_ADDR_rawport(sock_address); + BIO_ADDRINFO_free(res); res = NULL; - if (BIO_ADDR_rawport(sock_address) == 0) { + if (sock_port == 0) { /* dynamically allocated port, report which one */ union BIO_sock_info_u info; char *hostname = NULL; |