summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNoel Grandin <noel.grandin@collabora.co.uk>2023-01-13 09:20:39 +0200
committerNoel Grandin <noel.grandin@collabora.co.uk>2023-01-14 17:24:33 +0000
commit7666648ab2f420caafa6739cc5bb41d5cc04bf91 (patch)
tree25bbf34e5ba1efb3a6499cbc52bc87f72e6fc7d7
parent47ff51eabb58c7aca82968dc31c0ca54e3da2661 (diff)
XUnoTunnel->dynamic_cast in CertificateImpl
Change-Id: I31c03fd894691681401e2ea104ba0d45c95a3c3d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145502 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
-rw-r--r--xmlsecurity/source/gpg/CertificateImpl.cxx13
-rw-r--r--xmlsecurity/source/gpg/CertificateImpl.hxx7
-rw-r--r--xmlsecurity/source/gpg/SecurityEnvironment.cxx4
3 files changed, 2 insertions, 22 deletions
diff --git a/xmlsecurity/source/gpg/CertificateImpl.cxx b/xmlsecurity/source/gpg/CertificateImpl.cxx
index a0a08fc807a8..b771c1282fa0 100644
--- a/xmlsecurity/source/gpg/CertificateImpl.cxx
+++ b/xmlsecurity/source/gpg/CertificateImpl.cxx
@@ -193,19 +193,6 @@ sal_Int32 SAL_CALL CertificateImpl::getCertificateUsage()
return KeyUsage::DIGITAL_SIGNATURE | KeyUsage::NON_REPUDIATION | KeyUsage::KEY_ENCIPHERMENT | KeyUsage::DATA_ENCIPHERMENT;
}
-/* XUnoTunnel */
-sal_Int64 SAL_CALL CertificateImpl::getSomething(const Sequence< sal_Int8 >& aIdentifier)
-{
- return comphelper::getSomethingImpl(aIdentifier, this);
-}
-
-/* XUnoTunnel extension */
-
-const Sequence< sal_Int8>& CertificateImpl::getUnoTunnelId() {
- static const comphelper::UnoIdInit theCertificateImplUnoTunnelId;
- return theCertificateImplUnoTunnelId.getSeq();
-}
-
void CertificateImpl::setCertificate(GpgME::Context* ctx, const GpgME::Key& key)
{
m_pKey = key;
diff --git a/xmlsecurity/source/gpg/CertificateImpl.hxx b/xmlsecurity/source/gpg/CertificateImpl.hxx
index b6c50cb2ed5c..b0856ca0109f 100644
--- a/xmlsecurity/source/gpg/CertificateImpl.hxx
+++ b/xmlsecurity/source/gpg/CertificateImpl.hxx
@@ -18,7 +18,6 @@
#include <com/sun/star/uno/Sequence.hxx>
#include <com/sun/star/lang/XServiceInfo.hpp>
-#include <com/sun/star/lang/XUnoTunnel.hpp>
#include <com/sun/star/security/CertificateKind.hpp>
#include <com/sun/star/security/XCertificate.hpp>
@@ -32,7 +31,6 @@
#endif
class CertificateImpl : public cppu::WeakImplHelper< css::security::XCertificate,
- css::lang::XUnoTunnel,
css::lang::XServiceInfo >,
public xmlsecurity::Certificate
{
@@ -76,11 +74,6 @@ public:
virtual sal_Int32 SAL_CALL getCertificateUsage() override;
- //Methods from XUnoTunnel
- virtual sal_Int64 SAL_CALL getSomething(const css::uno::Sequence< sal_Int8 >& aIdentifier) override;
-
- static const css::uno::Sequence< sal_Int8 >& getUnoTunnelId();
-
/// @see xmlsecurity::Certificate::getSHA256Thumbprint().
virtual css::uno::Sequence<sal_Int8> getSHA256Thumbprint() override;
/// @see xmlsecurity::Certificate::getSignatureMethodAlgorithm().
diff --git a/xmlsecurity/source/gpg/SecurityEnvironment.cxx b/xmlsecurity/source/gpg/SecurityEnvironment.cxx
index aff3b68c5ed8..38d0563b161e 100644
--- a/xmlsecurity/source/gpg/SecurityEnvironment.cxx
+++ b/xmlsecurity/source/gpg/SecurityEnvironment.cxx
@@ -203,7 +203,7 @@ Reference< XCertificate > SecurityEnvironmentGpg::createCertificateFromAscii( co
sal_Int32 SecurityEnvironmentGpg::verifyCertificate( const Reference< XCertificate >& aCert,
const Sequence< Reference< XCertificate > >& /*intermediateCerts*/ )
{
- const CertificateImpl* xCert = comphelper::getFromUnoTunnel<CertificateImpl>(aCert);
+ const CertificateImpl* xCert = dynamic_cast<CertificateImpl*>(aCert.get());
if (xCert == nullptr) {
// Can't find the key locally -> unknown owner
return security::CertificateValidity::ISSUER_UNKNOWN;
@@ -223,7 +223,7 @@ sal_Int32 SecurityEnvironmentGpg::verifyCertificate( const Reference< XCertifica
sal_Int32 SecurityEnvironmentGpg::getCertificateCharacters(
const Reference< XCertificate >& aCert)
{
- if (comphelper::getFromUnoTunnel<CertificateImpl>(aCert) == nullptr)
+ if (dynamic_cast<CertificateImpl*>(aCert.get()) == nullptr)
throw RuntimeException();
// we only listed private keys anyway, up in