Fix minor memory leak in pg_dump
authorMichael Paquier
Wed, 15 Apr 2020 06:56:43 +0000 (15:56 +0900)
committerMichael Paquier
Wed, 15 Apr 2020 06:56:43 +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 dd2e58e0b6b6b2ef960ea5513daef3a72bec6e9d..52153484f1fa21bb0d06a2cb810e8b44213b2c56 100644 (file)
@@ -9463,6 +9463,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
    {