sc/source/ui/unoobj/dapiuno.cxx |   12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

New commits:
commit c27e334661b62897a81a450ea9188f80e053261e
Author: Kohei Yoshida <kohei.yosh...@gmail.com>
Date:   Wed Jan 2 16:03:35 2013 -0500

    fdo#58539: Avoid throwing exception not specified in the signature.
    
    This should fix the crasher with the fdo#45266 document.
    
    Change-Id: I41cf02f211e289b85c31b2d2d60e0c4d849b7e8e

diff --git a/sc/source/ui/unoobj/dapiuno.cxx b/sc/source/ui/unoobj/dapiuno.cxx
index feb8b4c..07c2693 100644
--- a/sc/source/ui/unoobj/dapiuno.cxx
+++ b/sc/source/ui/unoobj/dapiuno.cxx
@@ -2697,8 +2697,16 @@ Reference< XDataPilotField > SAL_CALL 
ScDataPilotFieldObj::createNameGroup( cons
         Reference< XNameAccess > xFields(mrParent.getDataPilotFields(), 
UNO_QUERY);
         if (xFields.is())
         {
-            xRet.set(xFields->getByName(sNewDim), UNO_QUERY);
-            OSL_ENSURE(xRet.is(), "there is a name, so there should be also a 
field");
+            try
+            {
+                xRet.set(xFields->getByName(sNewDim), UNO_QUERY);
+                OSL_ENSURE(xRet.is(), "there is a name, so there should be 
also a field");
+            }
+            catch (const container::NoSuchElementException&)
+            {
+                // Avoid throwing exception that's not specified in the method 
signature.
+                throw RuntimeException();
+            }
         }
     }
     return xRet;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to