diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2016-04-03 21:35:26 +0100 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2016-04-04 08:20:00 +0000 |
| commit | f25eca3f230c170b46d7ce9f40842161a50b1883 (patch) | |
| tree | f69b7d90914c01e6fb3ffeb7e55d1e55b72cb752 | |
| parent | 527f2cd0b75f901efc683efd92a51af771b860b5 (diff) | |
subtable corruption is possible
Change-Id: Ifbf4312df583724a22328491bea3b6e05b82418f
(cherry picked from commit 9814cf713c57b211f5632733a853150d1d1e8da9)
Reviewed-on: https://gerrit.libreoffice.org/23767
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
| -rw-r--r-- | lotuswordpro/qa/cppunit/data/fail/subtable-1.lwp | bin | 0 -> 83439 bytes | |||
| -rw-r--r-- | lotuswordpro/source/filter/xfilter/xfcell.cxx | 5 |
2 files changed, 2 insertions, 3 deletions
diff --git a/lotuswordpro/qa/cppunit/data/fail/subtable-1.lwp b/lotuswordpro/qa/cppunit/data/fail/subtable-1.lwp Binary files differnew file mode 100644 index 000000000000..3d906ee68459 --- /dev/null +++ b/lotuswordpro/qa/cppunit/data/fail/subtable-1.lwp diff --git a/lotuswordpro/source/filter/xfilter/xfcell.cxx b/lotuswordpro/source/filter/xfilter/xfcell.cxx index 7dde717989f3..523bb1a94162 100644 --- a/lotuswordpro/source/filter/xfilter/xfcell.cxx +++ b/lotuswordpro/source/filter/xfilter/xfcell.cxx @@ -89,10 +89,9 @@ void XFCell::Add(XFContent *pContent) Reset(); m_eValueType = enumXFValueTypeNone; } - if( m_pSubTable ) + if (m_pSubTable) { - assert(false); - return; + throw std::runtime_error("subtable already set"); } if (!pContent) { |