summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMiklos Vajna <vmiklos@suse.cz>2013-07-22 09:10:15 +0200
committerMiklos Vajna <vmiklos@suse.cz>2013-07-22 09:16:50 +0200
commit9f7676033585ab3bf352d5dc2ef43a3a9d8d5c46 (patch)
treebfd1b8892897509620ab2f85f356841769fc89bd
parente77b0fe093ed0bc94c7d03d45838e590fb8219b2 (diff)
writerfilter: make sure the redline stack always has at least one element
One element is pushed to that stack in the constructor, so calling top() on it always OK, as long as we check during pop() that the stack won't be empty. Change-Id: I010bd77897ab2a3684e7d9c5c0450e7bda626fc0
-rw-r--r--writerfilter/source/dmapper/DomainMapper_Impl.cxx5
1 files changed, 5 insertions, 0 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index cc1dfba29550..fb7b8fc44253 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -1635,6 +1635,11 @@ void DomainMapper_Impl::PopFootOrEndnote()
if (!m_aTextAppendStack.empty())
m_aTextAppendStack.pop();
+ if (m_aRedlines.size() == 1)
+ {
+ SAL_WARN("writerfilter", "PopFootOrEndnote() is called without PushFootOrEndnote()?");
+ return;
+ }
m_aRedlines.pop();
}