Re: Leak repairs - Mailing list pgsql-odbc

From Dave Page
Subject Re: Leak repairs
Date
Msg-id E7F85A1B5FF8D44C8A1AF6885BC9A0E4AC932E@ratbert.vale-housing.co.uk
Whole thread Raw
In response to Leak repairs  ("Dave Page" <dpage@vale-housing.co.uk>)
Responses Re: Leak repairs
List pgsql-odbc

> -----Original Message-----
> From: pgsql-odbc-owner@postgresql.org
> [mailto:pgsql-odbc-owner@postgresql.org] On Behalf Of Marko Ristola
> Sent: 19 July 2005 08:37
> Cc: Anoop Kumar; pgsql-odbc@postgresql.org
> Subject: Re: [ODBC] Leak repairs
>
>
> Have you seen this one?
>
> 1121712844: 17186:  not freed: '0x404ef808|s1' (800 bytes) from
> 'qresult.c:421'
> 1121712844: 17186:  not freed: '0x404f0c08|s1' (800 bytes) from
> 'qresult.c:421'
> 1121712844: 17186:  not freed: '0x404fd008|s1' (3200 bytes) from
> 'qresult.c:421'
> 1121712844: 17186:  not freed: '0x404fe008|s1' (3200 bytes) from
> 'qresult.c:421'
> 1121712844: 17186:  not freed: '0x404ff008|s1' (3200 bytes) from
> 'qresult.c:421'
> 1121712844: 17186:  not freed: '0x40500008|s1' (3200 bytes) from
> 'qresult.c:421'
> 1121712844: 17186:  not freed: '0x40501008|s1' (3200 bytes) from
> 'qresult.c:421'
>
> Line 421 allocates a buffer into QResultClass.backend_tuples.
>
> backend_tuples will be set into NULL in QR_Destructor() with
> "self->backend_tuples = NULL". I suspect, that the memory leak.
>

Nicely spotted :-)

That would appear to be the last of them (at least for my simple testing
at the moment). I'll commit a patch shortly.

BTW, I assume you got that output from the VC++ debugger - care to share
how? I spent quite some time trying to persuade it to give me that and
didn't get anywhere :-(

Regards, Dave.

pgsql-odbc by date:

Previous
From: Marko Ristola
Date:
Subject: Re: Libpq driver: thread problem
Next
From: Marko Ristola
Date:
Subject: Re: Leak repairs