diff options
author | Caolán McNamara <caolanm@redhat.com> | 2018-04-07 20:24:40 +0100 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2018-04-12 10:54:43 +0200 |
commit | d0349c2e0248b87fcc8d8db8466b14902cf6b644 (patch) | |
tree | a4dbe109899f95ba511da920be79389ae972576a /package | |
parent | 06fc11cc7ccb2eb93811419113ad30097bfa1730 (diff) |
forcepoint#37 revert tdf#108572 let remove storage element
reverts
commit 26957fc325c27cf61e8ef5555b380d627def4931
Date: Wed Sep 6 10:54:50 2017 +0200
tdf#108572 let remove storage element
void OStorage_Impl::RemoveElement( SotElement_Impl* pElement )
expects that m_pAntiImpl will be nullptr, if not throws an exception.
This caused that in case of deleting embedded database used for
mail merge it wasn't fully removed and
The problem is that by allowing the removal of the element, existing readonly
accessors are left pointing to the deleted element which causes quite some problems
as seen in this sample document
Change-Id: Ie6ac36e6dea771b02637d77b78eb8026e3bb7ae8
Reviewed-on: https://gerrit.libreoffice.org/52600
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
(cherry picked from commit 8182037871daf1d94f4beff63afeb56b20055480)
Diffstat (limited to 'package')
-rw-r--r-- | package/source/xstor/xstorage.cxx | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/package/source/xstor/xstorage.cxx b/package/source/xstor/xstorage.cxx index 5c494a586197..48bc0d6aeb04 100644 --- a/package/source/xstor/xstorage.cxx +++ b/package/source/xstor/xstorage.cxx @@ -2813,8 +2813,6 @@ void SAL_CALL OStorage::removeElement( const OUString& aElementName ) if ( !pElement ) throw container::NoSuchElementException( THROW_WHERE ); //??? - if ( pElement->m_xStorage ) - pElement->m_xStorage->m_pAntiImpl = nullptr; m_pImpl->RemoveElement( pElement ); m_pImpl->m_bIsModified = true; |