[PHP-CVS] cvs: php-src /ext/spl spl_iterators.c /ext/spl/tests bug46088.phpt
felipe Mon Sep 22 13:06:29 2008 UTC Added files: /php-src/ext/spl/tests bug46088.phpt Modified files: /php-src/ext/splspl_iterators.c Log: - Fixed bug #46088 (RegexIterator::accept - segfault) http://cvs.php.net/viewvc.cgi/php-src/ext/spl/spl_iterators.c?r1=1.178&r2=1.179&diff_format=u Index: php-src/ext/spl/spl_iterators.c diff -u php-src/ext/spl/spl_iterators.c:1.178 php-src/ext/spl/spl_iterators.c:1.179 --- php-src/ext/spl/spl_iterators.c:1.178 Tue Sep 9 19:43:52 2008 +++ php-src/ext/spl/spl_iterators.c Mon Sep 22 13:06:28 2008 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: spl_iterators.c,v 1.178 2008/09/09 19:43:52 lbarnaud Exp $ */ +/* $Id: spl_iterators.c,v 1.179 2008/09/22 13:06:28 felipe Exp $ */ #ifdef HAVE_CONFIG_H # include "config.h" @@ -1746,6 +1746,10 @@ intsubject_len, use_copy, count, result_len; zval subject_copy, zcount, *replacement; + if (intern->current.data == NULL) { + RETURN_FALSE; + } + if (intern->u.regex.flags & REGIT_USE_KEY) { if (intern->current.key_type == HASH_KEY_IS_LONG) { subject_len = snprintf(tmp, sizeof(tmp), "%ld", intern->current.int_key); http://cvs.php.net/viewvc.cgi/php-src/ext/spl/tests/bug46088.phpt?view=markup&rev=1.1 Index: php-src/ext/spl/tests/bug46088.phpt +++ php-src/ext/spl/tests/bug46088.phpt --TEST-- Bug #46088 (RegexIterator::accept - segfault) --FILE-- accept()); ?> --EXPECT-- bool(false) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/spl spl_iterators.c
felipe Mon Sep 22 13:09:09 2008 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/splspl_iterators.c Log: - MFH: Fixed bug #46088 (RegexIterator::accept - segfault) http://cvs.php.net/viewvc.cgi/php-src/ext/spl/spl_iterators.c?r1=1.73.2.30.2.28.2.14&r2=1.73.2.30.2.28.2.15&diff_format=u Index: php-src/ext/spl/spl_iterators.c diff -u php-src/ext/spl/spl_iterators.c:1.73.2.30.2.28.2.14 php-src/ext/spl/spl_iterators.c:1.73.2.30.2.28.2.15 --- php-src/ext/spl/spl_iterators.c:1.73.2.30.2.28.2.14 Tue Sep 9 19:44:15 2008 +++ php-src/ext/spl/spl_iterators.c Mon Sep 22 13:09:09 2008 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: spl_iterators.c,v 1.73.2.30.2.28.2.14 2008/09/09 19:44:15 lbarnaud Exp $ */ +/* $Id: spl_iterators.c,v 1.73.2.30.2.28.2.15 2008/09/22 13:09:09 felipe Exp $ */ #ifdef HAVE_CONFIG_H # include "config.h" @@ -1689,7 +1689,11 @@ char *subject, tmp[32], *result; intsubject_len, use_copy, count, result_len; zval subject_copy, zcount, *replacement; - + + if (intern->current.data == NULL) { + RETURN_FALSE; + } + if (intern->u.regex.flags & REGIT_USE_KEY) { if (intern->current.key_type == HASH_KEY_IS_LONG) { subject_len = slprintf(tmp, sizeof(tmp), "%ld", intern->current.int_key); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/spl/tests bug46088.phpt
felipe Mon Sep 22 13:09:50 2008 UTC Added files: (Branch: PHP_5_3) /php-src/ext/spl/tests bug46088.phpt Log: - New test http://cvs.php.net/viewvc.cgi/php-src/ext/spl/tests/bug46088.phpt?view=markup&rev=1.1 Index: php-src/ext/spl/tests/bug46088.phpt +++ php-src/ext/spl/tests/bug46088.phpt --TEST-- Bug #46088 (RegexIterator::accept - segfault) --FILE-- accept()); ?> --EXPECT-- bool(false) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) /ext/spl spl_iterators.c /ext/spl/tests bug46088.phpt
felipe Mon Sep 22 13:15:16 2008 UTC Added files: (Branch: PHP_5_2) /php-src/ext/spl/tests bug46088.phpt Modified files: /php-src/ext/splspl_iterators.c Log: - MFH: Fixed bug #46088 (RegexIterator::accept - segfault) http://cvs.php.net/viewvc.cgi/php-src/ext/spl/spl_iterators.c?r1=1.73.2.30.2.32&r2=1.73.2.30.2.33&diff_format=u Index: php-src/ext/spl/spl_iterators.c diff -u php-src/ext/spl/spl_iterators.c:1.73.2.30.2.32 php-src/ext/spl/spl_iterators.c:1.73.2.30.2.33 --- php-src/ext/spl/spl_iterators.c:1.73.2.30.2.32 Tue Sep 9 19:44:25 2008 +++ php-src/ext/spl/spl_iterators.c Mon Sep 22 13:15:16 2008 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: spl_iterators.c,v 1.73.2.30.2.32 2008/09/09 19:44:25 lbarnaud Exp $ */ +/* $Id: spl_iterators.c,v 1.73.2.30.2.33 2008/09/22 13:15:16 felipe Exp $ */ #ifdef HAVE_CONFIG_H # include "config.h" @@ -1363,6 +1363,10 @@ intsubject_len, use_copy, count, result_len; zval subject_copy, zcount, *replacement; + if (intern->current.data == NULL) { + RETURN_FALSE; + } + if (intern->u.regex.flags & REGIT_USE_KEY) { if (intern->current.key_type == HASH_KEY_IS_LONG) { subject_len = slprintf(tmp, sizeof(tmp), "%ld", intern->current.int_key); http://cvs.php.net/viewvc.cgi/php-src/ext/spl/tests/bug46088.phpt?view=markup&rev=1.1 Index: php-src/ext/spl/tests/bug46088.phpt +++ php-src/ext/spl/tests/bug46088.phpt --TEST-- Bug #46088 (RegexIterator::accept - segfault) --FILE-- accept()); ?> --EXPECT-- bool(false) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) / NEWS
felipe Mon Sep 22 13:17:58 2008 UTC Modified files: (Branch: PHP_5_2) /php-srcNEWS Log: - BFN #46088 http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.1230&r2=1.2027.2.547.2.1231&diff_format=u Index: php-src/NEWS diff -u php-src/NEWS:1.2027.2.547.2.1230 php-src/NEWS:1.2027.2.547.2.1231 --- php-src/NEWS:1.2027.2.547.2.1230Mon Sep 22 01:25:44 2008 +++ php-src/NEWSMon Sep 22 13:17:58 2008 @@ -18,8 +18,9 @@ (Scott) - Fixed a crash on invalid method in ReflectionParameter constructor. (Christian Seiler) -- Fixed Bug #46147 (after stream seek, appending stream filter reads incorrect data). +- Fixed bug #46147 (after stream seek, appending stream filter reads incorrect data). (Greg) +- Fixed bug #46088 (RegexIterator::accept - segfault). (Felipe) - Fixed bug #46059 (Compile failure under IRIX 6.5.30 building posix.c). (Arnaud) - Fixed bug #46053 (SplFileObject::seek - Endless loop). (Arnaud) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src /ext/dom domconfiguration.c node.c
rrichards Mon Sep 22 15:09:07 2008 UTC Modified files: /php-src/ext/domdomconfiguration.c node.c Log: fix hints causing phpdoc crash http://cvs.php.net/viewvc.cgi/php-src/ext/dom/domconfiguration.c?r1=1.12&r2=1.13&diff_format=u Index: php-src/ext/dom/domconfiguration.c diff -u php-src/ext/dom/domconfiguration.c:1.12 php-src/ext/dom/domconfiguration.c:1.13 --- php-src/ext/dom/domconfiguration.c:1.12 Thu Aug 14 08:38:37 2008 +++ php-src/ext/dom/domconfiguration.c Mon Sep 22 15:09:06 2008 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: domconfiguration.c,v 1.12 2008/08/14 08:38:37 tony2001 Exp $ */ +/* $Id: domconfiguration.c,v 1.13 2008/09/22 15:09:06 rrichards Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -31,7 +31,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_configuration_set_parameter, 0, 0, 2) ZEND_ARG_INFO(0, name) - ZEND_ARG_OBJ_INFO(0, value, DOMUserData, 0) + ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); static @@ -42,7 +42,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_configuration_can_set_parameter, 0, 0, 0) ZEND_ARG_INFO(0, name) - ZEND_ARG_OBJ_INFO(0, value, DOMUserData, 0) + ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); /* }}} */ http://cvs.php.net/viewvc.cgi/php-src/ext/dom/node.c?r1=1.62&r2=1.63&diff_format=u Index: php-src/ext/dom/node.c diff -u php-src/ext/dom/node.c:1.62 php-src/ext/dom/node.c:1.63 --- php-src/ext/dom/node.c:1.62 Thu Aug 14 08:38:37 2008 +++ php-src/ext/dom/node.c Mon Sep 22 15:09:06 2008 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: node.c,v 1.62 2008/08/14 08:38:37 tony2001 Exp $ */ +/* $Id: node.c,v 1.63 2008/09/22 15:09:06 rrichards Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -112,7 +112,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_node_set_user_data, 0, 0, 3) ZEND_ARG_INFO(0, key) - ZEND_ARG_OBJ_INFO(0, data, DOMUserData, 0) + ZEND_ARG_INFO(0, data) ZEND_ARG_INFO(0, handler) ZEND_END_ARG_INFO(); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/dom domconfiguration.c node.c
rrichards Mon Sep 22 15:09:46 2008 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/domdomconfiguration.c node.c Log: MFH: fix hints causing phpdoc crash http://cvs.php.net/viewvc.cgi/php-src/ext/dom/domconfiguration.c?r1=1.5.2.1.2.1.2.4&r2=1.5.2.1.2.1.2.5&diff_format=u Index: php-src/ext/dom/domconfiguration.c diff -u php-src/ext/dom/domconfiguration.c:1.5.2.1.2.1.2.4 php-src/ext/dom/domconfiguration.c:1.5.2.1.2.1.2.5 --- php-src/ext/dom/domconfiguration.c:1.5.2.1.2.1.2.4 Thu Aug 14 08:39:04 2008 +++ php-src/ext/dom/domconfiguration.c Mon Sep 22 15:09:46 2008 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: domconfiguration.c,v 1.5.2.1.2.1.2.4 2008/08/14 08:39:04 tony2001 Exp $ */ +/* $Id: domconfiguration.c,v 1.5.2.1.2.1.2.5 2008/09/22 15:09:46 rrichards Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -31,7 +31,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_configuration_set_parameter, 0, 0, 2) ZEND_ARG_INFO(0, name) - ZEND_ARG_OBJ_INFO(0, value, DOMUserData, 0) + ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); static @@ -42,7 +42,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_configuration_can_set_parameter, 0, 0, 0) ZEND_ARG_INFO(0, name) - ZEND_ARG_OBJ_INFO(0, value, DOMUserData, 0) + ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); /* }}} */ http://cvs.php.net/viewvc.cgi/php-src/ext/dom/node.c?r1=1.37.2.3.2.8.2.6&r2=1.37.2.3.2.8.2.7&diff_format=u Index: php-src/ext/dom/node.c diff -u php-src/ext/dom/node.c:1.37.2.3.2.8.2.6 php-src/ext/dom/node.c:1.37.2.3.2.8.2.7 --- php-src/ext/dom/node.c:1.37.2.3.2.8.2.6 Thu Aug 14 08:39:04 2008 +++ php-src/ext/dom/node.c Mon Sep 22 15:09:46 2008 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: node.c,v 1.37.2.3.2.8.2.6 2008/08/14 08:39:04 tony2001 Exp $ */ +/* $Id: node.c,v 1.37.2.3.2.8.2.7 2008/09/22 15:09:46 rrichards Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -112,7 +112,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_node_set_user_data, 0, 0, 3) ZEND_ARG_INFO(0, key) - ZEND_ARG_OBJ_INFO(0, data, DOMUserData, 0) + ZEND_ARG_INFO(0, data) ZEND_ARG_INFO(0, handler) ZEND_END_ARG_INFO(); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) /ext/dom domconfiguration.c node.c
rrichards Mon Sep 22 15:10:24 2008 UTC Modified files: (Branch: PHP_5_2) /php-src/ext/domdomconfiguration.c node.c Log: MFH: fix hints causing phpdoc crash http://cvs.php.net/viewvc.cgi/php-src/ext/dom/domconfiguration.c?r1=1.5.2.1.2.3&r2=1.5.2.1.2.4&diff_format=u Index: php-src/ext/dom/domconfiguration.c diff -u php-src/ext/dom/domconfiguration.c:1.5.2.1.2.3 php-src/ext/dom/domconfiguration.c:1.5.2.1.2.4 --- php-src/ext/dom/domconfiguration.c:1.5.2.1.2.3 Mon Feb 4 15:23:10 2008 +++ php-src/ext/dom/domconfiguration.c Mon Sep 22 15:10:24 2008 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: domconfiguration.c,v 1.5.2.1.2.3 2008/02/04 15:23:10 sebastian Exp $ */ +/* $Id: domconfiguration.c,v 1.5.2.1.2.4 2008/09/22 15:10:24 rrichards Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -32,7 +32,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_configuration_set_parameter, 0, 0, 2) ZEND_ARG_INFO(0, name) - ZEND_ARG_OBJ_INFO(0, value, DOMUserData, 0) + ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); static @@ -43,7 +43,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_configuration_can_set_parameter, 0, 0, 0) ZEND_ARG_INFO(0, name) - ZEND_ARG_OBJ_INFO(0, value, DOMUserData, 0) + ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); /* }}} */ http://cvs.php.net/viewvc.cgi/php-src/ext/dom/node.c?r1=1.37.2.3.2.11&r2=1.37.2.3.2.12&diff_format=u Index: php-src/ext/dom/node.c diff -u php-src/ext/dom/node.c:1.37.2.3.2.11 php-src/ext/dom/node.c:1.37.2.3.2.12 --- php-src/ext/dom/node.c:1.37.2.3.2.11Wed Jan 30 06:48:20 2008 +++ php-src/ext/dom/node.c Mon Sep 22 15:10:24 2008 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: node.c,v 1.37.2.3.2.11 2008/01/30 06:48:20 sebastian Exp $ */ +/* $Id: node.c,v 1.37.2.3.2.12 2008/09/22 15:10:24 rrichards Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -112,7 +112,7 @@ static ZEND_BEGIN_ARG_INFO_EX(arginfo_dom_node_set_user_data, 0, 0, 3) ZEND_ARG_INFO(0, key) - ZEND_ARG_OBJ_INFO(0, data, DOMUserData, 0) + ZEND_ARG_INFO(0, data) ZEND_ARG_INFO(0, handler) ZEND_END_ARG_INFO(); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src /ext/gd gd.c
rrichards Mon Sep 22 15:30:59 2008 UTC Modified files: /php-src/ext/gd gd.c Log: sz is long not zval** http://cvs.php.net/viewvc.cgi/php-src/ext/gd/gd.c?r1=1.393&r2=1.394&diff_format=u Index: php-src/ext/gd/gd.c diff -u php-src/ext/gd/gd.c:1.393 php-src/ext/gd/gd.c:1.394 --- php-src/ext/gd/gd.c:1.393 Fri Aug 29 11:09:05 2008 +++ php-src/ext/gd/gd.c Mon Sep 22 15:30:59 2008 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: gd.c,v 1.393 2008/08/29 11:09:05 pajoye Exp $ */ +/* $Id: gd.c,v 1.394 2008/09/22 15:30:59 rrichards Exp $ */ /* gd 1.2 is copyright 1994, 1995, Quest Protein Database Center, Cold Spring Harbor Labs. */ @@ -4231,10 +4231,10 @@ /* printf("%d %d %d %d\n", str_bbox.llx, str_bbox.lly, str_bbox.urx, str_bbox.ury); */ - add_next_index_long(return_value, (int) ceil(((double) str_bbox.llx)*Z_LVAL_PP(sz)/1000)); - add_next_index_long(return_value, (int) ceil(((double) str_bbox.lly)*Z_LVAL_PP(sz)/1000)); - add_next_index_long(return_value, (int) ceil(((double) str_bbox.urx)*Z_LVAL_PP(sz)/1000)); - add_next_index_long(return_value, (int) ceil(((double) str_bbox.ury)*Z_LVAL_PP(sz)/1000)); + add_next_index_long(return_value, (int) ceil(((double) str_bbox.llx)*sz/1000)); + add_next_index_long(return_value, (int) ceil(((double) str_bbox.lly)*sz/1000)); + add_next_index_long(return_value, (int) ceil(((double) str_bbox.urx)*sz/1000)); + add_next_index_long(return_value, (int) ceil(((double) str_bbox.ury)*sz/1000)); } /* }}} */ #endif -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php