Re: Assert name/short_desc to prevent SHOW ALL segfault

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: Assert name/short_desc to prevent SHOW ALL segfault
Дата
Msg-id Yo2/FfbtzTrwKgsM@paquier.xyz
обсуждение исходный текст
Ответ на Assert name/short_desc to prevent SHOW ALL segfault  (Steve Chavez <steve@supabase.io>)
Список pgsql-hackers
Дерево обсуждения 
On Wed, May 25, 2022 at 12:05:55AM -0500, Steve Chavez wrote:
> Thank you for the reviews Nathan, Michael.
>
> I agree with handling NULL in ShowAllGUCConfig() instead.
>
> I've attached the updated patch.

Shouldn't the same check as extra_desc be done in
GetConfigOptionByNum() for short_desc (point of upthread)?  See
3ac7d024, though -fsanitize=undefined does not apply here, I guess.
--
Michael
Вложения

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

Предыдущее
От: Steve Chavez
Дата:
Сообщение: Re: Assert name/short_desc to prevent SHOW ALL segfault
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Assert name/short_desc to prevent SHOW ALL segfault