Re: Fix error handling in be_tls_open_server()

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: Fix error handling in be_tls_open_server()
Дата
Msg-id E3921399-FAE7-4B1F-B1BF-B3357DDC9F19@yesql.se
обсуждение исходный текст
Ответ на Fix error handling in be_tls_open_server()  (Sergey Shinderuk <s.shinderuk@postgrespro.ru>)
Ответы Re: Fix error handling in be_tls_open_server()  (Jacob Champion <jchampion@timescale.com>)
Re: Fix error handling in be_tls_open_server()  (Sergey Shinderuk <s.shinderuk@postgrespro.ru>)
Список pgsql-hackers
> On 1 Aug 2023, at 16:44, Sergey Shinderuk <s.shinderuk@postgrespro.ru> wrote:

> A static analyzer reported a possible pfree(NULL) in be_tls_open_server().

This has the smell of a theoretical problem, I can't really imagine a
certificate where which would produce this.  Have you been able to trigger it?

Wouldn't a better fix be to error out on len == -1 as in the attached, maybe
with a "Shouldn't happen" comment?

--
Daniel Gustafsson


Вложения

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

Предыдущее
От: David Rowley
Дата:
Сообщение: meson uses stale pg_config_paths.h left over from make
Следующее
От: Pavel Stehule
Дата:
Сообщение: Re: Schema variables - new implementation for Postgres 15