You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2019/10/23 17:43:42 UTC

[commons-beanutils] branch master updated: [BEANUTILS-402] Double-Checked Locking anti pattern in WeakFastHashMap. (#15)

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-beanutils.git


The following commit(s) were added to refs/heads/master by this push:
     new 0c6b5f1  [BEANUTILS-402] Double-Checked Locking anti pattern in WeakFastHashMap. (#15)
0c6b5f1 is described below

commit 0c6b5f191fb892a223b5fed4257f8fa2e23925ca
Author: Melloware <me...@gmail.com>
AuthorDate: Wed Oct 23 13:43:37 2019 -0400

    [BEANUTILS-402] Double-Checked Locking anti pattern in WeakFastHashMap. (#15)
---
 src/changes/changes.xml                                          | 3 +++
 src/main/java/org/apache/commons/beanutils2/WeakFastHashMap.java | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index 5816f70..99d899c 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -30,6 +30,9 @@
   <body>
 
     <release version="2.0.0" date="2018-MM-DD" description="Update from Java 6 to 7, update from Apache Commons Collection 3 to 4.">
+      <action issue="BEANUTILS-402" dev="melloware" type="update" due-to="Melloware">
+        Double-Checked Locking anti pattern in WeakFastHashMap.
+      </action>
       <action issue="BEANUTILS-504" dev="ggregory" type="update" due-to="Gary Gregory">
         Update from Java 6 to 7.
       </action>
diff --git a/src/main/java/org/apache/commons/beanutils2/WeakFastHashMap.java b/src/main/java/org/apache/commons/beanutils2/WeakFastHashMap.java
index d7f9cf3..5c16de3 100644
--- a/src/main/java/org/apache/commons/beanutils2/WeakFastHashMap.java
+++ b/src/main/java/org/apache/commons/beanutils2/WeakFastHashMap.java
@@ -67,7 +67,7 @@ public class WeakFastHashMap<K, V> extends HashMap<K, V> {
     /**
      * The underlying map we are managing.
      */
-    private Map<K, V> map = null;
+    private volatile Map<K, V> map = null;
 
     /**
      * Are we currently operating in "fast" mode?