github/arthas
4
0
Fork 0
mirror of https://github.com/alibaba/arthas.git synced 2026-10-21 08:04:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: race condition in AdviceListenerManager causing lost listeners (#3237)

Browse Source
This commit is contained in:
Roger3581321
2026-07-17 10:45:28 +08:00
committed by GitHub
parent 7b74eb6a1b
commit 89fed559e8
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
core/src/main/java/com/taobao/arthas/core/advisor/AdviceListenerManager.java
+12 -3
View File
@@ -197,7 +197,10 @@ public class AdviceListenerManager {
if (manager == null) {
manager = new ClassLoaderAdviceListenerManager();
adviceListenerMap.put(classLoader, manager);
ClassLoaderAdviceListenerManager existing = adviceListenerMap.putIfAbsent(classLoader, manager);
if (existing != null) {
manager = existing;
}
}
manager.registerAdviceListener(className, methodName, methodDesc, listener);
}
@@ -228,7 +231,10 @@ public class AdviceListenerManager {
if (manager == null) {
manager = new ClassLoaderAdviceListenerManager();
adviceListenerMap.put(classLoader, manager);
ClassLoaderAdviceListenerManager existing = adviceListenerMap.putIfAbsent(classLoader, manager);
if (existing != null) {
manager = existing;
}
}
manager.registerTraceAdviceListener(className, owner, methodName, methodDesc, listener);
}
@@ -258,7 +264,10 @@ public class AdviceListenerManager {
if (manager == null) {
manager = new ClassLoaderAdviceListenerManager();
adviceListenerMap.put(classLoader, manager);
ClassLoaderAdviceListenerManager existing = adviceListenerMap.putIfAbsent(classLoader, manager);
if (existing != null) {
manager = existing;
}
}
manager.registerLineAdviceListener(className, methodName, methodDesc, lineNumber, listener);
}