BIO_sock_init() returns 1 for success and -1 for failure, not 0;
authorBodo Möller <bodo@openssl.org>
Mon, 9 Oct 2000 16:40:16 +0000 (16:40 +0000)
committerBodo Möller <bodo@openssl.org>
Mon, 9 Oct 2000 16:40:16 +0000 (16:40 +0000)
thus the condition '!BIO_sock_init()' doesn't make sense.

crypto/bio/b_sock.c

index 6431005..e971a07 100644 (file)
@@ -113,8 +113,8 @@ int BIO_get_host_ip(const char *str, unsigned char *ip)
 
        /* At this point, we have something that is most probably correct
           in some way, so let's init the socket. */
 
        /* At this point, we have something that is most probably correct
           in some way, so let's init the socket. */
-       if (!BIO_sock_init())
-               return(0); /* don't generate another error code here */
+       if (BIO_sock_init() != 1)
+               return 0; /* don't generate another error code here */
 
        /* If the string actually contained an IP address, we need not do
           anything more */
 
        /* If the string actually contained an IP address, we need not do
           anything more */
@@ -527,7 +527,7 @@ int BIO_get_accept_socket(char *host, int bind_mode)
        unsigned long l;
        int err_num;
 
        unsigned long l;
        int err_num;
 
-       if (!BIO_sock_init()) return(INVALID_SOCKET);
+       if (BIO_sock_init() != 1) return(INVALID_SOCKET);
 
        if ((str=BUF_strdup(host)) == NULL) return(INVALID_SOCKET);
 
 
        if ((str=BUF_strdup(host)) == NULL) return(INVALID_SOCKET);