From: aclement <aclement>
Date: Sat, 8 Jan 2011 01:17:40 +0000 (+0000)
Subject: 333469
X-Git-Tag: V1_6_11RC1~34
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=75e44f31e461eaca930a2f662eadbbfa3e7c93a2;p=aspectj.git

333469
---

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<String> and the ITD is on MyList - after resolution it may be:
+			// ArrayList<String>.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);