diff options
author | Marc Mutz <marc.mutz@qt.io> | 2022-06-10 16:57:52 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2022-06-14 01:30:48 +0000 |
commit | de005e7976e4b110d6b5f32a8bab5141713acb7b (patch) | |
tree | f49e428a018aa5f7efd52209ed8738129d5b1f38 /qmake | |
parent | 246f17d47061b61f3ad9a1711568767ed2254a9b (diff) |
QByteArray: more GCC 12 -Werror=array-bound whack-a-mole
This one came up on a tsan build, but not on an asan/ubsan one!?!:
qbytearray.cpp: In member function ‘QByteArray QByteArray::toBase64(Base64Options) const’:
qbytearray.cpp:3884:42: error: array subscript 1 is outside array bounds of ‘const char [1]’ [-Werror=array-bounds]
3884 | chunk |= int(uchar(data()[i++])) << 8;
| ~~~~~~~~~~^
qbytearray.cpp:37:24: note: at offset 1 into object ‘QByteArray::_empty’ of size 1
37 | Q_CONSTINIT const char QByteArray::_empty = '\0';
| ^~~~~~~~~~
cc1plus: all warnings being treated as errors
Fix, as so often, by caching the size().
The code in 5.15 is different, but similar, so picking all the way.
Pick-to: 6.4 6.3 6.2 5.15
Task-number: QTBUG-103923
Change-Id: Iac30e964c8d7d367620d16db65ceeaade33ee6b4
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
Diffstat (limited to 'qmake')
0 files changed, 0 insertions, 0 deletions