aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosé Bollo <jose.bollo@iot.bzh>2019-06-13 16:12:22 +0200
committerJosé Bollo <jose.bollo@iot.bzh>2019-06-13 16:12:22 +0200
commit547374849a76225b1a6a0dd8bb84ae27a601f19d (patch)
treec2def8dc16fc13c7ed24c937a42a707c8a0a86e2
parent886b1208a88e33030e462afcca3b535a9bac5cd9 (diff)
Don't passing a callback is a valid model. But the library should not crash in that case. This fix the uissue. Bug-AGL: SPEC-2520 Change-Id: I6e6f3e6b617cfcf251ac910db5fa27832c2713a1 Signed-off-by: José Bollo <jose.bollo@iot.bzh>
-rw-r--r--qafbwebsocketclient.cpp7
1 files changed, 3 insertions, 4 deletions
diff --git a/qafbwebsocketclient.cpp b/qafbwebsocketclient.cpp
index 0cbc274..c0e5e71 100644
--- a/qafbwebsocketclient.cpp
+++ b/qafbwebsocketclient.cpp
@@ -85,8 +85,9 @@ void QAfbWebsocketClient::close()
*/
void QAfbWebsocketClient::call(const QString& api, const QString& verb, const QJsonValue& arg, closure_t closure)
{
- QString callId = QString::number(m_nextCallId);
- m_closures[callId] = closure;
+ QString callId = QString::number(m_nextCallId++);
+ if (closure)
+ m_closures[callId] = closure;
QJsonArray msg;
msg.append(2); // Call
@@ -94,8 +95,6 @@ void QAfbWebsocketClient::call(const QString& api, const QString& verb, const QJ
msg.append(api + "/" + verb);
msg.append(arg);
- m_nextCallId++;
-
QJsonDocument value;
value.setArray(msg);