[jboss-jira] [JBoss JIRA] Commented: (JBAOP-618) AspectManager.getTempAdvisorIfNotExist causes infinite loop

AspectManager.getTempAdvisorIfNotExist causes infinite loop ----------------------------------------------------------- Key: JBAOP-618 URL: https://jira.jboss.org/jira/browse/JBAOP-618 Project: JBoss AOP Issue Type: Bug Security Level: Public(Everyone can see) Affects Versions: 2.0.0.CR15 Reporter: Flavia Rainone Assignee: Flavia Rainone Fix For: 2.0.0.GA This method contains a loop (lines 1207-1227) while (superClass != null) { try { Field field = superClass.getDeclaredField(Instrumentor.HELPER_FIELD_NAME); SecurityActions.setAccessible(field); advisor = (ClassAdvisor) field.get(null); if (advisor != null) { return advisor; } else { break; } } catch (NoSuchFieldException e) { superClass = clazz.getSuperclass(); } } With an infinite loop. Before entering this loop, superClazz value is assigned to be equal to clazz. If clazz is not advised, we get a NSFE, and go to clazz superclass. But, if this class is not advised, we will enter infinite loop, as we will check clazz's superclass again and again and again.