diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2020-11-07 08:08:31 -0800 |
---|---|---|
committer | Thiago Macieira <thiago.macieira@intel.com> | 2020-12-07 22:54:39 +0000 |
commit | 58bea7f3a2d0a88a56793c3878a7fcef74b4c646 (patch) | |
tree | d082ba10bcdd6e199651aa4705cc47821310f9fa /tests/auto/corelib | |
parent | bab2cd1125a21885bea97079219031ab45861826 (diff) |
QCborStreamReader::next: don't allocate too much memory in a QBA
Because CBOR strings are encoded in UTF-8, it's possible that the string
that won't fit a QString in UTF-16 would still fit QByteArray in UTF-8
(e.g., anything US-ASCII and most Latin text).
The previous solution was an improvement because we used to read into a
QByteArray then convert the QByteArray to QString, thus using 3x the
amount of memory (1x in QByteArray, 2x in QString). The previous commit
skipped the middle allocation and made the regular readString() function
do the decoding either directly on source memory or by reading in small
chunks (16 kB).
Future improvement for Qt 6.1: add readStringChunk(char16_t *, qsizetype)
so we can do the validation / skipping at O(1) memory.
Pick-to: 5.15 6.0
Change-Id: I7b9b97ae9b32412abdc6fffd1645458c655cc566
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'tests/auto/corelib')
-rw-r--r-- | tests/auto/corelib/serialization/cborlargedatavalidation.cpp | 19 | ||||
-rw-r--r-- | tests/auto/corelib/serialization/qcborstreamreader/tst_qcborstreamreader.cpp | 12 |
2 files changed, 19 insertions, 12 deletions
diff --git a/tests/auto/corelib/serialization/cborlargedatavalidation.cpp b/tests/auto/corelib/serialization/cborlargedatavalidation.cpp index c5cf1dc04e..a9a31ddf57 100644 --- a/tests/auto/corelib/serialization/cborlargedatavalidation.cpp +++ b/tests/auto/corelib/serialization/cborlargedatavalidation.cpp @@ -117,6 +117,7 @@ void addValidationHugeDevice(qsizetype byteArrayInvalid, qsizetype stringInvalid qRegisterMetaType<QSharedPointer<QIODevice>>(); QTest::addColumn<QSharedPointer<QIODevice>>("device"); QTest::addColumn<CborError>("expectedError"); + QTest::addColumn<CborError>("expectedValidationError"); char buf[1 + sizeof(quint64)]; auto device = [&buf](QCborStreamReader::Type t, quint64 size) { @@ -130,21 +131,21 @@ void addValidationHugeDevice(qsizetype byteArrayInvalid, qsizetype stringInvalid // do the exact limits QTest::newRow("bytearray-just-too-big") - << device(QCborStreamReader::ByteArray, byteArrayInvalid) << CborErrorDataTooLarge; - - // TODO: Fix this to work for 64-bit. The test tries to allocate too much data and fails. - if (sizeof(size_t) == 4) { - QTest::newRow("string-just-too-big") - << device(QCborStreamReader::String, stringInvalid) << CborErrorDataTooLarge; - } + << device(QCborStreamReader::ByteArray, byteArrayInvalid) + << CborErrorDataTooLarge << CborNoError; + QTest::newRow("string-just-too-big") + << device(QCborStreamReader::String, stringInvalid) + << CborErrorDataTooLarge << CborErrorDataTooLarge; auto addSize = [=](const char *sizename, qint64 size) { if (byteArrayInvalid < size) QTest::addRow("bytearray-%s", sizename) - << device(QCborStreamReader::ByteArray, size) << CborErrorDataTooLarge; + << device(QCborStreamReader::ByteArray, size) + << CborErrorDataTooLarge << CborNoError; if (stringInvalid < size) QTest::addRow("string-%s", sizename) - << device(QCborStreamReader::String, size) << CborErrorDataTooLarge; + << device(QCborStreamReader::String, size) + << CborErrorDataTooLarge << CborErrorDataTooLarge; }; addSize("1GB", quint64(1) << 30); addSize("2GB", quint64(1) << 31); diff --git a/tests/auto/corelib/serialization/qcborstreamreader/tst_qcborstreamreader.cpp b/tests/auto/corelib/serialization/qcborstreamreader/tst_qcborstreamreader.cpp index 437d8c40ce..f04f26b42f 100644 --- a/tests/auto/corelib/serialization/qcborstreamreader/tst_qcborstreamreader.cpp +++ b/tests/auto/corelib/serialization/qcborstreamreader/tst_qcborstreamreader.cpp @@ -958,6 +958,7 @@ void tst_QCborStreamReader::hugeDeviceValidation() QFETCH(QSharedPointer<QIODevice>, device); QFETCH(CborError, expectedError); + QFETCH(CborError, expectedValidationError); QCborError error = { QCborError::Code(expectedError) }; device->open(QIODevice::ReadOnly | QIODevice::Unbuffered); @@ -966,10 +967,15 @@ void tst_QCborStreamReader::hugeDeviceValidation() QVERIFY(parseOne(reader).isEmpty()); QCOMPARE(reader.lastError(), error); - // next() should fail reader.reset(); - QVERIFY(!reader.next()); - QCOMPARE(reader.lastError(), error); + error = { QCborError::Code(expectedValidationError) }; + if (error == QCborError{}) { + // this test actually succeeds, so don't do it to avoid large memory consumption + } else { + // next() should fail + QVERIFY(!reader.next()); + QCOMPARE(reader.lastError(), error); + } } static const int Recursions = 3; |