From 75e44f31e461eaca930a2f662eadbbfa3e7c93a2 Mon Sep 17 00:00:00 2001 From: aclement Date: Sat, 8 Jan 2011 01:17:40 +0000 Subject: [PATCH] 333469 --- .../org/aspectj/weaver/CrosscuttingMembers.java | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/org.aspectj.matcher/src/org/aspectj/weaver/CrosscuttingMembers.java b/org.aspectj.matcher/src/org/aspectj/weaver/CrosscuttingMembers.java index 0b514d6ca..eb4b4444e 100644 --- a/org.aspectj.matcher/src/org/aspectj/weaver/CrosscuttingMembers.java +++ b/org.aspectj.matcher/src/org/aspectj/weaver/CrosscuttingMembers.java @@ -209,9 +209,25 @@ public class CrosscuttingMembers { // Looking it up ensures we get the annotations - the accessedMembers are just retrieved from the attribute and // don't have that information ResolvedMember resolvedMember = world.resolve(member); + + // pr333469 + // If the member is for an ITD (e.g. serialVersionUID) then during resolution we may resolve it on + // a supertype because it doesn't yet exist on the target. + // For example: MyList extends ArrayList and the ITD is on MyList - after resolution it may be: + // ArrayList.serialVersionUID, we need to avoid that happening + if (resolvedMember == null) { // can happen for ITDs - are there many privileged access ITDs?? resolvedMember = member; + if (resolvedMember.hasBackingGenericMember()) { + resolvedMember = resolvedMember.getBackingGenericMember(); + } + } else { + UnresolvedType unresolvedDeclaringType = member.getDeclaringType().getRawType(); + UnresolvedType resolvedDeclaringType = resolvedMember.getDeclaringType().getRawType(); + if (!unresolvedDeclaringType.equals(resolvedDeclaringType)) { + resolvedMember = member; + } } PrivilegedAccessMunger privilegedAccessMunger = new PrivilegedAccessMunger(resolvedMember, version >= WeaverVersionInfo.WEAVER_VERSION_AJ169); -- 2.39.5