Re: Converting tab-complete.c's else-if chain to a switch

Поиск
Список
Период
Сортировка
От Daniel Gustafsson
Тема Re: Converting tab-complete.c's else-if chain to a switch
Дата
Msg-id 901EFF66-B387-4E69-BC63-C6A0512C2AF1@yesql.se
обсуждение исходный текст
Ответ на Converting tab-complete.c's else-if chain to a switch  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
> On 10 Oct 2024, at 09:22, Anthonin Bonnefoy <anthonin.bonnefoy@datadoghq.com> wrote:

> psql tries to read the previous word when matches are NULL. However,
> there's no previous word set here, leading to the segfault.
>
> I've attached a patch that checks if the previous word does exist
> before trying to use it, along with an additional test for this
> codepath.

Nice, I was just typing up an email with the same fix when your landed in my
inbox after having ended up with a segfaulting psql testing a patch.  (I didn't
include a test though so yours is better.) From reading the code I agree that
we must check previous_words_count here so +1 on this patch.

--
Daniel Gustafsson




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