From 5060fc2349447fe17b59184a0147e73feb0e7a30 Mon Sep 17 00:00:00 2001 From: Simon JAILLET Date: Sat, 6 May 2017 16:29:34 +0200 Subject: [PATCH] Fixes #68948 related to a BC break introduced by #68532 fix. --- ext/standard/tests/streams/bug68948.phpt | 30 +++++++++++++++++++ .../tests/streams/stream_set_chunk_size.phpt | 10 ++++--- main/streams/streams.c | 2 +- 3 files changed, 37 insertions(+), 5 deletions(-) create mode 100644 ext/standard/tests/streams/bug68948.phpt diff --git a/ext/standard/tests/streams/bug68948.phpt b/ext/standard/tests/streams/bug68948.phpt new file mode 100644 index 0000000000..7ce9d7ad03 --- /dev/null +++ b/ext/standard/tests/streams/bug68948.phpt @@ -0,0 +1,30 @@ +--TEST-- +Bug #68948: feof() on temporary streams broken +--FILE-- + +--EXPECT-- +string(10) "0123456789" +int(10) +bool(false) +string(10) "0123456789" +int(10) +bool(true) + diff --git a/ext/standard/tests/streams/stream_set_chunk_size.phpt b/ext/standard/tests/streams/stream_set_chunk_size.phpt index ce272519c4..88f4897e93 100644 --- a/ext/standard/tests/streams/stream_set_chunk_size.phpt +++ b/ext/standard/tests/streams/stream_set_chunk_size.phpt @@ -39,7 +39,7 @@ var_dump(fwrite($f, str_repeat('b', 3))); echo "should return previous chunk size (1)\n"; var_dump(stream_set_chunk_size($f, 100)); -echo "should elicit one read of size 100 (chunk size)\n"; +echo "should elicit 3 reads of size 100 (chunk size)\n"; var_dump(strlen(fread($f, 250))); echo "should elicit one read of size 100 (chunk size)\n"; var_dump(strlen(fread($f, 50))); @@ -67,13 +67,15 @@ write with size: 1 int(3) should return previous chunk size (1) int(1) -should elicit one read of size 100 (chunk size) +should elicit 3 reads of size 100 (chunk size) read with size: 100 -int(100) -should elicit one read of size 100 (chunk size) read with size: 100 +read with size: 100 +int(250) +should elicit one read of size 100 (chunk size) int(50) should elicit no read because there is sufficient cached data +read with size: 100 int(50) should elicit 2 writes of size 100 and one of size 50 write with size: 100 diff --git a/main/streams/streams.c b/main/streams/streams.c index dab8505d41..b1099b6fcc 100644 --- a/main/streams/streams.c +++ b/main/streams/streams.c @@ -710,7 +710,7 @@ PHPAPI size_t _php_stream_read(php_stream *stream, char *buf, size_t size) } /* just break anyway, to avoid greedy read */ - if (stream->wrapper != &php_plain_files_wrapper) { + if (!stream->wrapper || stream->wrapper->is_url) { break; } } -- 2.50.1