Обсуждение: Abort if dup fail (src/bin/pg_dump/compress_none.c)

Поиск
Список
Период
Сортировка

Abort if dup fail (src/bin/pg_dump/compress_none.c)

От
Ranier Vilela
Дата:
Hi.

Per Coverity.
CID 1506240: (#1 of 1): Argument cannot be negative (NEGATIVE_RETURNS)
3. negative_returns: dup(fd) is passed to a parameter that cannot be negative.

pg_dump function open_none, tries to associate a stream to a file descriptor,
using function dup, which may fail and return negative value.

fdopen cannot receive negative parameters, in this case fail and return EBADF.

This can be confusing for the user, who will be trying to figure out what's wrong.
Better abort and report the correct failure to the user.

Patch attached.

Best regards,
Ranier Vilela


Вложения