summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKonstantin Ritt <ritt.ks@gmail.com>2020-05-02 04:58:18 +0300
committerKonstantin Ritt <ritt.ks@gmail.com>2020-05-02 15:15:14 +0300
commitab2e985e3c8687812445b4da5a9bc32ccf5f73cd (patch)
tree5b566b28e7ae7c2d1419a4c6c48fda064d9d7384
parent70a7791c36bb846d1b8d89c85dcc705ed92c0d96 (diff)
Exit more gracefuly on protocol violations
Change-Id: Ie797b505e67f02e65cfff3b25d49ddae523f4131 Reviewed-by: Maurice Kalinowski <maurice.kalinowski@qt.io>
-rw-r--r--src/mqtt/qmqttconnection.cpp5
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);