framework/inc/uiconfiguration/imagemanager.hxx          |    5 ++++-
 framework/inc/uiconfiguration/moduleimagemanager.hxx    |    5 ++++-
 framework/source/uiconfiguration/imagemanager.cxx       |    5 +++--
 framework/source/uiconfiguration/moduleimagemanager.cxx |    4 +++-
 offapi/com/sun/star/ui/XImageManager.idl                |    2 +-
 pyuno/source/module/pyuno_callable.cxx                  |    2 +-
 pyuno/source/module/pyuno_runtime.cxx                   |    4 ++--
 7 files changed, 18 insertions(+), 9 deletions(-)

New commits:
commit 86598a4e185007958117d2fbf1a2200c7b982808
Author: Caolán McNamara <caol...@redhat.com>
Date:   Tue Jun 3 17:08:49 2014 +0100

    coverity#982753 Dereference null return value
    
    Change-Id: I011a600a27c801dca32734e680ea5a13dfebdfe3

diff --git a/pyuno/source/module/pyuno_callable.cxx 
b/pyuno/source/module/pyuno_callable.cxx
index f91aa75..22b8cd1 100644
--- a/pyuno/source/module/pyuno_callable.cxx
+++ b/pyuno/source/module/pyuno_callable.cxx
@@ -115,7 +115,7 @@ PyObject* PyUNO_callable_call(
         PyRef temp = runtime.any2PyObject (ret_value);
         if( aOutParam.getLength() )
         {
-            PyRef return_list( PyTuple_New (1+aOutParam.getLength()), 
SAL_NO_ACQUIRE );
+            PyRef return_list( PyTuple_New (1+aOutParam.getLength()), 
SAL_NO_ACQUIRE, NOT_NULL );
             PyTuple_SetItem (return_list.get(), 0, temp.getAcquired());
 
             // initialize with defaults in case of exceptions
commit 2ca70c3b5c1959c4aa052dbfae1db25a379b0579
Author: Caolán McNamara <caol...@redhat.com>
Date:   Tue Jun 3 17:08:06 2014 +0100

    coverity#982755 Dereference null return value
    
    Change-Id: I652b308d087c1ee9fff95f81297b079e3f816e07

diff --git a/pyuno/source/module/pyuno_runtime.cxx 
b/pyuno/source/module/pyuno_runtime.cxx
index a4b81fc..cb2ba90 100644
--- a/pyuno/source/module/pyuno_runtime.cxx
+++ b/pyuno/source/module/pyuno_runtime.cxx
@@ -156,7 +156,7 @@ static void getRuntimeImpl( PyRef & globalDict, PyRef 
&runtimeImpl )
 static PyRef importUnoModule( ) throw ( RuntimeException )
 {
     // import the uno module
-    PyRef module( PyImport_ImportModule( "uno" ), SAL_NO_ACQUIRE );
+    PyRef module( PyImport_ImportModule( "uno" ), SAL_NO_ACQUIRE, NOT_NULL );
     if( PyErr_Occurred() )
     {
         PyRef excType, excValue, excTraceback;
commit b75b626ee52b7dec91f48271aa58d66b27604e51
Author: Caolán McNamara <caol...@redhat.com>
Date:   Tue Jun 3 17:07:18 2014 +0100

    coverity#982756 Dereference null return value
    
    Change-Id: Ia29c6167df0ce061fae88d81bef9a98ca37fcce3

diff --git a/pyuno/source/module/pyuno_runtime.cxx 
b/pyuno/source/module/pyuno_runtime.cxx
index 5ef20df..a4b81fc 100644
--- a/pyuno/source/module/pyuno_runtime.cxx
+++ b/pyuno/source/module/pyuno_runtime.cxx
@@ -524,7 +524,7 @@ PyRef Runtime::any2PyObject (const Any &a ) const
             Reference< XTypeConverter > tc = getImpl()->cargo->xTypeConverter;
             Reference< XSingleServiceFactory > ssf = 
getImpl()->cargo->xInvocation;
             tc->convertTo (a, ::getCppuType (&s)) >>= s;
-            PyRef tuple( PyTuple_New (s.getLength()), SAL_NO_ACQUIRE);
+            PyRef tuple( PyTuple_New (s.getLength()), SAL_NO_ACQUIRE, 
NOT_NULL);
             int i=0;
             try
             {
commit 229abf40ac4ee551063cdefabede3c9b31146e07
Author: Caolán McNamara <caol...@redhat.com>
Date:   Tue Jun 3 16:51:48 2014 +0100

    coverity#1219794 Uncaught exception
    
    Change-Id: Icc9e5faf9ef15840c359b7766112278094d52dea

diff --git a/dbaccess/source/ui/dlg/indexfieldscontrol.cxx 
b/dbaccess/source/ui/dlg/indexfieldscontrol.cxx
index b156470..f241969 100644
--- a/dbaccess/source/ui/dlg/indexfieldscontrol.cxx
+++ b/dbaccess/source/ui/dlg/indexfieldscontrol.cxx
@@ -206,8 +206,6 @@ namespace dbaui
 
     void IndexFieldsControl::Init(const Sequence< OUString >& 
_rAvailableFields)
     {
-        fprintf(stderr, " _rAvailableFields len is %d\n",  
_rAvailableFields.getLength());
-
         RemoveColumns();
 
         // for the width: both columns together should be somewhat smaller 
than the whole window (without the scrollbar)
@@ -237,9 +235,6 @@ namespace dbaui
             m_pSortingCell->InsertEntry(m_sAscendingText);
             m_pSortingCell->InsertEntry(m_sDescendingText);
             m_pSortingCell->SetHelpId( HID_DLGINDEX_INDEXDETAILS_SORTORDER );
-            m_pSortingCell->Show();
-
-            fprintf(stderr, "created m_pSortingCell %p\n", m_pSortingCell);
 
             nFieldNameWidth -= nSortOrderColumnWidth;
         }
@@ -258,11 +253,7 @@ namespace dbaui
         const OUString* pFields = _rAvailableFields.getConstArray();
         const OUString* pFieldsEnd = pFields + _rAvailableFields.getLength();
         for (;pFields < pFieldsEnd; ++pFields)
-        {
-            fprintf(stderr, "foo is %s\n", OUStringToOString(*pFields, 
RTL_TEXTENCODING_UTF8).getStr());
             m_pFieldNameCell->InsertEntry(*pFields);
-        }
-        m_pFieldNameCell->Show();
     }
 
     CellController* IndexFieldsControl::GetController(long _nRow, sal_uInt16 
_nColumnId)
diff --git a/framework/inc/uiconfiguration/imagemanager.hxx 
b/framework/inc/uiconfiguration/imagemanager.hxx
index 9ad1c45..1a18839 100644
--- a/framework/inc/uiconfiguration/imagemanager.hxx
+++ b/framework/inc/uiconfiguration/imagemanager.hxx
@@ -84,7 +84,10 @@ namespace framework
             virtual void SAL_CALL initialize( const 
::com::sun::star::uno::Sequence< ::com::sun::star::uno::Any >& aArguments ) 
throw (::com::sun::star::uno::Exception, 
::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
 
             // XImageManager
-            virtual void SAL_CALL reset() throw 
(::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
+            virtual void SAL_CALL reset()
+                throw (css::lang::IllegalAccessException,
+                       css::uno::RuntimeException,
+                       std::exception) SAL_OVERRIDE;
             virtual ::com::sun::star::uno::Sequence< OUString > SAL_CALL 
getAllImageNames( ::sal_Int16 nImageType ) throw 
(::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
             virtual sal_Bool SAL_CALL hasImage( ::sal_Int16 nImageType, const 
OUString& aCommandURL ) throw 
(::com::sun::star::lang::IllegalArgumentException, 
::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
             virtual ::com::sun::star::uno::Sequence< 
::com::sun::star::uno::Reference< ::com::sun::star::graphic::XGraphic > > 
SAL_CALL getImages( ::sal_Int16 nImageType, const 
::com::sun::star::uno::Sequence< OUString >& aCommandURLSequence ) throw 
(::com::sun::star::lang::IllegalArgumentException, 
::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
diff --git a/framework/source/uiconfiguration/imagemanager.cxx 
b/framework/source/uiconfiguration/imagemanager.cxx
index 60770cb..20cac0f 100644
--- a/framework/source/uiconfiguration/imagemanager.cxx
+++ b/framework/source/uiconfiguration/imagemanager.cxx
@@ -102,9 +102,10 @@ void SAL_CALL ImageManager::initialize( const Sequence< 
Any >& aArguments ) thro
 
 // XImageManager
 void SAL_CALL ImageManager::reset()
-throw (::com::sun::star::uno::RuntimeException, std::exception)
+    throw (css::lang::IllegalAccessException,
+           css::uno::RuntimeException,
+           std::exception)
 {
-
     /* SAFE AREA 
-----------------------------------------------------------------------------------------------
 */
     m_pImpl->reset();
 }
commit 0301abbbc33c794fccd4e84f75b6de4c296f957c
Author: Caolán McNamara <caol...@redhat.com>
Date:   Tue Jun 3 16:49:15 2014 +0100

    coverity#1219801 Uncaught exception
    
    Change-Id: I372c1891e41510c24daf256d00f3ca98e8a374a9

diff --git a/dbaccess/source/ui/dlg/indexfieldscontrol.cxx 
b/dbaccess/source/ui/dlg/indexfieldscontrol.cxx
index f241969..b156470 100644
--- a/dbaccess/source/ui/dlg/indexfieldscontrol.cxx
+++ b/dbaccess/source/ui/dlg/indexfieldscontrol.cxx
@@ -206,6 +206,8 @@ namespace dbaui
 
     void IndexFieldsControl::Init(const Sequence< OUString >& 
_rAvailableFields)
     {
+        fprintf(stderr, " _rAvailableFields len is %d\n",  
_rAvailableFields.getLength());
+
         RemoveColumns();
 
         // for the width: both columns together should be somewhat smaller 
than the whole window (without the scrollbar)
@@ -235,6 +237,9 @@ namespace dbaui
             m_pSortingCell->InsertEntry(m_sAscendingText);
             m_pSortingCell->InsertEntry(m_sDescendingText);
             m_pSortingCell->SetHelpId( HID_DLGINDEX_INDEXDETAILS_SORTORDER );
+            m_pSortingCell->Show();
+
+            fprintf(stderr, "created m_pSortingCell %p\n", m_pSortingCell);
 
             nFieldNameWidth -= nSortOrderColumnWidth;
         }
@@ -253,7 +258,11 @@ namespace dbaui
         const OUString* pFields = _rAvailableFields.getConstArray();
         const OUString* pFieldsEnd = pFields + _rAvailableFields.getLength();
         for (;pFields < pFieldsEnd; ++pFields)
+        {
+            fprintf(stderr, "foo is %s\n", OUStringToOString(*pFields, 
RTL_TEXTENCODING_UTF8).getStr());
             m_pFieldNameCell->InsertEntry(*pFields);
+        }
+        m_pFieldNameCell->Show();
     }
 
     CellController* IndexFieldsControl::GetController(long _nRow, sal_uInt16 
_nColumnId)
diff --git a/framework/inc/uiconfiguration/moduleimagemanager.hxx 
b/framework/inc/uiconfiguration/moduleimagemanager.hxx
index 5afe454..0775d1d 100644
--- a/framework/inc/uiconfiguration/moduleimagemanager.hxx
+++ b/framework/inc/uiconfiguration/moduleimagemanager.hxx
@@ -72,7 +72,10 @@ namespace framework
             virtual void SAL_CALL initialize( const 
::com::sun::star::uno::Sequence< ::com::sun::star::uno::Any >& aArguments ) 
throw (::com::sun::star::uno::Exception, 
::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
 
             // XImageManager
-            virtual void SAL_CALL reset() throw 
(::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
+            virtual void SAL_CALL reset()
+                throw (css::lang::IllegalAccessException,
+                       css::uno::RuntimeException,
+                       std::exception) SAL_OVERRIDE;
             virtual ::com::sun::star::uno::Sequence< OUString > SAL_CALL 
getAllImageNames( ::sal_Int16 nImageType ) throw 
(::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
             virtual sal_Bool SAL_CALL hasImage( ::sal_Int16 nImageType, const 
OUString& aCommandURL ) throw 
(::com::sun::star::lang::IllegalArgumentException, 
::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
             virtual ::com::sun::star::uno::Sequence< 
::com::sun::star::uno::Reference< ::com::sun::star::graphic::XGraphic > > 
SAL_CALL getImages( ::sal_Int16 nImageType, const 
::com::sun::star::uno::Sequence< OUString >& aCommandURLSequence ) throw 
(::com::sun::star::lang::IllegalArgumentException, 
::com::sun::star::uno::RuntimeException, std::exception) SAL_OVERRIDE;
diff --git a/framework/source/uiconfiguration/moduleimagemanager.cxx 
b/framework/source/uiconfiguration/moduleimagemanager.cxx
index 1047878..d9979c6 100644
--- a/framework/source/uiconfiguration/moduleimagemanager.cxx
+++ b/framework/source/uiconfiguration/moduleimagemanager.cxx
@@ -100,7 +100,9 @@ void SAL_CALL ModuleImageManager::initialize( const 
Sequence< Any >& aArguments
 
 // XImageManager
 void SAL_CALL ModuleImageManager::reset()
-throw (::com::sun::star::uno::RuntimeException, std::exception)
+    throw (css::lang::IllegalAccessException,
+           css::uno::RuntimeException,
+           std::exception)
 {
     m_pImpl->reset();
 }
diff --git a/offapi/com/sun/star/ui/XImageManager.idl 
b/offapi/com/sun/star/ui/XImageManager.idl
index cad89a1..73eef67 100644
--- a/offapi/com/sun/star/ui/XImageManager.idl
+++ b/offapi/com/sun/star/ui/XImageManager.idl
@@ -49,7 +49,7 @@ interface XImageManager
         This means that all user images of the instance will be removed.
         </p>
     */
-    void reset();
+    void reset() raises ( com::sun::star::lang::IllegalArgumentException );
 
     /** retrieves the list of command URLs which have images associated.
 
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

Reply via email to