* packet.h, main.h, sig-check.c (signature_check2, check_key_signature2,
[gnupg.git] / g10 / sig-check.c
index 8dce3cb..53363f8 100644 (file)
@@ -39,8 +39,8 @@ struct cmp_help_context_s {
     MD_HANDLE md;
 };
 
-static int do_check( PKT_public_key *pk, PKT_signature *sig,
-                                        MD_HANDLE digest, int *r_expired );
+static int do_check( PKT_public_key *pk, PKT_signature *sig, MD_HANDLE digest,
+                    int *r_expired, PKT_public_key *ret_pk);
 
 /****************
  * Check the signature which is contained in SIG.
@@ -52,12 +52,12 @@ signature_check( PKT_signature *sig, MD_HANDLE digest )
 {
     u32 dummy;
     int dum2;
-    return signature_check2( sig, digest, &dummy, &dum2 );
+    return signature_check2( sig, digest, &dummy, &dum2, NULL );
 }
 
 int
-signature_check2( PKT_signature *sig, MD_HANDLE digest,
-                 u32 *r_expiredate, int *r_expired )
+signature_check2( PKT_signature *sig, MD_HANDLE digest, u32 *r_expiredate, 
+                 int *r_expired, PKT_public_key *ret_pk )
 {
     PKT_public_key *pk = m_alloc_clear( sizeof *pk );
     int rc=0;
@@ -71,18 +71,16 @@ signature_check2( PKT_signature *sig, MD_HANDLE digest,
 
     if(!md_algo_present(digest,sig->digest_algo)) {
         log_info(_("WARNING: signature digest conflict in message\n"));
-       rc=G10ERR_BAD_SIGN;
+       rc=G10ERR_GENERAL;
     }
     else if( get_pubkey( pk, sig->keyid ) )
        rc = G10ERR_NO_PUBKEY;
-    else if(!pk->is_valid &&
-           (pk->main_keyid[0]!=pk->keyid[0] ||
-            pk->main_keyid[1]!=pk->keyid[1]))
+    else if(!pk->is_valid && !pk->is_primary)
         rc=G10ERR_BAD_PUBKEY; /* you cannot have a good sig from an
                                 invalid subkey */
     else {
        *r_expiredate = pk->expiredate;
-       rc = do_check( pk, sig, digest, r_expired );
+       rc = do_check( pk, sig, digest, r_expired, ret_pk );
     }
 
     free_public_key( pk );
@@ -137,7 +135,7 @@ signature_check2( PKT_signature *sig, MD_HANDLE digest,
 static int
 cmp_help( void *opaque, MPI result )
 {
-  #if 0 /* we do not use this anymore */
+#if 0 /* we do not use this anymore */
     int rc=0, i, j, c, old_enc;
     byte *dp;
     const byte *asn;
@@ -200,34 +198,30 @@ cmp_help( void *opaque, MPI result )
            return G10ERR_BAD_SIGN;
     }
     return 0;
-  #else
+#else
     return -1;
-  #endif
+#endif
 }
 
-
 static int
-do_check( PKT_public_key *pk, PKT_signature *sig, MD_HANDLE digest,
-                                                   int *r_expired )
+do_check_messages( PKT_public_key *pk, PKT_signature *sig, int *r_expired )
 {
-    MPI result = NULL;
-    int rc=0;
-    struct cmp_help_context_s ctx;
     u32 cur_time;
 
     *r_expired = 0;
     if( pk->version == 4 && pk->pubkey_algo == PUBKEY_ALGO_ELGAMAL_E ) {
-       log_info(_("this is a PGP generated "
-                 "ElGamal key which is NOT secure for signatures!\n"));
+       log_info(_("key %08lX: this is a PGP generated "
+                  "ElGamal key which is NOT secure for signatures!\n"),
+                 (ulong)keyid_from_pk(pk,NULL));
        return G10ERR_PUBKEY_ALGO;
     }
 
     if( pk->timestamp > sig->timestamp ) {
        ulong d = pk->timestamp - sig->timestamp;
        log_info( d==1
-                 ? _("public key is %lu second newer than the signature\n")
-                 : _("public key is %lu seconds newer than the signature\n"),
-                      d );
+            ? _("public key %08lX is %lu second newer than the signature\n")
+            : _("public key %08lX is %lu seconds newer than the signature\n"),
+               (ulong)keyid_from_pk(pk,NULL),d );
        if( !opt.ignore_time_conflict )
            return G10ERR_TIME_CONFLICT; /* pubkey newer than signature */
     }
@@ -235,10 +229,11 @@ do_check( PKT_public_key *pk, PKT_signature *sig, MD_HANDLE digest,
     cur_time = make_timestamp();
     if( pk->timestamp > cur_time ) {
        ulong d = pk->timestamp - cur_time;
-       log_info( d==1 ? _("key has been created %lu second "
+       log_info( d==1 ? _("key %08lX has been created %lu second "
                           "in future (time warp or clock problem)\n")
-                      : _("key has been created %lu seconds "
-                          "in future (time warp or clock problem)\n"), d );
+                      : _("key %08lX has been created %lu seconds "
+                          "in future (time warp or clock problem)\n"),
+                      (ulong)keyid_from_pk(pk,NULL),d );
        if( !opt.ignore_time_conflict )
            return G10ERR_TIME_CONFLICT;
     }
@@ -259,7 +254,20 @@ do_check( PKT_public_key *pk, PKT_signature *sig, MD_HANDLE digest,
        *r_expired = 1;
     }
 
+    return 0;
+}
+
 
+static int
+do_check( PKT_public_key *pk, PKT_signature *sig, MD_HANDLE digest,
+         int *r_expired, PKT_public_key *ret_pk )
+{
+    MPI result = NULL;
+    int rc=0;
+    struct cmp_help_context_s ctx;
+
+    if( (rc=do_check_messages(pk,sig,r_expired)) )
+        return rc;
     if( (rc=check_digest_algo(sig->digest_algo)) )
        return rc;
     if( (rc=check_pubkey_algo(sig->pubkey_algo)) )
@@ -335,10 +343,13 @@ do_check( PKT_public_key *pk, PKT_signature *sig, MD_HANDLE digest,
     }
 
     if( !rc && sig->flags.unknown_critical ) {
-       log_info(_("assuming bad signature due to an unknown critical bit\n"));
+      log_info(_("assuming bad signature from key %08lX due to an unknown critical bit\n"),(ulong)keyid_from_pk(pk,NULL));
        rc = G10ERR_BAD_SIGN;
     }
 
+    if(!rc && ret_pk)
+      copy_public_key(ret_pk,pk);
+
     return rc;
 }
 
@@ -392,6 +403,71 @@ cache_sig_result ( PKT_signature *sig, int result )
     }
 }
 
