Fix issue #1 (#2)

Makes the iteration to clean old ack data in ClockServer::updateEntry dependent on previous entry deletion. Co-authored-by: Fabian Dreer <[email protected]>
camilleg · Nov 17, 2020 · ab395fd · ab395fd
1 parent 9bc7454
commit ab395fd
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/ClockKit/ClockServer.cpp b/ClockKit/ClockServer.cpp
@@ -104,11 +104,14 @@ void ClockServer::updateEntry(string addr, int offset, int rtt)
             map<string, Entry>::iterator it;
 
             // purge old entries
-            for(it = ackData_.begin(); it != ackData_.end(); it++)
+            for(it = ackData_.begin(); it != ackData_.end();)
             {
                 timestamp_t entryTime = (it->second).time;
-                if (now - entryTime > SYSTEM_STATE_PURGE_TIME)
-                    ackData_.erase(it);
+                if (now - entryTime > SYSTEM_STATE_PURGE_TIME) {
+                    it = ackData_.erase(it);
+                } else {
+                    ++it;
+                }
             }
 
             // calculate maximum offset