From: Felix Dörre <felix@dogcraft.de>
Date: Thu, 19 Feb 2015 20:38:11 +0000 (+0100)
Subject: FIX: coverity resource leaks.
X-Git-Url: https://code.wpia.club/?p=gigi.git;a=commitdiff_plain;h=14ce2292d15b5d43a285804991f9b7d94207e274

FIX: coverity resource leaks.
---

diff --git a/src/org/cacert/gigi/DevelLauncher.java b/src/org/cacert/gigi/DevelLauncher.java
index 6a462f21..5d168672 100644
--- a/src/org/cacert/gigi/DevelLauncher.java
+++ b/src/org/cacert/gigi/DevelLauncher.java
@@ -32,7 +32,9 @@ public class DevelLauncher {
 
     public static void main(String[] args) throws Exception {
         Properties mainProps = new Properties();
-        mainProps.load(new FileInputStream("config/gigi.properties"));
+        try (FileInputStream inStream = new FileInputStream("config/gigi.properties")) {
+            mainProps.load(inStream);
+        }
         for (int i = 0; i < args.length; i++) {
             if (args[i].equals("--port")) {
                 mainProps.setProperty("port", args[i + 1]);
diff --git a/src/org/cacert/gigi/database/DatabaseConnection.java b/src/org/cacert/gigi/database/DatabaseConnection.java
index 49658fb7..ef9b19cc 100644
--- a/src/org/cacert/gigi/database/DatabaseConnection.java
+++ b/src/org/cacert/gigi/database/DatabaseConnection.java
@@ -120,11 +120,12 @@ public class DatabaseConnection {
         try {
             Statement s = getInstance().c.createStatement();
             while (version < CURRENT_SCHEMA_VERSION) {
-                InputStream resourceAsStream = DatabaseConnection.class.getResourceAsStream("upgrade/from_" + version + ".sql");
-                if (resourceAsStream == null) {
-                    throw new Error("Upgrade script from version " + version + " was not found.");
+                try (InputStream resourceAsStream = DatabaseConnection.class.getResourceAsStream("upgrade/from_" + version + ".sql")) {
+                    if (resourceAsStream == null) {
+                        throw new Error("Upgrade script from version " + version + " was not found.");
+                    }
+                    SQLFileManager.addFile(s, resourceAsStream, ImportType.PRODUCTION);
                 }
-                SQLFileManager.addFile(s, resourceAsStream, ImportType.PRODUCTION);
                 version++;
             }
             s.addBatch("INSERT INTO schemeVersion SET version='" + version + "'");
diff --git a/src/org/cacert/gigi/dbObjects/Domain.java b/src/org/cacert/gigi/dbObjects/Domain.java
index 8efa4ee1..9df2e482 100644
--- a/src/org/cacert/gigi/dbObjects/Domain.java
+++ b/src/org/cacert/gigi/dbObjects/Domain.java
@@ -1,6 +1,7 @@
 package org.cacert.gigi.dbObjects;
 
 import java.io.IOException;
+import java.io.InputStream;
 import java.net.IDN;
 import java.util.Arrays;
 import java.util.Collections;
@@ -70,8 +71,8 @@ public class Domain implements IdCachable {
     private static final Set<String> IDNEnabledTLDs;
     static {
         Properties CPS = new Properties();
-        try {
-            CPS.load(Domain.class.getResourceAsStream("CPS.properties"));
+        try (InputStream resourceAsStream = Domain.class.getResourceAsStream("CPS.properties")) {
+            CPS.load(resourceAsStream);
             IDNEnabledTLDs = Collections.unmodifiableSet(new HashSet<>(Arrays.asList(CPS.getProperty("IDN-enabled").split(","))));
         } catch (IOException e) {
             throw new Error(e);
diff --git a/src/org/cacert/gigi/util/DNSUtil.java b/src/org/cacert/gigi/util/DNSUtil.java
index 816b2473..7ebbf83d 100644
--- a/src/org/cacert/gigi/util/DNSUtil.java
+++ b/src/org/cacert/gigi/util/DNSUtil.java
@@ -40,6 +40,7 @@ public class DNSUtil {
         Attributes dnsLookup = context.getAttributes(name, new String[] {
             "TXT"
         });
+        context.close();
 
         return extractTextEntries(dnsLookup.get("TXT"));
     }