+
+/* Check the revocation keys to see if any of them have revoked our
+   pk.  sig is the revocation sig.  pk is the key it is on.  This code
+   will need to be modified if gpg ever becomes multi-threaded.  Note
+   that this guarantees that a designated revocation sig will never be
+   considered valid unless it is actually valid, as well as being
+   issued by a revocation key in a valid direct signature.  Note that
+   this is written so that a revoked revoker can still issue
+   revocations: i.e. If A revokes B, but A is revoked, B is still
+   revoked.  I'm not completely convinced this is the proper behavior,
+   but it matches how PGP does it. -dms */
+
+/* Returns 0 if sig is valid (i.e. pk is revoked), non-0 if not
+   revoked */
+int
+check_revocation_keys(PKT_public_key *pk,PKT_signature *sig)
+{
+  static int busy=0;
+  int i,rc=G10ERR_GENERAL;
+
+  assert(IS_KEY_REV(sig));
+  assert((sig->keyid[0]!=pk->keyid[0]) || (sig->keyid[0]!=pk->keyid[1]));
+
+  if(busy)
+    {
+      /* return -1 (i.e. not revoked), but mark the pk as uncacheable
+         as we don't really know its revocation status until it is
+         checked directly. */
+
+      pk->dont_cache=1;
+      return rc;
+    }
+
+  busy=1;
+
+  /*  printf("looking at %08lX with a sig from %08lX\n",(ulong)pk->keyid[1],
+      (ulong)sig->keyid[1]); */
+
+  /* is the issuer of the sig one of our revokers? */
+  if( !pk->revkey && pk->numrevkeys )
+     BUG();
+  else
+      for(i=0;i<pk->numrevkeys;i++)
+       {
+          u32 keyid[2];
+    
+          keyid_from_fingerprint(pk->revkey[i].fpr,MAX_FINGERPRINT_LEN,keyid);
+    
+          if(keyid[0]==sig->keyid[0] && keyid[1]==sig->keyid[1])
+           {
+              MD_HANDLE md;
+    
+              md=md_open(sig->digest_algo,0);
+              hash_public_key(md,pk);
+              rc=signature_check(sig,md);
+             cache_sig_result(sig,rc);
+             break;
+           }
+       }
+
+  busy=0;
+
+  return rc;
+} 
+
 /****************
  * check the signature pointed to by NODE. This is a key signature.
  * If the function detects a self-signature, it uses the PK from
@@ -402,12 +478,18 @@ check_key_signature( KBNODE root, KBNODE node, int *is_selfsig )
 {
     u32 dummy;
     int dum2;
-    return check_key_signature2(root, node, is_selfsig, &dummy, &dum2 );
+    return check_key_signature2(root, node, NULL, NULL,
+                               is_selfsig, &dummy, &dum2 );
 }
 
+/* If check_pk is set, then use it to check the signature in node
+   rather than getting it from root or the keydb.  If ret_pk is set,
+   fill in the public key that was used to verify the signature.
+   ret_pk is only meaningful when the verification was successful. */
 int
