* options.h, g10.c (main), keyserver.c (keyserver_refresh): Maintain and
[gnupg.git] / g10 / keyid.c
index ef7b1ba..43e531e 100644 (file)
@@ -1,14 +1,14 @@
-/* keyid.c - jeyid and fingerprint handling
- *     Copyright (C) 1998 Free Software Foundation, Inc.
+/* keyid.c - key ID and fingerprint handling
+ * Copyright (C) 1998, 1999, 2000, 2001 Free Software Foundation, Inc.
  *
- * This file is part of GNUPG.
+ * This file is part of GnuPG.
  *
- * GNUPG 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.
  *
- * GNUPG 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.
@@ -31,6 +31,7 @@
 #include "options.h"
 #include "mpi.h"
 #include "keydb.h"
+#include "i18n.h"
 
 
 int
@@ -40,163 +41,92 @@ pubkey_letter( int algo )
       case PUBKEY_ALGO_RSA:    return 'R' ;
       case PUBKEY_ALGO_RSA_E:  return 'r' ;
       case PUBKEY_ALGO_RSA_S:  return 's' ;
+      case PUBKEY_ALGO_ELGAMAL_E: return 'g';
       case PUBKEY_ALGO_ELGAMAL: return 'G' ;
       case PUBKEY_ALGO_DSA:    return 'D' ;
       default: return '?';
     }
 }
 
-/* this is special code for V3 which uses ElGamal and
- * calculates a fingerprint like V4, but with rmd160
- * and a version byte of 3. Returns an md handle, caller must
- * do md_close()
- */
-
 static MD_HANDLE
