dont attempt to reschedule timers that are deleted

.vscode/settings.json

@@ -81,6 +81,8 @@
 		"semaphore": "cpp",
 		"stop_token": "cpp",
 		"charconv": "cpp",
-		"any": "cpp"
+		"any": "cpp",
+		"ranges": "cpp",
+		"span": "cpp"
 	}
 }

src/dpp/cluster/timer.cpp

@@ -97,10 +97,21 @@ void cluster::tick_timers() {
 		}
 	}
 	for (auto & t : scheduled) {
+		timer handle = t->handle;
 		/* Call handler */
 		t->on_tick(t->handle);
-		/* Reschedule for next tick */
-		timer_reschedule(t);
+		/* Reschedule if it wasn't deleted.
+		 * Note: We wrap the .contains() check in a lambda as it needs locking
+		 * for thread safety, but timer_rescheudle also locks the container, so this
+		 * is the cleanest way to do it.
+		 */
+		bool not_deleted = ([handle, this]() -> bool {
+			std::lock_guard<std::mutex> l(timer_guard);
+			return timer_list.contains(handle);
+		}());
+		if (not_deleted) {
+			timer_reschedule(t);
+		}
 	}
 }