[PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ NEWS Zend/zend_builtin_functions.c

2011-09-15 Thread Dmitry Stogov 
dmitry   Thu, 15 Sep 2011 09:58:33 +

Revision: http://svn.php.net/viewvc?view=revisionrevision=316810

Log:
Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)

Bug: https://bugs.php.net/55475 (Assigned) is_a() triggers autoloader
  
Changed paths:
U   php/php-src/branches/PHP_5_3/NEWS
U   php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c

Modified: php/php-src/branches/PHP_5_3/NEWS
===
--- php/php-src/branches/PHP_5_3/NEWS   2011-09-15 09:15:07 UTC (rev 316809)
+++ php/php-src/branches/PHP_5_3/NEWS   2011-09-15 09:58:33 UTC (rev 316810)
@@ -3,19 +3,20 @@
 ?? ??? 2011, PHP 5.3.9

 - Core:
-  . Fixed Bug #55649 (Undefined function Bug()). (Laruence)
+  . Fixed bug #55649 (Undefined function Bug()). (Laruence)
   . Fixed bug #55576: Cannot conditionally move uploaded file without race
 condition. (Gustavo)
-  . Fixed bug #55366: keys lost when using substr_replace an array. (Arpad)
-  . Fixed bug #55273 (base64_decode() with strict rejects whitespace after
-pad). (Ilia)
   . Fixed bug #55510: $_FILES 'name' missing first character after upload.
 (Arpad)
   . Fixed bug #55509 (segfault on x86_64 using more than 2G memory). (Laruence)
   . Fixed bug #55504 (Content-Type header is not parsed correctly on
 HTTP POST request). (Hannes)
+  . Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)
   . Fixed bug #52461 (Incomplete doctype and missing xmlns).
 (virsacer at web dot de, Pierre)
+  . Fixed bug #55366: keys lost when using substr_replace an array. (Arpad)
+  . Fixed bug #55273 (base64_decode() with strict rejects whitespace after
+pad). (Ilia)

 - Curl:
   . Fixed bug #54798 (Segfault when CURLOPT_STDERR file pointer is closed

Modified: php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c
===
--- php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c  2011-09-15 
09:15:07 UTC (rev 316809)
+++ php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c  2011-09-15 
09:58:33 UTC (rev 316810)
@@ -822,7 +822,7 @@
return;
}

