Re: possible memory leak in Server Status window

Поиск
Список
Период
Сортировка
От Guillaume Lelarge
Тема Re: possible memory leak in Server Status window
Дата
Msg-id 4D564398.4040006@lelarge.info
обсуждение исходный текст
Ответ на Re: possible memory leak in Server Status window  (Julius Tuskenis <julius@nsoft.lt>)
Ответы Re: possible memory leak in Server Status window  (Julius Tuskenis <julius@nsoft.lt>)
Re: possible memory leak in Server Status window  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-support
Le 11/02/2011 11:42, Julius Tuskenis a écrit :
> Hello, Guillaume
> 
> Please look at frmStatus::fillLogfileCombo(). Please note, that the set
> is defined and assigned, but then as it has not enough rows to be
> "interesting" the function returns 0 and exits. So "delete set;" is
> never called. Can this be the issue?
> 

Yes, it's definitely a memory leak. As I don't have the adminpack
installed, I had no way to find this (other than reading the complete
frmStatus code).

Anyway, I pushed a fix for this.

I know Peter and Dave talked about some stuff I actually didn't
understand, but it seems it would take some time to do. So, I prefer
having this simple and effective fix right now :)

Thanks, Julius, for reporting the leak and caring enough to actually
read the code :)


-- 
Guillaumehttp://www.postgresql.frhttp://dalibo.com


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

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: possible memory leak in Server Status window
Следующее
От: Julius Tuskenis
Дата:
Сообщение: Re: possible memory leak in Server Status window