tree 99159b0bf8d73fbedf76997f6d50c4e8b89cd5b0
parent aa55640b8c3704de79b576337605d36d93fadfc3
author Willy Tarreau <w@1wt.eu> 1655977574 +0200
committer Willy Tarreau <w@1wt.eu> 1655977749 +0200
encoding latin1

BUG/MINOR: stream: only free the req/res captures when set

There's a subtle bug in stream_free() when releasing captures. The
pools may be NULL when no capture is defined, and the calls to
pool_free() are inconditional. The only reason why this doesn't
cause trouble is because the pointer to be freed is always NULL in
this case and we don't go further down the chain. That's particularly
ugly and it complicates debugging, so let's only call these ones when
the pointers are set.

There's no impact on running code, it only fools those trying to debug
pools manually. There's no need to backport it though it unless it helps
for debugging sessions.