Nitpicking: unnecessary NULL-pointer check in pg_upgrade's controldata.c

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Nitpicking: unnecessary NULL-pointer check in pg_upgrade's controldata.c
Дата
Msg-id CAB7nPqTmLvKZKrtNQ0+jc7SAnKDe0w_udkt8JCO6z_5Lgq5H_A@mail.gmail.com
обсуждение исходный текст
Ответы Re: Nitpicking: unnecessary NULL-pointer check in pg_upgrade's controldata.c
Список pgsql-hackers
Дерево обсуждения 
Hi,

Coverity is nitpickingly pointing out the following thing:
--- 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);
The thing is that output can never be NULL, pg_upgrade leaving with
pg_fatal before coming to this code path. Hence this NULL check could
be simply removed.
Regards,
-- 
Michael

В списке pgsql-hackers по дате отправления: