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 fdfefb52fcc35e6f08e597ad4775aaf413647354..f924320b67f5b25fb26f2a6d14d67119909c244c 100644 (file)
--- a/src/HttpHeader.cc
+++ b/src/HttpHeader.cc
@@ -1534,8 +1534,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 << ": " << value << "'");
 
     return new HttpHeaderEntry(id, name.termedBuf(), value.termedBuf());