From 711f203d1eeaeb0c2c75569c8606e98c394fd301 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <caolanm@redhat.com>
Date: Sat, 21 Jan 2017 17:37:42 +0000
Subject: coverity#1399444 UwF: Unwritten field
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

and

coverity#1399441 NP: Null pointer dereference

since

commit 10cb59eb914ba722c203242272de244d795a51e8
Date:   Mon Jan 16 16:41:53 2017 +0200

    remove some unused Java code

Change-Id: Ieab0325c8685cdc89458570ac8e4d3292af84dff
Reviewed-on: https://gerrit.libreoffice.org/33377
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
---
 qadevOOo/runner/helper/ProcessHandler.java | 42 +++++++-----------------------
 1 file changed, 9 insertions(+), 33 deletions(-)

(limited to 'qadevOOo/runner')

diff --git a/qadevOOo/runner/helper/ProcessHandler.java b/qadevOOo/runner/helper/ProcessHandler.java
index 7ec20a570420..1e136d97095e 100644
--- a/qadevOOo/runner/helper/ProcessHandler.java
+++ b/qadevOOo/runner/helper/ProcessHandler.java
@@ -109,9 +109,7 @@ class Pump extends Thread
  */
 public class ProcessHandler
 {
-
     private String cmdLine;
-    private String[] cmdLineArray;
     private String[] envVars = null;
     private File workDir = null;
     private PrintWriter log;
@@ -254,47 +252,25 @@ public class ProcessHandler
         final Runtime runtime = Runtime.getRuntime();
         try
         {
-            if (cmdLine == null)
+            if (workDir != null)
             {
-                log.println(utils.getDateTime() + "execute: Starting command from array: ");
-                for (int i = 0; i < cmdLineArray.length; i++)
-                {
-                    log.println(cmdLineArray[i]);
-                }
+                log.println(utils.getDateTime() + "execute: Starting command: ");
+                log.println(cmdLine + " path=" + workDir.getAbsolutePath());
                 showEnvVars();
-                log.println("");
-                initializeProcessKiller();
-                m_aProcess = runtime.exec(cmdLineArray, envVars);
+                m_aProcess = runtime.exec(cmdLine, envVars, workDir);
             }
             else
             {
-                if (workDir != null)
-                {
-                    log.println(utils.getDateTime() + "execute: Starting command: ");
-                    log.println(cmdLine + " path=" + workDir.getAbsolutePath());
-                    showEnvVars();
-                    m_aProcess = runtime.exec(cmdLine, envVars, workDir);
-                }
-                else
-                {
-                    log.println(utils.getDateTime() + "execute: Starting command: ");
-                    log.println(cmdLine);
-                    showEnvVars();
-                    m_aProcess = runtime.exec(cmdLine, envVars);
-                }
+                log.println(utils.getDateTime() + "execute: Starting command: ");
+                log.println(cmdLine);
+                showEnvVars();
+                m_aProcess = runtime.exec(cmdLine, envVars);
             }
             isStarted = true;
         }
         catch (java.io.IOException e)
         {
-            if (cmdLine == null)
-            {
-                log.println(utils.getDateTime() + "execute: The command array can't be started: " + e);
-            }
-            else
-            {
-                log.println(utils.getDateTime() + "execute: The command " + cmdLine + " can't be started: " + e);
-            }
+            log.println(utils.getDateTime() + "execute: The command " + cmdLine + " can't be started: " + e);
             return;
         }
         dbg("execute: pump io-streams");
-- 
cgit