Check to see if request->path is actually allocated before trying to free

it.

2 files changed, 10 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index 271d4e6..7bafb66 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -2,6 +2,8 @@
 
 	* src/reqs.c (remove_connection_headers): Handle both the
 	Connection header _and_ the Proxy-Connection header.
+	(free_request_struct): Check to see if request->path is actually
+	allocated before trying to delete it.
 
 	* src/hashmap.c: The hashmap will now handle multiple keys with
 	the same value.  This change was need because some sites like
diff --git a/src/reqs.c b/src/reqs.c
index 853474c..18bc675 100644
--- a/src/reqs.c
+++ b/src/reqs.c
@@ -1,4 +1,4 @@
-/* $Id: reqs.c,v 1.66 2002-04-25 18:58:08 rjkaes Exp $
+/* $Id: reqs.c,v 1.67 2002-04-25 19:20:56 rjkaes Exp $
  *
  * This is where all the work in tinyproxy is actually done. Incoming
  * connections have a new thread created for them. The thread then
@@ -192,7 +192,13 @@ free_request_struct(struct request_s *request)
 	safefree(request->protocol);
 
 	safefree(request->host);
-	safefree(request->path);
+
+	/*
+	 * The path is not used for SSL connection, so don't try to delete
+	 * it.
+	 */
+	if (request->path)
+		safefree(request->path);
 
 	safefree(request);
 }