Browse Source

Refine null-safety in the spring-aop module

See gh-34154
pull/34171/head
Sébastien Deleuze 12 months ago
parent
commit
4fa33dfece
  1. 6
      spring-aop/src/main/java/org/springframework/aop/aspectj/AbstractAspectJAdvice.java
  2. 2
      spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/BeanFactoryAspectJAdvisorsBuilder.java

6
spring-aop/src/main/java/org/springframework/aop/aspectj/AbstractAspectJAdvice.java

@ -570,7 +570,7 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence @@ -570,7 +570,7 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence
for (PointcutParameter parameter : parameterBindings) {
String name = parameter.getName();
Integer index = this.argumentBindings.get(name);
Assert.notNull(index, "Index must not be null");
Assert.state(index != null, "Index must not be null");
adviceInvocationArgs[index] = parameter.getBinding();
numBound++;
}
@ -578,14 +578,14 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence @@ -578,14 +578,14 @@ public abstract class AbstractAspectJAdvice implements Advice, AspectJPrecedence
// binding from returning clause
if (this.returningName != null) {
Integer index = this.argumentBindings.get(this.returningName);
Assert.notNull(index, "Index must not be null");
Assert.state(index != null, "Index must not be null");
adviceInvocationArgs[index] = returnValue;
numBound++;
}
// binding from thrown exception
if (this.throwingName != null) {
Integer index = this.argumentBindings.get(this.throwingName);
Assert.notNull(index, "Index must not be null");
Assert.state(index != null, "Index must not be null");
adviceInvocationArgs[index] = ex;
numBound++;
}

2
spring-aop/src/main/java/org/springframework/aop/aspectj/annotation/BeanFactoryAspectJAdvisorsBuilder.java

@ -157,7 +157,7 @@ public class BeanFactoryAspectJAdvisorsBuilder { @@ -157,7 +157,7 @@ public class BeanFactoryAspectJAdvisorsBuilder {
}
else {
MetadataAwareAspectInstanceFactory factory = this.aspectFactoryCache.get(aspectName);
Assert.notNull(factory, "Factory must not be null");
Assert.state(factory != null, "Factory must not be null");
advisors.addAll(this.advisorFactory.getAdvisors(factory));
}
}

Loading…
Cancel
Save