Fix some additional file-handle-leaks reported by unit-tests via an enhanced version of the file-leak-detector

Also add a test for Bug 64045

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1872287 13f79535-47bb-0310-9956-ffa450edef68

src/integrationtest/org/apache/poi/TestAllFiles.java

@@ -317,6 +317,7 @@ public class TestAllFiles {
         "spreadsheet/poc-shared-strings.xlsx",  // contains shared-string-entity-expansion
         "document/61612a.docx",
         "document/word2.doc",
+        "spreadsheet/xlsx-corrupted.xlsx",
 
         // old Excel files, which we only support simple text extraction of
         "spreadsheet/testEXCEL_2.xls",

src/ooxml/java/org/apache/poi/xslf/util/EMFHandler.java

@@ -43,8 +43,9 @@ class EMFHandler extends MFProxy {
 
     @Override
     public void parse(File file) throws IOException {
-        // stream needs to be kept open
-        parse(file.toURI().toURL().openStream());
+        // stream needs to be kept open until the instance is closed
+        is = file.toURI().toURL().openStream();
+        parse(is);
     }
 
     @Override
@@ -66,7 +67,6 @@ class EMFHandler extends MFProxy {
 //                }
             }
         }
-
     }
 
     protected String getContentType() {

src/ooxml/testcases/org/apache/poi/xssf/XSSFMemoryLeakTests.java

@@ -17,6 +17,9 @@
 
 package org.apache.poi.xssf;
 
+import org.apache.poi.ooxml.POIXMLException;
+import org.apache.poi.openxml4j.exceptions.InvalidFormatException;
+import org.apache.poi.openxml4j.opc.OPCPackage;
 import org.apache.poi.util.MemoryLeakVerifier;
 import org.apache.poi.xssf.usermodel.XSSFCell;
 import org.apache.poi.xssf.usermodel.XSSFRow;
@@ -27,12 +30,14 @@ import org.junit.Test;
 import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTCell;
 
 import java.io.ByteArrayOutputStream;
+import java.io.File;
 import java.io.IOException;
 import java.io.OutputStream;
 import java.util.ArrayList;
 import java.util.List;
 
 import static org.junit.Assert.assertSame;
+import static org.junit.Assert.fail;
 
 /**
  * A test which uses {@link MemoryLeakVerifier} to ensure that certain
@@ -141,4 +146,24 @@ public class XSSFMemoryLeakTests {
 
         wb1.close();
     }
+
+    @Test(expected = POIXMLException.class)
+    public void testFileLeak() throws IOException, InvalidFormatException {
+        File file = XSSFTestDataSamples.getSampleFile("xlsx-corrupted.xlsx");
+        verifier.addObject(file);
+        try (XSSFWorkbook ignored = new XSSFWorkbook(file)) {
+            fail("Should catch exception as the file is corrupted");
+        }
+    }
+
+    @Test(expected = POIXMLException.class)
+    public void testFileLeak2() throws IOException, InvalidFormatException {
+        File file = XSSFTestDataSamples.getSampleFile("xlsx-corrupted.xlsx");
+        verifier.addObject(file);
+        try (OPCPackage pkg = OPCPackage.open(file)) {
+            try (XSSFWorkbook ignored = new XSSFWorkbook(pkg)) {
+                fail("Should catch exception as the file is corrupted");
+            }
+        }
+    }
 }

src/ooxml/testcases/org/apache/poi/xssf/extractor/TestXSSFBEventBasedExcelExtractor.java

@@ -113,8 +113,8 @@ public class TestXSSFBEventBasedExcelExtractor {
             extractor.setIncludeCellComments(true);
             String[] rows = extractor.getText().split("[\r\n]+");
             assertEquals(283, rows.length);
-            BufferedReader reader = Files.newBufferedReader(XSSFTestDataSamples.getSampleFile("62815.xlsb.txt").toPath(),
-                    StandardCharsets.UTF_8);
+            try (BufferedReader reader = Files.newBufferedReader(XSSFTestDataSamples.getSampleFile("62815.xlsb.txt").toPath(),
+                    StandardCharsets.UTF_8)) {
                 String line = reader.readLine();
                 for (String row : rows) {
                     assertEquals(line, row);
@@ -125,5 +125,5 @@ public class TestXSSFBEventBasedExcelExtractor {
                 }
             }
         }
-
+    }
 }

src/ooxml/testcases/org/apache/poi/xssf/model/TestSharedStringsTable.java

@@ -27,6 +27,7 @@ import java.nio.file.Path;
 import java.util.List;
 import java.util.function.Predicate;
 import java.util.stream.Collectors;
+import java.util.stream.Stream;
 
 import org.apache.poi.ss.usermodel.Sheet;
 import org.apache.poi.ss.usermodel.Workbook;
@@ -233,11 +234,12 @@ public final class TestSharedStringsTable {
 
         Path path = XSSFTestDataSamples.getSampleFile("48936-strings.txt").toPath();
 
-        List<String> lst = Files
-            .lines(path, StandardCharsets.UTF_8)
+        Stream<String> lines = Files.lines(path, StandardCharsets.UTF_8);
+        List<String> lst = lines
             .map(String::trim)
             .filter(((Predicate<String>)String::isEmpty).negate())
             .collect(Collectors.toList());
+        lines.close();
 
         for (String str : lst) {
             s.createRow(i++).createCell(0).setCellValue(str);

src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java

@@ -3455,4 +3455,12 @@ public final class TestXSSFBugs extends BaseTestBugzillaIssues {
             }*/
         }
     }
+
+    @Test(expected = POIXMLException.class)
+    public void test64045() throws IOException, InvalidFormatException {
+        File file = XSSFTestDataSamples.getSampleFile("xlsx-corrupted.xlsx");
+        try (XSSFWorkbook ignored = new XSSFWorkbook(file)) {
+            fail("Should catch exception as the file is corrupted");
+        }
+    }
 }

src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java

@@ -2914,7 +2914,7 @@ public final class TestBugs extends BaseTestBugzillaIssues {
 
     @Test
     public void test46515() throws IOException {
-        Workbook wb = HSSFTestDataSamples.openSampleWorkbook("46515.xls");
+        try (Workbook wb = HSSFTestDataSamples.openSampleWorkbook("46515.xls")) {
 
             // Get structure from webservice
             String urlString = "http://poi.apache.org/components/spreadsheet/images/calendar.jpg";
@@ -2925,8 +2925,6 @@ public final class TestBugs extends BaseTestBugzillaIssues {
             } catch (IOException e) {
                 Assume.assumeNoException("Downloading a jpg from poi.apache.org should work", e);
                 return;
-        } finally {
-            wb.close();
             }
 
             // Convert BufferedImage to byte[]
@@ -2949,8 +2947,7 @@ public final class TestBugs extends BaseTestBugzillaIssues {
 //        FileOutputStream fileOut = new FileOutputStream("/tmp/46515.xls");
 //        wb.write(fileOut);
 //        fileOut.close();
-
-        wb.close();
+        }
     }
 
     @Test