Re: cvs: php4 /ext/pgsql pgsql.c

This patch won't always work (it may cause the link not to be closed when it's supposed to). I committed a different patch to the CVS - can you please check it? At 18:54 13/2/2001, Thies C. Arntzen wrote:

thies           Tue Feb 13 08:54:48 2001 EDT

Modified files:

    /php4/ext/pgsql     pgsql.c

Log: @- fixes PostgeSQL pg_connect() bug. We would sometimes close the default @ link by acciedent. (Patch by: aja@nlgroup.ca) Index: php4/ext/pgsql/pgsql.c diff -u php4/ext/pgsql/pgsql.c:1.85 php4/ext/pgsql/pgsql.c:1.86 --- php4/ext/pgsql/pgsql.c:1.85 Wed Jan 31 13:53:29 2001

+++ php4/ext/pgsql/pgsql.c      Tue Feb 13 08:54:48 2001

@@ -17,7 +17,7 @@

    +----------------------------------------------------------------------+

*/ -/* $Id: pgsql.c,v 1.85 2001/01/31 21:53:29 andi Exp $ */ +/* $Id: pgsql.c,v 1.86 2001/02/13 16:54:48 thies Exp $ */ #include <stdlib.h> @@ -114,9 +114,10 @@ {

        PGLS_FETCH();

-    if (PGG(default_link)!=-1) {
+    if ((PGG(default_link) != -1) && (PGG(default_link) != id)) {
         zend_list_delete(PGG(default_link));
     }

+

     if (PGG(default_link) != id) {
         PGG(default_link) = id;
         zend_list_addref(id);

-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, e-mail: php-cvs-unsubscribe@lists.php.net For additional commands, e-mail: php-cvs-help@lists.php.net To contact the list administrators, e-mail: php-list-admin@lists.php.net

-- Zeev Suraski <zeev@zend.com> CTO & co-founder, Zend Technologies Ltd. http://www.zend.com/