Skip to content

Commit f8e26d9

Browse files
committed
Merge branch 'PHP-5.3' into PHP-5.4
* PHP-5.3: Fix bug #63240 on stream_get_line()
2 parents 9eff1a2 + 76601c4 commit f8e26d9

File tree

2 files changed

+27
-2
lines changed

2 files changed

+27
-2
lines changed
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
--TEST--
2+
Bug #63240: stream_get_line() return contains delimiter string
3+
--FILE--
4+
<?php
5+
$fd = fopen('php://temp', 'r+');
6+
$delimiter = 'MM';
7+
$str = str_repeat('.', 8191) . $delimiter . "rest";
8+
fwrite($fd, $str);
9+
rewind($fd);
10+
$line = stream_get_line($fd, 9000, $delimiter);
11+
var_dump(strlen($line));
12+
$line = stream_get_line($fd, 9000, $delimiter);
13+
var_dump($line);
14+
?>
15+
--EXPECT--
16+
int(8191)
17+
string(4) "rest"

main/streams/streams.c

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1060,9 +1060,17 @@ PHPAPI char *php_stream_get_record(php_stream *stream, size_t maxlen, size_t *re
10601060
if (has_delim) {
10611061
/* search for delimiter, but skip buffered_len (the number of bytes
10621062
* buffered before this loop iteration), as they have already been
1063-
* searched for the delimiter */
1063+
* searched for the delimiter.
1064+
* The left part of the delimiter may still remain in the buffer,
1065+
* so subtract up to <delim_len - 1> from buffered_len, which is
1066+
* the ammount of data we skip on this search as an optimization
1067+
*/
10641068
found_delim = _php_stream_search_delim(
1065-
stream, maxlen, buffered_len, delim, delim_len TSRMLS_CC);
1069+
stream, maxlen,
1070+
buffered_len >= (delim_len - 1)
1071+
? buffered_len - (delim_len - 1)
1072+
: 0,
1073+
delim, delim_len TSRMLS_CC);
10661074
if (found_delim) {
10671075
break;
10681076
}

0 commit comments

Comments
 (0)