From c01a2e897d410efbcbaef36ffd0f56208e276970 Mon Sep 17 00:00:00 2001 From: Phillip Webb Date: Tue, 7 Jun 2022 10:06:50 -0700 Subject: [PATCH] Fix null assignment to non-nullable field Update `BeanRegistrationCodeFragments` package-private constructor to set `codeFragments` to `this` rather than `null`. The field is never actually read by `DefaultBeanRegistrationCodeFragments` since all methods are overridden. Closes gh-28578 --- .../beans/factory/aot/BeanRegistrationCodeFragments.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/spring-beans/src/main/java/org/springframework/beans/factory/aot/BeanRegistrationCodeFragments.java b/spring-beans/src/main/java/org/springframework/beans/factory/aot/BeanRegistrationCodeFragments.java index b409d4d71d8..b252c2949ff 100644 --- a/spring-beans/src/main/java/org/springframework/beans/factory/aot/BeanRegistrationCodeFragments.java +++ b/spring-beans/src/main/java/org/springframework/beans/factory/aot/BeanRegistrationCodeFragments.java @@ -60,10 +60,11 @@ public abstract class BeanRegistrationCodeFragments { /** * Package-private constructor exclusively for - * {@link DefaultBeanRegistrationCodeFragments}. + * {@link DefaultBeanRegistrationCodeFragments}. All methods are overridden + * so {@code this.codeFragments} is never actually used. */ BeanRegistrationCodeFragments() { - this.codeFragments = null; + this.codeFragments = this; } /**