From b424120b743f491d588270a7dad0ba79c76e7467 Mon Sep 17 00:00:00 2001
From: Amos Jeffries <amosjeffries@squid-cache.org>
Date: Wed, 14 Jul 2010 17:42:01 -0600
Subject: [PATCH] Author: Alex Rousskov <rousskov@measurement-factory.com>
 Fixed memory leak related to retried requests.

ErrorState object was not destroyed if the failed request to the origin
server or peer was retried.
---
 src/forward.cc | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/forward.cc b/src/forward.cc
index 9b0006af4e..92c6c55886 100644
--- a/src/forward.cc
+++ b/src/forward.cc
@@ -527,7 +527,10 @@ FwdState::retryOrBail()
 
         /* Ditch error page if it was created before.
          * A new one will be created if there's another problem */
-        err = NULL;
+        if (err) {
+            errorStateFree(err);
+            err = NULL;
+        }
 
         /* use eventAdd to break potential call sequence loops and to slow things down a little */
         eventAdd("fwdConnectStart", fwdConnectStartWrapper, this, originserver ? 0.05 : 0.005, 0);
-- 
2.47.2