diff options
author | Konstantin Ritt <ritt.ks@gmail.com> | 2020-05-02 04:58:18 +0300 |
---|---|---|
committer | Konstantin Ritt <ritt.ks@gmail.com> | 2020-05-02 15:15:14 +0300 |
commit | ab2e985e3c8687812445b4da5a9bc32ccf5f73cd (patch) | |
tree | 5b566b28e7ae7c2d1419a4c6c48fda064d9d7384 | |
parent | 70a7791c36bb846d1b8d89c85dcc705ed92c0d96 (diff) |
Exit more gracefuly on protocol violations
Change-Id: Ie797b505e67f02e65cfff3b25d49ddae523f4131
Reviewed-by: Maurice Kalinowski <maurice.kalinowski@qt.io>
-rw-r--r-- | src/mqtt/qmqttconnection.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/mqtt/qmqttconnection.cpp b/src/mqtt/qmqttconnection.cpp index 3c33d7d..29c734c 100644 --- a/src/mqtt/qmqttconnection.cpp +++ b/src/mqtt/qmqttconnection.cpp @@ -1749,9 +1749,12 @@ bool QMqttConnection::processDataHelper() default: qCDebug(lcMqttConnection) << "Unknown packet to finalize."; closeConnection(QMqttClient::ProtocolViolation); - return false; + break; } + if (m_internalState == BrokerDisconnected) + return false; + Q_ASSERT(m_missingData == 0); m_readBuffer = m_readBuffer.mid(m_readPosition); |