Re: possible memory leak in Server Status window - Mailing list pgadmin-support

From Julius Tuskenis
Subject Re: possible memory leak in Server Status window
Date
Msg-id 4D55129D.1080508@nsoft.lt
Whole thread Raw
In response to Re: possible memory leak in Server Status window  (Julius Tuskenis <julius@nsoft.lt>)
Responses Re: possible memory leak in Server Status window  (Peter Geoghegan <peter.geoghegan86@gmail.com>)
Re: possible memory leak in Server Status window  (Guillaume Lelarge <guillaume@lelarge.info>)
List pgadmin-support
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?

int frmStatus::fillLogfileCombo()
{    int count = cbLogfiles->GetCount();    if (!count)        cbLogfiles->Append(_("Current log"));    else
count--;
    pgSet *set = connection->ExecuteSet(                     wxT("SELECT filename, filetime\n")
wxT(" FROM pg_logdir_ls() AS A(filetime 
 
timestamp, filename text)\n")                     wxT(" ORDER BY filetime DESC"));    if (set)    {        if
(set->NumRows()<= count)            return 0;
 

-- 
Julius Tuskenis
Programavimo skyriaus vadovas
UAB nSoft
mob. +37068233050



pgadmin-support by date:

Previous
From: Julius Tuskenis
Date:
Subject: Re: possible memory leak in Server Status window
Next
From: Peter Geoghegan
Date:
Subject: Re: possible memory leak in Server Status window