-   if (Z_TYPE_P(obj) == IS_STRING) {
+   if (only_subclass  Z_TYPE_P(obj) == IS_STRING) {
zend_class_entry **the_ce;
if (zend_lookup_class(Z_STRVAL_P(obj), Z_STRLEN_P(obj), the_ce 
TSRMLS_CC) == FAILURE) {
RETURN_FALSE;

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

Re: [PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ NEWS Zend/zend_builtin_functions.c

2011-09-15 Thread Ferenc Kovacs 
On Thu, Sep 15, 2011 at 11:58 AM, Dmitry Stogov dmi...@php.net wrote:
 dmitry                                   Thu, 15 Sep 2011 09:58:33 +

 Revision: http://svn.php.net/viewvc?view=revisionrevision=316810

 Log:
 Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)

 Bug: https://bugs.php.net/55475 (Assigned) is_a() triggers autoloader

 Changed paths:
    U   php/php-src/branches/PHP_5_3/NEWS
    U   php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c

 Modified: php/php-src/branches/PHP_5_3/NEWS
 ===
 --- php/php-src/branches/PHP_5_3/NEWS   2011-09-15 09:15:07 UTC (rev 316809)
 +++ php/php-src/branches/PHP_5_3/NEWS   2011-09-15 09:58:33 UTC (rev 316810)
 @@ -3,19 +3,20 @@
  ?? ??? 2011, PHP 5.3.9

  - Core:
 -  . Fixed Bug #55649 (Undefined function Bug()). (Laruence)
 +  . Fixed bug #55649 (Undefined function Bug()). (Laruence)
   . Fixed bug #55576: Cannot conditionally move uploaded file without race
     condition. (Gustavo)
 -  . Fixed bug #55366: keys lost when using substr_replace an array. (Arpad)
 -  . Fixed bug #55273 (base64_decode() with strict rejects whitespace after
 -    pad). (Ilia)
   . Fixed bug #55510: $_FILES 'name' missing first character after upload.
     (Arpad)
   . Fixed bug #55509 (segfault on x86_64 using more than 2G memory). 
 (Laruence)
   . Fixed bug #55504 (Content-Type header is not parsed correctly on
     HTTP POST request). (Hannes)
 +  . Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)
   . Fixed bug #52461 (Incomplete doctype and missing xmlns).
     (virsacer at web dot de, Pierre)
 +  . Fixed bug #55366: keys lost when using substr_replace an array. (Arpad)
 +  . Fixed bug #55273 (base64_decode() with strict rejects whitespace after
 +    pad). (Ilia)

  - Curl:
   . Fixed bug #54798 (Segfault when CURLOPT_STDERR file pointer is closed

 Modified: php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c
 ===
 --- php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c  2011-09-15 
 09:15:07 UTC (rev 316809)
 +++ php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c  2011-09-15 
 09:58:33 UTC (rev 316810)
 @@ -822,7 +822,7 @@
                return;
        }

 -       if (Z_TYPE_P(obj) == IS_STRING) {
 +       if (only_subclass  Z_TYPE_P(obj) == IS_STRING) {
                zend_class_entry **the_ce;
                if (zend_lookup_class(Z_STRVAL_P(obj), Z_STRLEN_P(obj), 
 the_ce TSRMLS_CC) == FAILURE) {
                        RETURN_FALSE;


so we are now reverting back to the old behavior?
I didn't expected that by the discussion on the mailing list.

-- 
Ferenc Kovács
@Tyr43l - http://tyrael.hu

--
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/ NEWS Zend/zend_builtin_functions.c

2011-09-15 Thread Dmitry Stogov 
dmitry   Thu, 15 Sep 2011 10:59:39 +

Revision: http://svn.php.net/viewvc?view=revisionrevision=316811

Log:
Reverted the fix for #55475 (is_a() triggers autoloader) before the common 
decision

Bug: https://bugs.php.net/55475 (Closed) is_a() triggers autoloader
  
Changed paths:
U   php/php-src/branches/PHP_5_3/NEWS
U   php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c

Modified: php/php-src/branches/PHP_5_3/NEWS
===
--- php/php-src/branches/PHP_5_3/NEWS   2011-09-15 09:58:33 UTC (rev 316810)
+++ php/php-src/branches/PHP_5_3/NEWS   2011-09-15 10:59:39 UTC (rev 316811)
@@ -11,7 +11,6 @@
   . Fixed bug #55509 (segfault on x86_64 using more than 2G memory). (Laruence)
   . Fixed bug #55504 (Content-Type header is not parsed correctly on
 HTTP POST request). (Hannes)
-  . Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)
   . Fixed bug #52461 (Incomplete doctype and missing xmlns).
 (virsacer at web dot de, Pierre)
   . Fixed bug #55366: keys lost when using substr_replace an array. (Arpad)

Modified: php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c
===
--- php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c  2011-09-15 
09:58:33 UTC (rev 316810)
+++ php/php-src/branches/PHP_5_3/Zend/zend_builtin_functions.c  2011-09-15 
10:59:39 UTC (rev 316811)
@@ -822,7 +822,7 @@
return;
}

-   if (only_subclass  Z_TYPE_P(obj) == IS_STRING) {
+   if (Z_TYPE_P(obj) == IS_STRING) {
zend_class_entry **the_ce;
if (zend_lookup_class(Z_STRVAL_P(obj), Z_STRLEN_P(obj), the_ce 
TSRMLS_CC) == FAILURE) {
RETURN_FALSE;

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

Re: [PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ NEWS Zend/zend_builtin_functions.c

2011-09-15 Thread Johannes Schlüter 
On Thu, 2011-09-15 at 12:28 +0200, Ferenc Kovacs wrote:
 so we are now reverting back to the old behavior?
 I didn't expected that by the discussion on the mailing list.

ack, we shouldn't have changed the behavior in the fist place. But going
back and forth is creating an even bigger mess.

johannes



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