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 44afec3eccd31fc640cb8bc3c8a98833b7911f4c..7a2f01aeba05fb80ca6b5a6c5c1c6963c9703b17 100644 (file)
--- 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());