last local commit
[gnupg.git] / g10 / mainproc.c
index 3ca23cc..dd1546c 100644 (file)
@@ -23,6 +23,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <assert.h>
+#include <time.h>
 
 #include "packet.h"
 #include "iobuf.h"
 #include "cipher.h"
 #include "main.h"
 #include "status.h"
-
+#include "i18n.h"
+#include "trustdb.h"
 
 /****************
  * Structure to hold the context
  */
 
 typedef struct {
-    PKT_public_cert *last_pubkey;
-    PKT_secret_cert *last_seckey;
+    PKT_public_key *last_pubkey;
+    PKT_secret_key *last_seckey;
     PKT_user_id     *last_user_id;
     md_filter_context_t mfx;
     int sigs_only;   /* process only signatures and reject all other stuff */
-    int encrypt_only; /* process onyl encrytion messages */
+    int encrypt_only; /* process only encrytion messages */
     STRLIST signed_data;
+    const char *sigfilename;
     DEK *dek;
-    int last_was_pubkey_enc;
+    int last_was_session_key;
     KBNODE list;   /* the current list of packets */
     int have_data;
     IOBUF iobuf;    /* used to get the filename etc. */
+    int trustletter; /* temp usage in list_node */
+    ulong local_id;    /* ditto */
 } *CTX;
 
 
@@ -83,8 +88,10 @@ add_onepass_sig( CTX c, PACKET *pkt )
        if( c->list->pkt->pkttype != PKT_ONEPASS_SIG ) {
           log_error("add_onepass_sig: another packet is in the way\n");
           release_list( c );
+          c->list = new_kbnode( pkt );
        }
-       add_kbnode( c->list, new_kbnode( pkt ));
+       else
+          add_kbnode( c->list, new_kbnode( pkt ));
     }
     else /* insert the first one */
        c->list = node = new_kbnode( pkt );
@@ -93,28 +100,23 @@ add_onepass_sig( CTX c, PACKET *pkt )
 }
 
 
-static int
-add_public_cert( CTX c, PACKET *pkt )
-{
-    release_list( c );
-    c->list = new_kbnode( pkt );
-    return 1;
-}
 
 static int
-add_secret_cert( CTX c, PACKET *pkt )
+add_user_id( CTX c, PACKET *pkt )
 {
-    release_list( c );
-    c->list = new_kbnode( pkt );
+    if( !c->list ) {
+       log_error("orphaned user id\n" );
+       return 0;
+    }
+    add_kbnode( c->list, new_kbnode( pkt ) );
     return 1;
 }
 
-
 static int
