From: Matt Caswell Date: Mon, 24 Apr 2017 10:45:42 +0000 (+0100) Subject: Fix some variable references in init_client X-Git-Tag: OpenSSL_1_1_0f~49 X-Git-Url: https://git.openssl.org/gitweb/?p=openssl.git;a=commitdiff_plain;h=bb22c4057f31346b3d8f43929c759f692a7e7ef9 Fix some variable references in init_client We were incorrectly using "res" when we meant "ai" Reviewed-by: Richard Levitte (Merged from https://github.com/openssl/openssl/pull/3287) --- diff --git a/apps/s_socket.c b/apps/s_socket.c index d16f5ad817..0cbe88514d 100644 --- a/apps/s_socket.c +++ b/apps/s_socket.c @@ -75,11 +75,12 @@ int init_client(int *sock, const char *host, const char *port, /* Admittedly, these checks are quite paranoid, we should not get * anything in the BIO_ADDRINFO chain that we haven't * asked for. */ - OPENSSL_assert((family == AF_UNSPEC || family == BIO_ADDRINFO_family(res)) - && (type == 0 || type == BIO_ADDRINFO_socktype(res))); + OPENSSL_assert((family == AF_UNSPEC + || family == BIO_ADDRINFO_family(ai)) + && (type == 0 || type == BIO_ADDRINFO_socktype(ai))); *sock = BIO_socket(BIO_ADDRINFO_family(ai), BIO_ADDRINFO_socktype(ai), - BIO_ADDRINFO_protocol(res), 0); + BIO_ADDRINFO_protocol(ai), 0); if (*sock == INVALID_SOCKET) { /* Maybe the kernel doesn't support the socket family, even if * BIO_lookup() added it in the returned result...