diff options
author | Samuel Rødal <sroedal@trolltech.com> | 2010-05-06 17:07:39 (GMT) |
---|---|---|
committer | Samuel Rødal <sroedal@trolltech.com> | 2010-05-07 08:36:01 (GMT) |
commit | f380ab3106cb6d39087adacd77f618184f6f90c3 (patch) | |
tree | ab917cf7bbc2304a33750f10d719c8395e1a116f /src/3rdparty | |
parent | 19db30f5b0a8a9a068a9acf9a20f08c86165393e (diff) | |
download | Qt-f380ab3106cb6d39087adacd77f618184f6f90c3.zip Qt-f380ab3106cb6d39087adacd77f618184f6f90c3.tar.gz Qt-f380ab3106cb6d39087adacd77f618184f6f90c3.tar.bz2 |
Fixed bug in QIODevice::read after first reading 0 bytes.
Change 02532ec80375c686503c4250c6ad6bb211515ec8 removed the early-exit
for 0 byte reads, causing us to hit code that assumed the buffer was
empty since nothing was read. It would thus read more into the end of
the buffer, causing the buffer to grow bigger than QIODEVICE_BUFFERSIZE.
Next, if the actual number of bytes we wanted to read was bigger than
the original buffer size we'd read the same data twice.
Reviewed-by: João Abecasis
Reviewed-by: Thiago Macieira
Diffstat (limited to 'src/3rdparty')
0 files changed, 0 insertions, 0 deletions