summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2013-06-06 11:41:03 +0100
committerMiklos Vajna <vmiklos@suse.cz>2013-06-06 19:20:31 +0200
commit81b23a2aad0ab1a72cc292f1b3fca663363c4fee (patch)
tree39c02e25414e4548ecb5ed0855965b0ddc7d70b6
parentecde9ba00dcf8c1bf3413afd0281236e9429148f (diff)
add some checks against NULL derefs
Change-Id: I85fbf279991ee361ea081d92d9acc84d36677af1 (cherry picked from commit fa671cecd4bb2f38a0a9171cfdbb56efddda1cfa) Signed-off-by: Miklos Vajna <vmiklos@suse.cz>
-rw-r--r--writerfilter/source/ooxml/OOXMLFastContextHandler.cxx3
-rw-r--r--writerfilter/source/ooxml/OOXMLParserState.cxx3
2 files changed, 6 insertions, 0 deletions
diff --git a/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx b/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx
index f4a7924432ea..205f82152dcf 100644
--- a/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx
+++ b/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx
@@ -1335,6 +1335,9 @@ void OOXMLFastContextHandlerProperties::lcl_endFastElement
{
if (isForwardEvents())
{
+ SAL_WARN_IF(!mpStream, "writerfilter", "no stream to set properties on");
+ if (!mpStream)
+ return;
mpStream->props(mpPropertySet);
}
}
diff --git a/writerfilter/source/ooxml/OOXMLParserState.cxx b/writerfilter/source/ooxml/OOXMLParserState.cxx
index c2dc79c3e847..49423756784f 100644
--- a/writerfilter/source/ooxml/OOXMLParserState.cxx
+++ b/writerfilter/source/ooxml/OOXMLParserState.cxx
@@ -125,6 +125,9 @@ OOXMLDocument * OOXMLParserState::getDocument() const
void OOXMLParserState::setXNoteId(const sal_Int32 nId)
{
+ SAL_WARN_IF(!mpDocument, "writerfilter", "no document to set note id on");
+ if (!mpDocument)
+ return;
mpDocument->setXNoteId(nId);
}