From: David von Oheimb <dev@ddvo.net>
Date: Wed, 31 Jul 2024 17:32:44 +0000 (+0200)
Subject: http_server.c: fix checks of error return code in http_server_init()
X-Git-Tag: openssl-3.5.0-alpha1~1092
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7ec5d5916bc8563935901c027fe56b6644787d10;p=thirdparty%2Fopenssl.git

http_server.c: fix checks of error return code in http_server_init()

Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/25010)
---

diff --git a/apps/lib/http_server.c b/apps/lib/http_server.c
index e80ad14eb66..a5a86ecbc54 100644
--- a/apps/lib/http_server.c
+++ b/apps/lib/http_server.c
@@ -204,8 +204,8 @@ BIO *http_server_init(const char *prog, const char *port, int verb)
         goto err;
     acbio = BIO_new(BIO_s_accept());
     if (acbio == NULL
-        || BIO_set_bind_mode(acbio, BIO_BIND_REUSEADDR) < 0
-        || BIO_set_accept_name(acbio, name) < 0) {
+        || BIO_set_bind_mode(acbio, BIO_BIND_REUSEADDR) <= 0
+        || BIO_set_accept_name(acbio, name) <= 0) {
         log_HTTP(prog, LOG_ERR, "error setting up accept BIO");
         goto err;
     }