runner: Avoid a possible NullPointerException with tCase Object

Change-Id: Ie1202b7fa89306ea2a99031b9347b657039b839e
Reviewed-on: https://gerrit.libreoffice.org/11750
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Tested-by: Michael Stahl <mstahl@redhat.com>

1 files changed, 14 insertions, 21 deletions

diff --git a/qadevOOo/runner/base/java_fat_service.java b/qadevOOo/runner/base/java_fat_service.java
index f9eefaa41dac..27ae90a7368c 100644
--- a/qadevOOo/runner/base/java_fat_service.java
+++ b/qadevOOo/runner/base/java_fat_service.java
@@ -217,32 +217,25 @@ public class java_fat_service implements TestBase {
             log = (LogWriter)dcl.getInstance((String)param.get("LogWriter"));
             param.getMSF();
 
-            TestCase tCase = null;
+            TestEnvironment tEnv = null;
 
-            try
-            {
-                tCase = (TestCase) dcl.getInstance("mod._"+entry.entryName);
-            }
-            catch (java.lang.IllegalArgumentException ie)
-            {
+            try {
+                TestCase tCase = (TestCase) dcl.getInstance("mod._"+entry.entryName);
+                log.println("Creating: " + tCase.getObjectName());
+                log.initialize(entry, true);
+                entry.UserDefinedParams = param;
+                tCase.setLogWriter((PrintWriter) log);
+                try {
+                    tCase.initializeTestCase(param);
+                    tEnv = tCase.getTestEnvironment(param);
+                } catch (com.sun.star.lang.DisposedException de) {
+                    log.println("Office disposed");
+                }
+            } catch (java.lang.IllegalArgumentException ie) {
                 entry.ErrorMsg=ie.getMessage();
                 entry.hasErrorMsg=true;
             }
 
-            log.println("Creating: "+tCase.getObjectName());
-            log.initialize(entry,true);
-            entry.UserDefinedParams = param;
-            tCase.setLogWriter((PrintWriter) log);
-            TestEnvironment tEnv = null;
-            try
-            {
-                tCase.initializeTestCase(param);
-                tEnv = tCase.getTestEnvironment(param);
-            }
-            catch (com.sun.star.lang.DisposedException de)
-            {
-                log.println("Office disposed");
-            }
             return tEnv;
     }