Declare proxyMetadataCache as volatile and ProxiedInterfacesCache fields as final

See gh-30499

Author: jhoeller

spring-aop/src/main/java/org/springframework/aop/framework/AdvisedSupport.java

@@ -117,7 +117,7 @@ public class AdvisedSupport extends ProxyConfig implements Advised {
 	 * @see JdkDynamicAopProxy#JdkDynamicAopProxy(AdvisedSupport)
 	 */
 	@Nullable
-	transient Object proxyMetadataCache;
+	transient volatile Object proxyMetadataCache;
 
 
 	/**

spring-aop/src/main/java/org/springframework/aop/framework/JdkDynamicAopProxy.java

@@ -316,31 +316,41 @@ private void readObject(ObjectInputStream ois) throws IOException, ClassNotFound
 	 */
 	static final class ProxiedInterfacesCache {
 
-		Class<?>[] proxiedInterfaces;
+		final Class<?>[] proxiedInterfaces;
 
-		boolean equalsDefined;
+		final boolean equalsDefined;
 
-		boolean hashCodeDefined;
+		final boolean hashCodeDefined;
 
 		ProxiedInterfacesCache(AdvisedSupport config) {
 			this.proxiedInterfaces = AopProxyUtils.completeProxiedInterfaces(config, true);
 
 			// Find any {@link #equals} or {@link #hashCode} method that may be defined
-			//on the supplied set of interfaces.
+			// on the supplied set of interfaces.
+			boolean equalsDefined = false;
+			boolean hashCodeDefined = false;
 			for (Class<?> proxiedInterface : this.proxiedInterfaces) {
 				Method[] methods = proxiedInterface.getDeclaredMethods();
 				for (Method method : methods) {
 					if (AopUtils.isEqualsMethod(method)) {
-						this.equalsDefined = true;
+						equalsDefined = true;
+						if (hashCodeDefined) {
+							break;
+						}
 					}
 					if (AopUtils.isHashCodeMethod(method)) {
-						this.hashCodeDefined = true;
-					}
-					if (this.equalsDefined && this.hashCodeDefined) {
-						return;
+						hashCodeDefined = true;
+						if (equalsDefined) {
+							break;
+						}
 					}
 				}
+				if (equalsDefined && hashCodeDefined) {
+					break;
+				}
 			}
+			this.equalsDefined = equalsDefined;
+			this.hashCodeDefined = hashCodeDefined;
 		}
 	}