See ChangeLog: Tue Oct 26 14:10:21 CEST 1999 Werner Koch
[gnupg.git] / g10 / sig-check.c
index cc13367..972c877 100644 (file)
@@ -1,14 +1,14 @@
 /* sig-check.c -  Check a signature
- *     Copyright (c) 1997 by Werner Koch (dd9jn)
+ *     Copyright (C) 1998 Free Software Foundation, Inc.
  *
- * This file is part of G10.
+ * This file is part of GnuPG.
  *
- * G10 is free software; you can redistribute it and/or modify
+ * GnuPG is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
  * (at your option) any later version.
  *
- * G10 is distributed in the hope that it will be useful,
+ * GnuPG is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
 #include <stdlib.h>
 #include <string.h>
 #include <assert.h>
+
+#include <gcrypt.h>
 #include "util.h"
 #include "packet.h"
 #include "memory.h"
 #include "mpi.h"
 #include "keydb.h"
-#include "cipher.h"
 #include "main.h"
+#include "status.h"
+#include "i18n.h"
+
+struct cmp_help_context_s {
+    PKT_signature *sig;
+    GCRY_MD_HD md;
+};
+
+
+static int do_signature_check( PKT_signature *sig, GCRY_MD_HD digest,
+                                                     u32 *r_expire );
+static int do_check( PKT_public_key *pk, PKT_signature *sig,
+                                               GCRY_MD_HD digest );
 
 
 /****************
- * Check the signature which is contained in the rsa_integer.
- * The md5handle should be currently open, so that this function
- * is able to append some data, before getting the digest.
+ * Check the signature which is contained in SIG.
+ * The GCRY_MD_HD should be currently open, so that this function
+ * is able to append some data, before finalizing the digest.
  */
 int
