Fix double use of ZCL sequence numbers
The problem here is that the sequence number in the
sendConfigureReportingRequest() is reused in the next request,
since the zclSeq++ will yield the same number and afterwards increment.
@@ -680,12 +680,11 @@ bool DeRestPluginPrivate::sendConfigureReportingRequest(BindingTask &bt, const s
return false;
}
- zclSeq++;
if (zclSeq == 0) // don't use zero, simplify matching
{
zclSeq = 1;
}
- const quint8 zclSeqNum = zclSeq; // to match in configure reporting response handler
+ const quint8 zclSeqNum = zclSeq++; // to match in configure reporting response handler
LightNode *lightNode = dynamic_cast<LightNode*>(bt.restNode);
QDateTime now = QDateTime::currentDateTime();
std::vector<ConfigureReportingRequest> out;
