summaryrefslogtreecommitdiffstats
path: root/openecomp-be/lib/openecomp-core-lib
diff options
context:
space:
mode:
authorsiddharth0905 <siddharth.singh4@amdocs.com>2018-08-13 22:09:54 +0530
committerVitaly Emporopulo <Vitaliy.Emporopulo@amdocs.com>2018-08-16 19:27:58 +0000
commit054dea4806c2d9a37b4f3cf72510dfbe6d47c6f9 (patch)
tree6006e56902efa0dd28ab96f53037f276f68205b7 /openecomp-be/lib/openecomp-core-lib
parent3a9c18a098c3f5c519fb0dbed36384af6ccaf151 (diff)
Fortify SDC issue
Fortify SDC issue Change-Id: I7acf3420080ed4c7045aeda1cb1ff0d31d9b5228 Issue-ID: SDC-1644 Signed-off-by: siddharth0905 <siddharth.singh4@amdocs.com>
Diffstat (limited to 'openecomp-be/lib/openecomp-core-lib')
-rw-r--r--openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-api/src/main/java/org/openecomp/core/factory/impl/AbstractFactoryBase.java30
1 files changed, 15 insertions, 15 deletions
diff --git a/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-api/src/main/java/org/openecomp/core/factory/impl/AbstractFactoryBase.java b/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-api/src/main/java/org/openecomp/core/factory/impl/AbstractFactoryBase.java
index cff9da0d5c..9d25a09758 100644
--- a/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-api/src/main/java/org/openecomp/core/factory/impl/AbstractFactoryBase.java
+++ b/openecomp-be/lib/openecomp-core-lib/openecomp-facade-lib/openecomp-facade-api/src/main/java/org/openecomp/core/factory/impl/AbstractFactoryBase.java
@@ -16,21 +16,20 @@
package org.openecomp.core.factory.impl;
-
-import org.apache.commons.lang3.StringUtils;
-import org.openecomp.sdc.common.errors.CoreException;
-import org.openecomp.sdc.common.errors.ErrorCategory;
-import org.openecomp.sdc.common.errors.ErrorCode;
+import static org.openecomp.core.utilities.CommonMethods.newInstance;
import java.util.Collection;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
-import static org.openecomp.core.utilities.CommonMethods.newInstance;
+import org.apache.commons.lang3.StringUtils;
+import org.openecomp.sdc.common.errors.CoreException;
+import org.openecomp.sdc.common.errors.ErrorCategory;
+import org.openecomp.sdc.common.errors.ErrorCode;
public abstract class AbstractFactoryBase {
- /**
+ /**
* Temporary registry of default implementations. The map keeps class names rather then class
* types to allow unloading of those classes from memory by garbage collector if factory is not
* actually used.
@@ -41,8 +40,9 @@ public abstract class AbstractFactoryBase {
* Cached factory instances.
*/
private static final Map<String, AbstractFactoryBase> FACTORY_MAP = new ConcurrentHashMap<>();
+ public static final String E0001 = "E0001";
- /**
+ /**
* Registers implementor for an abstract factory. The method accepts Java classes rather then
* class names to ensure type safety at compilation time.
*
@@ -55,13 +55,13 @@ public abstract class AbstractFactoryBase {
Class<? extends F> impl) {
if (factory == null) {
throw new CoreException(
- new ErrorCode.ErrorCodeBuilder().withId("E0001").withMessage("Mandatory input factory.")
+ new ErrorCode.ErrorCodeBuilder().withId(E0001).withMessage("Mandatory input factory.")
.withCategory(ErrorCategory.SYSTEM).build());
}
if (impl == null) {
throw new CoreException(
- new ErrorCode.ErrorCodeBuilder().withId("E0001").withMessage("Mandatory input impl.")
+ new ErrorCode.ErrorCodeBuilder().withId(E0001).withMessage("Mandatory input impl.")
.withCategory(ErrorCategory.SYSTEM).build());
}
if (FACTORY_MAP.containsKey(factory.getName())) {
@@ -84,7 +84,7 @@ public abstract class AbstractFactoryBase {
public static <F extends AbstractFactoryBase> void unregisterFactory(Class<F> factory) {
if (factory == null) {
throw new CoreException(
- new ErrorCode.ErrorCodeBuilder().withId("E0001").withMessage("Mandatory input factory.")
+ new ErrorCode.ErrorCodeBuilder().withId(E0001).withMessage("Mandatory input factory.")
.withCategory(ErrorCategory.SYSTEM).build());
}
@@ -103,7 +103,7 @@ public abstract class AbstractFactoryBase {
public static <I, F extends AbstractFactoryBase> F getInstance(Class<F> factoryType) {
if (factoryType == null) {
throw new CoreException(
- new ErrorCode.ErrorCodeBuilder().withId("E0001")
+ new ErrorCode.ErrorCodeBuilder().withId(E0001)
.withMessage("Mandatory input factory type.").withCategory(ErrorCategory.SYSTEM)
.build());
@@ -113,7 +113,7 @@ public abstract class AbstractFactoryBase {
// Check for the first time access
if (factory == null) {
// Synchronize factory instantiation
- synchronized (factoryType) {
+ synchronized (FACTORY_MAP) {
// Re-check the factory instance
factory = (F) FACTORY_MAP.get(factoryType.getName());
if (factory == null) {
@@ -122,7 +122,7 @@ public abstract class AbstractFactoryBase {
if (StringUtils.isEmpty(implName)) {
throw new CoreException(
- new ErrorCode.ErrorCodeBuilder().withId("E0001")
+ new ErrorCode.ErrorCodeBuilder().withId(E0001)
.withMessage("Mandatory input factory implementation.")
.withCategory(ErrorCategory.SYSTEM).build());
}
@@ -153,7 +153,7 @@ public abstract class AbstractFactoryBase {
boolean isFactoryRegistered = false;
if (factoryType == null) {
throw new CoreException(
- new ErrorCode.ErrorCodeBuilder().withId("E0001")
+ new ErrorCode.ErrorCodeBuilder().withId(E0001)
.withMessage("Mandatory input factory type.").withCategory(ErrorCategory.SYSTEM)
.build());
}