Re: not null validation option in contrib/file_fdw

Поиск
Список
Период
Сортировка
От Shigeru HANADA
Тема Re: not null validation option in contrib/file_fdw
Дата
Msg-id 4F880C50.9090107@gmail.com
обсуждение исходный текст
Ответ на not null validation option in contrib/file_fdw  ("Etsuro Fujita" <fujita.etsuro@lab.ntt.co.jp>)
Ответы Re: not null validation option in contrib/file_fdw  (Andrew Dunstan <andrew@dunslane.net>)
Список pgsql-hackers
(2012/04/13 16:59), Etsuro Fujita wrote:
> I updated the patch added to CF 2012-Next [1].  Attached is the updated
> version of the patch.

I applied the patch and ran regression tests of file_fdw, and I got
SIGSEGV X-(

The failure occurs in fileGetOptions, and it is caused by
list_delete_cell used in foreach loop; ListCell points delete target has
been free-ed in list_delete_cell, but foreach accesses it to get next
element.

Some of backend functions which use list_delete_cell in loop use "for"
loop instead of foreach, and other functions exit the loop after calling
list_delete_cell.  Since we can't stop searching non-COPY options until
meeting the end of the options list, we would need to choose former
("for" loop), or create another list which contains only valid COPY
options and return it via other_options parameter.

Regards,
-- 
Shigeru HANADA


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

Предыдущее
От: Dimitri Fontaine
Дата:
Сообщение: Re: Last gasp
Следующее
От: Christoph Berg
Дата:
Сообщение: [trivial patch] grammar fixes in doc/src/sgml/high-availability.sgml