Fix minor memory leak in pg_dump
authorMichael Paquier <michael@paquier.xyz>
Wed, 15 Apr 2020 06:57:00 +0000 (15:57 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 15 Apr 2020 06:57:00 +0000 (15:57 +0900)
A query used to read default ACL information from the catalogs did not
free a set of PQExpBuffer.

Oversight in commit e2090d9, so backpatch down to 9.6.

Author: Jie Zhang
Reviewed-by: Sawada Masahiko
Discussion: https://postgr.es/m/05bcbc5857f948efa0b451b85a48ae10@G08CNEXMBPEKD06.g08.fujitsu.local
Backpatch-through: 9.6

src/bin/pg_dump/pg_dump.c

index f36065f4a3ce9ec1846cd702a77e91f28f5c4b8a..3339df8ada4dfd9609a0a511ee053896c2631279 100644 (file)
@@ -9082,6 +9082,11 @@ getDefaultACLs(Archive *fout, int *numDefaultACLs)
                          racl_subquery->data,
                          initacl_subquery->data,
                          initracl_subquery->data);
+
+       destroyPQExpBuffer(acl_subquery);
+       destroyPQExpBuffer(racl_subquery);
+       destroyPQExpBuffer(initacl_subquery);
+       destroyPQExpBuffer(initracl_subquery);
    }
    else
    {