diff options
| author | Marc Mutz <marc.mutz@qt.io> | 2022-05-06 09:35:42 +0200 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@qt.io> | 2022-05-07 00:51:15 +0200 |
| commit | dffba486d1e63e3034596607f58801b2719a7b6c (patch) | |
| tree | 067144e86a7acf1c3e3f68e05624a51b8e68ac44 /src/corelib/serialization/qxmlstream.cpp | |
| parent | 61c83dfb220f2e0d452a5e87a5a9b6fa45a9e6a9 (diff) | |
QXmlStreamReader: port Value::prefix from int to qint16
The prefix is a part of a name, the length of which is bounded to 4k
in fastScanName(), so qint16 suffices.
The length field also shouldn't stay int, but that's a different
patch, because it's just a relative offset to pos, and so isn't as
easily overflown.
This shrinks the Value struct a tiny bit; created QTBUG-103306 to
track ideas to shrink it further.
Pick-to: 6.3 6.2
Task-number: QTBUG-102465
Change-Id: I579815e72501a091360f55e750af63cb4dc5a5a7
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Diffstat (limited to 'src/corelib/serialization/qxmlstream.cpp')
| -rw-r--r-- | src/corelib/serialization/qxmlstream.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/corelib/serialization/qxmlstream.cpp b/src/corelib/serialization/qxmlstream.cpp index 17f75bb142b..3323ff07073 100644 --- a/src/corelib/serialization/qxmlstream.cpp +++ b/src/corelib/serialization/qxmlstream.cpp @@ -1279,14 +1279,14 @@ inline int QXmlStreamReaderPrivate::fastScanContentCharList() return n; } -inline int QXmlStreamReaderPrivate::fastScanName(int *prefix) +inline int QXmlStreamReaderPrivate::fastScanName(qint16 *prefix) { int n = 0; uint c; while ((c = getChar()) != StreamEOF) { if (n >= 4096) { // This is too long to be a sensible name, and - // can exhaust memory + // can exhaust memory, or the range of decltype(*prefix) return 0; } switch (c) { @@ -1325,7 +1325,7 @@ inline int QXmlStreamReaderPrivate::fastScanName(int *prefix) case ':': if (prefix) { if (*prefix == 0) { - *prefix = n+2; + *prefix = qint16(n + 2); } else { // only one colon allowed according to the namespace spec. putChar(c); return n; |