summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjan iversen <jani@documentfoundation.org>2016-01-23 18:38:24 +0100
committerjan iversen <jani@documentfoundation.org>2016-01-29 10:05:52 +0000
commit4b368564091a0023e1981bd8346716db2474b068 (patch)
tree662248575f1490c6c75bc891a9c9bf0db05929c8
parent3c28a0e3864edc2812a02dec2f80f09d6cf2bc73 (diff)
cid#1326176, 1326270, 1326271, 1326272, 1326273, 1326274, 1326275
null pointer used to dereference, add "if" Change-Id: I37173f252a3b3da44976581ccefd0ed54aa8aa0d Reviewed-on: https://gerrit.libreoffice.org/21735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
-rw-r--r--qadevOOo/tests/java/ifc/frame/_XSynchronousFrameLoader.java2
-rw-r--r--xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java6
2 files changed, 7 insertions, 1 deletions
diff --git a/qadevOOo/tests/java/ifc/frame/_XSynchronousFrameLoader.java b/qadevOOo/tests/java/ifc/frame/_XSynchronousFrameLoader.java
index 322138d11215..faa0274e4d98 100644
--- a/qadevOOo/tests/java/ifc/frame/_XSynchronousFrameLoader.java
+++ b/qadevOOo/tests/java/ifc/frame/_XSynchronousFrameLoader.java
@@ -110,6 +110,8 @@ public class _XSynchronousFrameLoader extends MultiMethodTest {
URL[] urlS = new URL[1];
urlS[0] = new URL();
urlS[0].Complete = url;
+ if (xURLTrans == null)
+ throw new IllegalStateException("xURLTrans unexpected null");
boolean res = xURLTrans.parseStrict(urlS);
log.println("Parsing URL '" + url + "': " + res);
descr = new PropertyValue[1] ;
diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java
index b549743fe308..de377ec43f37 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java
@@ -153,6 +153,10 @@ public class ConverterInfo {
String version, String vendor, String impl)
throws RegistryException {
+ if (officeMime == null || displayName == null || description == null ||
+ version == null || vendor == null || impl == null)
+ throw new IllegalArgumentException("arguments unexpected null");
+
if (!isValidOfficeType(officeMime.trim())) {
RegistryException re = new RegistryException(
"Invalid office type");
@@ -387,4 +391,4 @@ public class ConverterInfo {
public String getXsltDeserial() {
return piXsltDeserial;
}
-} \ No newline at end of file
+}