From: Fabian Keil <fk@fabiankeil.de>
Date: Tue, 9 Sep 2025 11:39:58 +0000 (+0200)
Subject: accept_connection(): Fix memory and socket leak
X-Git-Url: http://www.privoxy.org/gitweb/@user-manual@@actions-help-prefix@FORCE-TEXT-MODE?a=commitdiff_plain;h=a2af69c67f04e2fe09574ff8f92ca8b106aeb6c2;p=privoxy.git

accept_connection(): Fix memory and socket leak

... if the server name and port number ASCII decimal
representation don't fit. This is not expected to happen.

Reported by Joshua Rogers.
---

diff --git a/jbsockets.c b/jbsockets.c
index 3ec64a8c..1ff73b34 100644
--- a/jbsockets.c
+++ b/jbsockets.c
@@ -1442,6 +1442,9 @@ int accept_connection(struct client_state * csp, jb_socket fds[])
          "Server name (%s) and port number (%d) ASCII decimal representation "
          "don't fit into %lu bytes",
          host_addr, csp->config->hport[i], listen_addr_size);
+      freez(csp->ip_addr_str);
+      freez(csp->listen_addr_str);
+      close_socket(csp->cfd);
       return 0;
    }