Fix minor memory leak in pg_dump
authorMichael Paquier <michael@paquier.xyz>
Wed, 15 Apr 2020 06:56:52 +0000 (15:56 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 15 Apr 2020 06:56:52 +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 8b3b1bf91ec468f967442b64798c2d27bad80d40..bd14d387403ea05f94106a70dae0f1330f70923f 100644 (file)
@@ -8982,6 +8982,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
    {