From 6992c74bba22981acd866d4b3521b0e31ad15fae Mon Sep 17 00:00:00 2001 From: Robert James Kaes Date: Thu, 25 Apr 2002 19:20:56 +0000 Subject: Check to see if request->path is actually allocated before trying to free it. --- ChangeLog | 2 ++ src/reqs.c | 10 ++++++++-- 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); } -- cgit v1.2.3