Commit: a4e6394a06fb11f80426e94874d0702cf5235726 Author: Gustavo André dos Santos Lopes <cataphr...@php.net> Sat, 7 Apr 2012 16:32:19 +0100 Committer: Johannes Schlüter <johan...@php.net> Thu, 12 Apr 2012 11:04:20 +0200 Parents: 9ad0b878d91064d78bcd093a4660ea2bf99b03fb Branches: PHP-5.3.11
Link: http://git.php.net/?p=php-src.git;a=commitdiff;h=a4e6394a06fb11f80426e94874d0702cf5235726 Log: Fixed bug in new stream_get_line() when using NUL as a delimiter. This is the issue Derick spotted a few days ago.. Changed paths: A ext/standard/tests/streams/stream_get_line_NUL_delimiter.phpt M main/streams/streams.c Diff: diff --git a/ext/standard/tests/streams/stream_get_line_NUL_delimiter.phpt b/ext/standard/tests/streams/stream_get_line_NUL_delimiter.phpt new file mode 100644 index 0000000..32756d7 --- /dev/null +++ b/ext/standard/tests/streams/stream_get_line_NUL_delimiter.phpt @@ -0,0 +1,27 @@ +--TEST-- +Bug #60455: stream_get_line and \0 as a delimiter +--FILE-- +<?php +class TestStream { + private $s = 0; + function stream_open($path, $mode, $options, &$opened_path) { + return true; + } + function stream_read($count) { + if ($this->s++ == 0) + return "a\0"; + + return ""; + } + function stream_eof() { + return $this->s >= 2; + } + +} + +stream_wrapper_register("test", "TestStream"); + +$f = fopen("test://", "r"); +var_dump(stream_get_line($f, 100, "\0")); +--EXPECT-- +string(1) "a" diff --git a/main/streams/streams.c b/main/streams/streams.c index 639ea07..116c0aa 100755 --- a/main/streams/streams.c +++ b/main/streams/streams.c @@ -957,8 +957,8 @@ PHPAPI char *php_stream_get_record(php_stream *stream, size_t maxlen, size_t *re char *ret_buf, /* returned buffer */ *found_delim = NULL; size_t buffered_len, - tent_ret_len; /* tentative returned length*/ - int has_delim = delim_len > 0 && delim[0] != '\0'; + tent_ret_len; /* tentative returned length */ + int has_delim = delim_len > 0; if (maxlen == 0) { return NULL; -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php