Issue #265: Prevent JVM crash when wrong datapath given

Author: lasselindqvist

src/main/java/net/sourceforge/tess4j/ITesseract.java

@@ -136,6 +136,7 @@ default String doOCR(BufferedImage bi, String filename, List<Rectangle> rects) t
      * Sets tessdata path.
      *
      * @param datapath the tessdata path to set
+     * @throws IllegalArgumentException if the given datapath is not an existing directory
      */
     void setDatapath(String datapath);
 

src/main/java/net/sourceforge/tess4j/Tesseract.java

@@ -24,6 +24,7 @@
 import java.nio.ByteBuffer;
 import java.nio.FloatBuffer;
 import java.nio.IntBuffer;
+import java.nio.file.Paths;
 import java.util.*;
 import javax.imageio.IIOImage;
 import javax.imageio.ImageIO;
@@ -87,6 +88,10 @@ public Tesseract() {
                 datapath = "./";
             }
         }
+        File datapathFile = new File(datapath);
+        if (!datapathFile.exists() || !datapathFile.isDirectory()) {
+        	throw new IllegalArgumentException("Given datapath " + datapath + " is not an existing directory.");
+        }
     }
 
     /**
@@ -114,6 +119,10 @@ protected TessBaseAPI getHandle() {
      */
     @Override
     public void setDatapath(String datapath) {
+        File datapathFile = new File(datapath);
+        if (!datapathFile.exists() || !datapathFile.isDirectory()) {
+        	throw new IllegalArgumentException("Given datapath " + datapath + " is not an existing directory.");
+        }
         this.datapath = datapath;
     }
 

src/test/java/net/sourceforge/tess4j/TesseractTest.java

@@ -407,4 +407,11 @@ public void testGetOSD() {
         assertEquals("Latin", result.getScriptName());
         assertTrue(result.getScriptConf() > 0);
     }
+    
+    @Test
+    public void testSetDatapath() {
+        logger.info("testSetDatapath");
+        IllegalArgumentException e = assertThrows(IllegalArgumentException.class, () ->  instance.setDatapath("non-existent-datapath"));
+        assertEquals("Given datapath non-existent-datapath is not an existing directory.", e.getMessage());
+    }
 }