pgsql: Fix more memory leaks in failure path in buildACLCommands. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix more memory leaks in failure path in buildACLCommands.
Date
Msg-id E1YLgp0-0001eC-18@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix more memory leaks in failure path in buildACLCommands.

We already had one go at this issue in commit d73b7f973db5ec7e, but we
failed to notice that buildACLCommands also leaked several PQExpBuffers
along with a simply malloc'd string.  This time let's try to make the
fix a bit more future-proof by eliminating the separate exit path.

It's still not exactly critical because pg_dump will curl up and die on
failure; but since the amount of the potential leak is now several KB,
it seems worth back-patching as far as 9.2 where the previous fix landed.

Per Coverity, which evidently is smarter than clang's static analyzer.

Branch
------
REL9_4_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/ee8d3778311e08d810f79c97f90ffb678063a59f

Modified Files
--------------
src/bin/pg_dump/dumputils.c |    7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix more memory leaks in failure path in buildACLCommands.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix more memory leaks in failure path in buildACLCommands.