Fixes for Coverity issues of type Resource Leaks - Remaining

1 files changed, 6 insertions, 4 deletions

diff --git a/base/java-tools/src/com/netscape/cmstools/ExtJoiner.java b/base/java-tools/src/com/netscape/cmstools/ExtJoiner.java
index 95c3162f4..5aac69e60 100644
--- a/base/java-tools/src/com/netscape/cmstools/ExtJoiner.java
+++ b/base/java-tools/src/com/netscape/cmstools/ExtJoiner.java
@@ -20,11 +20,11 @@ package com.netscape.cmstools;
 import java.io.FileInputStream;
 import java.io.IOException;
 
-import com.netscape.cmsutil.util.Utils;
-
 import netscape.security.util.DerOutputStream;
 import netscape.security.util.DerValue;
 
+import com.netscape.cmsutil.util.Utils;
+
 /**
  * This program joins a sequence of extensions together
  * so that the final output can be used in configuration
@@ -93,11 +93,13 @@ public class ExtJoiner {
             throws IOException {
         FileInputStream fis = new FileInputStream(fileName);
 
-        byte data[] = new byte[fis.available()];
+        byte data[] = null;
         try {
+            data = new byte[fis.available()];
             fis.read(data);
         } finally {
-            fis.close();
+            if (fis != null)
+                fis.close();
         }
         return Utils.base64decode(new String(data));
     }