summaryrefslogtreecommitdiff
path: root/qadevOOo
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2016-01-21 14:34:46 +0000
committerCaolán McNamara <caolanm@redhat.com>2016-01-21 15:15:52 +0000
commit683d193327b49971098517758d6d3f79c5d94a4f (patch)
treec4355991d1e2426374dc94d3c9752d85cba8dd58 /qadevOOo
parent2043556e6c9ebe4bfaaa887309a3637493b63caf (diff)
coverity#1326456 Dereference null return value
and coverity#1326455 Dereference null return value Change-Id: I48f18483c82dafed6352434e98d5b49120fdcfc5
Diffstat (limited to 'qadevOOo')
-rw-r--r--qadevOOo/tests/java/ifc/configuration/backend/_XBackend.java32
1 files changed, 13 insertions, 19 deletions
diff --git a/qadevOOo/tests/java/ifc/configuration/backend/_XBackend.java b/qadevOOo/tests/java/ifc/configuration/backend/_XBackend.java
index c55c4579a918..eb8a09cc65a3 100644
--- a/qadevOOo/tests/java/ifc/configuration/backend/_XBackend.java
+++ b/qadevOOo/tests/java/ifc/configuration/backend/_XBackend.java
@@ -118,6 +118,9 @@ public class _XBackend extends MultiMethodTest {
} catch (com.sun.star.container.NoSuchElementException e) {
log.println("unexpected Exception " + e + " -- FAILED");
res = false;
+ } catch (com.sun.star.uno.Exception e) {
+ log.println("unexpected Exception " + e + " -- FAILED");
+ res = false;
}
tRes.tested("getUpdateHandler()", res);
@@ -127,8 +130,7 @@ public class _XBackend extends MultiMethodTest {
boolean res = true;
try {
- XStringSubstitution sts = createStringSubstitution(
- tParam.getMSF());
+ XStringSubstitution sts = createStringSubstitution(tParam.getMSF());
String ent = sts.getSubstituteVariableValue("$(inst)") +
"/share/registry";
XLayer[] Layers = oObj.listLayers(
@@ -147,6 +149,9 @@ public class _XBackend extends MultiMethodTest {
} catch (com.sun.star.container.NoSuchElementException e) {
log.println("unexpected Exception " + e + " -- FAILED");
res = false;
+ } catch (com.sun.star.uno.Exception e) {
+ log.println("unexpected Exception " + e + " -- FAILED");
+ res = false;
}
tRes.tested("listLayers()", res);
@@ -236,21 +241,10 @@ public class _XBackend extends MultiMethodTest {
return res;
}
- public static XStringSubstitution createStringSubstitution(XMultiServiceFactory xMSF) {
- Object xPathSubst = null;
-
- try {
- xPathSubst = xMSF.createInstance(
- "com.sun.star.util.PathSubstitution");
- } catch (com.sun.star.uno.Exception e) {
- e.printStackTrace();
- }
-
- if (xPathSubst != null) {
- return UnoRuntime.queryInterface(
- XStringSubstitution.class, xPathSubst);
- } else {
- return null;
- }
+ public static XStringSubstitution createStringSubstitution(XMultiServiceFactory xMSF) throws com.sun.star.uno.Exception
+ {
+ Object xPathSubst = xMSF.createInstance(
+ "com.sun.star.util.PathSubstitution");
+ return UnoRuntime.queryInterface(XStringSubstitution.class, xPathSubst);
}
-} \ No newline at end of file
+}