From: Stanislav Malyshev Date: Tue, 5 Aug 2008 21:23:20 +0000 (+0000) Subject: fix memnstr bug by Laurent Gaffie X-Git-Tag: BEFORE_HEAD_NS_CHANGE~858 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=fb9945422ea057511d6be34ad84130025867fc92;p=php fix memnstr bug by Laurent Gaffie --- diff --git a/Zend/zend_operators.h b/Zend/zend_operators.h index d23e0f342c..2d1614b14f 100644 --- a/Zend/zend_operators.h +++ b/Zend/zend_operators.h @@ -234,6 +234,9 @@ zend_memnstr(char *haystack, char *needle, int needle_len, char *end) if (needle_len == 1) { return (char *)memchr(p, *needle, (end-p)); } + if(needle_len > end-haystack) { + return NULL; + } end -= needle_len; diff --git a/ext/standard/tests/strings/explode_bug.phpt b/ext/standard/tests/strings/explode_bug.phpt new file mode 100644 index 0000000000..9766f0b8f4 --- /dev/null +++ b/ext/standard/tests/strings/explode_bug.phpt @@ -0,0 +1,15 @@ +--TEST-- +Explode/memnstr bug +--INI-- +error_reporting=2047 +memory_limit=256M +--FILE-- + +--EXPECTF-- +array(1) { + [0]=> + string(1) "1" +}