diff options
author | Kai Koehne <kai.koehne@qt.io> | 2020-10-13 15:47:31 +0200 |
---|---|---|
committer | Kai Koehne <kai.koehne@qt.io> | 2020-10-27 10:05:24 +0100 |
commit | 6485b6d45ad165cf976138cf8ab683c42515e794 (patch) | |
tree | 2e704c5958c861fda1f3f9d27d28df7f0136eed8 /src/corelib/io | |
parent | aebc0df4bd4bcf4c705302f45347db2c21ce1b24 (diff) |
Fix allocated memory of QByteArray returned by QIODevice::readLine
If the maxSize argument is 0 (the default), QIODevice::readLine will
allocate a QByteArray with the size of the next chunk of data, which
may be quite large. Before returning, it then resizes the byte array
to the actual size that was read.
But since change 6b884d2aa129, QByteArray::resize() does no
longer shrink the capacity. This means that the returned QByteArray
keeps it's maximum size as allocated memory. This can lead to
excessive memory consumption, especially if the returned QByteArray's
are stored for further processing in the client code.
Fix this by explicitly calling QByteArray::squeeze() before returning.
[ChangeLog][QtCore][QIODevice] Fixes a regression in Qt 5.15 causing
QByteArray's that are returned by QIODevice::readLine() to
consume large amounts of memory.
Fixes: QTBUG-87010
Change-Id: I1f95fc4098849e900680fc945238bfeda881022c
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
(cherry picked from commit 263b29eedb223dec1ecaee193302070af87a1852,
limited squeeze() call if bytes are actually read to preserve retVal.isNull()
behavior in 5.15)
Diffstat (limited to 'src/corelib/io')
-rw-r--r-- | src/corelib/io/qiodevice.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/corelib/io/qiodevice.cpp b/src/corelib/io/qiodevice.cpp index cc1d110252..0f11c2e805 100644 --- a/src/corelib/io/qiodevice.cpp +++ b/src/corelib/io/qiodevice.cpp @@ -1480,10 +1480,12 @@ QByteArray QIODevice::readLine(qint64 maxSize) } else readBytes = readLine(result.data(), result.size()); - if (readBytes <= 0) + if (readBytes <= 0) { result.clear(); - else + } else { result.resize(readBytes); + result.squeeze(); + } return result; } |