Fix minor memory leak in pg_dump
authorMichael Paquier
Wed, 15 Apr 2020 06:56:01 +0000 (15:56 +0900)
committerMichael Paquier
Wed, 15 Apr 2020 06:56:01 +0000 (15:56 +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 c579227b1974899d029753bb92560da86cdc039c..5db4f5761d7abaef7f8074d875764acbab21a44b 100644 (file)
@@ -9592,6 +9592,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
    {