-add_user_id( CTX c, PACKET *pkt )
+add_subkey( CTX c, PACKET *pkt )
 {
     if( !c->list ) {
-       log_error("orphaned user id\n" );
+       log_error("subkey w/o mainkey\n" );
        return 0;
     }
     add_kbnode( c->list, new_kbnode( pkt ) );
@@ -128,12 +130,12 @@ add_signature( CTX c, PACKET *pkt )
     KBNODE node;
 
     if( pkt->pkttype == PKT_SIGNATURE && !c->list ) {
-       /* This is the first signature for a following datafile.
-        * G10 does not write such packets, instead it always uses
+       /* This is the first signature for the following datafile.
+        * G10 does not write such packets; instead it always uses
         * onepass-sig packets.  The drawback of PGP's method
-        * of prepending the signtaure to the data is,
+        * of prepending the signature to the data is
         * that it is not possible to make a signature from data read
-        * from stdin.  (Anyway, G10 is able to read these stuff) */
+        * from stdin.  (G10 is able to read PGP stuff anyway.) */
        node = new_kbnode( pkt );
        c->list = node;
        return 1;
@@ -151,17 +153,33 @@ add_signature( CTX c, PACKET *pkt )
 
 
 static void
+proc_symkey_enc( CTX c, PACKET *pkt )
+{
+    PKT_symkey_enc *enc;
+
+    enc = pkt->pkt.symkey_enc;
+    if( enc->seskeylen )
+       log_error( "symkey_enc packet with session keys are not supported!\n");
+    else {
+       c->last_was_session_key = 2;
+       c->dek = passphrase_to_dek( NULL, enc->cipher_algo, &enc->s2k, 0 );
+    }
+    free_packet(pkt);
+}
+
+static void
 proc_pubkey_enc( CTX c, PACKET *pkt )
 {
     PKT_pubkey_enc *enc;
     int result = 0;
 
-    c->last_was_pubkey_enc = 1;
+    /* check whether the secret key is available and store in this case */
+    c->last_was_session_key = 1;
     enc = pkt->pkt.pubkey_enc;
     /*printf("enc: encrypted by a pubkey with keyid %08lX\n", enc->keyid[1] );*/
-    if( enc->pubkey_algo == PUBKEY_ALGO_ELGAMAL
+    if( is_ELGAMAL(enc->pubkey_algo)
        || enc->pubkey_algo == PUBKEY_ALGO_DSA
-       || enc->pubkey_algo == PUBKEY_ALGO_RSA  ) {
+       || is_RSA(enc->pubkey_algo)  ) {
        m_free(c->dek ); /* paranoid: delete a pending DEK */
        c->dek = m_alloc_secure( sizeof *c->dek );
        if( (result = get_session_key( enc, c->dek )) ) {
@@ -178,24 +196,24 @@ proc_pubkey_enc( CTX c, PACKET *pkt )
        if( opt.verbose > 1 )
            log_info( "pubkey_enc packet: Good DEK\n" );
     }
-    else
-       log_error( "pubkey_enc packet: %s\n", g10_errstr(result));
+    else {
+       log_error(_("public key decryption failed: %s\n"), g10_errstr(result));
+    }
     free_packet(pkt);
 }
 
 
-
 static void
 proc_encrypted( CTX c, PACKET *pkt )
 {
     int result = 0;
 
     /*printf("dat: %sencrypted data\n", c->dek?"":"conventional ");*/
-    if( !c->dek && !c->last_was_pubkey_enc ) {
-       /* assume this is conventional encrypted data */
-       c->dek = m_alloc_secure( sizeof *c->dek );
-       c->dek->algo = opt.def_cipher_algo;
-       result = make_dek_from_passphrase( c->dek, 0, NULL );
+    if( !c->dek && !c->last_was_session_key ) {
+       /* assume this is old conventional encrypted data */
+       c->dek = passphrase_to_dek( NULL,
+                   opt.def_cipher_algo ? opt.def_cipher_algo
+                                       : DEFAULT_CIPHER_ALGO, NULL, 0 );
     }
     else if( !c->dek )
        result = G10ERR_NO_SECKEY;
@@ -206,13 +224,15 @@ proc_encrypted( CTX c, PACKET *pkt )
        ;
     else if( !result ) {
        if( opt.verbose > 1 )
-           log_info("encryption okay\n");
+           log_info("decryption okay\n");
     }
     else {
-       log_error("encryption failed: %s\n", g10_errstr(result));
+       log_error(_("decryption failed: %s\n"), g10_errstr(result));
+       /* FIXME: if this is secret key not available, try with
+        * other keys */
     }
     free_packet(pkt);
-    c->last_was_pubkey_enc = 0;
+    c->last_was_session_key = 0;
 }
 
 
@@ -220,30 +240,66 @@ static void
 proc_plaintext( CTX c, PACKET *pkt )
 {
     PKT_plaintext *pt = pkt->pkt.plaintext;
-    int rc;
+    int any, clearsig, rc;
+    KBNODE n;
 
-    if( opt.verbose )
+    if( pt->namelen == 8 && !memcmp( pt->name, "_CONSOLE", 8 ) )
+       log_info(_("note: sender requested \"for-your-eyes-only\"\n"));
+    else if( opt.verbose )
        log_info("original file name='%.*s'\n", pt->namelen, pt->name);
     free_md_filter_context( &c->mfx );
-    /* fixme: take the digest algo(s) to use from the
-     * onepass_sig packet (if we have these)
-     * And look at the sigclass to check wether we should use the
-     * textmode filter (sigclass 0x01)
-     */
-    c->mfx.md = md_open( DIGEST_ALGO_RMD160, 0);
-    md_enable( c->mfx.md, DIGEST_ALGO_MD5 );
-    rc = handle_plaintext( pt, &c->mfx );
+    c->mfx.md = md_open( 0, 0);
+    /* fixme: we may need to push the textfilter if we have sigclass 1
+     * and no armoring - Not yet tested */
+    any = clearsig = 0;
+    for(n=c->list; n; n = n->next ) {
+       if( n->pkt->pkttype == PKT_ONEPASS_SIG ) {
+           if( n->pkt->pkt.onepass_sig->digest_algo ) {
+               md_enable( c->mfx.md, n->pkt->pkt.onepass_sig->digest_algo );
+               any = 1;
+           }
+           /* Check whether this is a cleartext signature.  We assume that
+            * we have one if the sig_class is 1 and the keyid is 0, that
+            * are the faked packets produced by armor.c.  There is a
+            * possibility that this fails, but there is no other easy way
+            * to do it. (We could use a special packet type to indicate
+            * this, but this may also be faked - it simply can't be verified
+            * and is _no_ security issue)
+            */
+           if( n->pkt->pkt.onepass_sig->sig_class == 0x01
+               && !n->pkt->pkt.onepass_sig->keyid[0]
+               && !n->pkt->pkt.onepass_sig->keyid[1]  )
+               clearsig = 1;
+       }
+    }
+    if( !any ) { /* no onepass sig packet: enable all standard algos */
+       md_enable( c->mfx.md, DIGEST_ALGO_RMD160 );
+       md_enable( c->mfx.md, DIGEST_ALGO_SHA1 );
+       md_enable( c->mfx.md, DIGEST_ALGO_MD5 );
+    }
+    if( c->mfx.md ) {
+       m_check(c->mfx.md);
+       if( c->mfx.md->list )
+           m_check( c->mfx.md->list );
+    }
+    rc = handle_plaintext( pt, &c->mfx, c->sigs_only, clearsig );
+    if( rc == G10ERR_CREATE_FILE && !c->sigs_only) {
+       /* can't write output but we hash it anyway to
+        * check the signature */
+       rc = handle_plaintext( pt, &c->mfx, 1, clearsig );
+    }
     if( rc )
        log_error( "handle plaintext failed: %s\n", g10_errstr(rc));
     free_packet(pkt);
-    c->last_was_pubkey_enc = 0;
+    c->last_was_session_key = 0;
 }
 
 
 static int
 proc_compressed_cb( IOBUF a, void *info )
 {
-    return proc_signature_packets( a, ((CTX)info)->signed_data );
+    return proc_signature_packets( a, ((CTX)info)->signed_data,
+                                     ((CTX)info)->sigfilename );
 }
 
 static int
@@ -268,7 +324,7 @@ proc_compressed( CTX c, PACKET *pkt )
     if( rc )
        log_error("uncompressing failed: %s\n", g10_errstr(rc));
     free_packet(pkt);
-    c->last_was_pubkey_enc = 0;
+    c->last_was_session_key = 0;
 }
 
 
@@ -279,21 +335,29 @@ proc_compressed( CTX c, PACKET *pkt )
  * Returns: 0 = valid signature or an error code
  */
 static int
-do_check_sig( CTX c, KBNODE node )
+do_check_sig( CTX c, KBNODE node, int *is_selfsig )
 {
     PKT_signature *sig;
     MD_HANDLE md;
     int algo, rc;
 
     assert( node->pkt->pkttype == PKT_SIGNATURE );
+    if( is_selfsig )
+       *is_selfsig = 0;
     sig = node->pkt->pkt.signature;
 
-    algo = digest_algo_from_sig( sig );
+    algo = sig->digest_algo;
     if( !algo )
        return G10ERR_PUBKEY_ALGO;
     if( (rc=check_digest_algo(algo)) )
        return rc;
 
+    if( c->mfx.md ) {
+       m_check(c->mfx.md);
+       if( c->mfx.md->list )
+           m_check( c->mfx.md->list );
+    }
+
     if( sig->sig_class == 0x00 ) {
        if( c->mfx.md )
            md = md_copy( c->mfx.md );
@@ -309,31 +373,12 @@ do_check_sig( CTX c, KBNODE node )
            md = md_open( 0, 0 ); /* signature_check() will enable the md*/
     }
     else if( (sig->sig_class&~3) == 0x10
+            || sig->sig_class == 0x18
             || sig->sig_class == 0x20
-            || sig->sig_class == 0x30  ) { /* classes 0x10..0x13,0x20,0x30 */
-       if( c->list->pkt->pkttype == PKT_PUBLIC_CERT ) {
-         #if 0
-           KBNODE n1;
-
-           if( sig->sig_class == 0x20 ) {
-               md = md_open( algo, 0 );
-               hash_public_cert( md, c->list->pkt->pkt.public_cert );
-           }
-           else if( (n1=find_prev_kbnode( c->list, node, PKT_USER_ID )) ) {
-               md = md_open( algo, 0 );
-               hash_public_cert( md, c->list->pkt->pkt.public_cert );
-               if( sig->sig_class != 0x20 )
-                   md_write( md, n1->pkt->pkt.user_id->name,
-                                 n1->pkt->pkt.user_id->len);
-           }
-           else {
-               log_error("invalid parent packet for sigclass %02x\n",
-                                                       sig->sig_class);
-               return G10ERR_SIG_CLASS;
-           }
-         #endif
-
-           return check_key_signature( c->list, node, NULL );
+            || sig->sig_class == 0x30  ) { /* classes 0x10..0x17,0x20,0x30 */
+       if( c->list->pkt->pkttype == PKT_PUBLIC_KEY
+           || c->list->pkt->pkttype == PKT_PUBLIC_SUBKEY ) {
+           return check_key_signature( c->list, node, is_selfsig );
        }
        else {
            log_error("invalid root packet for sigclass %02x\n",
@@ -366,15 +411,18 @@ print_userid( PACKET *pkt )
 
 
 static void
-print_fingerprint( PKT_public_cert *pkc, PKT_secret_cert *skc )
+print_fingerprint( PKT_public_key *pk, PKT_secret_key *sk )
 {
-    byte *array, *p;
+    byte array[MAX_FINGERPRINT_LEN], *p;
     size_t i, n;
 
-    p = array = skc? fingerprint_from_skc( skc, &n )
-                  : fingerprint_from_pkc( pkc, &n );
+    if( sk )
+       fingerprint_from_sk( sk, array, &n );
+    else
+       fingerprint_from_pk( pk, array, &n );
+    p = array;
     if( opt.with_colons ) {
-       printf("fpr::::::::");
+       printf("fpr:::::::::");
        for(i=0; i < n ; i++, p++ )
            printf("%02X", *p );
        putchar(':');
@@ -397,7 +445,6 @@ print_fingerprint( PKT_public_cert *pkc, PKT_secret_cert *skc )
        }
     }
     putchar('\n');
-    m_free(array);
 }
 
 
@@ -409,88 +456,152 @@ static void
 list_node( CTX c, KBNODE node )
 {
     int any=0;
+    int mainkey;
 
     if( !node )
        ;
-    else if( node->pkt->pkttype == PKT_PUBLIC_CERT ) {
-       PKT_public_cert *pkc = node->pkt->pkt.public_cert;
+    else if( (mainkey = (node->pkt->pkttype == PKT_PUBLIC_KEY) )
+            || node->pkt->pkttype == PKT_PUBLIC_SUBKEY ) {
+       PKT_public_key *pk = node->pkt->pkt.public_key;
 
        if( opt.with_colons ) {
            u32 keyid[2];
-           keyid_from_pkc( pkc, keyid );
-           printf("pub::%u:%d:%08lX%08lX:%s:::",
-                   /* fixme: add trust value here */
-                   nbits_from_pkc( pkc ),
-                   pkc->pubkey_algo,
+           keyid_from_pk( pk, keyid );
+           if( mainkey ) {
+               c->local_id = pk->local_id;
+               c->trustletter = query_trust_info( pk );
+           }
+           printf("%s:%c:%u:%d:%08lX%08lX:%s:%s:",
+                   mainkey? "pub":"sub",
+                   c->trustletter,
+                   nbits_from_pk( pk ),
+                   pk->pubkey_algo,
                    (ulong)keyid[0],(ulong)keyid[1],
-                   datestr_from_pkc( pkc )
-                   /* fixme: add LID and ownertrust here */
-                                           );
+                   datestr_from_pk( pk ),
+                   pk->expiredate? strtimestamp(pk->expiredate):"" );
+           if( c->local_id )
+               printf("%lu", c->local_id );
+           putchar(':');
+           /* fixme: add ownertrust here */
+           putchar(':');
        }
        else
-           printf("pub  %4u%c/%08lX %s ", nbits_from_pkc( pkc ),
-                                     pubkey_letter( pkc->pubkey_algo ),
-                                     (ulong)keyid_from_pkc( pkc, NULL ),
-                                     datestr_from_pkc( pkc )     );
-       /* and now list all userids with their signatures */
-       for( node = node->next; node; node = node->next ) {
-           if( any != 2 && node->pkt->pkttype == PKT_SIGNATURE ) {
-               if( !any ) {
-                   if( node->pkt->pkt.signature->sig_class == 0x20 )
-                       puts("[revoked]");
-                   else
+           printf("%s  %4u%c/%08lX %s ",
+                                     mainkey? "pub":"sub",
+                                     nbits_from_pk( pk ),
+                                     pubkey_letter( pk->pubkey_algo ),
+                                     (ulong)keyid_from_pk( pk, NULL ),
+                                     datestr_from_pk( pk )     );
+       if( mainkey ) {
+           /* and now list all userids with their signatures */
+           for( node = node->next; node; node = node->next ) {
+               if( node->pkt->pkttype == PKT_SIGNATURE ) {
+                   if( !any ) {
+                       if( node->pkt->pkt.signature->sig_class == 0x20 )
+                           puts("[revoked]");
+                       else
+                           putchar('\n');
+                       any = 1;
+                   }
+                   list_node(c,  node );
+               }
+               else if( node->pkt->pkttype == PKT_USER_ID ) {
+                   if( any ) {
+                       if( opt.with_colons )
+                           printf("uid:::::::::");
+                       else
+                           printf( "uid%*s", 28, "" );
+                   }
+                   print_userid( node->pkt );
+                   if( opt.with_colons )
+                       putchar(':');
+                   putchar('\n');
+                   if( opt.fingerprint && !any )
+                       print_fingerprint( pk, NULL );
+                   any=1;
+               }
+               else if( node->pkt->pkttype == PKT_PUBLIC_SUBKEY ) {
+                   if( !any ) {
                        putchar('\n');
+                       any = 1;
+                   }
+                   list_node(c,  node );
                }
-               list_node(c,  node );
-               any = 1;
            }
-           else if( node->pkt->pkttype == PKT_USER_ID ) {
-               KBNODE n;
+       }
+       if( !any )
+           putchar('\n');
+       if( !mainkey && opt.fingerprint > 1 )
+           print_fingerprint( pk, NULL );
+    }
+    else if( (mainkey = (node->pkt->pkttype == PKT_SECRET_KEY) )
+            || node->pkt->pkttype == PKT_SECRET_SUBKEY ) {
+       PKT_secret_key *sk = node->pkt->pkt.secret_key;
 
-               if( any ) {
+       if( opt.with_colons ) {
+           u32 keyid[2];
+           keyid_from_sk( sk, keyid );
+           printf("%s::%u:%d:%08lX%08lX:%s:%s:::",
+                   mainkey? "sec":"ssb",
+                   nbits_from_sk( sk ),
+                   sk->pubkey_algo,
+                   (ulong)keyid[0],(ulong)keyid[1],
+                   datestr_from_sk( sk ),
+                   sk->expiredate? strtimestamp(sk->expiredate):""
+                   /* fixme: add LID */ );
+       }
+       else
+           printf("%s  %4u%c/%08lX %s ",
+                                     mainkey? "sec":"ssb",
+                                     nbits_from_sk( sk ),
+                                     pubkey_letter( sk->pubkey_algo ),
+                                     (ulong)keyid_from_sk( sk, NULL ),
+                                     datestr_from_sk( sk )   );
+       if( mainkey ) {
+           /* and now list all userids with their signatures */
+           for( node = node->next; node; node = node->next ) {
+               if( node->pkt->pkttype == PKT_SIGNATURE ) {
+                   if( !any ) {
+                       if( node->pkt->pkt.signature->sig_class == 0x20 )
+                           puts("[revoked]");
+                       else
+                           putchar('\n');
+                       any = 1;
+                   }
+                   list_node(c,  node );
+               }
+               else if( node->pkt->pkttype == PKT_USER_ID ) {
+                   if( any ) {
+                       if( opt.with_colons )
+                           printf("uid:::::::::");
+                       else
+                           printf( "uid%*s", 28, "" );
+                   }
+                   print_userid( node->pkt );
                    if( opt.with_colons )
-                       printf("uid::::::::");
-                   else
-                       printf( "uid%*s", 28, "" );
+                       putchar(':');
+                   putchar('\n');
+                   if( opt.fingerprint && !any )
+                       print_fingerprint( NULL, sk );
+                   any=1;
                }
-               print_userid( node->pkt );
-               if( opt.with_colons )
-                   putchar(':');
-               putchar('\n');
-               if( opt.fingerprint && !any )
-                   print_fingerprint( pkc, NULL );
-               for( n=node->next; n; n = n->next ) {
-                   if( n->pkt->pkttype == PKT_USER_ID )
-                       break;
-                   if( n->pkt->pkttype == PKT_SIGNATURE )
-                       list_node(c,  n );
+               else if( node->pkt->pkttype == PKT_SECRET_SUBKEY ) {
+                   if( !any ) {
+                       putchar('\n');
+                       any = 1;
+                   }
+                   list_node(c,  node );
                }
-               any=2;
            }
        }
-       if( any != 2 )
-           printf("ERROR: no user id!\n");
-    }
-    else if( node->pkt->pkttype == PKT_SECRET_CERT ) {
-       PKT_secret_cert *skc = node->pkt->pkt.secret_cert;
-
-       printf("sec  %4u%c/%08lX %s ", nbits_from_skc( skc ),
-                                     pubkey_letter( skc->pubkey_algo ),
-                                     (ulong)keyid_from_skc( skc, NULL ),
-                                     datestr_from_skc( skc )     );
-       /* and now list all userids */
-       while( (node = find_next_kbnode(node, PKT_USER_ID)) ) {
-           print_userid( node->pkt );
-           putchar('\n');
-           if( opt.fingerprint && !any )
-               print_fingerprint( NULL, skc );
-           any=1;
-       }
        if( !any )
-           printf("ERROR: no user id!\n");
+           putchar('\n');
+       if( !mainkey && opt.fingerprint > 1 )
+           print_fingerprint( NULL, sk );
     }
     else if( node->pkt->pkttype == PKT_SIGNATURE  ) {
        PKT_signature *sig = node->pkt->pkt.signature;
+       int is_selfsig = 0;
        int rc2=0;
        size_t n;
        char *p;
@@ -505,18 +616,32 @@ list_node( CTX c, KBNODE node )
            fputs("sig", stdout);
        if( opt.check_sigs ) {
            fflush(stdout);
-           switch( (rc2=do_check_sig( c, node )) ) {
+           switch( (rc2=do_check_sig( c, node, &is_selfsig )) ) {
              case 0:                  sigrc = '!'; break;
              case G10ERR_BAD_SIGN:    sigrc = '-'; break;
              case G10ERR_NO_PUBKEY:   sigrc = '?'; break;
              default:                 sigrc = '%'; break;
            }
        }
+       else {  /* check whether this is a self signature */
+           u32 keyid[2];
+
+           if( c->list->pkt->pkttype == PKT_PUBLIC_KEY
+               || c->list->pkt->pkttype == PKT_SECRET_KEY ) {
+               if( c->list->pkt->pkttype == PKT_PUBLIC_KEY )
+                   keyid_from_pk( c->list->pkt->pkt.public_key, keyid );
+               else
+                   keyid_from_sk( c->list->pkt->pkt.secret_key, keyid );
+
+               if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] )
+                   is_selfsig = 1;
+           }
+       }
        if( opt.with_colons ) {
            putchar(':');
            if( sigrc != ' ' )
                putchar(sigrc);
-           printf(":::%08lX%08lX:%s:::", (ulong)sig->keyid[0],
+           printf(":::%08lX%08lX:%s::::", (ulong)sig->keyid[0],
                       (ulong)sig->keyid[1], datestr_from_sig(sig));
        }
        else
@@ -526,6 +651,13 @@ list_node( CTX c, KBNODE node )
            printf("[%s] ", g10_errstr(rc2) );
        else if( sigrc == '?' )
            ;
+       else if( is_selfsig ) {
+           if( opt.with_colons )
+               putchar(':');
+           fputs( sig->sig_class == 0x18? "[keybind]":"[selfsig]", stdout);
+           if( opt.with_colons )
+               putchar(':');
+       }
        else {
            p = get_user_id( sig->keyid, &n );
            print_string( stdout, p, n, opt.with_colons );
@@ -550,12 +682,13 @@ proc_packets( IOBUF a )
 }
 
 int
-proc_signature_packets( IOBUF a, STRLIST signedfiles )
+proc_signature_packets( IOBUF a, STRLIST signedfiles, const char *sigfilename )
 {
     CTX c = m_alloc_clear( sizeof *c );
     int rc;
     c->sigs_only = 1;
     c->signed_data = signedfiles;
+    c->sigfilename = sigfilename;
     rc = do_proc_packets( c, a );
     m_free( c );
     return rc;
@@ -585,7 +718,9 @@ do_proc_packets( CTX c, IOBUF a )
     while( (rc=parse_packet(a, pkt)) != -1 ) {
        /* cleanup if we have an illegal data structure */
        if( c->dek && pkt->pkttype != PKT_ENCRYPTED ) {
-           log_error("oops: valid pubkey enc packet not followed by data\n");
+           /* FIXME: do we need to ave it in case we have no secret
+            * key for one of the next reciepents- we should check it
+            * here. */
            m_free(c->dek); c->dek = NULL; /* burn it */
        }
 
@@ -606,9 +741,10 @@ do_proc_packets( CTX c, IOBUF a )
        }
        else if( c->sigs_only ) {
            switch( pkt->pkttype ) {
-             case PKT_PUBLIC_CERT:
-             case PKT_SECRET_CERT:
+             case PKT_PUBLIC_KEY:
+             case PKT_SECRET_KEY:
              case PKT_USER_ID:
+             case PKT_SYMKEY_ENC:
              case PKT_PUBKEY_ENC:
              case PKT_ENCRYPTED:
                rc = G10ERR_UNEXPECTED;
@@ -622,12 +758,13 @@ do_proc_packets( CTX c, IOBUF a )
        }
        else if( c->encrypt_only ) {
            switch( pkt->pkttype ) {
-             case PKT_PUBLIC_CERT:
-             case PKT_SECRET_CERT:
+             case PKT_PUBLIC_KEY:
+             case PKT_SECRET_KEY:
              case PKT_USER_ID:
                rc = G10ERR_UNEXPECTED;
                goto leave;
              case PKT_SIGNATURE:   newpkt = add_signature( c, pkt ); break;
+             case PKT_SYMKEY_ENC:  proc_symkey_enc( c, pkt ); break;
              case PKT_PUBKEY_ENC:  proc_pubkey_enc( c, pkt ); break;
              case PKT_ENCRYPTED:   proc_encrypted( c, pkt ); break;
              case PKT_PLAINTEXT:   proc_plaintext( c, pkt ); break;
@@ -638,11 +775,20 @@ do_proc_packets( CTX c, IOBUF a )
        }
        else {
            switch( pkt->pkttype ) {
-             case PKT_PUBLIC_CERT: newpkt = add_public_cert( c, pkt ); break;
-             case PKT_SECRET_CERT: newpkt = add_secret_cert( c, pkt ); break;
+             case PKT_PUBLIC_KEY:
+             case PKT_SECRET_KEY:
+               release_list( c );
+               c->list = new_kbnode( pkt );
+               newpkt = 1;
+               break;
+             case PKT_PUBLIC_SUBKEY:
+             case PKT_SECRET_SUBKEY:
+               newpkt = add_subkey( c, pkt );
+               break;
              case PKT_USER_ID:     newpkt = add_user_id( c, pkt ); break;
              case PKT_SIGNATURE:   newpkt = add_signature( c, pkt ); break;
              case PKT_PUBKEY_ENC:  proc_pubkey_enc( c, pkt ); break;
+             case PKT_SYMKEY_ENC:  proc_symkey_enc( c, pkt ); break;
              case PKT_ENCRYPTED:   proc_encrypted( c, pkt ); break;
              case PKT_PLAINTEXT:   proc_plaintext( c, pkt ); break;
              case PKT_COMPRESSED:  proc_compressed( c, pkt ); break;
@@ -689,30 +835,37 @@ static int
 check_sig_and_print( CTX c, KBNODE node )
 {
     PKT_signature *sig = node->pkt->pkt.signature;
+    const char *astr, *tstr;
     int rc;
 
-    rc = do_check_sig(c, node );
+    if( opt.skip_verify ) {
+       log_info("signature verification suppressed\n");
+       return 0;
+    }
+
+    tstr = asctimestamp(sig->timestamp);
+    astr = pubkey_algo_to_string( sig->pubkey_algo );
+    log_info(_("Signature made %.*s using %s key ID %08lX\n"),
+           (int)strlen(tstr)-1, tstr, astr? astr: "?", (ulong)sig->keyid[1] );
+
+    rc = do_check_sig(c, node, NULL );
     if( !rc || rc == G10ERR_BAD_SIGN ) {
-       char *p, *buf;
-
-       p = get_user_id_string( sig->keyid );
-       buf = m_alloc( 20 + strlen(p) );
-       sprintf(buf, "%lu %s", (ulong)sig->timestamp, p );
-       m_free(p);
-       if( (p=strchr(buf,'\n')) )
-           *p = 0; /* just in case ... */
-       write_status_text( rc? STATUS_BADSIG : STATUS_GOODSIG, buf );
-       m_free(buf);
-       log_info("%s signature from ", rc? "BAD":"Good");
+       char *us = get_user_id_string( sig->keyid );
+       write_status_text( rc? STATUS_BADSIG : STATUS_GOODSIG, us );
+       m_free(us);
+       log_info(rc? _("BAD signature from \"")
+                  : _("Good signature from \""));
        print_keyid( stderr, sig->keyid );
+       putc('\"', stderr);
        putc('\n', stderr);
-       if( opt.batch )
+       if( !rc )
+           rc = check_signatures_trust( sig );
+       if( opt.batch && rc )
            g10_exit(1);
     }
     else {
        write_status( STATUS_ERRSIG );
-       log_error("Can't check signature made by %08lX: %s\n",
-                  (ulong)sig->keyid[1], g10_errstr(rc) );
+       log_error(_("Can't check signature: %s\n"), g10_errstr(rc) );
     }
     return rc;
 }
@@ -730,9 +883,12 @@ proc_tree( CTX c, KBNODE node )
     if( opt.list_packets )
        return;
 
-    if( node->pkt->pkttype == PKT_PUBLIC_CERT )
+    c->local_id = 0;
+    c->trustletter = ' ';
+    if( node->pkt->pkttype == PKT_PUBLIC_KEY
+       || node->pkt->pkttype == PKT_PUBLIC_SUBKEY )
        list_node( c, node );
-    else if( node->pkt->pkttype == PKT_SECRET_CERT )
+    else if( node->pkt->pkttype == PKT_SECRET_KEY )
        list_node( c, node );
     else if( node->pkt->pkttype == PKT_ONEPASS_SIG ) {
        /* check all signatures */
@@ -742,13 +898,12 @@ proc_tree( CTX c, KBNODE node )
            c->mfx.md = md_open(0, 0);
            /* fixme: why looking for the signature packet and not 1passpacket*/
            for( n1 = node; (n1 = find_next_kbnode(n1, PKT_SIGNATURE )); ) {
-               md_enable( c->mfx.md,
-                          digest_algo_from_sig(n1->pkt->pkt.signature));
+               md_enable( c->mfx.md, n1->pkt->pkt.signature->digest_algo);
            }
            /* ask for file and hash it */
            if( c->sigs_only )
-               rc = hash_datafiles( c->mfx.md, c->signed_data,
-                           n1->pkt->pkt.onepass_sig->sig_class == 0x01 );
+               rc = hash_datafiles( c->mfx.md, c->signed_data, c->sigfilename,
+                       n1? (n1->pkt->pkt.onepass_sig->sig_class == 0x01):0 );
            else
                rc = ask_for_detached_datafile( &c->mfx,
                                            iobuf_get_fname(c->iobuf));
@@ -766,9 +921,9 @@ proc_tree( CTX c, KBNODE node )
 
        if( !c->have_data ) {
            free_md_filter_context( &c->mfx );
-           c->mfx.md = md_open(digest_algo_from_sig(sig), 0);
+           c->mfx.md = md_open(sig->digest_algo, 0);
            if( c->sigs_only )
-               rc = hash_datafiles( c->mfx.md, c->signed_data,
+               rc = hash_datafiles( c->mfx.md, c->signed_data, c->sigfilename,
                                     sig->sig_class == 0x01 );
            else
                rc = ask_for_detached_datafile( &c->mfx,