| commit | f1db20c473e1c5bfb20a8613325675d178499f84 | [log] [tgz] |
|---|---|---|
| author | Willy Tarreau <w@1wt.eu> | Sat Jul 17 18:46:30 2021 +0200 |
| committer | Willy Tarreau <w@1wt.eu> | Sat Jul 17 18:46:30 2021 +0200 |
| tree | 907c460ef4ad92ef029750fc09f839c9b549824d | |
| parent | 69a23ae0917750329fa5e6072f5cb9b68b949bdd [diff] |
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).