You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by fr...@apache.org on 2017/01/10 15:27:07 UTC

svn commit: r1778143 - /jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/Manifest.java

Author: frm
Date: Tue Jan 10 15:27:07 2017
New Revision: 1778143

URL: http://svn.apache.org/viewvc?rev=1778143&view=rev
Log:
OAK-5436 - Don't leak file descriptors when reading the manifest file

Modified:
    jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/Manifest.java

Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/Manifest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/Manifest.java?rev=1778143&r1=1778142&r2=1778143&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/Manifest.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/file/Manifest.java Tue Jan 10 15:27:07 2017
@@ -36,7 +36,9 @@ class Manifest {
      */
     static Manifest load(File file) throws IOException {
         Properties properties = new Properties();
-        properties.load(new FileReader(file));
+        try (FileReader r = new FileReader(file)) {
+            properties.load(r);
+        }
         return new Manifest(properties);
     }