aboutsummaryrefslogtreecommitdiffstats
path: root/g10/sig-check.c
diff options
context:
space:
mode:
authorDaniel Kahn Gillmor <[email protected]>2015-10-18 21:35:32 +0000
committerWerner Koch <[email protected]>2015-10-19 12:24:27 +0000
commit76afaed65e3b0ddfa4923cb577ada43217dd4b18 (patch)
tree6d4e201bbb13c88b29aaafe9b4eef3dfadf87d3f /g10/sig-check.c
parentdirmngr: Make --use-tor work - still leaks DNS. (diff)
downloadgnupg-76afaed65e3b0ddfa4923cb577ada43217dd4b18.tar.gz
gnupg-76afaed65e3b0ddfa4923cb577ada43217dd4b18.zip
gpg: Add option --weak-digest to gpg and gpgv.
* g10/options.h: Add additional_weak_digests linked list to opts. * g10/main.h: Declare weakhash linked list struct and additional_weak_digest() function to insert newly-declared weak digests into opts. * g10/misc.c: (additional_weak_digest): New function. (print_digest_algo_note): Check for deprecated digests; use proper gcry_md_algos type. * g10/sig-check.c: (do_check): Reject weak digests in addition to MD5. * g10/gpg.c: Add --weak-digest option to gpg. * doc/gpg.texi: Document gpg --weak-digest option. * g10/gpgv.c: Add --weak-digest option to gpgv. * doc/gpgv.texi: Document gpgv --weak-digest option. -- gpg and gpgv treat signatures made over MD5 as unreliable, unless the user supplies --allow-weak-digests to gpg. Signatures over any other digest are considered acceptable. Despite SHA-1 being a mandatory-to-implement digest algorithm in RFC 4880, the collision-resistance of SHA-1 is weaker than anyone would like it to be. Some operators of high-value targets that depend on OpenPGP signatures may wish to require their signers to use a stronger digest algorithm than SHA1, even if the OpenPGP ecosystem at large cannot deprecate SHA1 entirely today. This changeset adds a new "--weak-digest DIGEST" option for both gpg and gpgv, which makes it straightforward for anyone to treat any signature or certification made over the specified digest as unreliable. This option can be supplied multiple times if the operator wishes to deprecate multiple digest algorithms, and will be ignored completely if the operator supplies --allow-weak-digests (as before). MD5 is still always considered weak, regardless of any further --weak-digest options supplied. Signed-off-by: Daniel Kahn Gillmor <[email protected]> Capitialized some comments, shorted a line in do_check, and changed subject to name the option. -wk
Diffstat (limited to 'g10/sig-check.c')
-rw-r--r--g10/sig-check.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/g10/sig-check.c b/g10/sig-check.c
index d45a9f378..84930d615 100644
--- a/g10/sig-check.c
+++ b/g10/sig-check.c
@@ -274,15 +274,18 @@ do_check( PKT_public_key *pk, PKT_signature *sig, gcry_md_hd_t digest,
{
gcry_mpi_t result = NULL;
int rc = 0;
+ const struct weakhash *weak;
if( (rc=do_check_messages(pk,sig,r_expired,r_revoked)) )
return rc;
- if (sig->digest_algo == GCRY_MD_MD5
- && !opt.flags.allow_weak_digest_algos)
+ if (!opt.flags.allow_weak_digest_algos)
{
- print_md5_rejected_note ();
- return GPG_ERR_DIGEST_ALGO;
+ if (sig->digest_algo == GCRY_MD_MD5)
+ return GPG_ERR_DIGEST_ALGO;
+ for (weak = opt.additional_weak_digests; weak; weak = weak->next)
+ if (sig->digest_algo == weak->algo)
+ return GPG_ERR_DIGEST_ALGO;
}
/* Make sure the digest algo is enabled (in case of a detached