[PHP-CVS] svn: /php/php-src/branches/PHP_5_4/Zend/ tests/bug54910.phpt zend_API.c

2011-05-31 Thread Dmitry Stogov
dmitry   Tue, 31 May 2011 11:36:57 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311653

Log:
MFH: Fixed bug #54910 (Crash when calling call_user_func with unknown function 
name)

Bug: http://bugs.php.net/54910 (Closed) Crash when calling call_user_func with 
unknown function name
  
Changed paths:
A   php/php-src/branches/PHP_5_4/Zend/tests/bug54910.phpt
U   php/php-src/branches/PHP_5_4/Zend/zend_API.c

Added: php/php-src/branches/PHP_5_4/Zend/tests/bug54910.phpt
===
--- php/php-src/branches/PHP_5_4/Zend/tests/bug54910.phpt   
(rev 0)
+++ php/php-src/branches/PHP_5_4/Zend/tests/bug54910.phpt   2011-05-31 
11:36:57 UTC (rev 311653)
@@ -0,0 +1,28 @@
+--TEST--
+Bug #54910 (Crash when calling call_user_func with unknown function name)
+--FILE--
+get();
+}
+die("No such method - '$method'\n");
+}
+
+protected function get() {
+$class = get_class($this);
+$call = array($class, 'noSuchMethod');
+
+if (is_callable($call)) {
+call_user_func($call);
+}
+}
+}
+
+class B extends A {}
+
+$input = new B();
+echo $input->getEmail();
+--EXPECT--
+No such method - 'noSuchMethod'

Modified: php/php-src/branches/PHP_5_4/Zend/zend_API.c
===
--- php/php-src/branches/PHP_5_4/Zend/zend_API.c2011-05-31 11:28:37 UTC 
(rev 311652)
+++ php/php-src/branches/PHP_5_4/Zend/zend_API.c2011-05-31 11:36:57 UTC 
(rev 311653)
@@ -2773,6 +2773,11 @@
if (fcc->function_handler) {
retval = 1;
call_via_handler = 
(fcc->function_handler->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) != 0;
+   if (call_via_handler && !fcc->object_ptr && 
EG(This) &&
+   Z_OBJ_HT_P(EG(This))->get_class_entry &&
+   instanceof_function(Z_OBJCE_P(EG(This)), 
fcc->calling_scope TSRMLS_CC)) {
+   fcc->object_ptr = EG(This);
+   }
}
}
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/branches/PHP_5_3/sapi/litespeed/ lsapi_main.c

2011-05-31 Thread George Wang
gwangTue, 31 May 2011 22:40:42 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311680

Log:
fix CLI mode timeout

Changed paths:
U   php/php-src/branches/PHP_5_3/sapi/litespeed/lsapi_main.c

Modified: php/php-src/branches/PHP_5_3/sapi/litespeed/lsapi_main.c
===
--- php/php-src/branches/PHP_5_3/sapi/litespeed/lsapi_main.c2011-05-31 
21:38:23 UTC (rev 311679)
+++ php/php-src/branches/PHP_5_3/sapi/litespeed/lsapi_main.c2011-05-31 
22:40:42 UTC (rev 311680)
@@ -660,6 +660,7 @@
 "implicit_flush",   "1",
 "output_buffering", "0",
 "max_execution_time",   "0",
+"max_input_time",   "-1",
 NULL
 };


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/gd/tests/ imageloadfont_error1.phpt imageloadfont_error2.phpt

2011-05-31 Thread Rafael Machado Dohms
rdohms   Tue, 31 May 2011 21:38:23 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311679

Log:
Adding testcases to validate imageloadfont input parameter validation, tests 
were developed at #tek11 testfest

Changed paths:
A   php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error1.phpt
A   php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error2.phpt

Added: php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error1.phpt
===
--- php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error1.phpt 
(rev 0)
+++ php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error1.phpt 
2011-05-31 21:38:23 UTC (rev 311679)
@@ -0,0 +1,15 @@
+--TEST--
+Testing that imageloadfont() breaks on non-string first parameter
+--CREDITS--
+Neveo Harrison  #testfest #tek11
+--SKIPIF--
+
+--FILE--
+
+--EXPECTF--
+Warning: imageloadfont() expects parameter 1 to be string, array given in %s 
on line %d
+NULL
\ No newline at end of file

Added: php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error2.phpt
===
--- php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error2.phpt 
(rev 0)
+++ php/php-src/branches/PHP_5_4/ext/gd/tests/imageloadfont_error2.phpt 
2011-05-31 21:38:23 UTC (rev 311679)
@@ -0,0 +1,15 @@
+--TEST--
+Testing that imageloadfont() breaks on invalid file passed as first argument
+--CREDITS--
+Austin Drouare  #testfest #tek11
+--SKIPIF--
+
+--FILE--
+
+--EXPECTF--
+Warning: imageloadfont(\src\invalidfile.font): failed to open stream: No such 
file or directory in %s on line %d
+bool(false)

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ext/gd/tests/ imageloadfont_error1.phpt imageloadfont_error2.phpt

2011-05-31 Thread Rafael Machado Dohms
rdohms   Tue, 31 May 2011 21:24:10 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311678

Log:
Adding testcases to validate imageloadfont input parameter validation, tests 
were developed at #tek11 testfest

Changed paths:
A   php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error1.phpt
A   php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error2.phpt

Added: php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error1.phpt
===
--- php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error1.phpt 
(rev 0)
+++ php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error1.phpt 
2011-05-31 21:24:10 UTC (rev 311678)
@@ -0,0 +1,15 @@
+--TEST--
+Testing that imageloadfont() breaks on non-string first parameter
+--CREDITS--
+Neveo Harrison  #testfest #tek11
+--SKIPIF--
+
+--FILE--
+
+--EXPECTF--
+Warning: imageloadfont() expects parameter 1 to be string, array given in %s 
on line %d
+NULL
\ No newline at end of file

