[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - oox/source

[Armin Le Grand]

 oox/source/core/xmlfilterbase.cxx |   13 +++++++++++++
 1 file changed, 13 insertions(+)

New commits:
commit e1187562c15594447766828ac89098fc8c964ef7
Author: Armin Le Grand <a...@apache.org>
Date:   Mon Feb 24 20:25:26 2014 +0000

    Resolves: #i118640# force early destruction of last FragmentHandler...
    
    to not let it use an already destroyed RelationsMap instance
    (cherry picked from commit 190444f7fd3fa9cfcd1955b3f5dcd916184d02f4)
    
    Conflicts:
        oox/source/core/xmlfilterbase.cxx
    
    Change-Id: Ibd09ff9fcba8f919df0704c8e3a64423518b932f
    (cherry picked from commit b071ce32befcc84a57a7ecd39667cb6658c0c866)

diff --git a/oox/source/core/xmlfilterbase.cxx 
b/oox/source/core/xmlfilterbase.cxx
index 2ef319e..c0a016d 100644
--- a/oox/source/core/xmlfilterbase.cxx
+++ b/oox/source/core/xmlfilterbase.cxx
@@ -165,6 +165,7 @@ struct XmlFilterBaseImpl
     TextFieldStack                 maTextFieldStack;
 
     explicit            XmlFilterBaseImpl( const Reference< XComponentContext 
>& rxContext ) throw( RuntimeException );
+    ~XmlFilterBaseImpl();
 };
 
 
@@ -177,6 +178,10 @@ XmlFilterBaseImpl::XmlFilterBaseImpl( const Reference< 
XComponentContext >& rxCo
     registerNamespaces(maFastParser);
 }
 
+XmlFilterBaseImpl::~XmlFilterBaseImpl()
+{
+}
+
 XmlFilterBase::XmlFilterBase( const Reference< XComponentContext >& rxContext 
) throw( RuntimeException ) :
     FilterBase( rxContext ),
     mxImpl( new XmlFilterBaseImpl( rxContext ) ),
@@ -187,6 +192,14 @@ XmlFilterBase::XmlFilterBase( const Reference< 
XComponentContext >& rxContext )
 
 XmlFilterBase::~XmlFilterBase()
 {
+    // #i118640# Reset the DocumentHandler at the FastSaxParser manually; this 
is
+    // needed since the mechanism is that instances of FragmentHandler execute
+    // their stuff (creating objects, setting attributes, ...) on being 
destroyed.
+    // They get destroyed by setting a new DocumentHandler. This also happens 
in
+    // the following implicit destruction chain of ~XmlFilterBaseImpl, but in 
that
+    // case it's member RelationsMap maRelationsMap will be destroyed, but 
maybe
+    // still be used by ~FragmentHandler -> crash.
+    mxImpl->maFastParser.setDocumentHandler( 0 );
 }
 
 // ----------------------------------------------------------------------------
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits