[Libreoffice-commits] core.git: connectivity/source include/connectivity

[Stephan Bergmann]

 connectivity/source/commontools/dbtools.cxx |   10 ++++++++--
 include/connectivity/OSubComponent.hxx      |    4 ++--
 2 files changed, 10 insertions(+), 4 deletions(-)

New commits:
commit 9624bde1c36a3c1b86d8d88f97bc729ac4d65853
Author: Stephan Bergmann <sbergman at redhat.com>
Date:   Tue Jul 12 15:05:41 2016 +0200

    tdf#100866: Don't let exception pass connectivity::release
    
    ...which is only ever called from onexcept XInterface::release overrides:
    connectivity::release itself appears to be only called from
    connectivity::OSubComponent::relase_ChildImpl [sic], which in turn is only
    called from various XInterface::release overrides across connectivity.
    
    Change-Id: I94b682ec531acecd0ef9f8c100f67a71c361941e

diff --git a/connectivity/source/commontools/dbtools.cxx b/connectivity/source/commontools/dbtools.cxx
index 701c64d..1c1dbfe 100644
--- a/connectivity/source/commontools/dbtools.cxx
+++ b/connectivity/source/commontools/dbtools.cxx
@@ -1965,7 +1965,7 @@ namespace connectivity
 void release(oslInterlockedCount& _refCount,
              ::cppu::OBroadcastHelper& rBHelper,
              Reference< XInterface >& _xInterface,
-             css::lang::XComponent* _pObject)
+             css::lang::XComponent* _pObject) throw ()
 {
     if (osl_atomic_decrement( &_refCount ) == 0)
     {
@@ -1982,7 +1982,13 @@ void release(oslInterlockedCount& _refCount,
             }
 
             // First dispose
-            _pObject->dispose();
+            try {
+                _pObject->dispose();
+            } catch (css::uno::RuntimeException & e) {
+                SAL_WARN(
+                    "connectivity.commontools",
+                    "Caught exception during dispose, " << e.Message);
+            }
 
             // only the alive ref holds the object
             OSL_ASSERT( _refCount == 1 );
diff --git a/include/connectivity/OSubComponent.hxx b/include/connectivity/OSubComponent.hxx
index 8f80e8e..1955c54 100644
--- a/include/connectivity/OSubComponent.hxx
+++ b/include/connectivity/OSubComponent.hxx
@@ -43,7 +43,7 @@ namespace connectivity
     void release(oslInterlockedCount& _refCount,
                  ::cppu::OBroadcastHelper& rBHelper,
                  css::uno::Reference< css::uno::XInterface >& _xInterface,
-                 css::lang::XComponent* _pObject);
+                 css::lang::XComponent* _pObject) throw ();
 
     // OSubComponent
 
@@ -70,7 +70,7 @@ namespace connectivity
             ::osl::MutexGuard aGuard( m_pDerivedImplementation->WEAK::rBHelper.rMutex );
         m_xParent.clear();
         }
-        void relase_ChildImpl()
+        void relase_ChildImpl() throw ()
         {
 #if 0
             ::connectivity::release(m_pDerivedImplementation->m_refCount,