[PATCH] Fixed leaking configurable SSL error details.

From: Alex Rousskov <rousskov_at_measurement-factory.com>
Date: Fri, 10 May 2013 16:24:46 -0600

Hello,

    The attached patch fixes a memory leak related to parsing
configurable SSL error details.

Trunk r11496 "Configurable SSL error details messages" correctly
disabled collection of HTTP statistics for non-HTTP header fields, such
as configurable SSL error details. However, it also incorrectly disabled
deletion of those non-HTTP header fields.

Configurable SSL error details are only created during [re]configuration
time, so the leak went unnoticed since 2011-06-17. However, when
annotations were added to non-HTTP header fields (r12413), the old bug
resulted in a major leak for configurations that annotate transactions.

Fortunately, that major leak was removed by r12779 "helperNotes to
NotePairs" polishing, which stopped using HttpHeader for annotations.
However, any code based on trunk between r12413 and r12779 needs this
fix (or needs to be resynced with the current trunk).

HTH,

Alex.

Received on Fri May 10 2013 - 22:24:49 MDT

This archive was generated by hypermail 2.2.0 : Sat May 11 2013 - 12:00:12 MDT