[ntcore] Fix overlapping subscriber handling (wpilibsuite#6067)

srimanachanta · Dec 20, 2023 · 3d201c7 · 3d201c7
1 parent f029841
commit 3d201c7
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/ntcore/src/main/native/cpp/net/ServerImpl.cpp b/ntcore/src/main/native/cpp/net/ServerImpl.cpp
@@ -1708,7 +1708,9 @@ ServerImpl::TopicData* ServerImpl::CreateTopic(ClientData* client,
       auto& tcd = topic->clients[aClient.get()];
       bool added = false;
       for (auto subscriber : subscribers) {
-        added = added || tcd.AddSubscriber(subscriber);
+        if (tcd.AddSubscriber(subscriber)) {
+          added = true;
+        }
       }
       if (added) {
         aClient->UpdatePeriod(tcd, topic);