ETM-Schurig/powersync-mqtt
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge PR #5: Fix a crash in topic matching.

Browse Source
This commit is contained in:
Jenkins nymea 2019-10-10 22:55:35 +02:00
commit 3a1c840ca4
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libnymea-mqtt/mqttserver.cpp
View File

@ -628,6 +628,9 @@ bool MqttServerPrivate::matchTopic(const QString &topicFilter, const QString &to
if (filterParts.at(i) == QStringLiteral("#")) {
continue;
}
if (topicParts.length() <= i) {
return false;
}
if (topicParts.at(i) == filterParts.at(i)) {
continue;
}

1
tests/operation/test_operation.cpp
View File

@ -489,6 +489,7 @@ void OperationTests::testSubscriptionTopicMatching_data()
rows.append({ "a//+/", "a///", "1" });
rows.append({ "a//+/#", "a//b/c", "1" });
rows.append({ "a//+/#", "a/b/c/d", "0" });
rows.append({ "a/b/c", "a/b", "0"});
rows.append({ "$SYS/", "$SYS/", "0" });
rows.append({ "#", "$SYS/", "0" });