Added: php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error2.phpt
===
--- php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error2.phpt 
(rev 0)
+++ php/php-src/branches/PHP_5_3/ext/gd/tests/imageloadfont_error2.phpt 
2011-05-31 21:24:10 UTC (rev 311678)
@@ -0,0 +1,15 @@
+--TEST--
+Testing that imageloadfont() breaks on invalid file passed as first argument
+--CREDITS--
+Austin Drouare  #testfest #tek11
+--SKIPIF--
+
+--FILE--
+
+--EXPECTF--
+Warning: imageloadfont(\src\invalidfile.font): failed to open stream: No such 
file or directory in %s on line %d
+bool(false)

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/trunk/ext/gd/tests/ imageloadfont_error1.phpt imageloadfont_error2.phpt

2011-05-31 Thread Rafael Machado Dohms
rdohms   Tue, 31 May 2011 21:10:01 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311677

Log:
Adding testcases to validate imageloadfont input parameter validation, tests 
were developed at #tek11 testfest

Changed paths:
A   php/php-src/trunk/ext/gd/tests/imageloadfont_error1.phpt
A   php/php-src/trunk/ext/gd/tests/imageloadfont_error2.phpt

Added: php/php-src/trunk/ext/gd/tests/imageloadfont_error1.phpt
===
--- php/php-src/trunk/ext/gd/tests/imageloadfont_error1.phpt
(rev 0)
+++ php/php-src/trunk/ext/gd/tests/imageloadfont_error1.phpt2011-05-31 
21:10:01 UTC (rev 311677)
@@ -0,0 +1,15 @@
+--TEST--
+Testing that imageloadfont() breaks on non-string first parameter
+--CREDITS--
+Neveo Harrison  #testfest #tek11
+--SKIPIF--
+
+--FILE--
+
+--EXPECTF--
+Warning: imageloadfont() expects parameter 1 to be string, array given in %s 
on line %d
+NULL
\ No newline at end of file

Added: php/php-src/trunk/ext/gd/tests/imageloadfont_error2.phpt
===
--- php/php-src/trunk/ext/gd/tests/imageloadfont_error2.phpt
(rev 0)
+++ php/php-src/trunk/ext/gd/tests/imageloadfont_error2.phpt2011-05-31 
21:10:01 UTC (rev 311677)
@@ -0,0 +1,15 @@
+--TEST--
+Testing that imageloadfont() breaks on invalid file passed as first argument
+--CREDITS--
+Austin Drouare  #testfest #tek11
+--SKIPIF--
+
+--FILE--
+
+--EXPECTF--
+Warning: imageloadfont(\src\invalidfile.font): failed to open stream: No such 
file or directory in %s on line %d
+bool(false)

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/mysqlnd/mysqlnd.c branches/PHP_5_4/ext/mysqlnd/mysqlnd.c trunk/ext/mysqlnd/mysqlnd.c

2011-05-31 Thread Andrey Hristov
andrey   Tue, 31 May 2011 20:42:44 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311676

Log:
reuse code

Changed paths:
U   php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd.c
U   php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd.c
U   php/php-src/trunk/ext/mysqlnd/mysqlnd.c

Modified: php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd.c
===
--- php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd.c  2011-05-31 19:04:52 UTC 
(rev 311675)
+++ php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd.c  2011-05-31 20:42:44 UTC 
(rev 311676)
@@ -896,25 +896,18 @@
 static enum_func_status
 MYSQLND_METHOD(mysqlnd_conn, query)(MYSQLND * conn, const char * query, 
unsigned int query_len TSRMLS_DC)
 {
-   enum_func_status ret;
+   enum_func_status ret = FAIL;
DBG_ENTER("mysqlnd_conn::query");
DBG_INF_FMT("conn=%llu query=%s", conn->thread_id, query);

-   if (PASS != conn->m->simple_command(conn, COM_QUERY, query, query_len,
-  
PROT_LAST /* we will handle the OK packet*/,
-  
FALSE, FALSE TSRMLS_CC)) {
-   DBG_RETURN(FAIL);
+   if (PASS == conn->m->send_query(conn, query, query_len TSRMLS_CC) &&
+   PASS == conn->m->reap_query(conn TSRMLS_CC))
+   {
+   ret = PASS;
+   if (conn->last_query_type == QUERY_UPSERT && 
conn->upsert_status.affected_rows) {
+   MYSQLND_INC_CONN_STATISTIC_W_VALUE(conn->stats, 
STAT_ROWS_AFFECTED_NORMAL, conn->upsert_status.affected_rows);
+   }
}
-   CONN_SET_STATE(conn, CONN_QUERY_SENT);
-   /*
- Here read the result set. We don't do it in simple_command because it 
need
- information from the ok packet. We will fetch it ourselves.
-   */
-   ret = conn->m->query_read_result_set_header(conn, NULL TSRMLS_CC);
-   if (ret == PASS && conn->last_query_type == QUERY_UPSERT && 
conn->upsert_status.affected_rows) {
-   MYSQLND_INC_CONN_STATISTIC_W_VALUE(conn->stats, 
STAT_ROWS_AFFECTED_NORMAL, conn->upsert_status.affected_rows);
-   }
-
DBG_RETURN(ret);
 }
 /* }}} */
@@ -931,7 +924,9 @@
ret = conn->m->simple_command(conn, COM_QUERY, query, query_len,
 PROT_LAST /* 
we will handle the OK packet*/,
 FALSE, FALSE 
TSRMLS_CC);
-   CONN_SET_STATE(conn, CONN_QUERY_SENT);
+   if (PASS == ret) {
+   CONN_SET_STATE(conn, CONN_QUERY_SENT);
+   }
DBG_RETURN(ret);
 }
 /* }}} */
@@ -950,6 +945,10 @@
DBG_ERR_FMT("Connection not opened, clear or has been closed. 
State=%u", state);
DBG_RETURN(FAIL);
}
+   /*
+ Here read the result set. We don't do it in simple_command because it 
need
+ information from the ok packet. We will fetch it ourselves.
+   */
DBG_RETURN(conn->m->query_read_result_set_header(conn, NULL TSRMLS_CC));
 }
 /* }}} */

