pgsql: Fix inconsequential fd leak in the new mark_file_as_archived() f - Mailing list pgsql-committers

From Andres Freund
Subject pgsql: Fix inconsequential fd leak in the new mark_file_as_archived() f
Date
Msg-id E1Y7lSO-0006sd-1N@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix inconsequential fd leak in the new mark_file_as_archived() function.

As every error in mark_file_as_archived() will lead to a failure of
pg_basebackup the FD leak couldn't ever lead to a real problem.  It
seems better to fix the leak anyway though, rather than silence
Coverity, as the usage of the function might get extended or copied at
some point in the future.

Pointed out by Coverity.

Backpatch to 9.2, like the relevant part of the previous patch.

Branch
------
REL9_4_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/ff7d46b8572ce566230c53e3c937412b3d19d8cf

Modified Files
--------------
src/bin/pg_basebackup/receivelog.c |    3 +++
1 file changed, 3 insertions(+)


pgsql-committers by date:

Previous
From: Andres Freund
Date:
Subject: pgsql: Prevent WAL files created by pg_basebackup -x/X from being archi
Next
From: Andres Freund
Date:
Subject: pgsql: Fix inconsequential fd leak in the new mark_file_as_archived() f