Fixed memory leak related to retried requests.

ErrorState object was not destroyed if the failed request to the origin
server or peer was retried.

diff --git a/src/forward.cc b/src/forward.cc
index a954803d05bf26be40366d948e3cffb642187e51..fcc5d009f4b462e6967816aef6a81cdfa20187d7 100644 (file)
--- a/src/forward.cc
+++ b/src/forward.cc
@@ -528,7 +528,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);