tony2001 Fri Feb 15 09:33:09 2008 UTC
Modified files: (Branch: PHP_5_3)
/php-src/ext/standard array.c
Log:
MFH: recursion protection in count()
http://cvs.php.net/viewvc.cgi/php-src/ext/standard/array.c?r1=1.308.2.21.2.37.2.24&r2=1.308.2.21.2.37.2.25&diff_format=u
Index: php-src/ext/standard/array.c
diff -u php-src/ext/standard/array.c:1.308.2.21.2.37.2.24
php-src/ext/standard/array.c:1.308.2.21.2.37.2.25
--- php-src/ext/standard/array.c:1.308.2.21.2.37.2.24 Thu Feb 14 14:08:22 2008
+++ php-src/ext/standard/array.c Fri Feb 15 09:33:09 2008
@@ -21,7 +21,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: array.c,v 1.308.2.21.2.37.2.24 2008/02/14 14:08:22 felipe Exp $ */
+/* $Id: array.c,v 1.308.2.21.2.37.2.25 2008/02/15 09:33:09 tony2001 Exp $ */
#include "php.h"
#include "php_ini.h"
@@ -275,6 +275,11 @@
zval **element;
if (Z_TYPE_P(array) == IS_ARRAY) {
+ if (Z_ARRVAL_P(array)->nApplyCount > 1) {
+ php_error_docref(NULL TSRMLS_CC, E_WARNING, "recursion
detected");
+ return 0;
+ }
+
cnt = zend_hash_num_elements(Z_ARRVAL_P(array));
if (mode == COUNT_RECURSIVE) {
HashPosition pos;
@@ -283,7 +288,9 @@
zend_hash_get_current_data_ex(Z_ARRVAL_P(array), (void **) &element, &pos) ==
SUCCESS;
zend_hash_move_forward_ex(Z_ARRVAL_P(array),
&pos)
) {
+ Z_ARRVAL_P(array)->nApplyCount++;
cnt += php_count_recursive(*element,
COUNT_RECURSIVE TSRMLS_CC);
+ Z_ARRVAL_P(array)->nApplyCount--;
}
}
}
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