-signature_check( PKT_signature *sig, MD_HANDLE digest )
+signature_check( PKT_signature *sig, GCRY_MD_HD digest )
 {
-    PKT_public_cert *pkc = m_alloc_clear( sizeof *pkc );
-    MPI result = NULL;
+    u32 dummy;
+    return do_signature_check( sig, digest, &dummy );
+}
+
+static int
+do_signature_check( PKT_signature *sig, GCRY_MD_HD digest, u32 *r_expire )
+{
+    PKT_public_key *pk = m_alloc_clear( sizeof *pk );
     int rc=0;
 
+    if( is_RSA(sig->pubkey_algo) )
+       write_status(STATUS_RSA_OR_IDEA);
 
-    if( get_pubkey( pkc, sig->keyid ) ) {
+    *r_expire = 0;
+    if( get_pubkey( pk, sig->keyid ) )
        rc = G10ERR_NO_PUBKEY;
-       goto leave;
+    else {
+       *r_expire = pk->expiredate;
+       rc = do_check( pk, sig, digest );
     }
 
-    if( pkc->pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
-       ELG_public_key pkey;
-
-       if( (rc=check_digest_algo(sig->d.elg.digest_algo)) )
-           goto leave;
-       /* complete the digest */
-       md_putc( digest, sig->sig_class );
-       {   u32 a = sig->timestamp;
-           md_putc( digest, (a >> 24) & 0xff );
-           md_putc( digest, (a >> 16) & 0xff );
-           md_putc( digest, (a >>  8) & 0xff );
-           md_putc( digest,  a        & 0xff );
+    free_public_key( pk );
+
+    if( !rc && sig->sig_class < 2 && is_status_enabled() ) {
+       /* This signature id works best with DLP algorithms because
+        * they use a random parameter for every signature.  Instead of
+        * this sig-id we could have also used the hash of the document
+        * and the timestamp, but the drawback of this is, that it is
+        * not possible to sign more than one identical document within
+        * one second.  Some remote bacth processing applications might
+        * like this feature here */
+       GCRY_MD_HD md;
+       u32 a = sig->timestamp;
+       int i, nsig = pubkey_get_nsig( sig->pubkey_algo );
+       byte *p, *buffer;
+
+       if( !(md = gcry_md_open( DIGEST_ALGO_RMD160, 0)) )
+           BUG();
+       gcry_md_putc( digest, sig->pubkey_algo );
+       gcry_md_putc( digest, sig->digest_algo );
+       gcry_md_putc( digest, (a >> 24) & 0xff );
+       gcry_md_putc( digest, (a >> 16) & 0xff );
+       gcry_md_putc( digest, (a >>  8) & 0xff );
+       gcry_md_putc( digest,  a        & 0xff );
+       for(i=0; i < nsig; i++ ) {
+           unsigned n = mpi_get_nbits( sig->data[i]);
+
+           gcry_md_putc( md, n>>8);
+           gcry_md_putc( md, n );
+           p = mpi_get_buffer( sig->data[i], &n, NULL );
+           gcry_md_write( md, p, n );
+           m_free(p);
        }
-       md_final( digest );
-       result = encode_md_value( digest, mpi_get_nbits(pkc->d.elg.p));
-       pkey.p = pkc->d.elg.p;
-       pkey.g = pkc->d.elg.g;
-       pkey.y = pkc->d.elg.y;
-       if( !elg_verify( sig->d.elg.a, sig->d.elg.b, result, &pkey ) )
-           rc = G10ERR_BAD_SIGN;
+       gcry_md_final( md );
+       p = make_radix64_string( gcry_md_read( md, 0 ), 20 );
+       buffer = m_alloc( strlen(p) + 60 );
+       sprintf( buffer, "%s %s %lu",
+                p, strtimestamp( sig->timestamp ), (ulong)sig->timestamp );
+       write_status_text( STATUS_SIG_ID, buffer );
+       m_free(buffer);
+       m_free(p);
+       gcry_md_close(md);
     }
- #ifdef HAVE_RSA_CIPHER
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_RSA ) {
-       int i, j, c, old_enc;
-       byte *dp;
-       RSA_public_key pkey;
-
-       result = mpi_alloc(40);
-       pkey.n = pkc->d.rsa.rsa_n;
-       pkey.e = pkc->d.rsa.rsa_e;
-       rsa_public( result, sig->d.rsa.rsa_integer, &pkey );
-
-       old_enc = 0;
-       for(i=j=0; (c=mpi_getbyte(result, i)) != -1; i++ ) {
-           if( !j ) {
-               if( !i && c != 1 )
-                   break;
-               else if( i && c == 0xff )
-                   ; /* skip the padding */
-               else if( i && !c )
-                   j++;
-               else
-                   break;
-           }
-           else if( ++j == 18 && c != 1 )
-               break;
-           else if( j == 19 && c == 0 ) {
-               old_enc++;
-               break;
-           }
-       }
-       if( old_enc ) {
-           log_error("old encoding scheme is not supported\n");
-           rc = G10ERR_GENERAL;
-           goto leave;
+
+    return rc;
+}
+
+
+#if 0 /* not anymore used */
+/****************
+ * Check the MDC which is contained in SIG.
+ * The GCRY_MD_HD should be currently open, so that this function
+ * is able to append some data, before finalizing the digest.
+ */
+int
+mdc_kludge_check( PKT_signature *sig, GCRY_MD_HD digest )
+{
+    int rc=0;
+
+    if( (rc=check_digest_algo(sig->digest_algo)) )
+       return rc;
+
+    /* make sure the digest algo is enabled (in case of a detached mdc??) */
+    md_enable( digest, sig->digest_algo );
+
+    /* complete the digest */
+    if( sig->version >= 4 )
+       gcry_md_putc( digest, sig->version );
+    gcry_md_putc( digest, sig->sig_class );
+    if( sig->version < 4 ) {
+       u32 a = sig->timestamp;
+       gcry_md_putc( digest, (a >> 24) & 0xff );
+       gcry_md_putc( digest, (a >> 16) & 0xff );
+       gcry_md_putc( digest, (a >>  8) & 0xff );
+       gcry_md_putc( digest,  a        & 0xff );
+    }
+    else {
+       byte buf[6];
+       size_t n;
+       gcry_md_putc( digest, sig->pubkey_algo );
+       gcry_md_putc( digest, sig->digest_algo );
+       if( sig->hashed_data ) {
+           n = (sig->hashed_data[0] << 8) | sig->hashed_data[1];
+           gcry_md_write( digest, sig->hashed_data, n+2 );
+           n += 6;
        }
+       else
+           n = 6;
+       /* add some magic */
+       buf[0] = sig->version;
+       buf[1] = 0xff;
+       buf[2] = n >> 24;
+       buf[3] = n >> 16;
+       buf[4] = n >>  8;
+       buf[5] = n;
+       gcry_md_write( digest, buf, 6 );
+    }
+    md_final( digest );
 
-       if( sig->d.rsa.digest_algo == DIGEST_ALGO_RMD160 ) {
-           static byte asn[15] = /* stored reverse */
-                 { 0x14, 0x04, 0x00, 0x05, 0x01, 0x02, 0x03, 0x24, 0x2b,
-                   0x05, 0x06, 0x09, 0x30, 0x21, 0x30 };
-
-           for(i=20,j=0; (c=mpi_getbyte(result, i)) != -1 && j < 15; i++, j++ )
-               if( asn[j] != c )
-                   break;
-           if( j != 15 || mpi_getbyte(result, i) ) { /* ASN is wrong */
-               rc = G10ERR_BAD_PUBKEY;
-               goto leave;
-           }
-           for(i++; (c=mpi_getbyte(result, i)) != -1; i++ )
-               if( c != 0xff  )
-                   break;
-           i++;
-           if( c != DIGEST_ALGO_RMD160 || mpi_getbyte(result, i) ) {
-               /* Padding or leading bytes in signature is wrong */
-               rc = G10ERR_BAD_PUBKEY;
-               goto leave;
-           }
-           if( mpi_getbyte(result, 19) != sig->d.rsa.digest_start[0]
-               || mpi_getbyte(result, 18) != sig->d.rsa.digest_start[1] ) {
-               /* Wrong key used to check the signature */
-               rc = G10ERR_BAD_PUBKEY;
-               goto leave;
-           }
+    rc = G10ERR_BAD_SIGN;
+    {  const byte *s1 = md_read( digest, sig->digest_algo );
+       int s1len = md_digest_length( sig->digest_algo );
 
-           /* complete the digest */
-           md_putc( digest, sig->sig_class );
-           {   u32 a = sig->timestamp;
-               md_putc( digest, (a >> 24) & 0xff );
-               md_putc( digest, (a >> 16) & 0xff );
-               md_putc( digest, (a >>  8) & 0xff );
-               md_putc( digest,  a        & 0xff );
-           }
-           md_final( digest );
-           dp = md_read( digest, DIGEST_ALGO_RMD160 );
-           for(i=19; i >= 0; i--, dp++ )
-               if( mpi_getbyte( result, i ) != *dp ) {
-                   rc = G10ERR_BAD_SIGN;
-                   goto leave;
-               }
+       log_hexdump( "MDC calculated", s1, s1len );
+
+       if( !sig->data[0] )
+           log_debug("sig_data[0] is NULL\n");
+       else {
+           unsigned s2len;
+           byte *s2;
+           s2 = mpi_get_buffer( sig->data[0], &s2len, NULL );
+           log_hexdump( "MDC stored    ", s2, s2len );
+
+           if( s2len != s1len )
+               log_debug("MDC check: len differ: %d/%d\n", s1len, s2len);
+           else if( memcmp( s1, s2, s1len ) )
+               log_debug("MDC check: hashs differ\n");
+           else
+               rc = 0;
+           m_free(s2);
        }
-       else if( sig->d.rsa.digest_algo == DIGEST_ALGO_MD5 ) {
-           static byte asn[18] = /* stored reverse */
-                 { 0x10, 0x04, 0x00, 0x05, 0x05, 0x02, 0x0d, 0xf7, 0x86,
-                   0x48, 0x86, 0x2a, 0x08, 0x06, 0x0c, 0x30, 0x20, 0x30 };
-
-           for(i=16,j=0; j < 18 && (c=mpi_getbyte(result, i)) != -1; i++, j++ )
-               if( asn[j] != c )
-                   break;
-           if( j != 18 || mpi_getbyte(result, i) ) { /* ASN is wrong */
-               rc = G10ERR_BAD_PUBKEY;
-               goto leave;
-           }
-           for(i++; (c=mpi_getbyte(result, i)) != -1; i++ )
-               if( c != 0xff  )
-                   break;
-           i++;
-           if( c != DIGEST_ALGO_MD5 || mpi_getbyte(result, i) ) {
-               /* Padding or leading bytes in signature is wrong */
-               rc = G10ERR_BAD_PUBKEY;
-               goto leave;
-           }
-           if( mpi_getbyte(result, 15) != sig->d.rsa.digest_start[0]
-               || mpi_getbyte(result, 14) != sig->d.rsa.digest_start[1] ) {
-               /* Wrong key used to check the signature */
-               rc = G10ERR_BAD_PUBKEY;
-               goto leave;
-           }
+    }
 
-           /* complete the digest */
-           md_putc( digest, sig->sig_class );
-           {   u32 a = sig->timestamp;
-               md_putc( digest, (a >> 24) & 0xff );
-               md_putc( digest, (a >> 16) & 0xff );
-               md_putc( digest, (a >>  8) & 0xff );
-               md_putc( digest,  a        & 0xff );
-           }
-           md_final( digest );
-           dp = md_read( digest, DIGEST_ALGO_MD5 );
-           for(i=15; i >= 0; i--, dp++ )
-               if( mpi_getbyte( result, i ) != *dp ) {
-                   rc = G10ERR_BAD_SIGN;
-                   goto leave;
-               }
+    if( !rc && sig->flags.unknown_critical ) {
+       log_info(_("assuming bad MDC due to an unknown critical bit\n"));
+       rc = G10ERR_BAD_SIGN;
+    }
+    sig->flags.checked = 1;
+    sig->flags.valid = !rc;
+
+    /* FIXME: check that we are actually in an encrypted packet */
+
+    return rc;
+}
+#endif
+
+/****************
+ * This function gets called by pubkey_verify() if the algorithm needs it.
+ */
+static int
+cmp_help( void *opaque, MPI result )
+{
+  #if 0 /* we do not use this anymore */
+    int rc=0, i, j, c, old_enc;
+    byte *dp;
+    const byte *asn;
+    size_t mdlen, asnlen;
+    struct cmp_help_context_s *ctx = opaque;
+    PKT_signature *sig = ctx->sig;
+    GCRY_MD_HD digest = ctx->md;
+
+    old_enc = 0;
+    for(i=j=0; (c=mpi_getbyte(result, i)) != -1; i++ ) {
+       if( !j ) {
+           if( !i && c != 1 )
+               break;
+           else if( i && c == 0xff )
+               ; /* skip the padding */
+           else if( i && !c )
+               j++;
+           else
+               break;
        }
-       else {
-           rc = G10ERR_DIGEST_ALGO;
-           goto leave;
+       else if( ++j == 18 && c != 1 )
+           break;
+       else if( j == 19 && c == 0 ) {
+           old_enc++;
+           break;
        }
     }
-  #endif/*HAVE_RSA_CIPHER*/
+    if( old_enc ) {
+       log_error("old encoding scheme is not supported\n");
+       return G10ERR_GENERAL;
+    }
+
+    if( (rc=check_digest_algo(sig->digest_algo)) )
+       return rc; /* unsupported algo */
+    asn = md_asn_oid( sig->digest_algo, &asnlen, &mdlen );
+
+    for(i=mdlen,j=asnlen-1; (c=mpi_getbyte(result, i)) != -1 && j >= 0;
+                                                          i++, j-- )
+       if( asn[j] != c )
+           break;
+    if( j != -1 || mpi_getbyte(result, i) )
+       return G10ERR_BAD_PUBKEY;  /* ASN is wrong */
+    for(i++; (c=mpi_getbyte(result, i)) != -1; i++ )
+       if( c != 0xff  )
+           break;
+    i++;
+    if( c != sig->digest_algo || mpi_getbyte(result, i) ) {
+       /* Padding or leading bytes in signature is wrong */
+       return G10ERR_BAD_PUBKEY;
+    }
+    if( mpi_getbyte(result, mdlen-1) != sig->digest_start[0]
+       || mpi_getbyte(result, mdlen-2) != sig->digest_start[1] ) {
+       /* Wrong key used to check the signature */
+       return G10ERR_BAD_PUBKEY;
+    }
+
+    dp = md_read( digest, sig->digest_algo );
+    for(i=mdlen-1; i >= 0; i--, dp++ ) {
+       if( mpi_getbyte( result, i ) != *dp )
+           return G10ERR_BAD_SIGN;
+    }
+    return 0;
+  #else
+    return -1;
+  #endif
+}
+
+
+static int
+do_check( PKT_public_key *pk, PKT_signature *sig, GCRY_MD_HD digest )
+{
+    MPI result = NULL;
+    int rc=0;
+    struct cmp_help_context_s ctx;
+    u32 cur_time;
+
+    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"));
+       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 );
+       return G10ERR_TIME_CONFLICT; /* pubkey newer than signature */
+    }
+
+    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 "
+                          "in future (time warp or clock problem)\n")
+                      : _("key has been created %lu seconds "
+                          "in future (time warp or clock problem)\n"), d );
+       return G10ERR_TIME_CONFLICT;
+    }
+
+    if( pk->expiredate && pk->expiredate < cur_time ) {
+       log_info(_("NOTE: signature key expired %s\n"),
+                                       asctimestamp( pk->expiredate ) );
+       write_status(STATUS_SIGEXPIRED);
+    }
+
+
+    if( (rc=openpgp_md_test_algo(sig->digest_algo)) )
+       return rc;
+    if( (rc=openpgp_pk_test_algo(sig->pubkey_algo)) )
+       return rc;
+
+    /* make sure the digest algo is enabled (in case of a detached signature)*/
+    gcry_md_enable( digest, sig->digest_algo );
+
+    /* complete the digest */
+    if( sig->version >= 4 )
+       gcry_md_putc( digest, sig->version );
+    gcry_md_putc( digest, sig->sig_class );
+    if( sig->version < 4 ) {
+       u32 a = sig->timestamp;
+       gcry_md_putc( digest, (a >> 24) & 0xff );
+       gcry_md_putc( digest, (a >> 16) & 0xff );
+       gcry_md_putc( digest, (a >>  8) & 0xff );
+       gcry_md_putc( digest,  a        & 0xff );
+    }
     else {
-       /*log_debug("signature_check: unsupported pubkey algo %d\n",
-                       pkc->pubkey_algo );*/
-       rc = G10ERR_PUBKEY_ALGO;
-       goto leave;
+       byte buf[6];
+       size_t n;
+       gcry_md_putc( digest, sig->pubkey_algo );
+       gcry_md_putc( digest, sig->digest_algo );
+       if( sig->hashed_data ) {
+           n = (sig->hashed_data[0] << 8) | sig->hashed_data[1];
+           gcry_md_write( digest, sig->hashed_data, n+2 );
+           n += 6;
+       }
+       else
+           n = 6;
+       /* add some magic */
+       buf[0] = sig->version;
+       buf[1] = 0xff;
+       buf[2] = n >> 24;
+       buf[3] = n >> 16;
+       buf[4] = n >>  8;
+       buf[5] = n;
+       gcry_md_write( digest, buf, 6 );
     }
+    gcry_md_final( digest );
 
+    result = encode_md_value( pk->pubkey_algo, digest, sig->digest_algo,
+                                     mpi_get_nbits(pk->pkey[0]));
 
-  leave:
-    if( pkc )
-       free_public_cert( pkc );
+    ctx.sig = sig;
+    ctx.md = digest;
+    rc = pubkey_verify( pk->pubkey_algo, result, sig->data, pk->pkey,
+                       cmp_help, &ctx );
     mpi_free( result );
+    if( !rc && sig->flags.unknown_critical ) {
+       log_info(_("assuming bad signature due to an unknown critical bit\n"));
+       rc = G10ERR_BAD_SIGN;
+    }
+    sig->flags.checked = 1;
+    sig->flags.valid = !rc;
+
     return rc;
 }
 
 
+static void
+hash_uid_node( KBNODE unode, GCRY_MD_HD md, PKT_signature *sig )
+{
+    PKT_user_id *uid = unode->pkt->pkt.user_id;
+
+    assert( unode->pkt->pkttype == PKT_USER_ID );
+    if( sig->version >=4 ) {
+       byte buf[5];
+       buf[0] = 0xb4; /* indicates a userid packet */
+       buf[1] = uid->len >> 24;  /* always use 4 length bytes */
+       buf[2] = uid->len >> 16;
+       buf[3] = uid->len >>  8;
+       buf[4] = uid->len;
+       gcry_md_write( md, buf, 5 );
+    }
+    gcry_md_write( md, uid->name, uid->len );
+}
+
 /****************
- * check the signature pointed to by NODE. This is a key signatures
+ * check the signature pointed to by NODE. This is a key signature.
+ * If the function detects a self-signature, it uses the PK from
+ * ROOT and does not read any public key.
  */
 int
 check_key_signature( KBNODE root, KBNODE node, int *is_selfsig )
 {
-    KBNODE unode;
-    MD_HANDLE md;
-    PKT_public_cert *pkc;
+    u32 dummy;
+    return check_key_signature2(root, node, is_selfsig, &dummy );
+}
+
+int
+check_key_signature2( KBNODE root, KBNODE node, int *is_selfsig, u32 *r_expire)
+{
+    GCRY_MD_HD md;
+    PKT_public_key *pk;
     PKT_signature *sig;
     int algo;
     int rc;
 
     if( is_selfsig )
        *is_selfsig = 0;
+    *r_expire = 0;
     assert( node->pkt->pkttype == PKT_SIGNATURE );
-    assert( (node->pkt->pkt.signature->sig_class&~3) == 0x10 );
-    assert( root->pkt->pkttype == PKT_PUBLIC_CERT );
+    assert( root->pkt->pkttype == PKT_PUBLIC_KEY );
 
-    pkc = root->pkt->pkt.public_cert;
+    pk = root->pkt->pkt.public_key;
     sig = node->pkt->pkt.signature;
+    algo = sig->digest_algo;
 
-    if( sig->pubkey_algo == PUBKEY_ALGO_ELGAMAL )
-       algo = sig->d.elg.digest_algo;
-    else if(sig->pubkey_algo == PUBKEY_ALGO_RSA )
-       algo = sig->d.rsa.digest_algo;
-    else
-       return G10ERR_PUBKEY_ALGO;
-    if( (rc=check_digest_algo(algo)) )
+  #if 0 /* I am not sure whether this is a good thing to do */
+    if( sig->flags.checked )
+       log_debug("check_key_signature: already checked: %s\n",
+                     sig->flags.valid? "good":"bad" );
+  #endif
+
+    if( (rc=openpgp_md_test_algo(algo)) )
        return rc;
 
-    unode = find_prev_kbnode( root, node, PKT_USER_ID );
+    if( sig->sig_class == 0x20 ) {
+       if( !(md = gcry_md_open( algo, 0 )) )
+           BUG();
+       hash_public_key( md, pk );
+       rc = do_check( pk, sig, md );
+       gcry_md_close(md);
+    }
+    else if( sig->sig_class == 0x28 ) { /* subkey revocation */
+       KBNODE snode = find_prev_kbnode( root, node, PKT_PUBLIC_SUBKEY );
 
-    if( unode ) {
-       PKT_user_id *uid = unode->pkt->pkt.user_id;
+       if( snode ) {
+           if( !(md = gcry_md_open( algo, 0 )) )
+               BUG();
+           hash_public_key( md, pk );
+           hash_public_key( md, snode->pkt->pkt.public_key );
+           rc = do_check( pk, sig, md );
+           gcry_md_close(md);
+       }
+       else {
+           log_error("no subkey for subkey revocation packet\n");
+           rc = G10ERR_SIG_CLASS;
+       }
+    }
+    else if( sig->sig_class == 0x18 ) {
+       KBNODE snode = find_prev_kbnode( root, node, PKT_PUBLIC_SUBKEY );
 
-       if( is_selfsig ) {
-           u32 keyid[2];
+       if( snode ) {
+           if( is_selfsig ) {  /* does this make sense????? */
+               u32 keyid[2];   /* it should always be a selfsig */
 
-           keyid_from_pkc( pkc, keyid );
-           if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] )
-               *is_selfsig = 1;
+               keyid_from_pk( pk, keyid );
+               if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] )
+                   *is_selfsig = 1;
+           }
+           if( !(md = gcry_md_open( algo, 0 )) )
+               BUG();
+           hash_public_key( md, pk );
+           hash_public_key( md, snode->pkt->pkt.public_key );
+           rc = do_check( pk, sig, md );
+           gcry_md_close(md);
+       }
+       else {
+           log_error("no subkey for key signature packet\n");
+           rc = G10ERR_SIG_CLASS;
        }
-       md = md_open( algo, 0 );
-       hash_public_cert( md, pkc );
-       md_write( md, uid->name, uid->len );
-       rc = signature_check( sig, md );
-       md_close(md);
     }
     else {
-       log_error("no user id for key signature packet\n");
-       rc = G10ERR_SIG_CLASS;
+       KBNODE unode = find_prev_kbnode( root, node, PKT_USER_ID );
+
+       if( unode ) {
+           u32 keyid[2];
+
+           keyid_from_pk( pk, keyid );
+           if( !(md = gcry_md_open( algo, 0 )) )
+               BUG();
+           hash_public_key( md, pk );
+           hash_uid_node( unode, md, sig );
+           if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] ) {
+               if( is_selfsig )
+                   *is_selfsig = 1;
+               rc = do_check( pk, sig, md );
+           }
+           else
+               rc = do_signature_check( sig, md, r_expire );
+           gcry_md_close(md);
+       }
+       else {
+           log_error("no user ID for key signature packet\n");
+           rc = G10ERR_SIG_CLASS;
+       }
     }
 
     return rc;