-v3_elg_fingerprint_md( PKT_public_cert *pkc )
+do_fingerprint_md( PKT_public_key *pk )
 {
     MD_HANDLE md;
-    byte *buf1, *buf2, *buf3;
-    byte *p1, *p2, *p3;
-    unsigned n1, n2, n3;
-    unsigned nb1, nb2, nb3;
     unsigned n;
-
-    nb1 = mpi_get_nbits(pkc->d.elg.p);
-    p1 = buf1 = mpi_get_buffer( pkc->d.elg.p, &n1, NULL );
-    for( ; !*p1 && n1; p1++, n1-- )  /* skip leading null bytes */
-       ;
-    nb2 = mpi_get_nbits(pkc->d.elg.g);
-    p2 = buf2 = mpi_get_buffer( pkc->d.elg.g, &n2, NULL );
-    for( ; !*p2 && n2; p2++, n2-- )  /* skip leading null bytes */
-       ;
-    nb3 = mpi_get_nbits(pkc->d.elg.y);
-    p3 = buf3 = mpi_get_buffer( pkc->d.elg.y, &n3, NULL );
-    for( ; !*p3 && n3; p3++, n3-- )  /* skip leading null bytes */
-       ;
-
-    /* calculate length of packet (1+4+2+1+2+n1+2+n2+2+n3) */
-    n = 14 + n1 + n2 + n3;
-    md = md_open( DIGEST_ALGO_RMD160, 0);
+    unsigned nb[PUBKEY_MAX_NPKEY];
+    unsigned nn[PUBKEY_MAX_NPKEY];
+    byte *pp[PUBKEY_MAX_NPKEY];
+    int i;
+    int npkey = pubkey_get_npkey( pk->pubkey_algo );
+
+    md = md_open( pk->version < 4 ? DIGEST_ALGO_RMD160 : DIGEST_ALGO_SHA1, 0);
+    n = pk->version < 4 ? 8 : 6;
+    for(i=0; i < npkey; i++ ) {
+       nb[i] = mpi_get_nbits(pk->pkey[i]);
+       pp[i] = mpi_get_buffer( pk->pkey[i], nn+i, NULL );
+       n += 2 + nn[i];
+    }
 
     md_putc( md, 0x99 );     /* ctb */
     md_putc( md, n >> 8 );   /* 2 byte length header */
     md_putc( md, n );
-    md_putc( md, 3 );       /* version */
-    {  u32 a = pkc->timestamp;
+    if( pk->version < 4 )
+       md_putc( md, 3 );
+    else
+       md_putc( md, 4 );
+
+    {  u32 a = pk->timestamp;
        md_putc( md, a >> 24 );
        md_putc( md, a >> 16 );
        md_putc( md, a >>  8 );
        md_putc( md, a       );
     }
-    {  u16 a = pkc->valid_days;
+    if( pk->version < 4 ) {
+       u16 a;
+
+       if( pk->expiredate )
+           a = (u16)((pk->expiredate - pk->timestamp) / 86400L);
+       else
+           a = 0;
        md_putc( md, a >> 8 );
        md_putc( md, a      );
     }
-    md_putc( md, pkc->pubkey_algo );
-    md_putc( md, nb1>>8); md_putc( md, nb1 ); md_write( md, p1, n1 );
-    md_putc( md, nb2>>8); md_putc( md, nb2 ); md_write( md, p2, n2 );
-    md_putc( md, nb3>>8); md_putc( md, nb3 ); md_write( md, p3, n3 );
-    m_free(buf1);
-    m_free(buf2);
-    m_free(buf3);
-    md_final( md );
-
-    return md;
-}
-
-static MD_HANDLE
-dsa_fingerprint_md( PKT_public_cert *pkc )
-{
-    MD_HANDLE md;
-    byte *buf1, *buf2, *buf3, *buf4 ;
-    byte *p1, *p2, *p3, *p4;
-    unsigned n1, n2, n3, n4;
-    unsigned nb1, nb2, nb3, nb4;
-    unsigned n;
-
-    nb1 = mpi_get_nbits(pkc->d.dsa.p);
-    p1 = buf1 = mpi_get_buffer( pkc->d.dsa.p, &n1, NULL );
-    for( ; !*p1 && n1; p1++, n1-- )  /* skip leading null bytes */
-       ;
-    nb2 = mpi_get_nbits(pkc->d.dsa.q);
-    p2 = buf2 = mpi_get_buffer( pkc->d.dsa.q, &n2, NULL );
-    for( ; !*p2 && n2; p2++, n2-- )
-       ;
-    nb3 = mpi_get_nbits(pkc->d.dsa.g);
-    p3 = buf3 = mpi_get_buffer( pkc->d.dsa.g, &n3, NULL );
-    for( ; !*p3 && n3; p3++, n3-- )
-       ;
-    nb4 = mpi_get_nbits(pkc->d.dsa.y);
-    p4 = buf4 = mpi_get_buffer( pkc->d.dsa.y, &n4, NULL );
-    for( ; !*p4 && n4; p4++, n4-- )
-       ;
-
-    /* calculate length of packet */
-    n = 14 + n1 + n2 + n3 +n4 ;
-    md = md_open( DIGEST_ALGO_SHA1, 0);
-
-    md_putc( md, 0x99 );     /* ctb */
-    md_putc( md, n >> 8 );   /* 2 byte length header */
-    md_putc( md, n );
-    md_putc( md, 4 );       /* version */
-    {  u32 a = pkc->timestamp;
-       md_putc( md, a >> 24 );
-       md_putc( md, a >> 16 );
-       md_putc( md, a >>  8 );
-       md_putc( md, a       );
+    md_putc( md, pk->pubkey_algo );
+    for(i=0; i < npkey; i++ ) {
+       md_putc( md, nb[i]>>8);
+       md_putc( md, nb[i] );
+       md_write( md, pp[i], nn[i] );
+       m_free(pp[i]);
     }
-    md_putc( md, pkc->pubkey_algo );
-    md_putc( md, nb1>>8); md_putc( md, nb1 ); md_write( md, p1, n1 );
-    md_putc( md, nb2>>8); md_putc( md, nb2 ); md_write( md, p2, n2 );
-    md_putc( md, nb3>>8); md_putc( md, nb3 ); md_write( md, p3, n3 );
-    md_putc( md, nb4>>8); md_putc( md, nb4 ); md_write( md, p4, n4 );
-    m_free(buf1);
-    m_free(buf2);
-    m_free(buf3);
-    m_free(buf4);
     md_final( md );
 
     return md;
 }
 
 static MD_HANDLE
-v3_elg_fingerprint_md_skc( PKT_secret_cert *skc )
-{
-    PKT_public_cert pkc;
-
-    pkc.pubkey_algo = skc->pubkey_algo;
-    pkc.timestamp = skc->timestamp;
-    pkc.valid_days = skc->valid_days;
-    pkc.pubkey_algo = skc->pubkey_algo;
-    pkc.d.elg.p = skc->d.elg.p;
-    pkc.d.elg.g = skc->d.elg.g;
-    pkc.d.elg.y = skc->d.elg.y;
-    return v3_elg_fingerprint_md( &pkc );
-}
-
-static MD_HANDLE
-dsa_fingerprint_md_skc( PKT_secret_cert *skc )
+do_fingerprint_md_sk( PKT_secret_key *sk )
 {
-    PKT_public_cert pkc;
-
-    pkc.pubkey_algo = skc->pubkey_algo;
-    pkc.timestamp = skc->timestamp;
-    pkc.pubkey_algo = skc->pubkey_algo;
-    pkc.d.dsa.p = skc->d.dsa.p;
-    pkc.d.dsa.q = skc->d.dsa.q;
-    pkc.d.dsa.g = skc->d.dsa.g;
-    pkc.d.dsa.y = skc->d.dsa.y;
-    return dsa_fingerprint_md( &pkc );
+    PKT_public_key pk;
+    int npkey = pubkey_get_npkey( sk->pubkey_algo ); /* npkey is correct! */
+    int i;
+
+    pk.pubkey_algo = sk->pubkey_algo;
+    pk.version    = sk->version;
+    pk.timestamp = sk->timestamp;
+    pk.expiredate = sk->expiredate;
+    pk.pubkey_algo = sk->pubkey_algo;
+    for( i=0; i < npkey; i++ )
+       pk.pkey[i] = sk->skey[i];
+    return do_fingerprint_md( &pk );
 }
 
 
 /****************
- * Get the keyid from the secret key certificate and put it into keyid
+ * Get the keyid from the secret key and put it into keyid
  * if this is not NULL. Return the 32 low bits of the keyid.
  */
 u32
-keyid_from_skc( PKT_secret_cert *skc, u32 *keyid )
+keyid_from_sk( PKT_secret_key *sk, u32 *keyid )
 {
     u32 lowbits;
     u32 dummy_keyid[2];
@@ -204,42 +134,31 @@ keyid_from_skc( PKT_secret_cert *skc, u32 *keyid )
     if( !keyid )
        keyid = dummy_keyid;
 
-    if( skc->pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
-       const byte *dp;
-       MD_HANDLE md;
-       md = v3_elg_fingerprint_md_skc(skc);
-       dp = md_read( md, DIGEST_ALGO_RMD160 );
-       keyid[0] = dp[12] << 24 | dp[13] << 16 | dp[14] << 8 | dp[15] ;
-       keyid[1] = dp[16] << 24 | dp[17] << 16 | dp[18] << 8 | dp[19] ;
-       lowbits = keyid[1];
-       md_close(md);
+    if( sk->version < 4 && is_RSA(sk->pubkey_algo) ) {
+       lowbits = pubkey_get_npkey(sk->pubkey_algo) ?
+                    mpi_get_keyid( sk->skey[0], keyid ) : 0; /* take n */
     }
-    else if( skc->pubkey_algo == PUBKEY_ALGO_DSA ) {
+    else {
        const byte *dp;
        MD_HANDLE md;
-       md = dsa_fingerprint_md_skc(skc);
-       dp = md_read( md, DIGEST_ALGO_SHA1 );
+       md = do_fingerprint_md_sk(sk);
+       dp = md_read( md, 0 );
        keyid[0] = dp[12] << 24 | dp[13] << 16 | dp[14] << 8 | dp[15] ;
        keyid[1] = dp[16] << 24 | dp[17] << 16 | dp[18] << 8 | dp[19] ;
        lowbits = keyid[1];
        md_close(md);
     }
-    else if( skc->pubkey_algo == PUBKEY_ALGO_RSA ) {
-       lowbits = mpi_get_keyid( skc->d.rsa.rsa_n, keyid );
-    }
-    else {
-       keyid[0] = keyid[1] = lowbits = 0;
-    }
+
     return lowbits;
 }
 
 
 /****************
- * Get the keyid from the public key certificate and put it into keyid
+ * Get the keyid from the public key and put it into keyid
  * if this is not NULL. Return the 32 low bits of the keyid.
  */
 u32
-keyid_from_pkc( PKT_public_cert *pkc, u32 *keyid )
+keyid_from_pk( PKT_public_key *pk, u32 *keyid )
 {
     u32 lowbits;
     u32 dummy_keyid[2];
@@ -247,34 +166,68 @@ keyid_from_pkc( PKT_public_cert *pkc, u32 *keyid )
     if( !keyid )
        keyid = dummy_keyid;
 
-    if( pkc->pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
-       const byte *dp;
-       MD_HANDLE md;
-       md = v3_elg_fingerprint_md(pkc);
-       dp = md_read( md, DIGEST_ALGO_RMD160 );
-       keyid[0] = dp[12] << 24 | dp[13] << 16 | dp[14] << 8 | dp[15] ;
-       keyid[1] = dp[16] << 24 | dp[17] << 16 | dp[18] << 8 | dp[19] ;
+    if( pk->keyid[0] || pk->keyid[1] ) {
+       keyid[0] = pk->keyid[0];
+       keyid[1] = pk->keyid[1];
        lowbits = keyid[1];
-       md_close(md);
     }
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_DSA ) {
+    else if( pk->version < 4 && is_RSA(pk->pubkey_algo) ) {
+       lowbits = pubkey_get_npkey(pk->pubkey_algo) ?
+                    mpi_get_keyid( pk->pkey[0], keyid ) : 0 ; /* from n */
+       pk->keyid[0] = keyid[0];
+       pk->keyid[1] = keyid[1];
+    }
+    else {
        const byte *dp;
        MD_HANDLE md;
-       md = dsa_fingerprint_md(pkc);
-       dp = md_read( md, DIGEST_ALGO_SHA1 );
+       md = do_fingerprint_md(pk);
+       dp = md_read( md, 0 );
        keyid[0] = dp[12] << 24 | dp[13] << 16 | dp[14] << 8 | dp[15] ;
        keyid[1] = dp[16] << 24 | dp[17] << 16 | dp[18] << 8 | dp[19] ;
        lowbits = keyid[1];
        md_close(md);
+       pk->keyid[0] = keyid[0];
+       pk->keyid[1] = keyid[1];
     }
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_RSA ) {
-       lowbits = mpi_get_keyid( pkc->d.rsa.rsa_n, keyid );
+
+    return lowbits;
+}
+
+
+/****************
+ * Get the keyid from the fingerprint. This function is simple for most
+ * keys, but has to do a keylookup for old stayle keys.
+ */
+u32
+keyid_from_fingerprint( const byte *fprint, size_t fprint_len, u32 *keyid )
+{
+    u32 dummy_keyid[2];
+
+    if( !keyid )
+       keyid = dummy_keyid;
+
+    if( fprint_len != 20 ) {
+       /* This is special as we have to lookup the key first */
+       PKT_public_key pk;
+       int rc;
+
+       memset( &pk, 0, sizeof pk );
+       rc = get_pubkey_byfprint( &pk, fprint, fprint_len );
+       if( rc ) {
+           log_error("Oops: keyid_from_fingerprint: no pubkey\n");
+           keyid[0] = 0;
+           keyid[1] = 0;
+       }
+       else
+           keyid_from_pk( &pk, keyid );
     }
     else {
-       keyid[0] = keyid[1] = lowbits = 0;
+       const byte *dp = fprint;
+       keyid[0] = dp[12] << 24 | dp[13] << 16 | dp[14] << 8 | dp[15] ;
+       keyid[1] = dp[16] << 24 | dp[17] << 16 | dp[18] << 8 | dp[19] ;
     }
 
-    return lowbits;
+    return keyid[1];
 }
 
 
@@ -289,175 +242,256 @@ keyid_from_sig( PKT_signature *sig, u32 *keyid )
 }
 
 /****************
- * return the number of bits used in the pkc
+ * return the number of bits used in the pk
  */
 unsigned
-nbits_from_pkc( PKT_public_cert *pkc )
+nbits_from_pk( PKT_public_key *pk )
 {
-    if( pkc->pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
-       return mpi_get_nbits( pkc->d.elg.p );
-    }
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_DSA ) {
-       return mpi_get_nbits( pkc->d.dsa.p );
-    }
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_RSA ) {
-       return mpi_get_nbits( pkc->d.rsa.rsa_n );
-    }
-    else
-       return 0;
+    return pubkey_nbits( pk->pubkey_algo, pk->pkey );
 }
 
 /****************
- * return the number of bits used in the skc
+ * return the number of bits used in the sk
  */
 unsigned
-nbits_from_skc( PKT_secret_cert *skc )
+nbits_from_sk( PKT_secret_key *sk )
 {
-    if( skc->pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
-       return mpi_get_nbits( skc->d.elg.p );
-    }
-    else if( skc->pubkey_algo == PUBKEY_ALGO_DSA ) {
-       return mpi_get_nbits( skc->d.dsa.p );
-    }
-    else if( skc->pubkey_algo == PUBKEY_ALGO_RSA ) {
-       return mpi_get_nbits( skc->d.rsa.rsa_n );
+    return pubkey_nbits( sk->pubkey_algo, sk->skey );
+}
+
+static const char *
+mk_datestr (char *buffer, time_t atime)
+{
+    struct tm *tp;
+
+    if ( atime < 0 ) /* 32 bit time_t and after 2038-01-19 */
+        strcpy (buffer, "????" "-??" "-??"); /* mark this as invalid */
+    else {
+        tp = gmtime (&atime);
+        sprintf (buffer,"%04d-%02d-%02d",
+                 1900+tp->tm_year, tp->tm_mon+1, tp->tm_mday );
     }
-    else
-       return 0;
+    return buffer;
 }
 
 /****************
- * return a string with the creation date of the pkc
+ * return a string with the creation date of the pk
  * Note: this is alloced in a static buffer.
  *    Format is: yyyy-mm-dd
  */
 const char *
-datestr_from_pkc( PKT_public_cert *pkc )
+datestr_from_pk( PKT_public_key *pk )
 {
     static char buffer[11+5];
-    struct tm *tp;
-    time_t atime = pkc->timestamp;
+    time_t atime = pk->timestamp;
 
-    tp = gmtime( &atime );
-    sprintf(buffer,"%04d-%02d-%02d", 1900+tp->tm_year, tp->tm_mon+1, tp->tm_mday );
-    return buffer;
+    return mk_datestr (buffer, atime);
 }
 
 const char *
-datestr_from_skc( PKT_secret_cert *skc )
+datestr_from_sk( PKT_secret_key *sk )
 {
     static char buffer[11+5];
-    struct tm *tp;
-    time_t atime = skc->timestamp;
+    time_t atime = sk->timestamp;
 
-    tp = gmtime( &atime );
-    sprintf(buffer,"%04d-%02d-%02d", 1900+tp->tm_year, tp->tm_mon+1, tp->tm_mday );
-    return buffer;
+    return mk_datestr (buffer, atime);
 }
 
 const char *
 datestr_from_sig( PKT_signature *sig )
 {
     static char buffer[11+5];
-    struct tm *tp;
     time_t atime = sig->timestamp;
 
-    tp = gmtime( &atime );
-    sprintf(buffer,"%04d-%02d-%02d", 1900+tp->tm_year, tp->tm_mon+1, tp->tm_mday );
-    return buffer;
+    return mk_datestr (buffer, atime);
 }
 
+const char *
+expirestr_from_pk( PKT_public_key *pk )
+{
+    static char buffer[11+5];
+    time_t atime;
 
-/**************** .
- * Return a byte array with the fingerprint for the given PKC/SKC
- * The length of the array is returned in ret_len. Caller must free
- * the array.
- */
-byte *
-fingerprint_from_skc( PKT_secret_cert *skc, size_t *ret_len )
+    if( !pk->expiredate )
+       return _("never     ");
+    atime = pk->expiredate;
+    return mk_datestr (buffer, atime);
+}
+
+const char *
+expirestr_from_sk( PKT_secret_key *sk )
+{
+    static char buffer[11+5];
+    time_t atime;
+
+    if( !sk->expiredate )
+       return _("never     ");
+    atime = sk->expiredate;
+    return mk_datestr (buffer, atime);
+}
+
+const char *
+expirestr_from_sig( PKT_signature *sig )
+{
+    static char buffer[11+5];
+    time_t atime;
+
+    if(!sig->expiredate)
+      return _("never     ");
+    atime=sig->expiredate;
+    return mk_datestr (buffer, atime);
+}
+
+const char *
+colon_strtime (u32 t)
 {
-    PKT_public_cert pkc;
-    byte *p;
-
-    pkc.pubkey_algo = skc->pubkey_algo;
-    if( pkc.pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
-       pkc.timestamp = skc->timestamp;
-       pkc.valid_days = skc->valid_days;
-       pkc.pubkey_algo = skc->pubkey_algo;
-       pkc.d.elg.p = skc->d.elg.p;
-       pkc.d.elg.g = skc->d.elg.g;
-       pkc.d.elg.y = skc->d.elg.y;
+    if (!t)
+        return "";
+    if (opt.fixed_list_mode) {
+        static char buf[15];
+        sprintf (buf, "%lu", (ulong)t);
+        return buf;
     }
-    else if( pkc.pubkey_algo == PUBKEY_ALGO_DSA ) {
-       pkc.timestamp = skc->timestamp;
-       pkc.valid_days = skc->valid_days;
-       pkc.pubkey_algo = skc->pubkey_algo;
-       pkc.d.dsa.p = skc->d.dsa.p;
-       pkc.d.dsa.q = skc->d.dsa.q;
-       pkc.d.dsa.g = skc->d.dsa.g;
-       pkc.d.dsa.y = skc->d.dsa.y;
+    return strtimestamp(t);
+}
+
+const char *
+colon_datestr_from_pk (PKT_public_key *pk)
+{
+    if (opt.fixed_list_mode) {
+        static char buf[15];
+        sprintf (buf, "%lu", (ulong)pk->timestamp);
+        return buf;
     }
-    else if( pkc.pubkey_algo == PUBKEY_ALGO_RSA ) {
-       pkc.d.rsa.rsa_n = skc->d.rsa.rsa_n;
-       pkc.d.rsa.rsa_e = skc->d.rsa.rsa_e;
+    return datestr_from_pk (pk);
+}
+
+const char *
+colon_datestr_from_sk (PKT_secret_key *sk)
+{
+    if (opt.fixed_list_mode) {
+        static char buf[15];
+        sprintf (buf, "%lu", (ulong)sk->timestamp);
+        return buf;
     }
-    p = fingerprint_from_pkc( &pkc, ret_len );
-    memset(&pkc, 0, sizeof pkc); /* not really needed */
-    return p;
+    return datestr_from_sk (sk);
 }
 
+const char *
+colon_datestr_from_sig (PKT_signature *sig)
+{
+    if (opt.fixed_list_mode) {
+        static char buf[15];
+        sprintf (buf, "%lu", (ulong)sig->timestamp);
+        return buf;
+    }
+    return datestr_from_sig (sig);
+}
+
+const char *
+colon_expirestr_from_sig (PKT_signature *sig)
+{
+    if(!sig->expiredate)
+        return "";
+    if (opt.fixed_list_mode) {
+        static char buf[15];
+        sprintf (buf, "%lu", (ulong)sig->expiredate);
+        return buf;
+    }
+    return expirestr_from_sig (sig);
+}
 
 
+/**************** .
+ * Return a byte array with the fingerprint for the given PK/SK
+ * The length of the array is returned in ret_len. Caller must free
+ * the array or provide an array of length MAX_FINGERPRINT_LEN.
+ */
 
 byte *
-fingerprint_from_pkc( PKT_public_cert *pkc, size_t *ret_len )
+fingerprint_from_pk( PKT_public_key *pk, byte *array, size_t *ret_len )
 {
-    byte *p, *buf, *array;
-    const char *dp;
+    byte *p, *buf;
+    const byte *dp;
     size_t len;
-    unsigned n;
+    unsigned int n;
 
-    if( pkc->pubkey_algo == PUBKEY_ALGO_ELGAMAL ) {
+    if( pk->version < 4 && is_RSA(pk->pubkey_algo) ) {
+       /* RSA in version 3 packets is special */
        MD_HANDLE md;
-       md = v3_elg_fingerprint_md(pkc);
-       dp = md_read( md, DIGEST_ALGO_RMD160 );
-       array = m_alloc( 20 );
-       len = 20;
-       memcpy(array, dp, 20 );
+
+       md = md_open( DIGEST_ALGO_MD5, 0);
+       if( pubkey_get_npkey( pk->pubkey_algo ) > 1 ) {
+           p = buf = mpi_get_buffer( pk->pkey[0], &n, NULL );
+           md_write( md, p, n );
+           m_free(buf);
+           p = buf = mpi_get_buffer( pk->pkey[1], &n, NULL );
+           md_write( md, p, n );
+           m_free(buf);
+       }
+       md_final(md);
+       if( !array )
+           array = m_alloc( 16 );
+       len = 16;
+       memcpy(array, md_read(md, DIGEST_ALGO_MD5), 16 );
        md_close(md);
     }
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_DSA ) {
+    else {
        MD_HANDLE md;
-       md = dsa_fingerprint_md(pkc);
-       dp = md_read( md, DIGEST_ALGO_SHA1 );
-       array = m_alloc( 20 );
-       len = 20;
-       memcpy(array, dp, 20 );
+       md = do_fingerprint_md(pk);
+       dp = md_read( md, 0 );
+       len = md_digest_length( md_get_algo( md ) );
+       assert( len <= MAX_FINGERPRINT_LEN );
+       if( !array )
+           array = m_alloc( len );
+       memcpy(array, dp, len );
+       pk->keyid[0] = dp[12] << 24 | dp[13] << 16 | dp[14] << 8 | dp[15] ;
+       pk->keyid[1] = dp[16] << 24 | dp[17] << 16 | dp[18] << 8 | dp[19] ;
        md_close(md);
     }
-    else if( pkc->pubkey_algo == PUBKEY_ALGO_RSA ) {
+
+    *ret_len = len;
+    return array;
+}
+
+byte *
+fingerprint_from_sk( PKT_secret_key *sk, byte *array, size_t *ret_len )
+{
+    byte *p, *buf;
+    const char *dp;
+    size_t len;
+    unsigned n;
+
+    if( sk->version < 4 && is_RSA(sk->pubkey_algo) ) {
+       /* RSA in version 3 packets is special */
        MD_HANDLE md;
 
        md = md_open( DIGEST_ALGO_MD5, 0);
-       p = buf = mpi_get_buffer( pkc->d.rsa.rsa_n, &n, NULL );
-       for( ; !*p && n; p++, n-- )
-           ;
-       md_write( md, p, n );
-       m_free(buf);
-       p = buf = mpi_get_buffer( pkc->d.rsa.rsa_e, &n, NULL );
-       for( ; !*p && n; p++, n-- )
-           ;
-       md_write( md, p, n );
-       m_free(buf);
+       if( pubkey_get_npkey( sk->pubkey_algo ) > 1 ) {
+           p = buf = mpi_get_buffer( sk->skey[0], &n, NULL );
+           md_write( md, p, n );
+           m_free(buf);
+           p = buf = mpi_get_buffer( sk->skey[1], &n, NULL );
+           md_write( md, p, n );
+           m_free(buf);
+       }
        md_final(md);
-       array = m_alloc( 16 );
+       if( !array )
+           array = m_alloc( 16 );
        len = 16;
        memcpy(array, md_read(md, DIGEST_ALGO_MD5), 16 );
        md_close(md);
     }
     else {
-       array = m_alloc(1);
-       len = 0; /* ooops */
+       MD_HANDLE md;
+       md = do_fingerprint_md_sk(sk);
+       dp = md_read( md, 0 );
+       len = md_digest_length( md_get_algo( md ) );
+       assert( len <= MAX_FINGERPRINT_LEN );
+       if( !array )
+           array = m_alloc( len );
+       memcpy(array, dp, len );
+       md_close(md);
     }
 
     *ret_len = len;