Modified: php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd.c
===
--- php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd.c  2011-05-31 19:04:52 UTC 
(rev 311675)
+++ php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd.c  2011-05-31 20:42:44 UTC 
(rev 311676)
@@ -996,25 +996,18 @@
 static enum_func_status
 MYSQLND_METHOD(mysqlnd_conn, query)(MYSQLND * conn, const char * query, 
unsigned int query_len TSRMLS_DC)
 {
-   enum_func_status ret;
+   enum_func_status ret = FAIL;
DBG_ENTER("mysqlnd_conn::query");
DBG_INF_FMT("conn=%llu query=%s", conn->thread_id, query);

-   if (PASS != conn->m->simple_command(conn, COM_QUERY, (zend_uchar *) 
query, query_len,
-  
PROT_LAST /* we will handle the OK packet*/,
-  
FALSE, FALSE TSRMLS_CC)) {
-   DBG_RETURN(FAIL);
+   if (PASS == conn->m->send_query(conn, query, query_len TSRMLS_CC) &&
+   PASS == conn->m->reap_query(conn TSRMLS_CC))
+   {
+   ret = PASS;
+   if (conn->last_query_type == QUERY_UPSERT && 
conn->upsert_status.affected_rows) {
+   MYSQLND_INC_CONN_STATISTIC_W_VALUE(conn->stats, 
STAT_ROWS_AFFECTED_NORMAL, conn->upsert_status.affected_rows);
+   }
}
-   CONN_SET_STATE(conn, CONN_QUERY_SENT);
-   /*
- Here read the result set. We don't do it in simple_command because it 
need
- information from the ok packet. We will fetch it ourselves.
-   */
-   ret = conn->m->query_read_result_set_header(conn, NULL TSRM

[PHP-CVS] svn: /SVNROOT/ global_avail

2011-05-31 Thread Pierre Joye
pajoye   Tue, 31 May 2011 18:15:49 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311674

Log:
- amqp karma for jtansavatdi

Changed paths:
U   SVNROOT/global_avail

Modified: SVNROOT/global_avail
===
--- SVNROOT/global_avail2011-05-31 17:44:40 UTC (rev 311673)
+++ SVNROOT/global_avail2011-05-31 18:15:49 UTC (rev 311674)
@@ -339,6 +339,8 @@
 avail|tpenner4d|pecl/pdo_4d,phpdoc
 avail|ianb|pecl/svm,phpdoc
 avail|michelec|pecl/rsync,phpdoc
+avail|dsr|pecl/sam,phpdoc
+avail|jtansavatdi|pecl/amqp,phpdoc

 # Objective-C bridge
 avail|wez,jan|php/php-objc
@@ -355,7 +357,6 @@
 avail|wez,dbs|php/php-src/*/ext/pdo_odbc
 avail|gcc,cem,mfp,ansriniv,slaws,chm,rs|pecl/sdo
 avail|dbs,kraman,kfbombar,ambrish|pecl/pdo_ids,pecl/pdo_informix,pecl/pdo_ibm
-avail|dsr|pecl/sam,phpdoc

 # PDO Specs. CLA required to commit
 unavail||php/php-src/*/ext/pdo/specs

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/curl/tests/bug48207.phpt branches/PHP_5_4/ext/curl/tests/bug48207.phpt trunk/ext/curl/tests/bug48207.phpt

2011-05-31 Thread Shein Alexey
sheinTue, 31 May 2011 12:57:40 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311664

Log:
Fixed notice breaking the test.

Changed paths:
U   php/php-src/branches/PHP_5_3/ext/curl/tests/bug48207.phpt
U   php/php-src/branches/PHP_5_4/ext/curl/tests/bug48207.phpt
U   php/php-src/trunk/ext/curl/tests/bug48207.phpt

Modified: php/php-src/branches/PHP_5_3/ext/curl/tests/bug48207.phpt
===
--- php/php-src/branches/PHP_5_3/ext/curl/tests/bug48207.phpt   2011-05-31 
12:45:00 UTC (rev 311663)
+++ php/php-src/branches/PHP_5_3/ext/curl/tests/bug48207.phpt   2011-05-31 
12:57:40 UTC (rev 311664)
@@ -18,7 +18,7 @@
 if(!empty($host)) {

 // Use the set Environment variable
-$url = "$host/get.php";
+$url = "$host/get.php?test=1";

 } else {


Modified: php/php-src/branches/PHP_5_4/ext/curl/tests/bug48207.phpt
===
--- php/php-src/branches/PHP_5_4/ext/curl/tests/bug48207.phpt   2011-05-31 
12:45:00 UTC (rev 311663)
+++ php/php-src/branches/PHP_5_4/ext/curl/tests/bug48207.phpt   2011-05-31 
12:57:40 UTC (rev 311664)
@@ -18,7 +18,7 @@
 if(!empty($host)) {

 // Use the set Environment variable
-$url = "$host/get.php";
+$url = "$host/get.php?test=1";

 } else {


Modified: php/php-src/trunk/ext/curl/tests/bug48207.phpt
===
--- php/php-src/trunk/ext/curl/tests/bug48207.phpt  2011-05-31 12:45:00 UTC 
(rev 311663)
+++ php/php-src/trunk/ext/curl/tests/bug48207.phpt  2011-05-31 12:57:40 UTC 
(rev 311664)
@@ -18,7 +18,7 @@
 if(!empty($host)) {

 // Use the set Environment variable
-$url = "$host/get.php";
+$url = "$host/get.php?test=1";

 } else {


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/ext/curl/tests/curl_setopt_basic001.phpt trunk/ext/curl/tests/curl_setopt_basic001.phpt

2011-05-31 Thread Shein Alexey
sheinTue, 31 May 2011 12:45:00 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311663

Log:
Removed test since there's no more safe_mode in PHP >= 5.4.

Changed paths:
D   php/php-src/branches/PHP_5_4/ext/curl/tests/curl_setopt_basic001.phpt
D   php/php-src/trunk/ext/curl/tests/curl_setopt_basic001.phpt

Deleted: php/php-src/branches/PHP_5_4/ext/curl/tests/curl_setopt_basic001.phpt
===
--- php/php-src/branches/PHP_5_4/ext/curl/tests/curl_setopt_basic001.phpt   
2011-05-31 12:05:25 UTC (rev 311662)
+++ php/php-src/branches/PHP_5_4/ext/curl/tests/curl_setopt_basic001.phpt   
2011-05-31 12:45:00 UTC (rev 311663)
@@ -1,35 +0,0 @@
---TEST--
-curl_setopt basic tests.
---CREDITS--
-Paul Sohier
-#phptestfest utrecht
---INI--
-safe_mode=On
---SKIPIF--
-
---FILE--
-
---EXPECTF--
-Deprecated: Directive 'safe_mode' is deprecated in PHP 5.3 and greater in 
Unknown on line 0
-*** Testing curl_setopt with CURLOPT_FOLLOWLOCATION in safemode
-
-Warning: curl_setopt(): CURLOPT_FOLLOWLOCATION cannot be activated when 
safe_mode is enabled or an open_basedir is set in %s on line %d
-bool(false)
-

Deleted: php/php-src/trunk/ext/curl/tests/curl_setopt_basic001.phpt
===
--- php/php-src/trunk/ext/curl/tests/curl_setopt_basic001.phpt  2011-05-31 
12:05:25 UTC (rev 311662)
+++ php/php-src/trunk/ext/curl/tests/curl_setopt_basic001.phpt  2011-05-31 
12:45:00 UTC (rev 311663)
@@ -1,35 +0,0 @@
---TEST--
-curl_setopt basic tests.
---CREDITS--
-Paul Sohier
-#phptestfest utrecht
---INI--
-safe_mode=On
---SKIPIF--
-
---FILE--
-
---EXPECTF--
-Deprecated: Directive 'safe_mode' is deprecated in PHP 5.3 and greater in 
Unknown on line 0
-*** Testing curl_setopt with CURLOPT_FOLLOWLOCATION in safemode
-
-Warning: curl_setopt(): CURLOPT_FOLLOWLOCATION cannot be activated when 
safe_mode is enabled or an open_basedir is set in %s on line %d
-bool(false)
-

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/branches/PHP_5_4/Zend/ zend_alloc.c

2011-05-31 Thread Dmitry Stogov
dmitry   Tue, 31 May 2011 11:38:31 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311654

Log:
MFH: Fixed bug #54957 (Memleak in alloc_globals_ctor)

Bug: http://bugs.php.net/54957 (Closed) Memleak in alloc_globals_ctor
  
Changed paths:
U   php/php-src/branches/PHP_5_4/Zend/zend_alloc.c

Modified: php/php-src/branches/PHP_5_4/Zend/zend_alloc.c
===
--- php/php-src/branches/PHP_5_4/Zend/zend_alloc.c  2011-05-31 11:36:57 UTC 
(rev 311653)
+++ php/php-src/branches/PHP_5_4/Zend/zend_alloc.c  2011-05-31 11:38:31 UTC 
(rev 311654)
@@ -1602,6 +1602,9 @@
int internal;

if (!heap->use_zend_alloc) {
+   if (full_shutdown) {
+   free(heap);
+   }
return;
}


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/mysqlnd/mysqlnd_priv.h branches/PHP_5_4/ext/mysqlnd/mysqlnd_priv.h trunk/ext/mysqlnd/mysqlnd_priv.h

2011-05-31 Thread Andrey Hristov
andrey   Tue, 31 May 2011 10:35:07 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311643

Log:
fix macros

Changed paths:
U   php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd_priv.h
U   php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd_priv.h
U   php/php-src/trunk/ext/mysqlnd/mysqlnd_priv.h

Modified: php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd_priv.h
===
--- php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd_priv.h 2011-05-31 
10:31:23 UTC (rev 311642)
+++ php/php-src/branches/PHP_5_3/ext/mysqlnd/mysqlnd_priv.h 2011-05-31 
10:35:07 UTC (rev 311643)
@@ -113,7 +113,7 @@
if ((message)) { \
(buf) = mnd_pestrndup((message), (len), (persistent)); \
} else { \
-   buf = NULL; \
+   (buf) = NULL; \
} \
(buf_len) = (len); \
}
@@ -130,22 +130,22 @@

 #define SET_EMPTY_ERROR(error_info) \
{ \
-   error_info.error_no = 0; \
-   error_info.error[0] = '\0'; \
-   strlcpy(error_info.sqlstate, "0", 
sizeof(error_info.sqlstate)); \
+   (error_info).error_no = 0; \
+   (error_info).error[0] = '\0'; \
+   strlcpy((error_info).sqlstate, "0", 
sizeof((error_info).sqlstate)); \
}

 #define SET_CLIENT_ERROR(error_info, a, b, c) \
{ \
-   error_info.error_no = (a); \
-   strlcpy(error_info.sqlstate, (b), sizeof(error_info.sqlstate)); 
\
-   strlcpy(error_info.error, (c), sizeof(error_info.error)); \
+   (error_info).error_no = (a); \
+   strlcpy((error_info).sqlstate, (b), 
sizeof((error_info).sqlstate)); \
+   strlcpy((error_info).error, (c), sizeof((error_info).error)); \
}

-#define SET_OOM_ERROR(error_info) SET_CLIENT_ERROR(error_info, 
CR_OUT_OF_MEMORY, UNKNOWN_SQLSTATE, mysqlnd_out_of_memory)
+#define SET_OOM_ERROR(error_info) SET_CLIENT_ERROR((error_info), 
CR_OUT_OF_MEMORY, UNKNOWN_SQLSTATE, mysqlnd_out_of_memory)


-#define SET_STMT_ERROR(stmt, a, b, c)  SET_CLIENT_ERROR(stmt->error_info, a, 
b, c)
+#define SET_STMT_ERROR(stmt, a, b, c)  SET_CLIENT_ERROR((stmt)->error_info, a, 
b, c)


 #ifdef ZTS
@@ -153,7 +153,7 @@
 #define CONN_SET_STATE(c, s)   (c)->m->set_state((c), (s) TSRMLS_CC)
 #else
 #define CONN_GET_STATE(c)  ((c)->state)
-#define CONN_SET_STATE(c, s)   ((c)->state = s)
+#define CONN_SET_STATE(c, s)   ((c)->state = (s))
 #endif



Modified: php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd_priv.h
===
--- php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd_priv.h 2011-05-31 
10:31:23 UTC (rev 311642)
+++ php/php-src/branches/PHP_5_4/ext/mysqlnd/mysqlnd_priv.h 2011-05-31 
10:35:07 UTC (rev 311643)
@@ -114,7 +114,7 @@
if ((message)) { \
(buf) = mnd_pestrndup((message), (len), (persistent)); \
} else { \
-   buf = NULL; \
+   (buf) = NULL; \
} \
(buf_len) = (len); \
}
@@ -131,22 +131,22 @@

 #define SET_EMPTY_ERROR(error_info) \
{ \
-   error_info.error_no = 0; \
-   error_info.error[0] = '\0'; \
-   strlcpy(error_info.sqlstate, "0", 
sizeof(error_info.sqlstate)); \
+   (error_info).error_no = 0; \
+   (error_info).error[0] = '\0'; \
+   strlcpy((error_info).sqlstate, "0", 
sizeof((error_info).sqlstate)); \
}

 #define SET_CLIENT_ERROR(error_info, a, b, c) \
{ \
-   error_info.error_no = (a); \
-   strlcpy(error_info.sqlstate, (b), sizeof(error_info.sqlstate)); 
\
-   strlcpy(error_info.error, (c), sizeof(error_info.error)); \
+   (error_info).error_no = (a); \
+   strlcpy((error_info).sqlstate, (b), 
sizeof((error_info).sqlstate)); \
+   strlcpy((error_info).error, (c), sizeof((error_info).error)); \
}

