diff options
| author | Robert Haas <rhaas@postgresql.org> | 2015-06-26 14:45:32 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2015-06-26 14:46:48 -0400 |
| commit | 8a8c581a8c99b9beecbdc517957da866f427f297 (patch) | |
| tree | c22255a21035d74bb7c768bf5c98763f121f0b16 | |
| parent | 31c018ecda9f40fe80055d8ba95248c023593fb4 (diff) | |
| download | postgresql-8a8c581a8c99b9beecbdc517957da866f427f297.tar.gz postgresql-8a8c581a8c99b9beecbdc517957da866f427f297.zip | |
Remove unnecessary NULL test.
Spotted by Coverity and reported by Michael Paquier. Per discussion,
we don't necessarily care about making Coverity happy in all such
instances, but we can go ahead and change them where it otherwise
seems to improve the code.
| -rw-r--r-- | src/bin/pg_upgrade/controldata.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/bin/pg_upgrade/controldata.c b/src/bin/pg_upgrade/controldata.c index 142e623c091..6978ae991b2 100644 --- a/src/bin/pg_upgrade/controldata.c +++ b/src/bin/pg_upgrade/controldata.c @@ -402,8 +402,7 @@ get_control_data(ClusterInfo *cluster, bool live_check) } } - if (output) - pclose(output); + pclose(output); /* * Restore environment variables |