diff options
author | Caolán McNamara <caolanm@redhat.com> | 2014-05-27 13:12:02 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-05-27 13:59:53 +0100 |
commit | 42119194fd5f66e393b9b53692d2330016d144d6 (patch) | |
tree | bf7ef113c0dfa1a3a1d7b9d5aa1fd0629c07ae6f /desktop | |
parent | 65a54a19adcaa2271c5a502a752eb7e6191d08da (diff) |
coverity#706499 Uncaught exception
also coverity#706500 and coverity#706502 and coverity#706503
Change-Id: If8c76f9fea669f3397a3db65fefaabe101bf9c35
Diffstat (limited to 'desktop')
5 files changed, 33 insertions, 17 deletions
diff --git a/desktop/source/deployment/registry/dp_backend.cxx b/desktop/source/deployment/registry/dp_backend.cxx index fb5c05d2458d..69b918b87216 100644 --- a/desktop/source/deployment/registry/dp_backend.cxx +++ b/desktop/source/deployment/registry/dp_backend.cxx @@ -515,8 +515,10 @@ OUString Package::getDescription() throw ( } -OUString Package::getLicenseText() throw ( - deployment::ExtensionRemovedException,RuntimeException, std::exception) +OUString Package::getLicenseText()throw ( + deployment::DeploymentException, + deployment::ExtensionRemovedException, + RuntimeException, std::exception) { if (m_bRemoved) throw deployment::ExtensionRemovedException(); @@ -751,6 +753,7 @@ PackageRegistryBackend * Package::getMyBackend() const } return pBackend; } + OUString Package::getRepositoryName() throw (RuntimeException, std::exception) { @@ -759,7 +762,8 @@ OUString Package::getRepositoryName() } beans::Optional< OUString > Package::getRegistrationDataURL() - throw (deployment::ExtensionRemovedException, + throw (deployment::DeploymentException, + deployment::ExtensionRemovedException, css::uno::RuntimeException, std::exception) { if (m_bRemoved) @@ -773,8 +777,6 @@ sal_Bool Package::isRemoved() return m_bRemoved; } - - Package::TypeInfo::~TypeInfo() { } diff --git a/desktop/source/deployment/registry/help/dp_help.cxx b/desktop/source/deployment/registry/help/dp_help.cxx index c3ecbd4508eb..74259ae3a08c 100644 --- a/desktop/source/deployment/registry/help/dp_help.cxx +++ b/desktop/source/deployment/registry/help/dp_help.cxx @@ -84,7 +84,8 @@ class BackendImpl : public ::dp_registry::backend::PackageRegistryBackend //XPackage virtual css::beans::Optional< OUString > SAL_CALL getRegistrationDataURL() - throw (deployment::ExtensionRemovedException, css::uno::RuntimeException, std::exception) SAL_OVERRIDE; + throw (deployment::DeploymentException, deployment::ExtensionRemovedException, + css::uno::RuntimeException, std::exception) SAL_OVERRIDE; }; friend class PackageImpl; @@ -356,7 +357,8 @@ void BackendImpl::PackageImpl::processPackage_( } beans::Optional< OUString > BackendImpl::PackageImpl::getRegistrationDataURL() - throw (deployment::ExtensionRemovedException, + throw (deployment::DeploymentException, + deployment::ExtensionRemovedException, css::uno::RuntimeException, std::exception) { if (m_bRemoved) diff --git a/desktop/source/deployment/registry/inc/dp_backend.h b/desktop/source/deployment/registry/inc/dp_backend.h index 618fc46cb20f..8e1a75e93d49 100644 --- a/desktop/source/deployment/registry/inc/dp_backend.h +++ b/desktop/source/deployment/registry/inc/dp_backend.h @@ -227,7 +227,8 @@ public: throw (css::deployment::ExtensionRemovedException, css::uno::RuntimeException, std::exception) SAL_OVERRIDE; virtual OUString SAL_CALL getLicenseText() - throw (css::deployment::ExtensionRemovedException, + throw (css::deployment::DeploymentException, + css::deployment::ExtensionRemovedException, css::uno::RuntimeException, std::exception) SAL_OVERRIDE; virtual css::uno::Sequence< OUString > SAL_CALL getUpdateInformationURLs() @@ -253,7 +254,8 @@ public: virtual OUString SAL_CALL getRepositoryName() throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE; virtual css::beans::Optional< OUString > SAL_CALL getRegistrationDataURL() - throw (css::deployment::ExtensionRemovedException, + throw (css::deployment::DeploymentException, + css::deployment::ExtensionRemovedException, css::uno::RuntimeException, std::exception) SAL_OVERRIDE; virtual sal_Bool SAL_CALL isRemoved() throw (css::uno::RuntimeException, std::exception) SAL_OVERRIDE; diff --git a/desktop/source/deployment/registry/package/dp_package.cxx b/desktop/source/deployment/registry/package/dp_package.cxx index a852f23bfaac..875d4239ce36 100644 --- a/desktop/source/deployment/registry/package/dp_package.cxx +++ b/desktop/source/deployment/registry/package/dp_package.cxx @@ -182,7 +182,9 @@ class BackendImpl : public ImplBaseT throw (deployment::ExtensionRemovedException, RuntimeException, std::exception) SAL_OVERRIDE; virtual OUString SAL_CALL getLicenseText() - throw (deployment::ExtensionRemovedException, RuntimeException, std::exception) SAL_OVERRIDE; + throw (deployment::DeploymentException, + deployment::ExtensionRemovedException, + RuntimeException, std::exception) SAL_OVERRIDE; virtual void SAL_CALL exportTo( OUString const & destFolderURL, OUString const & newTitle, @@ -999,7 +1001,9 @@ OUString BackendImpl::PackageImpl::getDescription() OUString BackendImpl::PackageImpl::getLicenseText() - throw (deployment::ExtensionRemovedException, RuntimeException, std::exception) + throw (deployment::DeploymentException, + deployment::ExtensionRemovedException, + RuntimeException, std::exception) { if (m_bRemoved) throw deployment::ExtensionRemovedException(); diff --git a/desktop/source/deployment/registry/sfwk/dp_sfwk.cxx b/desktop/source/deployment/registry/sfwk/dp_sfwk.cxx index 86c079183652..bbf8dbfbf806 100644 --- a/desktop/source/deployment/registry/sfwk/dp_sfwk.cxx +++ b/desktop/source/deployment/registry/sfwk/dp_sfwk.cxx @@ -75,8 +75,12 @@ class BackendImpl : public ::dp_registry::backend::PackageRegistryBackend OUString const & url, OUString const & libType, bool bRemoved, OUString const & identifier); // XPackage - virtual OUString SAL_CALL getDescription() throw (RuntimeException, std::exception) SAL_OVERRIDE; - virtual OUString SAL_CALL getLicenseText() throw (RuntimeException, std::exception) SAL_OVERRIDE; + virtual OUString SAL_CALL getDescription() + throw (deployment::ExtensionRemovedException, + RuntimeException, std::exception) SAL_OVERRIDE; + virtual OUString SAL_CALL getLicenseText() + throw (deployment::ExtensionRemovedException, + RuntimeException, std::exception) SAL_OVERRIDE; }; friend class PackageImpl; @@ -116,7 +120,9 @@ BackendImpl * BackendImpl::PackageImpl::getMyBackend() const return pBackend; } -OUString BackendImpl::PackageImpl::getDescription() throw (RuntimeException, std::exception) +OUString BackendImpl::PackageImpl::getDescription() + throw (deployment::ExtensionRemovedException, + RuntimeException, std::exception) { if (m_descr.isEmpty()) return Package::getDescription(); @@ -124,13 +130,13 @@ OUString BackendImpl::PackageImpl::getDescription() throw (RuntimeException, std return m_descr; } - -OUString BackendImpl::PackageImpl::getLicenseText() throw (RuntimeException, std::exception) +OUString BackendImpl::PackageImpl::getLicenseText() + throw (deployment::ExtensionRemovedException, + RuntimeException, std::exception) { return Package::getDescription(); } - BackendImpl::PackageImpl::PackageImpl( ::rtl::Reference<BackendImpl> const & myBackend, OUString const & url, OUString const & libType, bool bRemoved, |