I've done some more code inspection and I think my fix is ok.
I'll make the change to my code base and I'll let you know if I have any problems with it.
I've submitted bug #1067 for this.
scot.
-----Original Message-----
From: Dave Page [mailto:dpage@vale-housing.co.uk]
Sent: Sat 11/27/2004 6:19 PM
To: Scot Loach; pgsql-odbc@postgresql.org
Cc:
Subject: RE: [ODBC] memory leak in ODBC driver
> -----Original Message-----
> From: pgsql-odbc-owner@postgresql.org
> [mailto:pgsql-odbc-owner@postgresql.org] On Behalf Of Scot Loach
> Sent: 27 November 2004 18:11
> To: pgsql-odbc@postgresql.org
> Subject: [ODBC] memory leak in ODBC driver
>
> There is a 16-byte leak for every statement handle that has
> SQLFetch called on it.
> I have tracked this leak down to its source:
> SC_Fetch calls extend_getdata_info, which allocates memory
> for the stmt.getdata_info.
> However, this memory is never freed when the statement is deallocated.
>
> This patch seems to fix the problem - however I don't know
> enough about the internals of the driver to really understand
> if this is the correct or safe fix.
> Can one of the developers help me out here?
It certainly *looks* OK to me, but I share the same concerns as you
(I've never really got that far inside the driver before). Anyone else
care to comment?
Regards, Dave.