Shared primitive variable "throwExceptionOnMissing" in one thread may

garydgregory · garydgregory · commit 3312ad79b01f · 2025-05-01T00:29:03.000-04:00
not yield the value of the most recent write from another thread
[org.apache.commons.configuration2.AbstractConfiguration] At
AbstractConfiguration.java:[line 1493]
AT_STALE_THREAD_WRITE_OF_PRIMITIVE
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -25,6 +25,7 @@
   <body>
     <release version="2.12.1" date="YYYY-MM-DD" description="Minor release with new features and updated dependencies; requires Java 8 or above.">
        <!-- FIX -->
+       <action type="fix" dev="ggregory" due-to="Gary Gregory, SpotBugs">Shared primitive variable "throwExceptionOnMissing" in one thread may not yield the value of the most recent write from another thread [org.apache.commons.configuration2.AbstractConfiguration] At AbstractConfiguration.java:[line 1493] AT_STALE_THREAD_WRITE_OF_PRIMITIVE.</action>
        <!-- ADD -->
        <!-- UPDATE -->
        <action type="update" dev="ggregory" due-to="Gary Gregory, Dependabot">Bump com.fasterxml.jackson.core:jackson-databind from 2.18.3 to 2.19.0 #565.</action>
diff --git a/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java b/src/main/java/org/apache/commons/configuration2/AbstractConfiguration.java
@@ -190,7 +190,7 @@ private static void throwMissingPropertyException(final String key) {
      * Whether the configuration should throw NoSuchElementExceptions or simply return null when a property does not exist.
      * Defaults to return null.
      */
-    private boolean throwExceptionOnMissing;
+    private volatile boolean throwExceptionOnMissing;
 
     /** Stores a reference to the object that handles variable interpolation. */
     private AtomicReference<ConfigurationInterpolator> interpolator;
