On Apr 18, 2008, at 12:33 AM, David Blevins wrote:

This looks pretty serious. How is it that we don't get stuck in this look all the time?

Ya. Was a bit concerned about that, myself. getMethods() is only called when getMethod() is unable to resolve an interceptor to an actual method. So, it's an exceptional condition and requires an error in an ejb-jar.xml to reproduce.

--kevan



-David

On Apr 17, 2008, at 10:52 AM, [EMAIL PROTECTED] wrote:

Author: kevan
Date: Thu Apr 17 10:52:30 2008
New Revision: 649205

URL: http://svn.apache.org/viewvc?rev=649205&view=rev
Log:
OPENEJB-793 fix infinite loop CheckCallbacks.getMethods(Class,String)

Modified:
openejb/trunk/openejb3/container/openejb-core/src/main/java/org/ apache/openejb/config/rules/CheckCallbacks.java

Modified: openejb/trunk/openejb3/container/openejb-core/src/main/ java/org/apache/openejb/config/rules/CheckCallbacks.java
URL: 
http://svn.apache.org/viewvc/openejb/trunk/openejb3/container/openejb-core/src/main/java/org/apache/openejb/config/rules/CheckCallbacks.java?rev=649205&r1=649204&r2=649205&view=diff
= = = = = = = = = ===================================================================== --- openejb/trunk/openejb3/container/openejb-core/src/main/java/org/ apache/openejb/config/rules/CheckCallbacks.java (original) +++ openejb/trunk/openejb3/container/openejb-core/src/main/java/org/ apache/openejb/config/rules/CheckCallbacks.java Thu Apr 17 10:52:30 2008
@@ -230,6 +230,7 @@
                   methods.add(method);
               }
           }
+            clazz = clazz.getSuperclass();
       }
       return methods;
   }





Reply via email to