perf_hooks: avoid memory leak on gc observer

Fixes: nodejs#22229
jasnell · Aug 10, 2018 · 977f9e6 · 977f9e6
1 parent fe47b8b
commit 977f9e6
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/node_perf.cc b/src/node_perf.cc
@@ -272,6 +272,9 @@ void MarkGarbageCollectionEnd(Isolate* isolate,
                               v8::GCCallbackFlags flags,
                               void* data) {
   Environment* env = static_cast<Environment*>(data);
+  // If no one is listening to gc performance entries, do not create them.
+  if (!env->performance_state()->observers[NODE_PERFORMANCE_ENTRY_TYPE_GC])
+    return;
   GCPerformanceEntry* entry =
       new GCPerformanceEntry(env,
                              static_cast<PerformanceGCKind>(type),