From 6085834c9009a4188ba009ec7585af0491622159 Mon Sep 17 00:00:00 2001 From: Art Cancro Date: Mon, 27 Dec 2021 15:51:58 -0500 Subject: [PATCH] Not sure what the point of that second call to SSL_accept() was for, but it was always failing, and removing it seems to be fine --- webcit/crypto.c | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/webcit/crypto.c b/webcit/crypto.c index 104945b04..c396e2d0e 100644 --- a/webcit/crypto.c +++ b/webcit/crypto.c @@ -110,25 +110,25 @@ int starttls(int sock) { else { syslog(LOG_WARNING, "first SSL_accept failed: %s", ssl_error_reason); } - sleeeeeeeeeep(1); - retval = SSL_accept(newssl); - } - if (retval < 1) { - long errval; - const char *ssl_error_reason = NULL; - - errval = SSL_get_error(newssl, retval); - ssl_error_reason = ERR_reason_error_string(ERR_get_error()); - if (ssl_error_reason == NULL) { - syslog(LOG_WARNING, "second SSL_accept failed: errval=%ld, retval=%d (%s)", errval, retval, strerror(errval)); - } - else { - syslog(LOG_WARNING, "second SSL_accept failed: %s", ssl_error_reason); - } - SSL_free(newssl); - newssl = NULL; - return(4); + // sleeeeeeeeeep(1); + // retval = SSL_accept(newssl); } + // if (retval < 1) { + // long errval; + // const char *ssl_error_reason = NULL; +// + // errval = SSL_get_error(newssl, retval); + // ssl_error_reason = ERR_reason_error_string(ERR_get_error()); + // if (ssl_error_reason == NULL) { + // syslog(LOG_WARNING, "second SSL_accept failed: errval=%ld, retval=%d (%s)", errval, retval, strerror(errval)); + // } + // else { + // syslog(LOG_WARNING, "second SSL_accept failed: %s", ssl_error_reason); + // } + // SSL_free(newssl); + // newssl = NULL; + // return(4); + // } else { syslog(LOG_INFO, "SSL_accept success"); } -- 2.30.2