-#define SET_OOM_ERROR(error_info) SET_CLIENT_ERROR(error_info, 
CR_OUT_OF_MEMORY, UNKNOWN_SQLSTATE, mysqlnd_out_of_memory)
+#define SET_OOM_ERROR(error_info) SET_CLIENT_ERROR((error_info), 
CR_OUT_OF_MEMORY, UNKNOWN_SQLSTATE, mysqlnd_out_of_memory)


-#define SET_STMT_ERROR(stmt, a, b, c)  SET_CLIENT_ERROR(stmt->error_info, a, 
b, c)
+#define SET_STMT_ERROR(stmt, a, b, c)  SET_CLIENT_ERROR((stmt)->error_info, a, 
b, c)


 #ifdef ZTS
@@ -154,7 +154,7 @@
 #define CONN_SET_STATE(c, s)   (c)->m->set_state((c), (s) TSRMLS_CC)
 #else
 #define CONN_GET_STATE(c)  ((c)->state)
-#define CONN_SET_STATE(c, s)   ((c)->state = s)
+#define CONN_SET_STATE(c, s)   ((c)->state = (s))
 #endif



Modified: php/php-src/trunk/ext/mysqlnd/mysqlnd_priv.h
=

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/NEWS branches/PHP_5_3/Zend/tests/bug54910.phpt branches/PHP_5_3/Zend/zend_API.c trunk/Zend/tests/bug54910.phpt trunk/Zend/zend_API.c

