Re: still memory leaks with libpgtcl - Mailing list pgsql-interfaces

From ljb
Subject Re: still memory leaks with libpgtcl
Date
Msg-id avg4mq$1ikp$1@news.hub.org
Whole thread Raw
In response to still memory leaks with libpgtcl  (Gerhard Hintermayer <g.hintermayer@inode.at>)
Responses Re: still memory leaks with libpgtcl
Re: still memory leaks with libpgtcl
List pgsql-interfaces
g.hintermayer@inode.at wrote:
>...
> If I'd track down the problem and sent a patch, has the wheel to be reinvented 
> for gborg ? Two concurrent developments is not a good thing.


Did you see Tom Lane's reply on this thread? The problem is known but fixing
the leak causes a crash. The code is in libpgtcl/pgtclId.c; look for this
comment:    * XXX Unfortunately, while this works fine if we are closing due to    * explicit pg_disconnect, Tcl
versionsthrough 8.3.3 dump core if we    * try to do it during interpreter shutdown.  Not clear why, or if    * there
isa workaround.  For now, accept leakage of the (fairly    * small) amount of memory taken for the channel state
representation.

I've confirmed that the crash occurs with Tcl 8.3.4 also. But I've got a
patch which seems to fix it - that is, it neither leaks on disconnect nor
crashes on shutdown after my patch is applied. At least it doesn't with my
extremely minimal testing. If you want to try this, go ahead. This patch is
seriously ugly and 'highly suspect', and I am NOT suggesting this be
included in the PostgreSQL release. Really. But it works. Maybe.

(This is for PostgreSQL-7.3.1)

--- src/interfaces/libpgtcl/pgtclId.c.bak    2002-10-17 10:53:32.000000000 -0400
+++ src/interfaces/libpgtcl/pgtclId.c    2003-01-07 21:51:53.000000000 -0500
@@ -300,12 +300,10 @@     * small) amount of memory taken for the channel state representation.     * Note we are not
leakinga socket, since libpq closed that already.     */
 
-#ifdef NOT_USED#if TCL_MAJOR_VERSION >= 8
-    if (connid->notifier_channel != NULL)
+    if (connid->notifier_channel != NULL && interp != NULL)        Tcl_UnregisterChannel(NULL,
connid->notifier_channel);#endif
-#endif    /*     * We must use Tcl_EventuallyFree because we don't want the connid


pgsql-interfaces by date:

Previous
From: "D'Arcy J.M. Cain"
Date:
Subject: Re: [ADMIN] pgdb.py is still wrong in Postgres 7.3.1 rpm
Next
From: Tom Lane
Date:
Subject: Re: [ADMIN] pgdb.py is still wrong in Postgres 7.3.1 rpm