From: Fabian Keil <fk@fabiankeil.de>
Date: Sat, 11 Apr 2009 10:37:23 +0000 (+0000)
Subject: When dropping connections due to ACL, don't leak csp->ip_addr_str.
X-Git-Tag: v_3_0_13~133
X-Git-Url: http://www.privoxy.org/gitweb/@default-cgi@/faq/%22https:/@default-cgi@show-url-info?a=commitdiff_plain;h=f0f85727cd3dfd64da91d334718a18d28c023cab;p=privoxy.git

When dropping connections due to ACL, don't leak csp->ip_addr_str.
---

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;
       }