summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAvinashS <avinash.s@huawei.com>2017-09-20 18:18:48 +0530
committerAvinashS <avinash.s@huawei.com>2017-09-20 18:18:48 +0530
commitce2b1e181d2ef074d4f12eeec4342c4020273327 (patch)
tree850a1df52c9bbd8dd685062ae61db78f32621e6c
parenta04f31a2b6aa49cc0646a35bf6f7f7566bb1c331 (diff)
File reader needs close
Potential file stream leak Change-Id: Ib72a7f07bd687c1858b2e06d0d2dcea2271cc519 IssueId: VNFSDK-93 Signed-off-by: AvinashS <avinash.s@huawei.com>
-rw-r--r--csarvalidation/src/main/java/org/onap/validation/csar/CsarValidator.java16
1 files changed, 2 insertions, 14 deletions
diff --git a/csarvalidation/src/main/java/org/onap/validation/csar/CsarValidator.java b/csarvalidation/src/main/java/org/onap/validation/csar/CsarValidator.java
index 7d252e1..891f83e 100644
--- a/csarvalidation/src/main/java/org/onap/validation/csar/CsarValidator.java
+++ b/csarvalidation/src/main/java/org/onap/validation/csar/CsarValidator.java
@@ -100,10 +100,8 @@ public class CsarValidator {
String cfile = csarFiles.get(CommonConstants.CSAR_META);
if (!cfile.isEmpty()) {
File file = new File(cfile);
- BufferedReader reader = null;
- try {
- reader = new BufferedReader(new FileReader(file));
+ try(BufferedReader reader = new BufferedReader(new FileReader(file))) {
String tempString = null;
while ((tempString = reader.readLine()) != null) {
if (!tempString.equals("")) {
@@ -133,17 +131,7 @@ public class CsarValidator {
}
} catch (IOException e2) {
LOG.error("Exception caught while validateCsarMeta ! " + e2.getMessage());
- } finally {
- if (reader != null) {
- try {
- reader.close();
-
- } catch (IOException e1) {
- LOG.error("close reader failed ! " + e1.getMessage());
- }
- }
-
- }
+ }
}
return false;