From 8dccacad79a970bcbb8e61816d3315bdd2181614 Mon Sep 17 00:00:00 2001
From: Francisco Sanchez <francisco.sanchez@bq.com>
Date: Fri, 25 Sep 2015 12:45:46 +0200
Subject: [PATCH] BitmapChecker removes each temporary file after use. Prevent
 hard drive filling up when launched as an internal library.

---
 .../adobe/epubcheck/bitmap/BitmapChecker.java | 50 +++++++++++++------
 1 file changed, 35 insertions(+), 15 deletions(-)

diff --git a/src/main/java/com/adobe/epubcheck/bitmap/BitmapChecker.java b/src/main/java/com/adobe/epubcheck/bitmap/BitmapChecker.java
index 56432a9ca..033e65c04 100755
--- a/src/main/java/com/adobe/epubcheck/bitmap/BitmapChecker.java
+++ b/src/main/java/com/adobe/epubcheck/bitmap/BitmapChecker.java
@@ -105,7 +105,12 @@ public ImageHeuristics getImageSizes(String imgFileName) throws
       tempFile = getImageFile(ocf, imgFileName);
       if (tempFile != null)
       {
-        return new ImageHeuristics(0, 0, tempFile.length());
+          long tempFileLength = tempFile.length();
+          if (ocf.getClass() == OCFZipPackage.class)
+          {
+              tempFile.delete();
+          }
+        return new ImageHeuristics(0, 0, tempFileLength);
       }
       return null;
     }
@@ -129,8 +134,10 @@ public ImageHeuristics getImageSizes(String imgFileName) throws
         }
         if (formatFromSuffix != null && formatFromSuffix.equals(formatFromInputStream)) break;
     }
-    
-    if (formatFromSuffix != null && formatFromSuffix.equals(formatFromInputStream)) {
+
+
+
+      if (formatFromSuffix != null && formatFromSuffix.equals(formatFromInputStream)) {
       // file format and file extension matches; read image file
       
       try {
@@ -155,18 +162,32 @@ public ImageHeuristics getImageSizes(String imgFileName) throws
         report.message(MessageId.PKG_021, EPUBLocation.create(imgFileName));
         return null;
       }
+      finally
+      {
+          if (ocf.getClass() == OCFZipPackage.class)
+          {
+              tempFile.delete();
+          }
+      }
   
-    } else if (formatFromSuffix != null) {
-      // file format and file extension differs
-      
-      report.message(MessageId.PKG_022, EPUBLocation.create(imgFileName), formatFromInputStream, suffix);
-      return null;
-      
-    } else {
-      // file format could not be determined
-      
-      throw new IOException("Not a known image file: " + imgFileName);
-    }
+    } else
+      {
+          if (ocf.getClass() == OCFZipPackage.class)
+          {
+              tempFile.delete();
+          }
+          if (formatFromSuffix != null) {
+              // file format and file extension differs
+
+              report.message(MessageId.PKG_022, EPUBLocation.create(imgFileName), formatFromInputStream, suffix);
+              return null;
+
+          } else {
+              // file format could not be determined
+
+              throw new IOException("Not a known image file: " + imgFileName);
+          }
+      }
   }
 
   private File getImageFile(OCFPackage ocf, String imgFileName) throws IOException
@@ -211,7 +232,6 @@ private File getTempImageFile(OCFZipPackage ocf, String imgFileName) throws IOEx
       String prefix = "img";
 
       file = File.createTempFile(prefix, suffix);
-      file.deleteOnExit();
       os = new FileOutputStream(file);
 
       is = ocf.getInputStream(imgFileName);