diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-03-18 22:09:41 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-03-18 22:09:41 -0400 |
| commit | 5058e95a6ef99e03dd5e41c9cda531cbed721d53 (patch) | |
| tree | 339d5e32082ba336c1948f41b6d0f5de7fcbf2c6 /src/backend/executor/nodeAgg.c | |
| parent | 0b618ddf8bb2fba5492445b56d21647722b8350e (diff) | |
| download | postgresql-5058e95a6ef99e03dd5e41c9cda531cbed721d53.tar.gz postgresql-5058e95a6ef99e03dd5e41c9cda531cbed721d53.zip | |
Don't leak malloc'd strings when a GUC setting is rejected.
Because guc.c prefers to keep all its string values in malloc'd
not palloc'd storage, it has to be more careful than usual to
avoid leaks. Error exits out of string GUC hook checks failed
to clear the proposed value string, and error exits out of
ProcessGUCArray() failed to clear the malloc'd results of
ParseLongOption().
Found via valgrind testing.
This problem is ancient, so back-patch to all supported branches.
Discussion: https://postgr.es/m/3816764.1616104288@sss.pgh.pa.us
Diffstat (limited to 'src/backend/executor/nodeAgg.c')
0 files changed, 0 insertions, 0 deletions