diff options
author | MÃ¥rten Nordheim <marten.nordheim@qt.io> | 2023-02-20 13:02:29 +0100 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2023-02-21 12:05:05 +0000 |
commit | 44eabe2f936c25c8746c3818e165224e0e00178e (patch) | |
tree | 7d2deea1fc4b67147091011e50641f23364f412a /examples | |
parent | f97235c459593c24d85c6cdc124f811348d0921e (diff) | |
download | qtbase-44eabe2f936c25c8746c3818e165224e0e00178e.tar.gz |
Torrent example: Fix some stack overflow issues
Because of the .size() used as part of the expression it
ends up copying sizeof(qsizetype) bytes to the array,
instead of the (rationally) expected 4 bytes.
Amends 69c31f6f68a1399c1fc1e545e318eaa40966fe85
Task-number: QTBUG-110622
Change-Id: I732e3f4bb5934ff3860087baa91f9bebbf044a7f
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
Reviewed-by: Konrad Kujawa <konrad.kujawa@qt.io>
(cherry picked from commit f3de92cdad7642e6c3425712823e1491768f2e56)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'examples')
-rw-r--r-- | examples/network/torrent/peerwireclient.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/examples/network/torrent/peerwireclient.cpp b/examples/network/torrent/peerwireclient.cpp index f6f81215a1..d4264ffc3a 100644 --- a/examples/network/torrent/peerwireclient.cpp +++ b/examples/network/torrent/peerwireclient.cpp @@ -177,7 +177,7 @@ void PeerWireClient::sendPieceList(const QBitArray &bitField) } char message[] = {0, 0, 0, 1, 5}; - qToBigEndian(bits.size() + 1, &message[0]); + qToBigEndian<qint32>(bits.size() + 1, &message[0]); write(message, sizeof(message)); write(bits); } @@ -227,7 +227,7 @@ void PeerWireClient::sendBlock(int piece, int offset, const QByteArray &data) QByteArray block; char message[] = {0, 0, 0, 1, 7}; - qToBigEndian(9 + data.size(), &message[0]); + qToBigEndian<qint32>(9 + data.size(), &message[0]); block += QByteArray(message, sizeof(message)); char numbers[4 * 2]; |