pgsql: Remove dead code in pg_dump. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Remove dead code in pg_dump.
Date
Msg-id E1aKqMI-0007V0-Pi@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Remove dead code in pg_dump.

Coverity quite reasonably complained that this check for fout==NULL
occurred after we'd already dereferenced fout.  However, the check
is just dead code since there is no code path by which CreateArchive
can return a null pointer.  Errors such as can't-open-that-file are
reported down inside CreateArchive, and control doesn't return.
So let's silence the warning by removing the dead code, rather than
continuing to pretend it does something.

Coverity didn't complain about this before 5b5fea2a1, so back-patch
to 9.5 like that patch.

Branch
------
REL9_5_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/6b3b3a502e5d96cd992470bd408dd7457139caf0

Modified Files
--------------
src/bin/pg_dump/pg_dump.c |    3 ---
1 file changed, 3 deletions(-)


pgsql-committers by date:

Previous
From: Peter Eisentraut
Date:
Subject: pgsql: psql: Add completion support for DROP INDEX CONCURRENTLY
Next
From: Tom Lane
Date:
Subject: pgsql: Remove dead code in pg_dump.