diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2016-01-15 10:23:01 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2016-01-15 20:24:51 +0000 |
| commit | d86963b5e84eebf102676efeb718b5bd05515f31 (patch) | |
| tree | cc06fa7a7b3e1d41228805119060fa5e80fa9aa3 /qadevOOo/tests | |
| parent | ba05bd50ffe87550e40c39b0ebd0443692f876a9 (diff) | |
coverity#1326294 Dereference after null check
Change-Id: I057b9ef27bd84ac9875fa6ce8321ff03e0f879e6
Diffstat (limited to 'qadevOOo/tests')
| -rw-r--r-- | qadevOOo/tests/java/ifc/configuration/backend/_XSchemaSupplier.java | 6 | ||||
| -rw-r--r-- | qadevOOo/tests/java/ifc/configuration/backend/_XSingleLayerStratum.java | 6 |
2 files changed, 5 insertions, 7 deletions
diff --git a/qadevOOo/tests/java/ifc/configuration/backend/_XSchemaSupplier.java b/qadevOOo/tests/java/ifc/configuration/backend/_XSchemaSupplier.java index d2738772c38f..aee858b6a647 100644 --- a/qadevOOo/tests/java/ifc/configuration/backend/_XSchemaSupplier.java +++ b/qadevOOo/tests/java/ifc/configuration/backend/_XSchemaSupplier.java @@ -36,9 +36,9 @@ public class _XSchemaSupplier extends MultiMethodTest { if (aSchema == null) { log.println("\treturned Layer is NULL -- FAILED"); + } else { + res &= checkSchema(aSchema); } - - res &= checkSchema(aSchema); } catch (com.sun.star.configuration.backend.BackendAccessException e) { log.println("Unexpected Exception (" + e + ") -- FAILED"); res &= false; @@ -106,4 +106,4 @@ public class _XSchemaSupplier extends MultiMethodTest { return res; } -}
\ No newline at end of file +} diff --git a/qadevOOo/tests/java/ifc/configuration/backend/_XSingleLayerStratum.java b/qadevOOo/tests/java/ifc/configuration/backend/_XSingleLayerStratum.java index 3ead3c9b0b5e..33eb0cfa8962 100644 --- a/qadevOOo/tests/java/ifc/configuration/backend/_XSingleLayerStratum.java +++ b/qadevOOo/tests/java/ifc/configuration/backend/_XSingleLayerStratum.java @@ -50,8 +50,7 @@ public class _XSingleLayerStratum extends MultiMethodTest { if (aLayer == null) { log.println("\treturned Layer is NULL -- FAILED"); - } - else { + } else { res &= checkLayer(aLayer); } } catch (com.sun.star.configuration.backend.BackendAccessException e) { @@ -89,8 +88,7 @@ public class _XSingleLayerStratum extends MultiMethodTest { if (aLayer == null) { log.println("\treturned Layer is NULL -- FAILED"); - } - else { + } else { res &= checkLayer(aLayer); } } catch (com.sun.star.configuration.backend.BackendAccessException e) { |