diff options
author | Mårten Nordheim <marten.nordheim@qt.io> | 2020-11-19 11:21:09 +0100 |
---|---|---|
committer | Mårten Nordheim <marten.nordheim@qt.io> | 2020-11-19 12:28:46 +0100 |
commit | 456699da93323ca3ad56ca8f16aa3197dec380e6 (patch) | |
tree | 879e816bfd20eb178bbb8396feff492040b61321 /src/network | |
parent | 28e4a8421ca618097bf8de35dab17ffbd423e468 (diff) |
QSocks5SocketEngine: Fix out-of-bounds access of QBA
This should've been caught a long time ago, but long story short: it wasn't.
This has been deprecated since 5.14 and is now asserting.
Pick-to: 5.15
Change-Id: Iddee22b1a68e5d1a03006831f795db8ec1c472be
Reviewed-by: Giuseppe D'Angelo <giuseppe.dangelo@kdab.com>
Diffstat (limited to 'src/network')
-rw-r--r-- | src/network/socket/qsocks5socketengine.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/network/socket/qsocks5socketengine.cpp b/src/network/socket/qsocks5socketengine.cpp index 3d40840c1b..d681bec2d5 100644 --- a/src/network/socket/qsocks5socketengine.cpp +++ b/src/network/socket/qsocks5socketengine.cpp @@ -1635,9 +1635,7 @@ qint64 QSocks5SocketEngine::writeDatagram(const char *data, qint64 len, const QI QByteArray outBuf; outBuf.reserve(270 + len); - outBuf[0] = 0x00; - outBuf[1] = 0x00; - outBuf[2] = 0x00; + outBuf.append(3, '\0'); if (!qt_socks5_set_host_address_and_port(header.destinationAddress, header.destinationPort, &outBuf)) { } outBuf += QByteArray(data, len); |