Fix sonar blocker issues in ccsdk/core module

Sonarqube report for CCSDK core identfied some blcoker issues. This fix addresses those bugs.

Issue-Id: CCSDK-84

Change-Id: Ie6ff19b7b0815a129f9809243e8e3eb0a64144ab
Signed-off-by: Arun S. Yerra <arun.yerra@huawei.com>

1 files changed, 18 insertions, 4 deletions

diff --git a/sli/common/src/main/java/org/onap/ccsdk/sli/core/sli/SvcLogicDblibStore.java b/sli/common/src/main/java/org/onap/ccsdk/sli/core/sli/SvcLogicDblibStore.java
index f11b362c..9216519a 100644
--- a/sli/common/src/main/java/org/onap/ccsdk/sli/core/sli/SvcLogicDblibStore.java
+++ b/sli/common/src/main/java/org/onap/ccsdk/sli/core/sli/SvcLogicDblibStore.java
@@ -132,7 +132,7 @@ public class SvcLogicDblibStore implements SvcLogicStore {
 			String mode) throws SvcLogicException {
 
 		DbLibService dbSvc = getDbLibService();
-
+		PreparedStatement fetchGraphStmt = null;
 		Connection dbConn = null;
 		SvcLogicGraph retval = null;
 		ResultSet results = null;
@@ -146,7 +146,7 @@ public class SvcLogicDblibStore implements SvcLogicStore {
 
 		try {
 			dbConn = ((DBResourceManager) dbSvc).getConnection();
-			PreparedStatement fetchGraphStmt;
+
 
 			ArrayList<String> args = new ArrayList<String>();
 			args.add(module);
@@ -193,6 +193,13 @@ public class SvcLogicDblibStore implements SvcLogicStore {
 		} catch (Exception e) {
 			throw new ConfigurationException("Graph processing failed", e);
 		} finally {
+			try {
+				if (fetchGraphStmt != null) {
+					fetchGraphStmt.close();
+				}
+			} catch (SQLException e) {
+				LOG.info(e.getMessage());
+			}
 			if (results != null) {
 				try {
 					results.close();
@@ -267,12 +274,12 @@ public class SvcLogicDblibStore implements SvcLogicStore {
 		}
 
 		Connection dbConn = null;
-
+		PreparedStatement storeGraphStmt = null;
 		try {
 			dbConn = ((DBResourceManager) dbSvc).getConnection();
 			boolean oldAutoCommit = dbConn.getAutoCommit();
 			dbConn.setAutoCommit(false);
-			PreparedStatement storeGraphStmt = dbConn
+			storeGraphStmt = dbConn
 					.prepareStatement(storeGraphSql);
 			storeGraphStmt.setString(1, graph.getModule());
 			storeGraphStmt.setString(2, graph.getRpc());
@@ -289,6 +296,13 @@ public class SvcLogicDblibStore implements SvcLogicStore {
 			throw new SvcLogicException("Could not write object to database", e);
 		} finally {
 			try {
+				if (storeGraphStmt != null) {
+					storeGraphStmt.close();
+				}
+			} catch (SQLException e) {
+				LOG.info(e.getMessage());
+			}
+			try {
 				if (dbConn != null && !dbConn.isClosed()) {
 					dbConn.close();
 				}