-check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
-                                      u32 *r_expiredate, int *r_expired )
+check_key_signature2( KBNODE root, KBNODE node, PKT_public_key *check_pk,
+                     PKT_public_key *ret_pk, int *is_selfsig,
+                     u32 *r_expiredate, int *r_expired )
 {
     MD_HANDLE md;
     PKT_public_key *pk;
@@ -436,6 +518,8 @@ check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
                if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] )
                    *is_selfsig = 1;
            }
+           if((rc=do_check_messages(pk,sig,r_expired)))
+             return rc;
             return sig->flags.valid? 0 : G10ERR_BAD_SIGN;
         }
     }
@@ -444,11 +528,20 @@ check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
        return rc;
 
     if( sig->sig_class == 0x20 ) { /* key revocation */
-       md = md_open( algo, 0 );
-       hash_public_key( md, pk );
-       rc = do_check( pk, sig, md, r_expired );
-        cache_sig_result ( sig, rc );
-       md_close(md);
+        u32 keyid[2];  
+       keyid_from_pk( pk, keyid );
+
+       /* is it a designated revoker? */
+        if(keyid[0]!=sig->keyid[0] || keyid[1]!=sig->keyid[1])
+         rc=check_revocation_keys(pk,sig);
+       else
+         {
+           md = md_open( algo, 0 );
+           hash_public_key( md, pk );
+           rc = do_check( pk, sig, md, r_expired, ret_pk );
+           cache_sig_result ( sig, rc );
+           md_close(md);
+         }
     }
     else if( sig->sig_class == 0x28 ) { /* subkey revocation */
        KBNODE snode = find_prev_kbnode( root, node, PKT_PUBLIC_SUBKEY );
@@ -457,13 +550,14 @@ check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
            md = md_open( algo, 0 );
            hash_public_key( md, pk );
            hash_public_key( md, snode->pkt->pkt.public_key );
-           rc = do_check( pk, sig, md, r_expired );
+           rc = do_check( pk, sig, md, r_expired, ret_pk );
             cache_sig_result ( sig, rc );
            md_close(md);
        }
        else {
             if (!opt.quiet)
-                log_info ("key %08lX: no subkey for subkey revocation packet\n",
+                log_info (_("key %08lX: no subkey for subkey "
+                           "revocation signature\n"),
                           (ulong)keyid_from_pk (pk, NULL));
            rc = G10ERR_SIG_CLASS;
        }
@@ -482,21 +576,22 @@ check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
            md = md_open( algo, 0 );
            hash_public_key( md, pk );
            hash_public_key( md, snode->pkt->pkt.public_key );
-           rc = do_check( pk, sig, md, r_expired );
+           rc = do_check( pk, sig, md, r_expired, ret_pk );
             cache_sig_result ( sig, rc );
            md_close(md);
        }
        else {
-            if (!opt.quiet)
-                log_info ("key %08lX: no subkey for subkey binding packet\n",
-                          (ulong)keyid_from_pk (pk, NULL));
+            if (opt.verbose)
+                log_info(_("key %08lX: no subkey for subkey "
+                          "binding signature\n"),
+                        (ulong)keyid_from_pk (pk, NULL));
            rc = G10ERR_SIG_CLASS;
        }
     }
     else if( sig->sig_class == 0x1f ) { /* direct key signature */
        md = md_open( algo, 0 );
        hash_public_key( md, pk );
-       rc = do_check( pk, sig, md, r_expired );
+       rc = do_check( pk, sig, md, r_expired, ret_pk );
         cache_sig_result ( sig, rc );
        md_close(md);
     }
@@ -510,14 +605,17 @@ check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
            md = md_open( algo, 0 );
            hash_public_key( md, pk );
            hash_uid_node( unode, md, sig );
-           if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] ) {
+           if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] )
+             {
                if( is_selfsig )
-                   *is_selfsig = 1;
-               rc = do_check( pk, sig, md, r_expired );
-           }
-           else {
-               rc = signature_check2( sig, md, r_expiredate, r_expired );
-           }
+                 *is_selfsig = 1;
+               rc = do_check( pk, sig, md, r_expired, ret_pk );
+             }
+           else if (check_pk)
+             rc=do_check(check_pk,sig,md,r_expired, ret_pk);
+           else
+             rc = signature_check2( sig, md, r_expiredate, r_expired, ret_pk);
+
             cache_sig_result ( sig, rc );
            md_close(md);
        }
@@ -532,5 +630,3 @@ check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig,
 
     return rc;
 }
-
-