diff options
author | Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> | 2017-07-11 16:18:19 +0200 |
---|---|---|
committer | Thorsten Behrens <Thorsten.Behrens@CIB.de> | 2017-07-14 18:22:33 +0200 |
commit | e0e8d86d861ca412cb0e7c8d713255f92ac79a9d (patch) | |
tree | bb3f5529c53adb5143b5c3d6f2d7f2e0778757e9 | |
parent | 0bb3f9a246ebe58edccf94b1de33267f8d922ce0 (diff) |
tdf#108831 PGP signature shown as invalid
It was valid, but not trusted.
We need to show the owner trust in another place.
gpg4libre
Change-Id: I344a7b064a22c16b647c73d52f7abd91cfc86be9
Reviewed-on: https://gerrit.libreoffice.org/39963
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
-rw-r--r-- | xmlsecurity/source/gpg/xmlsignature_gpgimpl.cxx | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/xmlsecurity/source/gpg/xmlsignature_gpgimpl.cxx b/xmlsecurity/source/gpg/xmlsignature_gpgimpl.cxx index 0700c43f1ea6..60983fc8e1b9 100644 --- a/xmlsecurity/source/gpg/xmlsignature_gpgimpl.cxx +++ b/xmlsecurity/source/gpg/xmlsignature_gpgimpl.cxx @@ -371,9 +371,7 @@ SAL_CALL XMLSignature_GpgImpl::validate( data_signature, data_text); // TODO: needs some more error handling, needs checking _all_ signatures - if( verify_res.isNull() || - verify_res.numSignatures() == 0 || - verify_res.signature(0).validity() < GpgME::Signature::Full ) + if( verify_res.isNull() || verify_res.numSignatures() == 0 ) { // let's try again, but this time import the public key payload // (avoiding that in a first cut for being a bit speedier) @@ -422,9 +420,7 @@ SAL_CALL XMLSignature_GpgImpl::validate( verify_res=rCtx.verifyDetachedSignature(data_signature, data_text); // TODO: needs some more error handling, needs checking _all_ signatures - if( verify_res.isNull() || - verify_res.numSignatures() == 0 || - verify_res.signature(0).validity() < GpgME::Signature::Full ) + if( verify_res.isNull() || verify_res.numSignatures() == 0 ) { clearErrorRecorder(); xmlFree(pSignatureValue); |