tony2001 Wed Sep 6 12:07:32 2006 UTC
Modified files: (Branch: PHP_5_2)
/php-src/ext/iconv iconv.c
/php-src/ext/iconv/tests iconv_strpos.phpt iconv_strrpos.phpt
Log:
do not search for empty strings with strpos() and strrpos()
http://cvs.php.net/viewvc.cgi/php-src/ext/iconv/iconv.c?r1=1.124.2.8.2.4&r2=1.124.2.8.2.5&diff_format=u
Index: php-src/ext/iconv/iconv.c
diff -u php-src/ext/iconv/iconv.c:1.124.2.8.2.4
php-src/ext/iconv/iconv.c:1.124.2.8.2.5
--- php-src/ext/iconv/iconv.c:1.124.2.8.2.4 Thu Aug 31 11:17:47 2006
+++ php-src/ext/iconv/iconv.c Wed Sep 6 12:07:32 2006
@@ -18,7 +18,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: iconv.c,v 1.124.2.8.2.4 2006/08/31 11:17:47 tony2001 Exp $ */
+/* $Id: iconv.c,v 1.124.2.8.2.5 2006/09/06 12:07:32 tony2001 Exp $ */
#ifdef HAVE_CONFIG_H
#include "config.h"
@@ -1946,6 +1946,10 @@
RETURN_FALSE;
}
+ if (ndl_len < 1) {
+ RETURN_FALSE;
+ }
+
err = _php_iconv_strpos(&retval, haystk, haystk_len, ndl, ndl_len,
offset, charset);
_php_iconv_show_error(err, GENERIC_SUPERSET_NAME, charset TSRMLS_CC);
@@ -1981,6 +1985,10 @@
RETURN_FALSE;
}
+ if (ndl_len < 1) {
+ RETURN_FALSE;
+ }
+
err = _php_iconv_strpos(&retval, haystk, haystk_len, ndl, ndl_len,
-1, charset);
_php_iconv_show_error(err, GENERIC_SUPERSET_NAME, charset TSRMLS_CC);
http://cvs.php.net/viewvc.cgi/php-src/ext/iconv/tests/iconv_strpos.phpt?r1=1.1&r2=1.1.6.1&diff_format=u
Index: php-src/ext/iconv/tests/iconv_strpos.phpt
diff -u php-src/ext/iconv/tests/iconv_strpos.phpt:1.1
php-src/ext/iconv/tests/iconv_strpos.phpt:1.1.6.1
--- php-src/ext/iconv/tests/iconv_strpos.phpt:1.1 Fri Jan 3 20:02:23 2003
+++ php-src/ext/iconv/tests/iconv_strpos.phpt Wed Sep 6 12:07:32 2006
@@ -31,6 +31,10 @@
foo(str_repeat("¤¢¤¤¤¦¤¨¤ª", 30)."¤¤¤¦¤ª¤¨¤¢".str_repeat("¤¢¤¤¤¨¤ª¤¦", 30),
"¤¦¤ª", 0, "EUC-JP");
$str = str_repeat("¤¢¤¤¤¦¤¨¤ª", 60).'$'.str_repeat("¤¢¤¤¤¨¤ª¤¦", 60);
foo($str, '$', 0, "ISO-2022-JP", "EUC-JP");
+
+var_dump(iconv_strpos("string", ""));
+var_dump(iconv_strpos("", "string"));
+
?>
--EXPECTF--
2: %s
@@ -50,3 +54,5 @@
int(151)
int(1)
int(300)
+bool(false)
+bool(false)
http://cvs.php.net/viewvc.cgi/php-src/ext/iconv/tests/iconv_strrpos.phpt?r1=1.1&r2=1.1.6.1&diff_format=u
Index: php-src/ext/iconv/tests/iconv_strrpos.phpt
diff -u php-src/ext/iconv/tests/iconv_strrpos.phpt:1.1
php-src/ext/iconv/tests/iconv_strrpos.phpt:1.1.6.1
--- php-src/ext/iconv/tests/iconv_strrpos.phpt:1.1 Fri Jan 3 20:02:23 2003
+++ php-src/ext/iconv/tests/iconv_strrpos.phpt Wed Sep 6 12:07:32 2006
@@ -32,6 +32,10 @@
$str = str_repeat("¤¢¤¤¤¦¤¨¤ª", 60).str_repeat('$',
$i).str_repeat("¤¢¤¤¤¨¤ª¤¦", 60);
foo($str, '$', "ISO-2022-JP", "EUC-JP");
}
+
+var_dump(iconv_strrpos("string", ""));
+var_dump(iconv_strrpos("", "string"));
+
?>
--EXPECT--
int(14)
@@ -54,3 +58,5 @@
int(304)
int(606)
int(305)
+bool(false)
+bool(false)
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