2011-05-31 Thread Ilia Alshanetsky
Please merge into 5.4 branch as well.

Thanks

On Tue, May 31, 2011 at 11:20 AM, Dmitry Stogov  wrote:
> dmitry                                   Tue, 31 May 2011 09:20:51 +
>
> Revision: http://svn.php.net/viewvc?view=revision&revision=311635
>
> Log:
> Fixed bug #54910 (Crash when calling call_user_func with unknown function 
> name)
>
> Bug: http://bugs.php.net/54910 (Assigned) Crash when calling call_user_func 
> with unknown function name
>
> Changed paths:
>    U   php/php-src/branches/PHP_5_3/NEWS
>    A   php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt
>    U   php/php-src/branches/PHP_5_3/Zend/zend_API.c
>    A   php/php-src/trunk/Zend/tests/bug54910.phpt
>    U   php/php-src/trunk/Zend/zend_API.c
>
> Modified: php/php-src/branches/PHP_5_3/NEWS
> ===
> --- php/php-src/branches/PHP_5_3/NEWS   2011-05-31 09:18:23 UTC (rev 311634)
> +++ php/php-src/branches/PHP_5_3/NEWS   2011-05-31 09:20:51 UTC (rev 311635)
> @@ -6,6 +6,8 @@
>  - Increased the backtrack limit from 10 to 100 (Rasmus)
>
>  - Zend Engine:
> +  . Fixed bug #54910 (Crash when calling call_user_func with unknown function
> +    name). (Dmitry)
>   . Fixed bug #54804 (__halt_compiler and imported namespaces).
>     (Pierrick, Felipe)
>   . Fixed bug #54585 (track_errors causes segfault). (Dmitry)
>
> Added: php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt
> ===
> --- php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt                     
>           (rev 0)
> +++ php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt       2011-05-31 
> 09:20:51 UTC (rev 311635)
> @@ -0,0 +1,28 @@
> +--TEST--
> +Bug #54910 (Crash when calling call_user_func with unknown function name)
> +--FILE--
> + +class A {
> +    public function __call($method, $args) {
> +        if (stripos($method, 'get') === 0) {
> +            return $this->get();
> +        }
> +        die("No such method - '$method'\n");
> +    }
> +
> +    protected function get() {
> +        $class = get_class($this);
> +        $call = array($class, 'noSuchMethod');
> +
> +        if (is_callable($call)) {
> +            call_user_func($call);
> +        }
> +    }
> +}
> +
> +class B extends A {}
> +
> +$input = new B();
> +echo $input->getEmail();
> +--EXPECT--
> +No such method - 'noSuchMethod'
>
> Modified: php/php-src/branches/PHP_5_3/Zend/zend_API.c
> ===
> --- php/php-src/branches/PHP_5_3/Zend/zend_API.c        2011-05-31 09:18:23 
> UTC (rev 311634)
> +++ php/php-src/branches/PHP_5_3/Zend/zend_API.c        2011-05-31 09:20:51 
> UTC (rev 311635)
> @@ -2590,6 +2590,11 @@
>                        if (fcc->function_handler) {
>                                retval = 1;
>                                call_via_handler = 
> (fcc->function_handler->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) != 0;
> +                               if (call_via_handler && !fcc->object_ptr && 
> EG(This) &&
> +                                   Z_OBJ_HT_P(EG(This))->get_class_entry &&
> +                                   instanceof_function(Z_OBJCE_P(EG(This)), 
> fcc->calling_scope TSRMLS_CC)) {
> +                                       fcc->object_ptr = EG(This);
> +                               }
>                        }
>                }
>        }
>
> Added: php/php-src/trunk/Zend/tests/bug54910.phpt
> ===
> --- php/php-src/trunk/Zend/tests/bug54910.phpt                          (rev 
> 0)
> +++ php/php-src/trunk/Zend/tests/bug54910.phpt  2011-05-31 09:20:51 UTC (rev 
> 311635)
> @@ -0,0 +1,28 @@
> +--TEST--
> +Bug #54910 (Crash when calling call_user_func with unknown function name)
> +--FILE--
> + +class A {
> +    public function __call($method, $args) {
> +        if (stripos($method, 'get') === 0) {
> +            return $this->get();
> +        }
> +        die("No such method - '$method'\n");
> +    }
> +
> +    protected function get() {
> +        $class = get_class($this);
> +        $call = array($class, 'noSuchMethod');
> +
> +        if (is_callable($call)) {
> +            call_user_func($call);
> +        }
> +    }
> +}
> +
> +class B extends A {}
> +
> +$input = new B();
> +echo $input->getEmail();
> +--EXPECT--
> +No such method - 'noSuchMethod'
>
> Modified: php/php-src/trunk/Zend/zend_API.c
> ===
> --- php/php-src/trunk/Zend/zend_API.c   2011-05-31 09:18:23 UTC (rev 311634)
> +++ php/php-src/trunk/Zend/zend_API.c   2011-05-31 09:20:51 UTC (rev 311635)
> @@ -2773,6 +2773,11 @@
>                        if (fcc->function_handler) {
>                                retval = 1;
>                                call_via_handler = 
> (fcc->function_handler->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) != 0;
> +                               i

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/NEWS branches/PHP_5_3/Zend/tests/bug54910.phpt branches/PHP_5_3/Zend/zend_API.c trunk/Zend/tests/bug54910.phpt trunk/Zend/zend_API.c

2011-05-31 Thread Dmitry Stogov
dmitry   Tue, 31 May 2011 09:20:51 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311635

Log:
Fixed bug #54910 (Crash when calling call_user_func with unknown function name)

Bug: http://bugs.php.net/54910 (Assigned) Crash when calling call_user_func 
with unknown function name
  
Changed paths:
U   php/php-src/branches/PHP_5_3/NEWS
A   php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt
U   php/php-src/branches/PHP_5_3/Zend/zend_API.c
A   php/php-src/trunk/Zend/tests/bug54910.phpt
U   php/php-src/trunk/Zend/zend_API.c

Modified: php/php-src/branches/PHP_5_3/NEWS
===
--- php/php-src/branches/PHP_5_3/NEWS   2011-05-31 09:18:23 UTC (rev 311634)
+++ php/php-src/branches/PHP_5_3/NEWS   2011-05-31 09:20:51 UTC (rev 311635)
@@ -6,6 +6,8 @@
 - Increased the backtrack limit from 10 to 100 (Rasmus)

 - Zend Engine:
+  . Fixed bug #54910 (Crash when calling call_user_func with unknown function
+name). (Dmitry)
   . Fixed bug #54804 (__halt_compiler and imported namespaces).
 (Pierrick, Felipe)
   . Fixed bug #54585 (track_errors causes segfault). (Dmitry)

