From f0f85727cd3dfd64da91d334718a18d28c023cab Mon Sep 17 00:00:00 2001 From: Fabian Keil Date: Sat, 11 Apr 2009 10:37:23 +0000 Subject: [PATCH] When dropping connections due to ACL, don't leak csp->ip_addr_str. --- jcc.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/jcc.c b/jcc.c index 8fb4e605..40829c08 100644 --- a/jcc.c +++ b/jcc.c @@ -1,4 +1,4 @@ -const char jcc_rcs[] = "$Id: jcc.c,v 1.239 2009/04/07 11:43:50 fabiankeil Exp $"; +const char jcc_rcs[] = "$Id: jcc.c,v 1.240 2009/04/09 10:12:54 fabiankeil Exp $"; /********************************************************************* * * File : $Source: /cvsroot/ijbswa/current/jcc.c,v $ @@ -33,6 +33,11 @@ const char jcc_rcs[] = "$Id: jcc.c,v 1.239 2009/04/07 11:43:50 fabiankeil Exp $" * * Revisions : * $Log: jcc.c,v $ + * Revision 1.240 2009/04/09 10:12:54 fabiankeil + * Fix two cases in which an invalid server response would result + * in the client connection being closed without sending an error + * message first. + * * Revision 1.239 2009/04/07 11:43:50 fabiankeil * If the server rudely resets the connection directly after sending the * headers, pass the mess to the client instead of sending an incorrect @@ -4281,6 +4286,7 @@ static void listen_loop(void) { log_error(LOG_LEVEL_CONNECT, "Connection from %s dropped due to ACL", csp->ip_addr_str); close_socket(csp->cfd); + freez(csp->ip_addr_str); freez(csp); continue; } -- 2.39.2