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 по дате отправления:

Предыдущее
От: Robert Haas
Дата:
Сообщение: Re: RFC: replace pg_stat_activity.waiting with something more descriptive
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Nitpicking: unnecessary NULL-pointer check in pg_upgrade's controldata.c