tree 907c460ef4ad92ef029750fc09f839c9b549824d
parent 69a23ae0917750329fa5e6072f5cb9b68b949bdd
author Willy Tarreau <w@1wt.eu> 1626540390 +0200
committer Willy Tarreau <w@1wt.eu> 1626540390 +0200
encoding latin1

BUG/MINOR: cfgcond: revisit the condition freeing mechanism to avoid a leak

The cfg_free_cond_{term,and,expr}() functions used to take a pointer to
the pointer to be freed in order to replace it with a NULL once done.
But this doesn't cope well with freeing lists as it would require
recursion which the current code tried to avoid.

Let's just change the API to free the area and let the caller set the NULL.

This leak was reported by oss-fuzz (issue 36265).