From ef8ffdda043f0f4f78fd98469dd256776e99653a Mon Sep 17 00:00:00 2001 From: Bram Matthys Date: Tue, 19 Dec 2006 19:37:41 +0000 Subject: [PATCH] - Showing even more SSL server errors now, hopefully all of them, also changed the error notice a bit so it's much more like non-SSL server link errors. Reported by vonitsanet (#0003150). --- Changes | 3 +++ src/ssl.c | 14 +++++++++++++- 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/Changes b/Changes index 07a4f01a8..33cb67010 100644 --- a/Changes +++ b/Changes @@ -1423,3 +1423,6 @@ (#0003143). ** 3.2.6-rc2 release ** - Fixed possible crash with using quarantine, reported by Sephiroth (#0003151). +- Showing even more SSL server errors now, hopefully all of them, also changed the + error notice a bit so it's much more like non-SSL server link errors. Reported by + vonitsanet (#0003150). diff --git a/src/ssl.c b/src/ssl.c index 47f6f1373..9919e53fd 100644 --- a/src/ssl.c +++ b/src/ssl.c @@ -670,7 +670,19 @@ static int fatal_ssl_error(int ssl_error, int where, aClient *sptr) char *myerr = ssl_errstr; if (ssl_error == SSL_ERROR_SYSCALL) myerr = STRERROR(errtmp); - sendto_failops_whoare_opers("Closing link: SSL_connect(): %s - %s", myerr, get_client_name(sptr, FALSE)); + /* sendto_failops_whoare_opers("Closing link: SSL_connect(): %s - %s", myerr, get_client_name(sptr, FALSE)); */ + sendto_umode(UMODE_OPER, "Lost connection to %s: %s: %s", + get_client_name(sptr, FALSE), ssl_func, myerr); + } else + if ((IsServer(sptr) || (sptr->serv && sptr->serv->conf)) && (where != SAFE_SSL_WRITE)) + { + /* if server (either judged by IsServer() or clearly an outgoing connect), + * and not writing (since otherwise deliver_it will take care of the error), THEN + * send a closing link error... + */ + sendto_umode(UMODE_OPER, "Lost connection to %s: %s: %s", + get_client_name(sptr, FALSE), ssl_func, ssl_errstr); + /* sendto_failops_whoare_opers("Closing link: %s: %s - %s", ssl_func, ssl_errstr, get_client_name(sptr, FALSE)); */ } if (errtmp)