Fix js-module timers leak

altmp · Dec 16, 2024 · e8744d8 · e8744d8
1 parent 5fd9c7a
commit e8744d8
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/shared/V8ResourceImpl.cpp b/shared/V8ResourceImpl.cpp
@@ -77,7 +77,16 @@ void V8ResourceImpl::OnTick()
     for(auto& nextTickCb : nextTickCallbacks) nextTickCb();
     nextTickCallbacks.clear();
 
-    for(auto& id : oldTimers) timers.erase(id);
+    for(auto& id : oldTimers)
+    {
+        auto it = timers.find(id);
+        if(it == timers.end())
+            continue;
+
+        auto timer = it->second;
+        timers.erase(it);
+        delete timer;
+    }
 
     oldTimers.clear();