diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2019-11-27 15:42:10 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2019-11-27 21:44:46 +0100 |
| commit | 6487eb6298ce8aafaa88b773538af71bcf63d44d (patch) | |
| tree | d1086e60d706d6a775c9fd1a0b8812e8ce6504c6 /writerfilter | |
| parent | 9b2f5d27cf0a0adcced813e18108d5878061d1ee (diff) | |
crashtesting: failure on import of tdf113230-1.docx
which is a favorite input document for crashes and assert apparently,
this is the third time it has triggered a different problem
this one is new since...
commit 9fdf8c0a5cc036ea9bd1e11dd8f2c1a6e601fae2
Author: Mike Kaganski <mike.kaganski@collabora.com>
Date: Sat Nov 16 16:34:25 2019 +0300
Also consider saved exceptions when terminating parse
Change-Id: I394b650613e8a835fe8a9f216a48864bdbc5065b
Reviewed-on: https://gerrit.libreoffice.org/83924
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'writerfilter')
| -rw-r--r-- | writerfilter/source/dmapper/DomainMapper_Impl.cxx | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx index 58d7c19604fc..8eebe7869dd2 100644 --- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx +++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx @@ -6384,7 +6384,11 @@ void DomainMapper_Impl::SetCurrentRedlineRevertProperties( const uno::Sequence<b // the context is destroyed. void DomainMapper_Impl::RemoveTopRedline( ) { - assert( m_aRedlines.top().size( ) > 0 ); + if (m_aRedlines.top().empty()) + { + SAL_WARN("writerfilter.dmapper", "RemoveTopRedline called with empty stack"); + throw uno::Exception("RemoveTopRedline failed", nullptr); + } m_aRedlines.top().pop_back( ); m_currentRedline.clear(); } |