Re: BUG #15105: OpenTransientFile() should be paired withCloseTransientFile() rather than close()

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Re: BUG #15105: OpenTransientFile() should be paired withCloseTransientFile() rather than close()
Дата
Msg-id 20180309044052.GC1416@paquier.xyz
обсуждение исходный текст
Ответ на BUG #15105: OpenTransientFile() should be paired withCloseTransientFile() rather than close()  (PG Bug reporting form <noreply@postgresql.org>)
Ответы Re: BUG #15105: OpenTransientFile() should be paired with CloseTransientFile() rather than close()
Список pgsql-bugs
On Fri, Mar 09, 2018 at 03:29:25AM +0000, PG Bug reporting form wrote:
> Details: The handler opened with OpenTransientFile() should be closed with
> CloseTransientFile(). However, in function dsm_impl_mmap(), on a certain
> path, the return value of OpenTransientFile() (at line 885) is passed to
> close() (at line 926). It is better to use CloseTransientFile() here, as
> done at line 909.

Good catch!  This is visibly a copy-paste error coming from
dsm_impl_posix().  As a patch it gives the attached.  I am adding also
Robert in CC for awareness.
--
Michael

Вложения

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

Предыдущее
От: PG Bug reporting form
Дата:
Сообщение: BUG #15105: OpenTransientFile() should be paired withCloseTransientFile() rather than close()
Следующее
От: Mehdi Rahman
Дата:
Сообщение: Re: BUG #15102: Performance problem when doing join, index are not used