git.postgresql.org Git - postgresql.git/commitdiff

git projects / postgresql.git / commitdiff

author Daniel Gustafsson <dgustafsson@postgresql.org>

Wed, 9 Feb 2022 13:12:55 +0000 (14:12 +0100)

committer Daniel Gustafsson <dgustafsson@postgresql.org>

Wed, 9 Feb 2022 13:12:55 +0000 (14:12 +0100)

ReadStr returns allocated memory which the caller is responsible for
freeing when done with the string. This commit ensures that memory is
freed in one case which used ReadStr in a conditional. While the leak
might not be too concerning, this makes the code consistent across all
ReadStr callsites in ReadToc. Due to the lack of complaints of issues
in production from this, no backpatch is performed at this point.

Author: Bharath Rupireddy, Georgios Kokolatos
Reviewed-by: Kyotaro Horiguchi
Discussion: https://postgr.es/m/oZwKiUxFsVaetG2xOJp7Hwao8F1AKIdfFDQLNJrnwoaxmjyB-45r_aYmhgXHKLcMI3GT24m9L6HafSi2ns7WFxXe0mw2_tIJpD-Z3vb_eyI=@pm.me

src/bin/pg_dump/pg_backup_archiver.c patch | blob | blame | history

diff --git a/src/bin/pg_dump/pg_backup_archiver.c b/src/bin/pg_dump/pg_backup_archiver.c

index 49bf0907cd20f6309db4a5642382e5ad59f71b2a..d41a99d6ea7d9b71a6d7816e7ad64f8a17cf74c8 100644 (file)

--- a/src/bin/pg_dump/pg_backup_archiver.c

+++ b/src/bin/pg_dump/pg_backup_archiver.c

@@ -2494,6 +2494,7 @@ ReadToc(ArchiveHandle *AH)

int depIdx;

int depSize;

TocEntry *te;

+ bool is_supported;

AH->tocCount = ReadInt(AH);

AH->maxDumpId = 0;