double-checked locking is not thread-safe in Java

found by PMD

Change-Id: Ibd4a9139c626932bec56c0b1dd32b4d59c8440b1

1 files changed, 3 insertions, 7 deletions

diff --git a/scripting/workben/ifc/scripting/ScriptingUtils.java b/scripting/workben/ifc/scripting/ScriptingUtils.java
index ce6071c78dd2..eb317036e2ab 100644
--- a/scripting/workben/ifc/scripting/ScriptingUtils.java
+++ b/scripting/workben/ifc/scripting/ScriptingUtils.java
@@ -34,13 +34,9 @@ public class ScriptingUtils {
     private ScriptingUtils() {
     }
 
-    public static ScriptingUtils getDefault() {
-        if (utils == null) {
-            synchronized (ScriptingUtils.class) {
-                if (utils == null)
-                    utils = new ScriptingUtils();
-            }
-        }
+    public static synchronized ScriptingUtils getDefault() {
+        if (utils == null)
+            utils = new ScriptingUtils();
         return utils;
     }