[refactor] Revise mechanism to execute external process

Author: dizzzz

extensions/exiftool/src/main/java/org/exist/exiftool/xquery/MetadataFunctions.java

@@ -154,7 +154,8 @@ private Sequence extractMetadataFromWebResource(String uri) throws XPathExceptio
     private Sequence exifToolExtract(final Path binaryFile) throws XPathException {
         final ExiftoolModule module = (ExiftoolModule) getParentModule();
         try {
-            final Process p = Runtime.getRuntime().exec(module.getPerlPath() + " " + module.getExiftoolPath() + " -X -struct " + binaryFile.toAbsolutePath());
+            final String[] command = new String[]{module.getPerlPath(), module.getExiftoolPath(), "-X", "-struct", binaryFile.toAbsolutePath().toString()};
+            final Process p = Runtime.getRuntime().exec(command);
             try(final InputStream stdIn = p.getInputStream();
                     final UnsynchronizedByteArrayOutputStream baos =  UnsynchronizedByteArrayOutputStream.builder().get()) {
 
@@ -176,7 +177,8 @@ private Sequence exifToolExtract(final Path binaryFile) throws XPathException {
     private Sequence exifToolWebExtract(final URI uri) throws XPathException {
         final ExiftoolModule module = (ExiftoolModule) getParentModule();
         try {
-            final Process p = Runtime.getRuntime().exec(module.getExiftoolPath()+" -fast -X -");
+            final String[] command = new String[]{module.getExiftoolPath(), "-fast", "-X", "-"};
+            final Process p = Runtime.getRuntime().exec(command);
 
             try(final InputStream stdIn = p.getInputStream();
                     final UnsynchronizedByteArrayOutputStream baos = UnsynchronizedByteArrayOutputStream.builder().get()) {