Added: php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt
===
--- php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt   
(rev 0)
+++ php/php-src/branches/PHP_5_3/Zend/tests/bug54910.phpt   2011-05-31 
09:20:51 UTC (rev 311635)
@@ -0,0 +1,28 @@
+--TEST--
+Bug #54910 (Crash when calling call_user_func with unknown function name)
+--FILE--
+get();
+}
+die("No such method - '$method'\n");
+}
+
+protected function get() {
+$class = get_class($this);
+$call = array($class, 'noSuchMethod');
+
+if (is_callable($call)) {
+call_user_func($call);
+}
+}
+}
+
+class B extends A {}
+
+$input = new B();
+echo $input->getEmail();
+--EXPECT--
+No such method - 'noSuchMethod'

Modified: php/php-src/branches/PHP_5_3/Zend/zend_API.c
===
--- php/php-src/branches/PHP_5_3/Zend/zend_API.c2011-05-31 09:18:23 UTC 
(rev 311634)
+++ php/php-src/branches/PHP_5_3/Zend/zend_API.c2011-05-31 09:20:51 UTC 
(rev 311635)
@@ -2590,6 +2590,11 @@
if (fcc->function_handler) {
retval = 1;
call_via_handler = 
(fcc->function_handler->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) != 0;
+   if (call_via_handler && !fcc->object_ptr && 
EG(This) &&
+   Z_OBJ_HT_P(EG(This))->get_class_entry &&
+   instanceof_function(Z_OBJCE_P(EG(This)), 
fcc->calling_scope TSRMLS_CC)) {
+   fcc->object_ptr = EG(This);
+   }
}
}
}

Added: php/php-src/trunk/Zend/tests/bug54910.phpt
===
--- php/php-src/trunk/Zend/tests/bug54910.phpt  (rev 0)
+++ php/php-src/trunk/Zend/tests/bug54910.phpt  2011-05-31 09:20:51 UTC (rev 
311635)
@@ -0,0 +1,28 @@
+--TEST--
+Bug #54910 (Crash when calling call_user_func with unknown function name)
+--FILE--
+get();
+}
+die("No such method - '$method'\n");
+}
+
+protected function get() {
+$class = get_class($this);
+$call = array($class, 'noSuchMethod');
+
+if (is_callable($call)) {
+call_user_func($call);
+}
+}
+}
+
+class B extends A {}
+
+$input = new B();
+echo $input->getEmail();
+--EXPECT--
+No such method - 'noSuchMethod'

Modified: php/php-src/trunk/Zend/zend_API.c
===
--- php/php-src/trunk/Zend/zend_API.c   2011-05-31 09:18:23 UTC (rev 311634)
+++ php/php-src/trunk/Zend/zend_API.c   2011-05-31 09:20:51 UTC (rev 311635)
@@ -2773,6 +2773,11 @@
if (fcc->function_handler) {
retval = 1;
call_via_handler = 
(fcc->function_handler->common.fn_flags & ZEND_ACC_CALL_VIA_HANDLER) != 0;
+   if (call_via_handler && !fcc->object_ptr && 
EG(This) &&
+   Z_OBJ_HT_P(EG(This))->get_class_entry &&
+   instanceof_function(Z_OBJCE_P(EG(This)), 
fcc->calling_scope TSRMLS_CC)) {
+   fcc->object_ptr = EG(This);
+   }
}
}
}

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ext/pdo_pgsql/tests/ is_in_transaction.phpt

