Fix use after free issue of EventDispatcher LatencyRecorder (#3267)

chenBright · web-flow · commit 771de31ed930 · 2026-04-10T13:38:38.000+08:00
diff --git a/src/brpc/event_dispatcher.cpp b/src/brpc/event_dispatcher.cpp
@@ -54,13 +54,11 @@ static void StopAndJoinGlobalDispatchers() {
             g_edisp[i * FLAGS_event_dispatcher_num + j].Join();
         }
     }
-    delete g_edisp_read_lantency;
-    delete g_edisp_write_lantency;
 }
 
 void InitializeGlobalDispatchers() {
-    g_edisp_read_lantency = new bvar::LatencyRecorder("event_dispatcher_read_latency");
-    g_edisp_write_lantency = new bvar::LatencyRecorder("event_dispatcher_write_latency");
+    g_edisp_read_lantency = new bvar::LatencyRecorder("event_dispatcher_read");
+    g_edisp_write_lantency = new bvar::LatencyRecorder("event_dispatcher_write");
 
     g_edisp = new EventDispatcher[FLAGS_task_group_ntags * FLAGS_event_dispatcher_num];
     for (int i = 0; i < FLAGS_task_group_ntags; ++i) {

Original file line number	Diff line number	Diff line change
`@@ -54,13 +54,11 @@ static void StopAndJoinGlobalDispatchers() {`
`54`	`54`	`g_edisp[i * FLAGS_event_dispatcher_num + j].Join();`
`55`	`55`	`}`
`56`	`56`	`}`
`57`		`- delete g_edisp_read_lantency;`
`58`		`- delete g_edisp_write_lantency;`
`59`	`57`	`}`
`60`	`58`
`61`	`59`	`void InitializeGlobalDispatchers() {`
`62`		`- g_edisp_read_lantency = new bvar::LatencyRecorder("event_dispatcher_read_latency");`
`63`		`- g_edisp_write_lantency = new bvar::LatencyRecorder("event_dispatcher_write_latency");`
	`60`	`+ g_edisp_read_lantency = new bvar::LatencyRecorder("event_dispatcher_read");`
	`61`	`+ g_edisp_write_lantency = new bvar::LatencyRecorder("event_dispatcher_write");`
`64`	`62`
`65`	`63`	`g_edisp = new EventDispatcher[FLAGS_task_group_ntags * FLAGS_event_dispatcher_num];`
`66`	`64`	`for (int i = 0; i < FLAGS_task_group_ntags; ++i) {`