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

From g.hintermayer@inode.at (Gerhard Hintermayer)
Subject Re: still memory leaks with libpgtcl
Date
Msg-id bd4db85f.0301160134.499958fb@posting.google.com
Whole thread Raw
In response to still memory leaks with libpgtcl  (Gerhard Hintermayer <g.hintermayer@inode.at>)
Responses Re: still memory leaks with libpgtcl
List pgsql-interfaces
ljb <lbayuk@mindspring.com> wrote in message 
> [...]
> (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 leaking a 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

Your patch does indeed fix the memory leakage but seems to introduce
another problem. Every once in a while I do get segfaults, when I
enter the event loop after pg_disconnect-ing (well, it looks like that
to me). I have to say, that this is the only script, where the error
occurs. All others (~500) work with no problem at all. I was trying to
reproduce that in a short script, but - no way. I am not even sure, if
this is postgreSQL related. I did build a static wish with
--enable-symbols=all and --enable-memory to track the error down. Hope
I'll see more then.

Gerhard


pgsql-interfaces by date:

Previous
From: "Key88 SF"
Date:
Subject: libpqxx: Cursors needed??!
Next
From: Laurette Cisneros
Date:
Subject: compiling Pg