From: Amos Jeffries <squid3@treenet.co.nz>
Date: Wed, 26 May 2010 04:55:21 +0000 (+1200)
Subject: Author: Alex Rousskov <rousskov@measurement-factory.com>
X-Git-Tag: SQUID_3_0_STABLE26~12
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=80948d4eb09986d58bec0445b5b546e5ad2f578d;p=thirdparty%2Fsquid.git

Author: Alex Rousskov <rousskov@measurement-factory.com>
Bug 2922: Fix assertion failed: HttpHeader.cc: "Headers[id].stat.aliveCount"

Fixed header accounting to avoid the "Headers[id].stat.aliveCount" assertion.

We were incrementing the alive header field counter twice for each decrement,
which probably resulted in the alive counter wrapping back to zero, triggering
the assertion.
---

diff --git a/src/HttpHeader.cc b/src/HttpHeader.cc
index 44afec3ecc..7a2f01aeba 100644
--- a/src/HttpHeader.cc
+++ b/src/HttpHeader.cc
@@ -1522,8 +1522,6 @@ HttpHeaderEntry::parse(const char *field_start, const char *field_end)
 
     Headers[id].stat.seenCount++;
 
-    Headers[id].stat.aliveCount++;
-
     debugs(55, 9, "parsed HttpHeaderEntry: '" << name.buf() << ": " << value.buf() << "'");
 
     return new HttpHeaderEntry(id, name.buf(), value.buf());