2011-05-31 Thread Ilia Alshanetsky
iliaaTue, 31 May 2011 09:01:44 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311632

Log:
Removed test for functionality available in 5.4+

Changed paths:
D   php/php-src/branches/PHP_5_3/ext/pdo_pgsql/tests/is_in_transaction.phpt

Deleted: php/php-src/branches/PHP_5_3/ext/pdo_pgsql/tests/is_in_transaction.phpt
===
--- php/php-src/branches/PHP_5_3/ext/pdo_pgsql/tests/is_in_transaction.phpt 
2011-05-31 08:59:32 UTC (rev 311631)
+++ php/php-src/branches/PHP_5_3/ext/pdo_pgsql/tests/is_in_transaction.phpt 
2011-05-31 09:01:44 UTC (rev 311632)
@@ -1,66 +0,0 @@
---TEST--
-PDO PgSQL isInTransaction
---SKIPIF--
-
---FILE--
-setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
-$db->setAttribute(PDO::ATTR_STRINGIFY_FETCHES, false);
-
-$db->exec('CREATE TABLE test (a integer not null primary key, b text)');
-
-$db->beginTransaction();
-try {
-echo "Test PDO::PGSQL_TRANSACTION_INTRANS\n";
-var_dump($db->inTransaction());
-
-$stmt = $db->prepare("INSERT INTO test (a, b) values (?, ?)");
-$stmt->bindValue(1, 1);
-$stmt->bindValue(2, "test insert");
-$stmt->execute();
-
-$db->commit();
-
-echo "Test PDO::PGSQL_TRANSACTION_IDLE\n";
-var_dump($db->inTransaction());
-
-$db->beginTransaction();
-
-try {
-$stmt = $db->prepare("INSERT INTO test (a, b) values (?, ?)");
-$stmt->bindValue(1, "error");
-$stmt->bindValue(2, "test insert");
-$stmt->execute();
-} catch (Exception $e) {
-   /* We catch the exception because the execute will give error and we 
must test the PDO::PGSQL_TRANSACTION_ERROR */
-   echo "Test PDO::PGSQL_TRANSACTION_INERROR\n";
-   var_dump($db->inTransaction());
-   $db->rollBack();
-}
-
-echo "Test PDO::PGSQL_TRANSACTION_IDLE\n";
-var_dump($db->inTransaction());
-
-} catch (Exception $e) {
-   /* catch exceptions so that we can show the relative error */
-   echo "Exception! at line ", $e->getLine(), "\n";
-   var_dump($e->getMessage());
-}
-
-?>
---EXPECT--
-Test PDO::PGSQL_TRANSACTION_INTRANS
-int(2)
-Test PDO::PGSQL_TRANSACTION_IDLE
-int(0)
-Test PDO::PGSQL_TRANSACTION_INERROR
-int(3)
-Test PDO::PGSQL_TRANSACTION_IDLE
-int(0)

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/pdo_pgsql/pdo_pgsql.c branches/PHP_5_3/ext/pdo_pgsql/php_pdo_pgsql.h branches/PHP_5_4/ext/pdo_pgsql/pdo_pgsql.c branches/PHP_5_4/ext/pdo_pgsql/php_pdo

2011-05-31 Thread Ilia Alshanetsky
iliaaTue, 31 May 2011 08:59:32 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311631

Log:
Removed unnecessary rinit/rshutdown

Changed paths:
U   php/php-src/branches/PHP_5_3/ext/pdo_pgsql/pdo_pgsql.c
U   php/php-src/branches/PHP_5_3/ext/pdo_pgsql/php_pdo_pgsql.h
U   php/php-src/branches/PHP_5_4/ext/pdo_pgsql/pdo_pgsql.c
U   php/php-src/branches/PHP_5_4/ext/pdo_pgsql/php_pdo_pgsql.h
U   php/php-src/trunk/ext/pdo_pgsql/pdo_pgsql.c
U   php/php-src/trunk/ext/pdo_pgsql/php_pdo_pgsql.h

Modified: php/php-src/branches/PHP_5_3/ext/pdo_pgsql/pdo_pgsql.c
===
--- php/php-src/branches/PHP_5_3/ext/pdo_pgsql/pdo_pgsql.c  2011-05-31 
08:58:28 UTC (rev 311630)
+++ php/php-src/branches/PHP_5_3/ext/pdo_pgsql/pdo_pgsql.c  2011-05-31 
08:59:32 UTC (rev 311631)
@@ -67,8 +67,8 @@
pdo_pgsql_functions,
PHP_MINIT(pdo_pgsql),
PHP_MSHUTDOWN(pdo_pgsql),
-   PHP_RINIT(pdo_pgsql),
-   PHP_RSHUTDOWN(pdo_pgsql),
+   NULL,
+   NULL,
PHP_MINFO(pdo_pgsql),
"1.0.2",
STANDARD_MODULE_PROPERTIES
@@ -106,24 +106,6 @@
 }
 /* }}} */

-/* {{{ PHP_RINIT_FUNCTION
- */
-PHP_RINIT_FUNCTION(pdo_pgsql)
-{
-   /*  php_pdo_register_driver(&pdo_pgsql_driver); */
-   return SUCCESS;
-}
-/* }}} */
-
-/* {{{ PHP_MSHUTDOWN_FUNCTION
- */
-PHP_RSHUTDOWN_FUNCTION(pdo_pgsql)
-{
-   /*  php_pdo_unregister_driver(&pdo_pgsql_driver); */
-   return SUCCESS;
-}
-/* }}} */
-
 /* {{{ PHP_MINFO_FUNCTION
  */
 PHP_MINFO_FUNCTION(pdo_pgsql)

