Avoid bug where stream_copy_to_stream() call for precisely 2000000 bytes would

result in entire resource being copied.
The mmap brake is already present inside mmap_range() and is set to 1MB.
This commit is contained in:
Wez Furlong 2005-02-06 23:17:09 +00:00
parent 1069cefada
commit 0569078901

View File

@ -407,10 +407,8 @@ END_EXTERN_C()
#define PHP_STREAM_OPTION_RETURN_NOTIMPL -2 /* underlying stream does not implement; streams can handle it instead */
/* copy up to maxlen bytes from src to dest. If maxlen is PHP_STREAM_COPY_ALL, copy until eof(src).
* Uses mmap if the src is a plain file and at offset 0
* To ensure we don't take up too much memory when reading large files, set the default mmap length
* at this many bytes */
#define PHP_STREAM_COPY_ALL 2000000
* Uses mmap if the src is a plain file and at offset 0 */
#define PHP_STREAM_COPY_ALL ((size_t)-1)
BEGIN_EXTERN_C()
PHPAPI size_t _php_stream_copy_to_stream(php_stream *src, php_stream *dest, size_t maxlen STREAMS_DC TSRMLS_DC);