From: avasseur <avasseur>
Date: Mon, 11 Jul 2005 15:09:52 +0000 (+0000)
Subject: 99861 temp fix for deadlock when Sun' Unsafe.defineClass API is used (deadlock in... 
X-Git-Tag: PRE_ANDY~25
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=aff35a1175201aab88a2f79bb4e13ca67eff125c;p=aspectj.git

99861 temp fix for deadlock when Sun' Unsafe.defineClass API is used (deadlock in JVMTI obviously, not tied to AJ, but bothers Ron)
---

diff --git a/loadtime/src/org/aspectj/weaver/loadtime/Aj.java b/loadtime/src/org/aspectj/weaver/loadtime/Aj.java
index 0d17dfdec..1c03f1496 100644
--- a/loadtime/src/org/aspectj/weaver/loadtime/Aj.java
+++ b/loadtime/src/org/aspectj/weaver/loadtime/Aj.java
@@ -74,13 +74,15 @@ public class Aj implements ClassPreProcessor {
         private static Map weavingAdaptors = new WeakHashMap();
 
         static WeavingAdaptor getWeaver(ClassLoader loader) {
-            synchronized (weavingAdaptors) {
-                WeavingAdaptor weavingAdaptor = (WeavingAdaptor) weavingAdaptors.get(loader);
-                if (weavingAdaptor == null) {
-                    weavingAdaptor = new ClassLoaderWeavingAdaptor(loader);
-                    weavingAdaptors.put(loader, weavingAdaptor);
+            synchronized(loader) {//FIXME AV - temp fix for #99861
+                synchronized (weavingAdaptors) {
+                    WeavingAdaptor weavingAdaptor = (WeavingAdaptor) weavingAdaptors.get(loader);
+                    if (weavingAdaptor == null) {
+                        weavingAdaptor = new ClassLoaderWeavingAdaptor(loader);
+                        weavingAdaptors.put(loader, weavingAdaptor);
+                    }
+                    return weavingAdaptor;
                 }
-                return weavingAdaptor;
             }
         }
     }