Modified: php/php-src/branches/PHP_5_3/ext/pdo_pgsql/php_pdo_pgsql.h
===
--- php/php-src/branches/PHP_5_3/ext/pdo_pgsql/php_pdo_pgsql.h  2011-05-31 
08:58:28 UTC (rev 311630)
+++ php/php-src/branches/PHP_5_3/ext/pdo_pgsql/php_pdo_pgsql.h  2011-05-31 
08:59:32 UTC (rev 311631)
@@ -32,8 +32,6 @@

 PHP_MINIT_FUNCTION(pdo_pgsql);
 PHP_MSHUTDOWN_FUNCTION(pdo_pgsql);
-PHP_RINIT_FUNCTION(pdo_pgsql);
-PHP_RSHUTDOWN_FUNCTION(pdo_pgsql);
 PHP_MINFO_FUNCTION(pdo_pgsql);

 #endif /* PHP_PDO_PGSQL_H */

Modified: php/php-src/branches/PHP_5_4/ext/pdo_pgsql/pdo_pgsql.c
===
--- php/php-src/branches/PHP_5_4/ext/pdo_pgsql/pdo_pgsql.c  2011-05-31 
08:58:28 UTC (rev 311630)
+++ php/php-src/branches/PHP_5_4/ext/pdo_pgsql/pdo_pgsql.c  2011-05-31 
08:59:32 UTC (rev 311631)
@@ -67,8 +67,8 @@
pdo_pgsql_functions,
PHP_MINIT(pdo_pgsql),
PHP_MSHUTDOWN(pdo_pgsql),
-   PHP_RINIT(pdo_pgsql),
-   PHP_RSHUTDOWN(pdo_pgsql),
+   NULL,
+   NULL,
PHP_MINFO(pdo_pgsql),
"1.0.2",
STANDARD_MODULE_PROPERTIES
@@ -106,24 +106,6 @@
 }
 /* }}} */

-/* {{{ PHP_RINIT_FUNCTION
- */
-PHP_RINIT_FUNCTION(pdo_pgsql)
-{
-   /*  php_pdo_register_driver(&pdo_pgsql_driver); */
-   return SUCCESS;
-}
-/* }}} */
-
-/* {{{ PHP_MSHUTDOWN_FUNCTION
- */
-PHP_RSHUTDOWN_FUNCTION(pdo_pgsql)
-{
-   /*  php_pdo_unregister_driver(&pdo_pgsql_driver); */
-   return SUCCESS;
-}
-/* }}} */
-
 /* {{{ PHP_MINFO_FUNCTION
  */
 PHP_MINFO_FUNCTION(pdo_pgsql)

Modified: php/php-src/branches/PHP_5_4/ext/pdo_pgsql/php_pdo_pgsql.h
===
--- php/php-src/branches/PHP_5_4/ext/pdo_pgsql/php_pdo_pgsql.h  2011-05-31 
08:58:28 UTC (rev 311630)
+++ php/php-src/branches/PHP_5_4/ext/pdo_pgsql/php_pdo_pgsql.h  2011-05-31 
08:59:32 UTC (rev 311631)
@@ -32,8 +32,6 @@

 PHP_MINIT_FUNCTION(pdo_pgsql);
 PHP_MSHUTDOWN_FUNCTION(pdo_pgsql);
-PHP_RINIT_FUNCTION(pdo_pgsql);
-PHP_RSHUTDOWN_FUNCTION(pdo_pgsql);
 PHP_MINFO_FUNCTION(pdo_pgsql);

 #endif /* PHP_PDO_PGSQL_H */

Modified: php/php-src/trunk/ext/pdo_pgsql/pdo_pgsql.c
===
--- php/php-src/trunk/ext/pdo_pgsql/pdo_pgsql.c 2011-05-31 08:58:28 UTC (rev 
311630)
+++ php/php-src/trunk/ext/pdo_pgsql/pdo_pgsql.c 2011-05-31 08:59:32 UTC (rev 
311631)
@@ -67,8 +67,8 @@
pdo_pgsql_functions,
PHP_MINIT(pdo_pgsql),
PHP_MSHUTDOWN(pdo_pgsql),
-   PHP_RINIT(pdo_pgsql),
-   PHP_RSHUTDOWN(pdo_pgsql),
+   NULL,
+   NULL,
PHP_MINFO(pdo_pgsql),
"1.0.2",
STANDARD_MODULE_PROPERTIES
@@ -106,24 +106,6 @@
 }
 /* }}} */

-/* {{{ PHP_RINIT_FUNCTION
- */
-PHP_RINIT_FUNCTION(pdo_pgsql)
-{
-   /*  php_pdo_register_driver(&pdo_pgsql_driver); */
-   return SUCCESS;
-}
-/* }}} */
-
-/* {{{ PHP_MSHUTDOWN_FUNCTION
- */
-PHP_RSHUTDOWN_FUNCTION(pdo_pgsql)
-{
-   /*  php_pdo_unregister_driver(&pdo_pgsql_driver); */
-   return SUCCESS;
-}
-/* }}} */
-
 /* {{{ PHP_MINFO_FUNCTION
  */
 PHP_MINFO_FUNCTION(pdo_pgsql)

Modified: php/php-src/trunk/ext/pdo_pgsql/php_pdo_pgsql.h
===
--- php/php-src/tru

[PHP-CVS] svn: /php/php-src/trunk/Zend/ zend_alloc.c

2011-05-31 Thread Dmitry Stogov
dmitry   Tue, 31 May 2011 08:26:53 +

Revision: http://svn.php.net/viewvc?view=revision&revision=311627

Log:
Fixed bug #54957 (Memleak in alloc_globals_ctor)

Bug: http://bugs.php.net/54957 (Assigned) Memleak in alloc_globals_ctor
  
Changed paths:
U   php/php-src/trunk/Zend/zend_alloc.c

Modified: php/php-src/trunk/Zend/zend_alloc.c
===
--- php/php-src/trunk/Zend/zend_alloc.c 2011-05-31 08:19:14 UTC (rev 311626)
+++ php/php-src/trunk/Zend/zend_alloc.c 2011-05-31 08:26:53 UTC (rev 311627)
@@ -1602,6 +1602,9 @@
int internal;

if (!heap->use_zend_alloc) {
+   if (full_shutdown) {
+   free(heap);
+   }
return;
}


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php