See ChangeLog: Thu Jan 21 06:22:10 CET 1999 Werner Koch
[gnupg.git] / g10 / trustdb.c
index 18726f1..674240a 100644 (file)
@@ -1,14 +1,14 @@
 /* trustdb.c
  *     Copyright (C) 1998 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.
 #include "i18n.h"
 #include "tdbio.h"
 
-
 #if MAX_FINGERPRINT_LEN > 20
   #error Must change structure of trustdb
 #endif
 
-typedef struct local_id_info *LOCAL_ID_INFO;
-struct local_id_info {
-    LOCAL_ID_INFO next;
+struct keyid_list {
+    struct keyid_list *next;
+    u32 keyid[2];
+};
+
+struct local_id_item {
+    struct local_id_item *next;
     ulong lid;
     unsigned flag;
 };
 
+struct local_id_table {
+    struct local_id_table *next; /* only used to keep a list of unused tables */
+    struct local_id_item *items[16];
+};
+
+
+typedef struct local_id_table *LOCAL_ID_TABLE;
+
 
 typedef struct trust_info TRUST_INFO;
 struct trust_info {
     ulong    lid;
-    unsigned trust;
+    byte     otrust; /* ownertrust (assigned trust) */
+    byte     trust;  /* calculated trust (validity) */
 };
 
-
 typedef struct trust_seg_list *TRUST_SEG_LIST;
 struct trust_seg_list {
     TRUST_SEG_LIST next;
-    int   nseg;     /* number of segmens */
-    int   dup;
-    TRUST_INFO seg[1];  /* segment list */
+    int  pathlen;
+    TRUST_INFO path[1];
 };
 
 
-typedef struct {
-    TRUST_SEG_LIST tsl;
-    int index;
-} ENUM_TRUST_WEB_CONTEXT;
+struct enum_cert_paths_ctx {
+   int init;
+   TRUST_SEG_LIST tsl_head;
+   TRUST_SEG_LIST tsl;
+   int idx;
+};
 
 
 struct recno_list_struct {
@@ -87,28 +99,31 @@ typedef struct recno_list_struct *RECNO_LIST;
 
 static int walk_sigrecs( SIGREC_CONTEXT *c );
 
-static LOCAL_ID_INFO *new_lid_table(void);
-static void release_lid_table( LOCAL_ID_INFO *tbl );
-static int ins_lid_table_item( LOCAL_ID_INFO *tbl, ulong lid, unsigned flag );
-static int qry_lid_table_flag( LOCAL_ID_INFO *tbl, ulong lid, unsigned *flag );
-static void upd_lid_table_flag( LOCAL_ID_INFO *tbl, ulong lid, unsigned flag );
+static LOCAL_ID_TABLE new_lid_table(void);
+static void release_lid_table( LOCAL_ID_TABLE tbl );
+static int ins_lid_table_item( LOCAL_ID_TABLE tbl, ulong lid, unsigned flag );
+static int qry_lid_table_flag( LOCAL_ID_TABLE tbl, ulong lid, unsigned *flag );
 
 static void print_user_id( const char *text, u32 *keyid );
-static int do_list_path( TRUST_INFO *stack, int depth, int max_depth,
-                        LOCAL_ID_INFO *lids, TRUST_SEG_LIST *tslist );
-
+static void sort_tsl_list( TRUST_SEG_LIST *trust_seg_list );
 static int list_sigs( ulong pubkey_id );
-static int propagate_trust( TRUST_SEG_LIST tslist );
 static int do_check( TRUSTREC *drec, unsigned *trustlevel );
 static int get_dir_record( PKT_public_key *pk, TRUSTREC *rec );
 
+static void upd_pref_record( TRUSTREC *urec, u32 *keyid, PKT_signature *sig );
+static void upd_cert_record( KBNODE keyblock, KBNODE signode, u32 *keyid,
+                TRUSTREC *drec, RECNO_LIST *recno_list, int recheck,
+                TRUSTREC *urec, const byte *uidhash, int revoke );
+
+static struct keyid_list *trusted_key_list;
 
 /* a table used to keep track of ultimately trusted keys
- * which are the ones from our secrings */
-static LOCAL_ID_INFO *ultikey_table;
+ * which are the ones from our secrings and the trusted keys */
+static LOCAL_ID_TABLE ultikey_table;
 
-static ulong last_trust_web_key;
-static TRUST_SEG_LIST last_trust_web_tslist;
+/* list of unused lid items and tables */
+static LOCAL_ID_TABLE unused_lid_tables;
+static struct local_id_item *unused_lid_items;
 
 
 #define HEXTOBIN(a) ( (a) >= '0' && (a) <= '9' ? ((a)-'0') : \
@@ -124,7 +139,7 @@ static void
 die_invalid_db()
 {
     log_error(_(
-       "The trust DB is corrupted; please run \"gpgm --fix-trust-db\".\n") );
+       "The trustdb is corrupted; please run \"gpgm --fix-trustdb\".\n") );
     g10_exit(2);
 }
 
@@ -137,7 +152,7 @@ read_record( ulong recno, TRUSTREC *rec, int rectype )
     int rc = tdbio_read_record( recno, rec, rectype );
     if( !rc )
        return;
-    log_error("trust record %lu, req type %d: read failed: %s\n",
+    log_error(_("trust record %lu, req type %d: read failed: %s\n"),
                                    recno, rectype,  g10_errstr(rc) );
     die_invalid_db();
 }
@@ -152,7 +167,7 @@ write_record( TRUSTREC *rec )
     int rc = tdbio_write_record( rec );
     if( !rc )
        return;
-    log_error("trust record %lu, type %d: write failed: %s\n",
+    log_error(_("trust record %lu, type %d: write failed: %s\n"),
                            rec->recnum, rec->rectype, g10_errstr(rc) );
     die_invalid_db();
 }
@@ -166,11 +181,24 @@ delete_record( ulong recno )
     int rc = tdbio_delete_record( recno );
     if( !rc )
        return;
-    log_error("trust record %lu: delete failed: %s\n",
+    log_error(_("trust record %lu: delete failed: %s\n"),
                                              recno, g10_errstr(rc) );
     die_invalid_db();
 }
 
+/****************
+ * sync the db
+ */
+static void
+do_sync( )
+{
+    int rc = tdbio_sync();
+    if( !rc )
+       return;
+    log_error(_("trustdb: sync failed: %s\n"), g10_errstr(rc) );
+    g10_exit(2);
+}
+
 
 \f
 /**********************************************
@@ -214,53 +242,67 @@ rel_recno_list( RECNO_LIST *head )
     *head = NULL;
 }
 
-static LOCAL_ID_INFO *
+static LOCAL_ID_TABLE
 new_lid_table(void)
 {
-    return m_alloc_clear( 16 * sizeof(LOCAL_ID_INFO));
+    LOCAL_ID_TABLE a;
+
+    a = unused_lid_tables;
+    if( a ) {
+       unused_lid_tables = a->next;
+       memset( a, 0, sizeof *a );
+    }
+    else
+       a = m_alloc_clear( sizeof *a );
+    return a;
 }
 
 static void
-release_lid_table( LOCAL_ID_INFO *tbl )
+release_lid_table( LOCAL_ID_TABLE tbl )
 {
-    LOCAL_ID_INFO a, a2;
+    struct local_id_item *a, *a2;
     int i;
 
     for(i=0; i < 16; i++ ) {
-       for(a=tbl[i]; a; a = a2 ) {
+       for(a=tbl->items[i]; a; a = a2 ) {
            a2 = a->next;
-           m_free(a);
+           a->next = unused_lid_items;
+           unused_lid_items = a;
        }
     }
-    m_free(tbl);
+    tbl->next = unused_lid_tables;
+    unused_lid_tables = tbl;
 }
 
 /****************
  * Add a new item to the table or return 1 if we already have this item
- * fixme: maybe it's a good idea to take items from an unused item list.
  */
 static int
-ins_lid_table_item( LOCAL_ID_INFO *tbl, ulong lid, unsigned flag )
+ins_lid_table_item( LOCAL_ID_TABLE tbl, ulong lid, unsigned flag )
 {
-    LOCAL_ID_INFO a;
+    struct local_id_item *a;
 
-    for( a = tbl[lid & 0x0f]; a; a = a->next )
+    for( a = tbl->items[lid & 0x0f]; a; a = a->next )
        if( a->lid == lid )
            return 1;
-    a = m_alloc( sizeof *a );
+    a = unused_lid_items;
+    if( a )
+       unused_lid_items = a->next;
+    else
+       a = m_alloc( sizeof *a );
     a->lid = lid;
     a->flag = flag;
-    a->next = tbl[lid & 0x0f];
-    tbl[lid & 0x0f] = a;
+    a->next = tbl->items[lid & 0x0f];
+    tbl->items[lid & 0x0f] = a;
     return 0;
 }
 
 static int
-qry_lid_table_flag( LOCAL_ID_INFO *tbl, ulong lid, unsigned *flag )
+qry_lid_table_flag( LOCAL_ID_TABLE tbl, ulong lid, unsigned *flag )
 {
-    LOCAL_ID_INFO a;
+    struct local_id_item *a;
 
-    for( a = tbl[lid & 0x0f]; a; a = a->next )
+    for( a = tbl->items[lid & 0x0f]; a; a = a->next )
        if( a->lid == lid ) {
            if( flag )
                *flag = a->flag;
@@ -269,19 +311,6 @@ qry_lid_table_flag( LOCAL_ID_INFO *tbl, ulong lid, unsigned *flag )
     return -1;
 }
 
-static void
-upd_lid_table_flag( LOCAL_ID_INFO *tbl, ulong lid, unsigned flag )
-{
-    LOCAL_ID_INFO a;
-
-    for( a = tbl[lid & 0x0f]; a; a = a->next )
-       if( a->lid == lid ) {
-           a->flag = flag;
-           return;
-       }
-    BUG();
-}
-
 
 
 /****************
@@ -293,19 +322,26 @@ keyid_from_lid( ulong lid, u32 *keyid )
     TRUSTREC rec;
     int rc;
 
-    rc = tdbio_read_record( lid, &rec, RECTYPE_DIR );
+    rc = tdbio_read_record( lid, &rec, 0 );
     if( rc ) {
-       log_error("error reading dir record for LID %lu: %s\n",
+       log_error(_("error reading dir record for LID %lu: %s\n"),
                                                    lid, g10_errstr(rc));
        return G10ERR_TRUSTDB;
     }
+    if( rec.rectype == RECTYPE_SDIR )
+       return 0;
+    if( rec.rectype != RECTYPE_DIR ) {
+       log_error(_("lid %lu: expected dir record, got type %d\n"),
+                                                   lid, rec.rectype );
+       return G10ERR_TRUSTDB;
+    }
     if( !rec.r.dir.keylist ) {
-       log_error("no primary key for LID %lu\n", lid );
+       log_error(_("no primary key for LID %lu\n"), lid );
        return G10ERR_TRUSTDB;
     }
     rc = tdbio_read_record( rec.r.dir.keylist, &rec, RECTYPE_KEY );
     if( rc ) {
-       log_error("error reading primary key for LID %lu: %s\n",
+       log_error(_("error reading primary key for LID %lu: %s\n"),
                                                    lid, g10_errstr(rc));
        return G10ERR_TRUSTDB;
     }
@@ -334,7 +370,6 @@ lid_from_keyblock( KBNODE keyblock )
 
 
 \f
-
 /****************
  * Walk through the signatures of a public key.
  * The caller must provide a context structure, with all fields set
@@ -356,9 +391,13 @@ walk_sigrecs( SIGREC_CONTEXT *c )
     r = &c->ctl.rec;
     if( !c->ctl.init_done ) {
        c->ctl.init_done = 1;
-       read_record( c->lid, r, RECTYPE_DIR );
+       read_record( c->lid, r, 0 );
+       if( r->rectype != RECTYPE_DIR ) {
+           c->ctl.eof = 1;
+           return -1;  /* return eof */
+       }
        c->ctl.nextuid = r->r.dir.uidlist;
-       /* force a read (what a bad bad hack) */
+       /* force a read */
        c->ctl.index = SIGS_PER_RECORD;
        r->r.sig.next = 0;
     }
@@ -398,9 +437,45 @@ walk_sigrecs( SIGREC_CONTEXT *c )
  ************* Trust  stuff  ******************
  ***********************************************/
 
+static int
+trust_letter( unsigned value )
+{
+    switch( value ) {
+      case TRUST_UNKNOWN:   return '-';
+      case TRUST_EXPIRED:   return 'e';
+      case TRUST_UNDEFINED: return 'q';
+      case TRUST_NEVER:     return 'n';
+      case TRUST_MARGINAL:  return 'm';
+      case TRUST_FULLY:     return 'f';
+      case TRUST_ULTIMATE:  return 'u';
+      default:             return  0 ;
+    }
+}
+
+
+void
+register_trusted_key( const char *string )
+{
+    u32 keyid[2];
+    struct keyid_list *r;
+
+    if( classify_user_id( string, keyid, NULL, NULL, NULL ) != 11 ) {
+       log_error(_("'%s' is not a valid long keyID\n"), string );
+       return;
+    }
+
+    for( r = trusted_key_list; r; r = r->next )
+       if( r->keyid[0] == keyid[0] && r->keyid[1] == keyid[1] )
+           return;
+    r = m_alloc( sizeof *r );
+    r->keyid[0] = keyid[0];
+    r->keyid[1] = keyid[1];
+    r->next = trusted_key_list;
+    trusted_key_list = r;
+}
 
 /****************
- * Verify that all our public keys are in the trustDB.
+ * Verify that all our public keys are in the trustdb.
  */
 static int
 verify_own_keys()
@@ -410,30 +485,76 @@ verify_own_keys()
     PKT_secret_key *sk = m_alloc_clear( sizeof *sk );
     PKT_public_key *pk = m_alloc_clear( sizeof *pk );
     u32 keyid[2];
+    struct keyid_list *kl;
+
+    /* put the trusted keys into the trusted key table */
+    for( kl = trusted_key_list; kl; kl = kl->next ) {
+       keyid[0] = kl->keyid[0];
+       keyid[1] = kl->keyid[1];
+       /* get the public key */
+       memset( pk, 0, sizeof *pk );
+       rc = get_pubkey( pk, keyid );
+       if( rc ) {
+           log_info(_("key %08lX: no public key for trusted key - skipped\n"),
+                                                           (ulong)keyid[1] );
+       }
+       else {
+           /* make sure that the pubkey is in the trustdb */
+           rc = query_trust_record( pk );
+           if( rc == -1 ) { /* put it into the trustdb */
+               rc = insert_trust_record( pk );
+               if( rc ) {
+                   log_error(_("key %08lX: can't put it into the trustdb\n"),
+                                                       (ulong)keyid[1] );
+               }
+           }
+           else if( rc ) {
+               log_error(_("key %08lX: query record failed\n"),
+                                                       (ulong)keyid[1] );
+           }
+           else {
+               if( ins_lid_table_item( ultikey_table, pk->local_id, 0 ) )
+                   log_error(_("key %08lX: already in trusted key table\n"),
+                                                         (ulong)keyid[1]);
+               else if( opt.verbose > 1 )
+                   log_info(_("key %08lX: accepted as trusted key.\n"),
+                                                         (ulong)keyid[1]);
+           }
+       }
+       release_public_key_parts( pk );
+    }
 
     while( !(rc=enum_secret_keys( &enum_context, sk, 0 ) ) ) {
+       int have_pk = 0;
+
        keyid_from_sk( sk, keyid );
 
        if( DBG_TRUST )
            log_debug("key %08lX: checking secret key\n", (ulong)keyid[1] );
 
        if( is_secret_key_protected( sk ) < 1 )
-           log_info("note: secret key %08lX is NOT protected.\n",
+           log_info(_("NOTE: secret key %08lX is NOT protected.\n"),
                                                            (ulong)keyid[1] );
 
+       for( kl = trusted_key_list; kl; kl = kl->next ) {
+           if( kl->keyid[0] == keyid[0] && kl->keyid[1] == keyid[1] )
+               goto skip; /* already in trusted key table */
+       }
+
        /* see whether we can access the public key of this secret key */
        memset( pk, 0, sizeof *pk );
        rc = get_pubkey( pk, keyid );
        if( rc ) {
-           log_error(_("key %08lX: secret key without public key\n"),
+           log_info(_("key %08lX: secret key without public key - skipped\n"),
                                                            (ulong)keyid[1] );
-           goto leave;
+           goto skip;
        }
+       have_pk=1;
+
        if( cmp_public_secret_key( pk, sk ) ) {
-           log_error(_("key %08lX: secret and public key don't match\n"),
+           log_info(_("key %08lX: secret and public key don't match\n"),
                                                            (ulong)keyid[1] );
-           rc = G10ERR_GENERAL;
-           goto leave;
+           goto skip;
        }
 
        /* make sure that the pubkey is in the trustdb */
@@ -443,12 +564,12 @@ verify_own_keys()
            if( rc ) {
                log_error(_("key %08lX: can't put it into the trustdb\n"),
                                                            (ulong)keyid[1] );
-               goto leave;
+               goto skip;
            }
        }
        else if( rc ) {
            log_error(_("key %08lX: query record failed\n"), (ulong)keyid[1] );
-           goto leave;
+           goto skip;
 
        }
 
@@ -456,18 +577,30 @@ verify_own_keys()
            log_debug("key %08lX.%lu: stored into ultikey_table\n",
                                    (ulong)keyid[1], pk->local_id );
        if( ins_lid_table_item( ultikey_table, pk->local_id, 0 ) )
-           log_error(_("key %08lX: already in ultikey_table\n"),
+           log_error(_("key %08lX: already in trusted key table\n"),
                                                        (ulong)keyid[1]);
-
+       else if( opt.verbose > 1 )
+           log_info(_("key %08lX: accepted as trusted key.\n"),
+                                                       (ulong)keyid[1]);
+      skip:
        release_secret_key_parts( sk );
-       release_public_key_parts( pk );
+       if( have_pk )
+           release_public_key_parts( pk );
     }
     if( rc != -1 )
-       log_error(_("enum_secret_keys failed: %s\n"), g10_errstr(rc) );
+       log_error(_("enumerate secret keys failed: %s\n"), g10_errstr(rc) );
     else
        rc = 0;
 
-  leave:
+    /* release the trusted keyid table */
+    {  struct keyid_list *kl2;
+       for( kl = trusted_key_list; kl; kl = kl2 ) {
+           kl2 = kl->next;
+           m_free( kl );
+       }
+       trusted_key_list = NULL;
+    }
+
     enum_secret_keys( &enum_context, NULL, 0 ); /* free context */
     free_secret_key( sk );
     free_public_key( pk );
@@ -493,17 +626,18 @@ print_user_id( const char *text, u32 *keyid )
     m_free(p);
 }
 
-static void
+#if 0
+static int
 print_keyid( FILE *fp, ulong lid )
 {
     u32 ki[2];
     if( keyid_from_lid( lid, ki ) )
-       fprintf(fp, "????????.%lu", lid );
+       return fprintf(fp, "????????.%lu", lid );
     else
-       fprintf(fp, "%08lX.%lu", (ulong)ki[1], lid );
+       return fprintf(fp, "%08lX.%lu", (ulong)ki[1], lid );
 }
 
-static void
+static int
 print_trust( FILE *fp, unsigned trust )
 {
     int c;
@@ -515,41 +649,62 @@ print_trust( FILE *fp, unsigned trust )
       case TRUST_MARGINAL:  c = 'm'; break;
       case TRUST_FULLY:     c = 'f'; break;
       case TRUST_ULTIMATE:  c = 'u'; break;
-      default: fprintf(fp, "%02x", trust ); return;
+      default: fprintf(fp, "%02x", trust ); return 2;
     }
     putc(c, fp);
+    return 1;
+}
+#endif
+
+static int
+print_sigflags( FILE *fp, unsigned flags )
+{
+    if( flags & SIGF_CHECKED ) {
+       fprintf(fp,"%c%c%c",
+          (flags & SIGF_VALID)   ? 'V':'-',
+          (flags & SIGF_EXPIRED) ? 'E':'-',
+          (flags & SIGF_REVOKED) ? 'R':'-');
+    }
+    else if( flags & SIGF_NOPUBKEY)
+       fputs("?--", fp);
+    else
+       fputs("---", fp);
+    return 3;
 }
 
 /* (a non-recursive algorithm would be easier) */
 static int
-do_list_sigs( ulong root, ulong pubkey, int depth,
-             LOCAL_ID_INFO *lids, unsigned *lineno )
+do_list_sigs( ulong root, ulong pk_lid, int depth,
+             LOCAL_ID_TABLE lids, unsigned *lineno )
 {
     SIGREC_CONTEXT sx;
     int rc;
     u32 keyid[2];
 
     memset( &sx, 0, sizeof sx );
-    sx.lid = pubkey;
+    sx.lid = pk_lid;
     for(;;) {
-       rc = walk_sigrecs( &sx );
-       if( rc )
+       rc = walk_sigrecs( &sx ); /* should we replace it and use */
+       if( rc )                  /* use a loop like in collect_paths ??*/
            break;
        rc = keyid_from_lid( sx.sig_lid, keyid );
        if( rc ) {
-           printf("%6u: %*s????????.%lu:%02x\n", *lineno, depth*4, "",
-                                                  sx.sig_lid, sx.sig_flag );
+           printf("%6u: %*s????????.%lu:", *lineno, depth*4, "", sx.sig_lid );
+           print_sigflags( stdout, sx.sig_flag );
+           putchar('\n');
            ++*lineno;
        }
        else {
-           printf("%6u: %*s%08lX.%lu:%02x ", *lineno, depth*4, "",
-                             (ulong)keyid[1], sx.sig_lid, sx.sig_flag );
-           /* check whether we already checked this pubkey */
+           printf("%6u: %*s%08lX.%lu:", *lineno, depth*4, "",
+                             (ulong)keyid[1], sx.sig_lid );
+           print_sigflags( stdout, sx.sig_flag );
+           putchar(' ');
+           /* check whether we already checked this pk_lid */
            if( !qry_lid_table_flag( ultikey_table, sx.sig_lid, NULL ) ) {
                print_user_id("[ultimately trusted]", keyid);
                ++*lineno;
            }
-           else if( sx.sig_lid == pubkey ) {
+           else if( sx.sig_lid == pk_lid ) {
                printf("[self-signature]\n");
                ++*lineno;
            }
@@ -587,7 +742,7 @@ list_sigs( ulong pubkey_id )
 {
     int rc;
     u32 keyid[2];
-    LOCAL_ID_INFO *lids;
+    LOCAL_ID_TABLE lids;
     unsigned lineno = 1;
 
     rc = keyid_from_lid( pubkey_id, keyid );
@@ -616,15 +771,16 @@ list_records( ulong lid )
 
     rc = tdbio_read_record( lid, &dr, RECTYPE_DIR );
     if( rc ) {
-       log_error("lid %lu: read dir record failed: %s\n", lid, g10_errstr(rc));
+       log_error(_("lid %lu: read dir record failed: %s\n"),
+                                               lid, g10_errstr(rc));
        return rc;
     }
     tdbio_dump_record( &dr, stdout );
 
     for( recno=dr.r.dir.keylist; recno; recno = rec.r.key.next ) {
-       rc = tdbio_read_record( recno, &rec, RECTYPE_KEY );
+       rc = tdbio_read_record( recno, &rec, 0 );
        if( rc ) {
-           log_error("lid %lu: read key record failed: %s\n",
+           log_error(_("lid %lu: read key record failed: %s\n"),
                                                lid, g10_errstr(rc));
            return rc;
        }
@@ -634,7 +790,7 @@ list_records( ulong lid )
     for( recno=dr.r.dir.uidlist; recno; recno = ur.r.uid.next ) {
        rc = tdbio_read_record( recno, &ur, RECTYPE_UID );
        if( rc ) {
-           log_error("lid %lu: read uid record failed: %s\n",
+           log_error(_("lid %lu: read uid record failed: %s\n"),
                                                lid, g10_errstr(rc));
            return rc;
        }
@@ -643,7 +799,7 @@ list_records( ulong lid )
        for(recno=ur.r.uid.prefrec; recno; recno = rec.r.pref.next ) {
            rc = tdbio_read_record( recno, &rec, RECTYPE_PREF );
            if( rc ) {
-               log_error("lid %lu: read pref record failed: %s\n",
+               log_error(_("lid %lu: read pref record failed: %s\n"),
                                                    lid, g10_errstr(rc));
                return rc;
            }
@@ -653,7 +809,7 @@ list_records( ulong lid )
        for(recno=ur.r.uid.siglist; recno; recno = rec.r.sig.next ) {
            rc = tdbio_read_record( recno, &rec, RECTYPE_SIG );
            if( rc ) {
-               log_error("lid %lu: read sig record failed: %s\n",
+               log_error(_("lid %lu: read sig record failed: %s\n"),
                                                    lid, g10_errstr(rc));
                return rc;
            }
@@ -670,281 +826,220 @@ list_records( ulong lid )
 
 
 
+
 /****************
- * Function to collect all trustpaths
+ * stack is an array of (max_path+1) elements. If trust_seg_head is not
+ * NULL it is a pointer to a variable which will receive a linked list
+ * of trust paths - The caller has to free the memory.
  */
 static int
-do_list_path( TRUST_INFO *stack, int depth, int max_depth,
-             LOCAL_ID_INFO *lids, TRUST_SEG_LIST *tslist )
+collect_paths( int depth, int max_depth, int all, TRUSTREC *drec,
+              TRUST_INFO *stack, TRUST_SEG_LIST *trust_seg_head )
 {
-    SIGREC_CONTEXT sx;
-    unsigned last_depth;
-    int rc;
-
-    assert(depth);
-
-    /*printf("%2lu/%d: scrutinizig\n", stack[depth-1], depth);*/
-    if( depth >= max_depth || depth >= MAX_LIST_SIGS_DEPTH-1 ) {
-       /*printf("%2lu/%d: too deeply nested\n", stack[depth-1], depth);*/
-       return 0;
-    }
-    memset( &sx, 0, sizeof sx );
-    sx.lid = stack[depth-1].lid;
-    /* loop over all signatures. If we do not have any, try to create them */
-    while( !(rc = walk_sigrecs( &sx )) ) {
-       TRUST_SEG_LIST tsl, t2, tl;
+    ulong rn, uidrn;
+    int marginal=0;
+    int fully=0;
+    LOCAL_ID_TABLE sigs_seen = NULL;
+
+    if( depth >= max_depth )  /* max cert_depth reached */
+       return TRUST_UNDEFINED;
+
+    stack[depth].lid = drec->r.dir.lid;
+    stack[depth].otrust = drec->r.dir.ownertrust;
+    stack[depth].trust = 0;
+    {  int i;
+
+       for(i=0; i < depth; i++ )
+           if( stack[i].lid == drec->r.dir.lid )
+               return TRUST_UNDEFINED; /* closed (we already visited this lid) */
+    }
+    if( !qry_lid_table_flag( ultikey_table, drec->r.dir.lid, NULL ) ) {
+       /* we are at the end of a path */
+       TRUST_SEG_LIST tsl;
        int i;
 
-       stack[depth].lid = sx.sig_lid;
-       stack[depth].trust = 0;
-       if( qry_lid_table_flag( lids, sx.sig_lid, &last_depth) ) {
-           /*printf("%2lu/%d: marked\n", sx.sig_lid, depth );*/
-           ins_lid_table_item( lids, sx.sig_lid, depth);
-           last_depth = depth;
-       }
-       else if( depth  < last_depth ) {
-           /*printf("%2lu/%d: last_depth=%u - updated\n", sx.sig_lid, depth, last_depth);*/
-           last_depth = depth;
-           upd_lid_table_flag( lids, sx.sig_lid, depth);
-       }
-
-       if( last_depth < depth )
-           /*printf("%2lu/%d: already visited\n", sx.sig_lid, depth)*/;
-       else if( !qry_lid_table_flag( ultikey_table, sx.sig_lid, NULL ) ) {
-           /* found end of path; store it, ordered by path length */
-           tsl = m_alloc( sizeof *tsl + depth*sizeof(TRUST_INFO) );
-           tsl->nseg = depth+1;
-           tsl->dup = 0;
+       stack[depth].trust = TRUST_ULTIMATE;
+       stack[depth].otrust = TRUST_ULTIMATE;
+       if( trust_seg_head ) {
+           /* we can now put copy our current stack to the trust_seg_list */
+           tsl = m_alloc( sizeof *tsl + (depth+1)*sizeof( TRUST_INFO ) );
            for(i=0; i <= depth; i++ )
-               tsl->seg[i] = stack[i];
-           for(t2=*tslist,tl=NULL; t2; tl=t2, t2 = t2->next )
-               if( depth < t2->nseg )
-                   break;
-           if( !tl ) {
-               tsl->next = t2;
-               *tslist = tsl;
-           }
-           else {
-               tsl->next = t2;
-               tl->next = tsl;
-           }
-           /*putchar('.'); fflush(stdout);*/
-           /*printf("%2lu/%d: found\n", sx.sig_lid, depth);*/
-       }
-       else {
-           rc = do_list_path( stack, depth+1, max_depth, lids, tslist);
-           if( rc && rc != -1 )
-               break;
-       }
-    }
-    return rc==-1? 0 : rc;
-}
+               tsl->path[i] = stack[i];
+           tsl->pathlen = i;
+           tsl->next = *trust_seg_head;
+           *trust_seg_head = tsl;
+       }
+       return TRUST_ULTIMATE;
+    }
+
+    /* loop over all user-ids */
+    if( !all )
+       sigs_seen = new_lid_table();
+    for( rn = drec->r.dir.uidlist; rn; rn = uidrn ) {
+       TRUSTREC rec;  /* used for uids and sigs */
+       ulong sigrn;
+
+       read_record( rn, &rec, RECTYPE_UID );
+       uidrn = rec.r.uid.next;
+       if( !(rec.r.uid.uidflags & UIDF_CHECKED) )
+           continue; /* user id has not been checked */
+       if( !(rec.r.uid.uidflags & UIDF_VALID) )
+           continue; /* user id is not valid */
+       if( (rec.r.uid.uidflags & UIDF_REVOKED) )
+           continue; /* user id has been revoked */
+
+       /* loop over all signature records */
+       for( rn = rec.r.uid.siglist; rn; rn = sigrn ) {
+           int i;
 
+           read_record( rn, &rec, RECTYPE_SIG );
+           sigrn = rec.r.sig.next;
 
-/****************
- * Make a list of trust paths
- */
-static int
-make_tsl( ulong lid, TRUST_SEG_LIST *ret_tslist )
-{
-    int i, rc;
-    LOCAL_ID_INFO *lids = new_lid_table();
-    TRUST_INFO stack[MAX_LIST_SIGS_DEPTH];
-    TRUST_SEG_LIST tsl, tslist;
-    int max_depth = 4;
-
-    tslist = *ret_tslist = NULL;
-
-    if( !qry_lid_table_flag( ultikey_table, lid, NULL ) ) {
-       tslist = m_alloc( sizeof *tslist );
-       tslist->nseg = 1;
-       tslist->dup = 0;
-       tslist->seg[0].lid = lid;
-       tslist->seg[0].trust = 0;
-       tslist->next = NULL;
-       rc = 0;
-    }
-    else {
-       stack[0].lid = lid;
-       stack[0].trust = 0;
-       rc = do_list_path( stack, 1, max_depth, lids, &tslist );
-    }
-    if( !rc ) { /* wipe out duplicates */
-       LOCAL_ID_INFO *work = new_lid_table();
-       for( tsl=tslist; tsl; tsl = tsl->next ) {
-           for(i=1; i < tsl->nseg-1; i++ ) {
-               if( ins_lid_table_item( work, tsl->seg[i].lid, 0 ) ) {
-                   tsl->dup = 1; /* mark as duplicate */
-                   break;
+           for(i=0; i < SIGS_PER_RECORD; i++ ) {
+               TRUSTREC tmp;
+               int ot, nt;
+               int unchecked = 0;
+
+               if( !rec.r.sig.sig[i].lid )
+                   continue; /* skip deleted sigs */
+               if( !(rec.r.sig.sig[i].flag & SIGF_CHECKED) ) {
+                   if( !all )
+                       continue; /* skip unchecked signatures */
+                   unchecked = 1;
+               }
+               else {
+                   if( !(rec.r.sig.sig[i].flag & SIGF_VALID) )
+                       continue; /* skip invalid signatures */
+                   if( (rec.r.sig.sig[i].flag & SIGF_EXPIRED) )
+                       continue; /* skip expired signatures */
+                   if( (rec.r.sig.sig[i].flag & SIGF_REVOKED) )
+                       continue; /* skip revoked signatures */
+               }
+
+               /* visit every signer only once (a signer may have
+                * signed more than one user ID) */
+               if( sigs_seen && ins_lid_table_item( sigs_seen,
+                                                    rec.r.sig.sig[i].lid, 0) )
+                   continue; /* we already have this one */
+
+               read_record( rec.r.sig.sig[i].lid, &tmp, 0 );
+               if( tmp.rectype != RECTYPE_DIR ) {
+                   if( tmp.rectype != RECTYPE_SDIR )
+                       log_info("oops: lid %lu: sig %lu has rectype %d"
+                            " - skipped\n",
+                           drec->r.dir.lid, tmp.recnum, tmp.rectype );
+                   continue;
+               }
+               ot = tmp.r.dir.ownertrust & TRUST_MASK;
+               if( ot >= TRUST_FULLY )
+                   ot = TRUST_FULLY;  /* just in case */
+               nt = collect_paths( depth+1, max_depth, all, &tmp, stack,
+                                                       trust_seg_head );
+               nt &= TRUST_MASK;
+
+               if( nt < TRUST_MARGINAL || unchecked ) {
+                   continue;
+               }
+
+               if( nt == TRUST_ULTIMATE ) {
+                   /* we have signed this key and only in this special case
+                    * we assume that this one is fully trusted */
+                   if( !all ) {
+                       if( sigs_seen )
+                           release_lid_table( sigs_seen );
+                       return (stack[depth].trust = TRUST_FULLY);
+                   }
+               }
+
+               if( nt > ot )
+                   nt = ot;
+
+               if( nt >= TRUST_FULLY )
+                   fully++;
+               if( nt >= TRUST_MARGINAL )
+                   marginal++;
+
+               if( fully >= opt.completes_needed
+                   || marginal >= opt.marginals_needed ) {
+                   if( !all ) {
+                       if( sigs_seen )
+                           release_lid_table( sigs_seen );
+                       return (stack[depth].trust = TRUST_FULLY);
+                   }
                }
            }
        }
-       release_lid_table(work);
-       *ret_tslist = tslist;
     }
-    else
-       ; /* FIXME: release tslist */
-    release_lid_table(lids);
-    return rc;
+    if( sigs_seen )
+       release_lid_table( sigs_seen );
+    if( all && ( fully >= opt.completes_needed
+                || marginal >= opt.marginals_needed ) ) {
+       return (stack[depth].trust = TRUST_FULLY );
+    }
+    if( marginal ) {
+       return (stack[depth].trust = TRUST_MARGINAL);
+    }
+    return (stack[depth].trust=TRUST_UNDEFINED);
 }
 
 
 /****************
- * Given a trust segment list tslist, walk over all paths and fill in
- * the trust information for each segment.  What this function does is
- * to assign a trustvalue to the first segment (which is the requested key)
- * of each path.
- *
- * FIXME: We have to do more thinking here. e.g. we should never increase
- *       the trust value.
- *
- * Do not do it for duplicates.
+ * Given the directory record of a key, check whether we can
+ * find a path to an ultimately trusted key.  We do this by
+ * checking all key signatures up to a some depth.
  */
 static int
-propagate_trust( TRUST_SEG_LIST tslist )
+verify_key( int max_depth, TRUSTREC *drec )
 {
-    int i;
-    unsigned trust, tr;
-    TRUST_SEG_LIST tsl;
+    TRUST_INFO *tmppath = m_alloc_clear( (max_depth+1)* sizeof *tmppath );
+    int tr;
 
-    for(tsl = tslist; tsl; tsl = tsl->next ) {
-       if( tsl->dup )
-           continue;
-       assert( tsl->nseg );
-       /* the last segment is always an ultimately trusted one, so we can
-        * assign a fully trust to the next one */
-       i = tsl->nseg-1;
-       tsl->seg[i].trust = TRUST_ULTIMATE;
-       trust = TRUST_FULLY;
-       for(i-- ; i >= 0; i-- ) {
-           tsl->seg[i].trust = trust;
-           if( i > 0 ) {
-               /* get the trust of this pubkey */
-               tr = get_ownertrust( tsl->seg[i].lid );
-               if( tr < trust )
-                   trust = tr;
-           }
-       }
-    }
-    return 0;
+    tr = collect_paths( 0, max_depth, 0, drec, tmppath, NULL );
+    m_free( tmppath );
+    return tr;
 }
 
 
+
+
 /****************
  * we have the pubkey record and all needed informations are in the trustdb
  * but nothing more is known.
- * (this function may re-read the dir record dr)
  */
 static int
-do_check( TRUSTREC *dr, unsigned *trustlevel )
+do_check( TRUSTREC *dr, unsigned *validity )
 {
-    int i, rc=0;
-    TRUST_SEG_LIST tsl, tsl2, tslist;
-    int marginal, fully;
-    int fully_needed = opt.completes_needed;
-    int marginal_needed = opt.marginals_needed;
-    unsigned tflags = 0;
-
-    assert( fully_needed > 0 && marginal_needed > 1 );
-
-
-    *trustlevel = TRUST_UNDEFINED;
-
     if( !dr->r.dir.keylist ) {
-       log_error("Ooops, no keys\n");
+       log_error(_("Ooops, no keys\n"));
        return G10ERR_TRUSTDB;
     }
     if( !dr->r.dir.uidlist ) {
-       log_error("Ooops, no user ids\n");
+       log_error(_("Ooops, no user ids\n"));
        return G10ERR_TRUSTDB;
     }
 
-    /* did we already check the signatures */
-    /* fixme:.... */
-
-    if( dr->r.dir.dirflags & DIRF_REVOKED )
-       tflags |= TRUST_FLAG_REVOKED;
-
-  #if 0
-    if( !rc && !dr->r.dir.siglist ) {
-       /* We do not have any signatures; check whether it is one of our
-        * secret keys */
-       if( !qry_lid_table_flag( ultikey_table, dr->r.dir.lid, NULL ) )
-           *trustlevel = tflags | TRUST_ULTIMATE;
-       return 0;
-    }
-  #endif
-    if( rc )
-       return rc;  /* error while looking for sigrec or building sigrecs */
-
-    /* fixme: take it from the cache if it is valid */
-
-    /* Make a list of all possible trust-paths */
-    rc = make_tsl( dr->r.dir.lid, &tslist );
-    if( rc )
-       return rc;
-    rc = propagate_trust( tslist );
-    if( rc )
-       return rc;
-    for(tsl = tslist; tsl; tsl = tsl->next ) {
-       if( tsl->dup )
-           continue;
-
-       if( opt.verbose ) {
-           log_info("trust path:" );
-           for(i=0; i < tsl->nseg; i++ ) {
-               putc(' ',stderr);
-               print_keyid( stderr, tsl->seg[i].lid );
-               putc(':',stderr);
-               print_trust( stderr, tsl->seg[i].trust );
-           }
-           putc('\n',stderr);
+    if( tdbio_db_matches_options()
+       && (dr->r.dir.dirflags & DIRF_VALVALID)
+       && dr->r.dir.validity )
+       *validity = dr->r.dir.validity;
+    else {
+       *validity = verify_key( opt.max_cert_depth, dr );
+       if( (*validity & TRUST_MASK) >= TRUST_UNDEFINED
+           && tdbio_db_matches_options() ) {
+           /* update the cached validity value */
+           dr->r.dir.validity = (*validity & TRUST_MASK);
+           dr->r.dir.dirflags |= DIRF_VALVALID;
+           write_record( dr );
        }
     }
 
-    /* and see whether there is a trusted path.
-     * We only have to look at the first segment, because
-     * propagate_trust has investigated all other segments */
-    marginal = fully = 0;
-    for(tsl = tslist; tsl; tsl = tsl->next ) {
-       if( tsl->dup )
-           continue;
-       if( tsl->seg[0].trust == TRUST_ULTIMATE ) {
-           *trustlevel = tflags | TRUST_ULTIMATE; /* our own key */
-           break;
-       }
-       if( tsl->seg[0].trust == TRUST_FULLY ) {
-           marginal++;
-           fully++;
-       }
-       else if( tsl->seg[0].trust == TRUST_MARGINAL )
-           marginal++;
-
-       if( fully >= fully_needed ) {
-           *trustlevel = tflags | TRUST_FULLY;
-           break;
-       }
-    }
-    if( !tsl && marginal >= marginal_needed )
-       *trustlevel = tflags | TRUST_MARGINAL;
+    if( dr->r.dir.dirflags & DIRF_REVOKED )
+       *validity |= TRUST_FLAG_REVOKED;
 
-    /* cache the tslist */
-    if( last_trust_web_key ) {
-       for( tsl = last_trust_web_tslist; tsl; tsl = tsl2 ) {
-           tsl2 = tsl->next;
-           m_free(tsl);
-       }
-    }
-    last_trust_web_key = dr->r.dir.lid;
-    last_trust_web_tslist = tslist;
     return 0;
 }
 
 \f
-/***********************************************
- ****************  API ************************
- ***********************************************/
-
 /****************
  * Perform some checks over the trustdb
  *  level 0: only open the db
@@ -990,25 +1085,29 @@ list_trustdb( const char *username )
        ulong lid = atoi(username+1);
 
        if( (rc = list_records( lid)) )
-           log_error("user '%s' read problem: %s\n", username, g10_errstr(rc));
+           log_error(_("user '%s' read problem: %s\n"),
+                                           username, g10_errstr(rc));
        else if( (rc = list_sigs( lid )) )
-           log_error("user '%s' list problem: %s\n", username, g10_errstr(rc));
+           log_error(_("user '%s' list problem: %s\n"),
+                                           username, g10_errstr(rc));
     }
     else if( username ) {
        PKT_public_key *pk = m_alloc_clear( sizeof *pk );
        int rc;
 
-       if( (rc = get_pubkey_byname( pk, username )) )
-           log_error("user '%s' not found: %s\n", username, g10_errstr(rc) );
+       if( (rc = get_pubkey_byname( NULL, pk, username, NULL )) )
+           log_error(_("user '%s' not found: %s\n"), username, g10_errstr(rc) );
        else if( (rc=tdbio_search_dir_bypk( pk, &rec )) && rc != -1 )
-           log_error("problem finding '%s' in trustdb: %s\n",
+           log_error(_("problem finding '%s' in trustdb: %s\n"),
                                                username, g10_errstr(rc));
        else if( rc == -1 )
-           log_error("user '%s' not in trustdb\n", username);
+           log_error(_("user '%s' not in trustdb\n"), username);
        else if( (rc = list_records( pk->local_id)) )
-           log_error("user '%s' read problem: %s\n", username, g10_errstr(rc));
+           log_error(_("user '%s' read problem: %s\n"),
+                                               username, g10_errstr(rc));
        else if( (rc = list_sigs( pk->local_id )) )
-           log_error("user '%s' list problem: %s\n", username, g10_errstr(rc));
+           log_error(_("user '%s' list problem: %s\n"),
+                                               username, g10_errstr(rc));
        free_public_key( pk );
     }
     else {
@@ -1037,17 +1136,20 @@ export_ownertrust()
     byte *p;
     int rc;
 
+    printf(_("# List of assigned trustvalues, created %s\n"
+            "# (Use \"gpgm --import-ownertrust\" to restore them)\n"),
+          asctimestamp( make_timestamp() ) );
     for(recnum=0; !tdbio_read_record( recnum, &rec, 0); recnum++ ) {
        if( rec.rectype == RECTYPE_DIR ) {
            if( !rec.r.dir.keylist ) {
-               log_error("Oops; directory record w/o primary key\n");
+               log_error(_("directory record w/o primary key\n"));
                continue;
            }
            if( !rec.r.dir.ownertrust )
                continue;
            rc = tdbio_read_record( rec.r.dir.keylist, &rec2, RECTYPE_KEY);
            if( rc ) {
-               log_error("error reading key record: %s\n", g10_errstr(rc));
+               log_error(_("error reading key record: %s\n"), g10_errstr(rc));
                continue;
            }
            p = rec2.r.key.fingerprint;
@@ -1087,23 +1189,24 @@ import_ownertrust( const char *fname )
            continue;
        n = strlen(line);
        if( line[n-1] != '\n' ) {
-           log_error_f(fname, "line to long\n" );
+           log_error_f(fname, _("line too long\n") );
+           /* ... or last line does not have a LF */
            break; /* can't continue */
        }
        for(p = line; *p && *p != ':' ; p++ )
            if( !isxdigit(*p) )
                break;
        if( *p != ':' ) {
-           log_error_f(fname, "error: missing colon\n" );
+           log_error_f(fname, _("error: missing colon\n") );
            continue;
        }
        fprlen = p - line;
        if( fprlen != 32 && fprlen != 40 ) {
-           log_error_f(fname, "error: invalid fingerprint\n" );
+           log_error_f(fname, _("error: invalid fingerprint\n") );
            continue;
        }
        if( sscanf(p, ":%u:", &otrust ) != 1 ) {
-           log_error_f(fname, "error: no otrust value\n" );
+           log_error_f(fname, _("error: no ownertrust value\n") );
            continue;
        }
        if( !otrust )
@@ -1128,127 +1231,161 @@ import_ownertrust( const char *fname )
        else if( rc == -1 ) { /* not found; get the key from the ring */
            PKT_public_key *pk = m_alloc_clear( sizeof *pk );
 
-           log_info_f(fname, "key not in trustdb, searching ring.\n");
+           log_info_f(fname, _("key not in trustdb, searching ring.\n"));
            rc = get_pubkey_byfprint( pk, line, fprlen );
            if( rc )
-               log_info_f(fname, "key not in ring: %s\n", g10_errstr(rc));
+               log_info_f(fname, _("key not in ring: %s\n"), g10_errstr(rc));
            else {
                rc = query_trust_record( pk );  /* only as assertion */
                if( rc != -1 )
-                   log_error_f(fname, "Oops: key is now in trustdb???\n");
+                   log_error_f(fname, _("Oops: key is now in trustdb???\n"));
                else {
                    rc = insert_trust_record( pk );
                    if( !rc )
                        goto repeat; /* update the ownertrust */
-                   log_error_f(fname, "insert trust record failed: %s\n",
+                   log_error_f(fname, _("insert trust record failed: %s\n"),
                                                           g10_errstr(rc) );
                }
            }
        }
        else /* error */
-           log_error_f(fname, "error finding dir record: %s\n",
+           log_error_f(fname, _("error finding dir record: %s\n"),
                                                    g10_errstr(rc));
     }
     if( ferror(fp) )
        log_error_f(fname, _("read error: %s\n"), strerror(errno) );
     if( !is_stdin )
        fclose(fp);
+    do_sync();
+}
+
+
+
+
+static void
+print_path( int pathlen, TRUST_INFO *path, FILE *fp, ulong highlight )
+{
+    int rc, c, i;
+    u32 keyid[2];
+    char *p;
+    size_t n;
+
+    for( i = 0; i < pathlen; i++ )  {
+       if( highlight )
+           fputs(highlight == path[i].lid? "* ":"  ", fp );
+       rc = keyid_from_lid( path[i].lid, keyid );
+       if( rc )
+           fprintf(fp, "????????.%lu:", path[i].lid );
+       else
+           fprintf(fp,"%08lX.%lu:", (ulong)keyid[1], path[i].lid );
+       c = trust_letter(path[i].otrust);
+       if( c )
+           putc( c, fp );
+       else
+           fprintf( fp, "%02x", path[i].otrust );
+       putc('/', fp);
+       c = trust_letter(path[i].trust);
+       if( c )
+           putc( c, fp );
+       else
+           fprintf( fp, "%02x", path[i].trust );
+       putc(' ', fp);
+       p = get_user_id( keyid, &n );
+       putc(' ', fp);
+       putc('\"', fp);
+       print_string( fp, p, n > 40? 40:n, 0 );
+       putc('\"', fp);
+       m_free(p);
+       putc('\n', fp );
+    }
+}
+
+
+static int
+cmp_tsl_array( const void *xa, const void *xb )
+{
+    TRUST_SEG_LIST a = *(TRUST_SEG_LIST*)xa;
+    TRUST_SEG_LIST b = *(TRUST_SEG_LIST*)xb;
+    return a->pathlen - b->pathlen;
+}
+
+
+static void
+sort_tsl_list( TRUST_SEG_LIST *trust_seg_list )
+{
+    TRUST_SEG_LIST *array, *tail, tsl;
+    size_t n;
+
+    for(n=0, tsl = *trust_seg_list; tsl; tsl = tsl->next )
+       n++;
+    array = m_alloc( (n+1) * sizeof *array );
+    for(n=0, tsl = *trust_seg_list; tsl; tsl = tsl->next )
+       array[n++] = tsl;
+    array[n] = NULL;
+    qsort( array, n, sizeof *array, cmp_tsl_array );
+    *trust_seg_list = NULL;
+    tail = trust_seg_list;
+    for(n=0; (tsl=array[n]); n++ ) {
+       *tail = tsl;
+       tail = &tsl->next;
+    }
+    m_free( array );
 }
 
 
 void
-list_trust_path( int max_depth, const char *username )
+list_trust_path( const char *username )
 {
     int rc;
-    int wipe=0;
-    int i;
     TRUSTREC rec;
+    TRUST_INFO *tmppath;
+    TRUST_SEG_LIST trust_seg_list, tsl, tsl2;
     PKT_public_key *pk = m_alloc_clear( sizeof *pk );
 
-    if( max_depth < 0 ) {
-       wipe = 1;
-       max_depth = -max_depth;
-    }
-
-    if( (rc = get_pubkey_byname( pk, username )) )
-       log_error("user '%s' not found: %s\n", username, g10_errstr(rc) );
+    if( (rc = get_pubkey_byname(NULL, pk, username, NULL )) )
+       log_error(_("user '%s' not found: %s\n"), username, g10_errstr(rc) );
     else if( (rc=tdbio_search_dir_bypk( pk, &rec )) && rc != -1 )
-       log_error("problem finding '%s' in trustdb: %s\n",
+       log_error(_("problem finding '%s' in trustdb: %s\n"),
                                            username, g10_errstr(rc));
     else if( rc == -1 ) {
-       log_info("user '%s' not in trustdb - inserting\n", username);
+       log_info(_("user '%s' not in trustdb - inserting\n"), username);
        rc = insert_trust_record( pk );
        if( rc )
-           log_error("failed to put '%s' into trustdb: %s\n", username, g10_errstr(rc));
+           log_error(_("failed to put '%s' into trustdb: %s\n"),
+                                                   username, g10_errstr(rc));
        else {
            assert( pk->local_id );
        }
     }
+    free_public_key( pk );
 
-    if( !rc ) {
-       TRUST_SEG_LIST tsl, tslist = NULL;
-
-       if( !qry_lid_table_flag( ultikey_table, pk->local_id, NULL ) ) {
-           tslist = m_alloc( sizeof *tslist );
-           tslist->nseg = 1;
-           tslist->dup = 0;
-           tslist->seg[0].lid = pk->local_id;
-           tslist->seg[0].trust = 0;
-           tslist->next = NULL;
-           rc = 0;
-       }
-       else {
-           LOCAL_ID_INFO *lids = new_lid_table();
-           TRUST_INFO stack[MAX_LIST_SIGS_DEPTH];
-
-           stack[0].lid = pk->local_id;
-           stack[0].trust = 0;
-           rc = do_list_path( stack, 1, max_depth, lids, &tslist );
-           if( wipe ) { /* wipe out duplicates */
-               LOCAL_ID_INFO *work;
-
-               work = new_lid_table();
-               for( tsl=tslist; tsl; tsl = tsl->next ) {
-                   for(i=1; i < tsl->nseg-1; i++ ) {
-                       if( ins_lid_table_item( work, tsl->seg[i].lid, 0 ) ) {
-                           tsl->dup = 1; /* mark as duplicate */
-                           break;
-                       }
-                   }
-               }
-               release_lid_table(work);
-           }
-           release_lid_table(lids);
-       }
-       if( rc )
-           log_error("user '%s' list problem: %s\n", username, g10_errstr(rc));
-       rc = propagate_trust( tslist );
-       if( rc )
-           log_error("user '%s' trust problem: %s\n", username, g10_errstr(rc));
-       for(tsl = tslist; tsl; tsl = tsl->next ) {
-           int i;
-
-           if( tsl->dup )
-               continue;
-           printf("trust path:" );
-           for(i=0; i < tsl->nseg; i++ ) {
-               putc(' ',stdout);
-               print_keyid( stdout, tsl->seg[i].lid );
-               putc(':',stdout);
-               print_trust( stdout, tsl->seg[i].trust );
-           }
+    /* collect the paths */
+    tmppath = m_alloc_clear( (opt.max_cert_depth+1)* sizeof *tmppath );
+    trust_seg_list = NULL;
+    collect_paths( 0, opt.max_cert_depth, 1, &rec, tmppath, &trust_seg_list );
+    m_free( tmppath );
+    sort_tsl_list( &trust_seg_list );
+    /* and now print them */
+    for(tsl = trust_seg_list; tsl; tsl = tsl->next ) {
+       print_path( tsl->pathlen, tsl->path, stdout, 0 );
+       if( tsl->next )
            putchar('\n');
-       }
     }
 
-    free_public_key( pk );
+    /* release the list */
+    for(tsl = trust_seg_list; tsl; tsl = tsl2 ) {
+       tsl2 = tsl->next;
+       m_free( tsl );
+    }
+    trust_seg_list = NULL;
 }
 
 
 /****************
  * Check the complete trustdb or only the entries for the given username.
- * Special hack:  A username "REBUILD" inserts all records from the public
- * key rings into the trustdb.
+ * We check the complete database. If a username is given or the special
+ * username "*" is used, a complete recheck is done.  With no user ID
+ * only the records which are not yet checkd are now checked.
  */
 void
 check_trustdb( const char *username )
@@ -1257,18 +1394,20 @@ check_trustdb( const char *username )
     KBNODE keyblock = NULL;
     KBPOS kbpos;
     int rc;
-    int rebuild = username && !strcmp(username, "REBUILD");
+    int recheck = username && *username == '*' && !username[1];
 
-    if( username && !rebuild ) {
+    if( username && !recheck ) {
        rc = find_keyblock_byname( &kbpos, username );
        if( !rc )
            rc = read_keyblock( &kbpos, &keyblock );
        if( rc ) {
-           log_error("%s: keyblock read problem: %s\n",
+           log_error(_("%s: keyblock read problem: %s\n"),
                                    username, g10_errstr(rc));
        }
        else {
-           rc = update_trust_record( keyblock );
+           int modified;
+
+           rc = update_trust_record( keyblock, 1, &modified );
            if( rc == -1 ) { /* not yet in trustdb: insert */
                rc = insert_trust_record(
                            find_kbnode( keyblock, PKT_PUBLIC_KEY
@@ -1276,22 +1415,30 @@ check_trustdb( const char *username )
 
            }
            if( rc )
-               log_error("%s: update failed: %s\n",
+               log_error(_("%s: update failed: %s\n"),
                                           username, g10_errstr(rc) );
+           else if( modified )
+               log_info(_("%s: updated\n"), username );
            else
-               log_info("%s: updated\n", username );
+               log_info(_("%s: okay\n"), username );
 
        }
        release_kbnode( keyblock ); keyblock = NULL;
     }
     else {
        ulong recnum;
+       ulong count=0, upd_count=0, err_count=0, skip_count=0;
 
        for(recnum=0; !tdbio_read_record( recnum, &rec, 0); recnum++ ) {
            if( rec.rectype == RECTYPE_DIR ) {
                TRUSTREC tmp;
+               int modified;
+
                if( !rec.r.dir.keylist ) {
-                   log_info("lid %lu: dir record w/o key - skipped\n", recnum);
+                   log_info(_("lid %lu: dir record w/o key - skipped\n"),
+                                                                 recnum);
+                   count++;
+                   skip_count++;
                    continue;
                }
 
@@ -1301,57 +1448,113 @@ check_trustdb( const char *username )
                                            tmp.r.key.fingerprint,
                                            tmp.r.key.fingerprint_len );
                if( rc ) {
-                   log_error("lid %lu: keyblock not found: %s\n",
+                   log_error(_("lid %lu: keyblock not found: %s\n"),
                                                 recnum, g10_errstr(rc) );
+                   count++;
+                   skip_count++;
                    continue;
                }
-               rc = update_trust_record( keyblock );
-               if( rc )
-                   log_error("lid %lu: update failed: %s\n",
-                                                recnum, g10_errstr(rc) );
-               else
-                   log_info("lid %lu: updated\n", recnum );
 
-               release_kbnode( keyblock ); keyblock = NULL;
+               rc = update_trust_record( keyblock, recheck, &modified );
+               if( rc ) {
+                   log_error(_("lid %lu: update failed: %s\n"),
+                                                recnum, g10_errstr(rc) );
+                   err_count++;
+               }
+               else if( modified ) {
+                   if( opt.verbose )
+                       log_info(_("lid %lu: updated\n"), recnum );
+                   upd_count++;
+               }
+               else if( opt.verbose > 1 )
+                   log_info(_("lid %lu: okay\n"), recnum );
+
+               release_kbnode( keyblock ); keyblock = NULL;
+               if( !(++count % 100) )
+                   log_info(_("%lu keys so far processed\n"), count);
            }
        }
-       if( rebuild ) {
-           log_info("adding new entries.\n");
+       log_info(_("%lu keys processed\n"), count);
+       if( skip_count )
+           log_info(_("\t%lu keys skipped\n"), skip_count);
+       if( err_count )
+           log_info(_("\t%lu keys with errors\n"), err_count);
+       if( upd_count )
+           log_info(_("\t%lu keys updated\n"), upd_count);
+    }
+}
 
-           rc = enum_keyblocks( 0, &kbpos, &keyblock );
-           if( !rc ) {
-               while( !(rc = enum_keyblocks( 1, &kbpos, &keyblock )) ) {
-                   rc = update_trust_record( keyblock );
-                   if( rc == -1 ) { /* not yet in trustdb: insert */
-                       PKT_public_key *pk =
-                                   find_kbnode( keyblock, PKT_PUBLIC_KEY
-                                              ) ->pkt->pkt.public_key;
-                       rc = insert_trust_record( pk );
-                       if( rc && !pk->local_id )
-                           log_error("lid ?: insert failed: %s\n",
-                                                            g10_errstr(rc) );
-                       else if( rc )
-                           log_error("lid %lu: insert failed: %s\n",
-                                              pk->local_id, g10_errstr(rc) );
-                       else
-                            log_info("lid %lu: inserted\n", pk->local_id );
-                   }
-                   else if( rc )
-                       log_error("lid %lu: update failed: %s\n",
-                                lid_from_keyblock(keyblock), g10_errstr(rc) );
-                   else
-                       log_info("lid %lu: updated\n",
-                                       lid_from_keyblock(keyblock) );
+
+/****************
+ * Put new entries  from the pubrings into the trustdb.
+ * This function honors the sig flags to speed up the check.
+ */
+void
+update_trustdb( )
+{
+    KBNODE keyblock = NULL;
+    KBPOS kbpos;
+    int rc;
+
+    rc = enum_keyblocks( 0, &kbpos, &keyblock );
+    if( !rc ) {
+       ulong count=0, upd_count=0, err_count=0, new_count=0;
+
+       while( !(rc = enum_keyblocks( 1, &kbpos, &keyblock )) ) {
+           int modified;
+
+           rc = update_trust_record( keyblock, 1, &modified );
+           if( rc == -1 ) { /* not yet in trustdb: insert */
+               PKT_public_key *pk =
+                           find_kbnode( keyblock, PKT_PUBLIC_KEY
+                                      ) ->pkt->pkt.public_key;
+               rc = insert_trust_record( pk );
+               if( rc && !pk->local_id ) {
+                   log_error(_("lid ?: insert failed: %s\n"),
+                                                    g10_errstr(rc) );
+                   err_count++;
+               }
+               else if( rc ) {
+                   log_error(_("lid %lu: insert failed: %s\n"),
+                                      pk->local_id, g10_errstr(rc) );
+                   err_count++;
                }
+               else {
+                   if( opt.verbose )
+                       log_info(_("lid %lu: inserted\n"), pk->local_id );
+                   new_count++;
+               }
+           }
+           else if( rc ) {
+               log_error(_("lid %lu: update failed: %s\n"),
+                        lid_from_keyblock(keyblock), g10_errstr(rc) );
+               err_count++;
            }
-           if( rc && rc != -1 )
-               log_error("enum_keyblocks failed: %s\n", g10_errstr(rc));
+           else if( modified ) {
+               if( opt.verbose )
+                   log_info(_("lid %lu: updated\n"), lid_from_keyblock(keyblock));
+               upd_count++;
+           }
+           else if( opt.verbose > 1 )
+               log_info(_("lid %lu: okay\n"), lid_from_keyblock(keyblock) );
 
-           enum_keyblocks( 2, &kbpos, &keyblock ); /* close */
-           release_kbnode( keyblock );
+           release_kbnode( keyblock ); keyblock = NULL;
+           if( !(++count % 100) )
+               log_info(_("%lu keys so far processed\n"), count);
        }
-
+       log_info(_("%lu keys processed\n"), count);
+       if( err_count )
+           log_info(_("\t%lu keys with errors\n"), err_count);
+       if( upd_count )
+           log_info(_("\t%lu keys updated\n"), upd_count);
+       if( new_count )
+           log_info(_("\t%lu keys inserted\n"), new_count);
     }
+    if( rc && rc != -1 )
+       log_error(_("enumerate keyblocks failed: %s\n"), g10_errstr(rc));
+
+    enum_keyblocks( 2, &kbpos, &keyblock ); /* close */
+    release_kbnode( keyblock );
 }
 
 
@@ -1393,7 +1596,7 @@ check_trust( PKT_public_key *pk, unsigned *r_trustlevel )
     }
     else { /* no local_id: scan the trustdb */
        if( (rc=tdbio_search_dir_bypk( pk, &rec )) && rc != -1 ) {
-           log_error("check_trust: search dir record failed: %s\n",
+           log_error(_("check_trust: search dir record failed: %s\n"),
                                                            g10_errstr(rc));
            return rc;
        }
@@ -1418,12 +1621,10 @@ check_trust( PKT_public_key *pk, unsigned *r_trustlevel )
        return G10ERR_TIME_CONFLICT;
     }
 
-    if( pk->valid_days && add_days_to_timestamp(pk->timestamp,
-                                               pk->valid_days) < cur_time ) {
+    if( pk->expiredate && pk->expiredate <= cur_time ) {
        log_info(_("key %08lX.%lu: expired at %s\n"),
                        (ulong)keyid[1], pk->local_id,
-                   asctimestamp( add_days_to_timestamp(pk->timestamp,
-                                                       pk->valid_days)));
+                            asctimestamp( pk->expiredate) );
         trustlevel = TRUST_EXPIRED;
     }
     else {
@@ -1444,6 +1645,8 @@ check_trust( PKT_public_key *pk, unsigned *r_trustlevel )
 }
 
 
+
+
 int
 query_trust_info( PKT_public_key *pk )
 {
@@ -1454,16 +1657,9 @@ query_trust_info( PKT_public_key *pk )
        return '?';
     if( trustlevel & TRUST_FLAG_REVOKED )
        return 'r';
-    switch( (trustlevel & TRUST_MASK) ) {
-      case TRUST_UNKNOWN:   c = 'o'; break;
-      case TRUST_EXPIRED:   c = 'e'; break;
-      case TRUST_UNDEFINED: c = 'q'; break;
-      case TRUST_NEVER:     c = 'n'; break;
-      case TRUST_MARGINAL:  c = 'm'; break;
-      case TRUST_FULLY:     c = 'f'; break;
-      case TRUST_ULTIMATE:  c = 'u'; break;
-      default: BUG();
-    }
+    c = trust_letter( (trustlevel & TRUST_MASK) );
+    if( !c )
+       c = '?';
     return c;
 }
 
@@ -1472,7 +1668,10 @@ query_trust_info( PKT_public_key *pk )
 /****************
  * Enumerate all keys, which are needed to build all trust paths for
  * the given key.  This function does not return the key itself or
- * the ultimate key.
+ * the ultimate key (the last point in cerificate chain).  Only
+ * certificate chains which ends up at an ultimately trusted key
+ * are listed. If ownertrust or validity is not NULL, the corresponding
+ * value for the returned LID is also returned in these variable(s).
  *
  *  1) create a void pointer and initialize it to NULL
  *  2) pass this void pointer by reference to this function.
@@ -1481,56 +1680,113 @@ query_trust_info( PKT_public_key *pk )
  *     to indicate EOF. LID does contain the next key used to build the web
  *  4) Always call this function a last time with LID set to NULL,
  *     so that it can free its context.
+ *
+ * Returns: -1 on EOF or the level of the returned LID
  */
 int
-enum_trust_web( void **context, ulong *lid )
+enum_cert_paths( void **context, ulong *lid,
+                unsigned *ownertrust, unsigned *validity )
 {
-    ENUM_TRUST_WEB_CONTEXT *c = *context;
+    struct enum_cert_paths_ctx *ctx;
+    TRUST_SEG_LIST tsl;
 
-    if( !c ) { /* make a new context */
-       c = m_alloc_clear( sizeof *c );
-       *context = c;
-       if( *lid == last_trust_web_key && last_trust_web_tslist )
-           c->tsl = last_trust_web_tslist;
-       else {
-           TRUST_SEG_LIST tsl, tsl2, tslist;
-           int rc;
+    if( !lid ) {  /* release the context */
+       if( *context ) {
+           TRUST_SEG_LIST tsl2;
 
-           rc = make_tsl( *lid, &tslist );
-           if( rc ) {
-               log_error("failed to build the TSL\n");
-               return rc;
+           ctx = *context;
+           for(tsl = ctx->tsl_head; tsl; tsl = tsl2 ) {
+               tsl2 = tsl->next;
+               m_free( tsl );
            }
-           /* cache the tslist, so that we do not need to free it */
-           if( last_trust_web_key ) {
-               for( tsl = last_trust_web_tslist; tsl; tsl = tsl2 ) {
-                   tsl2 = tsl->next;
-                   m_free(tsl);
-               }
-           }
-           last_trust_web_key = *lid;
-           last_trust_web_tslist = tslist;
-           c->tsl = last_trust_web_tslist;
+           *context = NULL;
        }
-       c->index = 1;
+       return -1;
     }
 
-    if( !lid ) { /* free the context */
-       m_free( c );
-       *context = NULL;
-       return 0;
+    if( !*context ) {
+       TRUST_INFO *tmppath;
+       TRUSTREC rec;
+
+       if( !*lid )
+           return -1;
+
+       ctx = m_alloc_clear( sizeof *ctx );
+       *context = ctx;
+       /* collect the paths */
+       read_record( *lid, &rec, RECTYPE_DIR );
+       tmppath = m_alloc_clear( (opt.max_cert_depth+1)* sizeof *tmppath );
+       tsl = NULL;
+       collect_paths( 0, opt.max_cert_depth, 1, &rec, tmppath, &tsl );
+       m_free( tmppath );
+       sort_tsl_list( &tsl );
+       /* setup the context */
+       ctx->tsl_head = tsl;
+       ctx->tsl = ctx->tsl_head;
+       ctx->idx = 0;
     }
+    else
+       ctx = *context;
+
+    while( ctx->tsl && ctx->idx >= ctx->tsl->pathlen ) {
+       ctx->tsl = ctx->tsl->next;
+       ctx->idx = 0;
+    }
+    tsl = ctx->tsl;
+    if( !tsl )
+       return -1; /* eof */
+
+    if( ownertrust )
+       *ownertrust = tsl->path[ctx->idx].otrust;
+    if( validity )
+       *validity = tsl->path[ctx->idx].trust;
+    *lid = tsl->path[ctx->idx].lid;
+    ctx->idx++;
+    return ctx->idx-1;
+}
 
-    while( c->tsl ) {
-       if( !c->tsl->dup && c->index < c->tsl->nseg-1 ) {
-           *lid = c->tsl->seg[c->index].lid;
-           c->index++;
-           return 0;
+
+/****************
+ * Print the current path
+ */
+void
+enum_cert_paths_print( void **context, FILE *fp,
+                                      int refresh, ulong selected_lid )
+{
+    struct enum_cert_paths_ctx *ctx;
+    TRUST_SEG_LIST tsl;
+
+    if( !*context )
+       return;
+    ctx = *context;
+    if( !ctx->tsl )
+       return;
+    tsl = ctx->tsl;
+
+    if( !fp )
+       fp = stderr;
+
+    if( refresh ) { /* update the ownertrust and if possible the validity */
+       int i;
+       int match = tdbio_db_matches_options();
+
+       for( i = 0; i < tsl->pathlen; i++ )  {
+           TRUSTREC rec;
+
+           read_record( tsl->path[i].lid, &rec, RECTYPE_DIR );
+           tsl->path[i].otrust = rec.r.dir.ownertrust;
+           /* update validity only if we have it in the cache
+            * calculation is too time consuming */
+           if( match && (rec.r.dir.dirflags & DIRF_VALVALID)
+                     && rec.r.dir.validity ) {
+               tsl->path[i].trust = rec.r.dir.validity;
+               if( rec.r.dir.dirflags & DIRF_REVOKED )
+                   tsl->path[i].trust = TRUST_FLAG_REVOKED;
+           }
        }
-       c->index = 1;
-       c->tsl = c->tsl->next;
     }
-    return -1; /* eof */
+
+    print_path( tsl->pathlen, tsl->path, fp, selected_lid );
 }
 
 
@@ -1553,17 +1809,21 @@ get_ownertrust_info( ulong lid )
     int c;
 
     otrust = get_ownertrust( lid );
-    switch( (otrust & TRUST_MASK) ) {
-      case TRUST_NEVER:     c = 'n'; break;
-      case TRUST_MARGINAL:  c = 'm'; break;
-      case TRUST_FULLY:     c = 'f'; break;
-      case TRUST_ULTIMATE:  c = 'u'; break;
-      default:             c = '-'; break;
-    }
+    c = trust_letter( (otrust & TRUST_MASK) );
+    if( !c )
+       c = '?';
     return c;
 }
 
-
+/*
+ * Return an allocated buffer with the preference values for
+ * the key with LID and the userid which is identified by the
+ * HAMEHASH or the firstone if namehash is NULL.  ret_n receives
+ * the length of the allcoated buffer. Structure of the buffer is
+ * a repeated sequences of 2 bytes; where the first byte describes the
+ * type of the preference and the second one the value.  The constants
+ * PREFTYPE_xxxx should be used to reference a type.
+ */
 byte *
 get_pref_data( ulong lid, const byte *namehash, size_t *ret_n )
 {
@@ -1579,7 +1839,7 @@ get_pref_data( ulong lid, const byte *namehash, size_t *ret_n )
            /* found the correct one or the first one */
            read_record( rec.r.uid.prefrec, &rec, RECTYPE_PREF );
            if( rec.r.pref.next )
-               log_info("warning: can't yet handle long pref records\n");
+               log_info(_("WARNING: can't yet handle long pref records\n"));
            buf = m_alloc( ITEMS_PER_PREF_RECORD );
            memcpy( buf, rec.r.pref.data, ITEMS_PER_PREF_RECORD );
            *ret_n = ITEMS_PER_PREF_RECORD;
@@ -1608,7 +1868,7 @@ is_algo_in_prefs( ulong lid, int preftype, int algo )
        if( rec.r.uid.prefrec ) {
            read_record( rec.r.uid.prefrec, &rec, RECTYPE_PREF );
            if( rec.r.pref.next )
-               log_info("warning: can't yet handle long pref records\n");
+               log_info(_("WARNING: can't yet handle long pref records\n"));
            pref = rec.r.pref.data;
            for(i=0; i+1 < ITEMS_PER_PREF_RECORD; i+=2 ) {
                if( pref[i] == preftype && pref[i+1] == algo )
@@ -1630,7 +1890,7 @@ get_dir_record( PKT_public_key *pk, TRUSTREC *rec )
     }
     else { /* no local_id: scan the trustdb */
        if( (rc=tdbio_search_dir_bypk( pk, rec )) && rc != -1 )
-           log_error("get_dir_record: search_record failed: %s\n",
+           log_error(_("get_dir_record: search_record failed: %s\n"),
                                                            g10_errstr(rc));
     }
     return rc;
@@ -1640,7 +1900,7 @@ get_dir_record( PKT_public_key *pk, TRUSTREC *rec )
 
 /****************
  * This function simply looks for the key in the trustdb
- * and makes sure that pk->local_id is set to the coreect value.
+ * and makes sure that pk->local_id is set to the correct value.
  * Return: 0 = found
  *        -1 = not found
  *       other = error
@@ -1663,12 +1923,16 @@ clear_trust_checked_flag( PKT_public_key *pk )
     if( rc )
        return rc;
 
-    if( !(rec.r.dir.dirflags & DIRF_CHECKED) )
+    /* check whether they are already reset */
+    if(   !(rec.r.dir.dirflags & DIRF_CHECKED)
+       && !(rec.r.dir.dirflags & DIRF_VALVALID) )
        return 0;
 
     /* reset the flag */
     rec.r.dir.dirflags &= ~DIRF_CHECKED;
+    rec.r.dir.dirflags &= ~DIRF_VALVALID;
     write_record( &rec );
+    do_sync();
     return 0;
 }
 
@@ -1680,21 +1944,42 @@ check_hint_sig( ulong lid, KBNODE keyblock, u32 *keyid, byte *uidrec_hash,
                TRUSTREC *sigrec, int sigidx, ulong hint_owner )
 {
     KBNODE node;
-    int rc, state=0;
+    int rc, state;
     byte uhash[20];
     int is_selfsig;
     PKT_signature *sigpkt = NULL;
+    TRUSTREC tmp;
+    u32 sigkid[2];
+    int revoke = 0;
 
     if( sigrec->r.sig.sig[sigidx].flag & SIGF_CHECKED )
-       log_info(_("note: sig rec %lu[%d] in hintlist "
+       log_info(_("NOTE: sig rec %lu[%d] in hintlist "
                   "of %lu but marked as checked\n"),
                    sigrec->recnum, sigidx, hint_owner );
     if( !(sigrec->r.sig.sig[sigidx].flag & SIGF_NOPUBKEY) )
-       log_info(_("note: sig rec %lu[%d] in hintlist "
+       log_info(_("NOTE: sig rec %lu[%d] in hintlist "
                   "of %lu but not marked\n"),
                    sigrec->recnum, sigidx, hint_owner );
 
+    read_record( sigrec->r.sig.sig[sigidx].lid, &tmp, 0 );
+    if( tmp.rectype != RECTYPE_DIR ) {
+       /* we need the dir record */
+       log_error(_("sig rec %lu[%d] in hintlist "
+                   "of %lu does not point to a dir record\n"),
+                   sigrec->recnum, sigidx, hint_owner );
+       return;
+    }
+    if( !tmp.r.dir.keylist ) {
+       log_error(_("lid %lu: no primary key\n"), tmp.r.dir.lid );
+       return;
+    }
+    read_record(tmp.r.dir.keylist, &tmp, RECTYPE_KEY );
+    keyid_from_fingerprint( tmp.r.key.fingerprint,
+                           tmp.r.key.fingerprint_len, sigkid );
+
+
     /* find the correct signature packet */
+    state = 0;
     for( node=keyblock; node; node = node->next ) {
        if( node->pkt->pkttype == PKT_USER_ID ) {
            PKT_user_id *uidpkt = node->pkt->pkt.user_id;
@@ -1707,15 +1992,22 @@ check_hint_sig( ulong lid, KBNODE keyblock, u32 *keyid, byte *uidrec_hash,
        }
        else if( state && node->pkt->pkttype == PKT_SIGNATURE ) {
            sigpkt = node->pkt->pkt.signature;
-           if( sigpkt->keyid[0] == keyid[0]
-               && sigpkt->keyid[1] == keyid[1]
-               && (sigpkt->sig_class&~3) == 0x10 )
+           if( sigpkt->keyid[0] == sigkid[0]
+               && sigpkt->keyid[1] == sigkid[1]
+               && ( (sigpkt->sig_class&~3) == 0x10
+                    || ( revoke = (sigpkt->sig_class == 0x30)) ) ) {
+               state = 2;
                break; /* found */
+           }
        }
     }
 
     if( !node ) {
-       log_error(_("lid %lu: user id not found in keyblock\n"), lid );
+       log_info(_("lid %lu: user id not found in keyblock\n"), lid );
+       return ;
+    }
+    if( state != 2 ) {
+       log_info(_("lid %lu: user id without signature\n"), lid );
        return ;
     }
 
@@ -1723,23 +2015,33 @@ check_hint_sig( ulong lid, KBNODE keyblock, u32 *keyid, byte *uidrec_hash,
     rc = check_key_signature( keyblock, node, &is_selfsig );
     if( is_selfsig ) {
        log_error(_("lid %lu: self-signature in hintlist\n"), lid );
-       return ;
+       return;
     }
+
+    /* FiXME: handling fo SIGF_REVOKED is not correct! */
+
     if( !rc ) { /* valid signature */
+       if( opt.verbose )
+           log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
+                   (ulong)keyid[1], lid, uhash[18], uhash[19],
+                   (ulong)sigpkt->keyid[1],
+                   revoke? _("Valid certificate revocation")
+                         : _("Good certificate") );
        sigrec->r.sig.sig[sigidx].flag = SIGF_CHECKED | SIGF_VALID;
+       if( revoke )
+           sigrec->r.sig.sig[sigidx].flag |= SIGF_REVOKED;
     }
     else if( rc == G10ERR_NO_PUBKEY ) {
-       log_info(_("key %08lX.%lu, uid %02X%02X: "
-          "very strange: no public key for signature %08lX\n"),
+       log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
              (ulong)keyid[1], lid, uhash[18], uhash[19],
-              (ulong)sigpkt->keyid[1] );
+              (ulong)sigpkt->keyid[1],
+                _("very strange: no public key\n") );
        sigrec->r.sig.sig[sigidx].flag = SIGF_NOPUBKEY;
     }
     else {
-       log_info(_("key %08lX.%lu, uid %02X%02X: "
-                  "invalid signature: %s\n"),
-                   (ulong)keyid[1], lid,
-                   uhash[18], uhash[19], g10_errstr(rc));
+       log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
+                   (ulong)keyid[1], lid, uhash[18], uhash[19],
+                   (ulong)sigpkt->keyid[1], g10_errstr(rc) );
        sigrec->r.sig.sig[sigidx].flag = SIGF_CHECKED;
     }
     sigrec->dirty = 1;
@@ -1840,14 +2142,103 @@ process_hintlist( ulong hintlist, ulong hint_owner )
 }
 
 
+/****************
+ * Create or update shadow dir record and return the LID of the record
+ */
+static ulong
+create_shadow_dir( PKT_signature *sig, ulong lid  )
+{
+    TRUSTREC sdir, hlst, tmphlst;
+    ulong recno, newlid;
+    int tmpidx=0; /* avoids gcc warnign - this is controlled by tmphlst */
+    int rc;
+
+    /* first see whether we already have such a record */
+    rc = tdbio_search_sdir( sig->keyid, sig->pubkey_algo, &sdir );
+    if( rc && rc != -1 ) {
+       log_error(_("tdbio_search_dir failed: %s\n"), g10_errstr(rc));
+       die_invalid_db();
+    }
+    if( rc == -1 ) { /* not found: create */
+       memset( &sdir, 0, sizeof sdir );
+       sdir.recnum = tdbio_new_recnum();
+       sdir.rectype= RECTYPE_SDIR;
+       sdir.r.sdir.lid = sdir.recnum;
+       sdir.r.sdir.keyid[0] = sig->keyid[0];
+       sdir.r.sdir.keyid[1] = sig->keyid[1];
+       sdir.r.sdir.pubkey_algo = sig->pubkey_algo;
+       sdir.r.sdir.hintlist = 0;
+       write_record( &sdir );
+    }
+    newlid = sdir.recnum;
+    /* Put the record number into the hintlist.
+     * (It is easier to use the lid and not the record number of the
+     * key to save some space (assuming that a signator has
+     * signed more than one user id - and it is easier to implement.)
+     */
+    tmphlst.recnum = 0;
+    for( recno=sdir.r.sdir.hintlist; recno; recno = hlst.r.hlst.next) {
+       int i;
+       read_record( recno, &hlst, RECTYPE_HLST );
+       for( i=0; i < ITEMS_PER_HLST_RECORD; i++ ) {
+           if( !hlst.r.hlst.rnum[i] ) {
+               if( !tmphlst.recnum ) {
+                   tmphlst = hlst;
+                   tmpidx = i;
+               }
+           }
+           else if( hlst.r.hlst.rnum[i] == lid )
+               return newlid; /* the signature is already in the hintlist */
+       }
+    }
+    /* not yet in the hint list, write it */
+    if( tmphlst.recnum ) { /* we have an empty slot */
+       tmphlst.r.hlst.rnum[tmpidx] = lid;
+       write_record( &tmphlst );
+    }
+    else { /* must append a new hlst record */
+       memset( &hlst, 0, sizeof hlst );
+       hlst.recnum = tdbio_new_recnum();
+       hlst.rectype = RECTYPE_HLST;
+       hlst.r.hlst.next = sdir.r.sdir.hintlist;
+       hlst.r.hlst.rnum[0] = lid;
+       write_record( &hlst );
+       sdir.r.sdir.hintlist = hlst.recnum;
+       write_record( &sdir );
+    }
+
+    return newlid;
+}
+
 
+/****************
+ * This function checks the given public key and inserts or updates
+ * the keyrecord from the trustdb.  Revocation certificates
+ * are handled here and the keybinding of subkeys is checked.
+ * Hmmm: Should we check here, that the key has at least one valid
+ * user ID or do we allow keys w/o user ID?
+ *
+ * keyblock points to the first node in the keyblock,
+ * keynode is the node with the public key to check
+ * (either primary or secondary), keyid is the keyid of
+ * the primary key, drec is the directory record and recno_list
+ * is a list used to keep track of visited records.
+ * Existing keyflags are recalculated if recheck is true.
+ */
 static void
-upd_key_record( PKT_public_key *pk, TRUSTREC *drec, RECNO_LIST *recno_list )
+upd_key_record( KBNODE keyblock, KBNODE keynode, u32 *keyid,
+               TRUSTREC *drec, RECNO_LIST *recno_list, int recheck )
 {
     TRUSTREC krec;
+    KBNODE  node;
+    PKT_public_key *pk = keynode->pkt->pkt.public_key;
+    ulong lid = drec->recnum;
     byte fpr[MAX_FINGERPRINT_LEN];
     size_t fprlen;
     ulong recno, newrecno;
+    int keybind_seen = 0;
+    int revoke_seen = 0;
+    int rc;
 
     fingerprint_from_pk( pk, fpr, &fprlen );
     /* do we already have this key? */
@@ -1859,12 +2250,12 @@ upd_key_record( PKT_public_key *pk, TRUSTREC *drec, RECNO_LIST *recno_list )
     }
     if( recno ) { /* yes */
        ins_recno_list( recno_list, recno, RECTYPE_KEY );
-       /* here we would compare/update the keyflags */
     }
     else { /* no: insert this new key */
+       recheck = 1;
        memset( &krec, 0, sizeof(krec) );
        krec.rectype = RECTYPE_KEY;
-       krec.r.key.lid = drec->recnum;
+       krec.r.key.lid = lid;
        krec.r.key.pubkey_algo = pk->pubkey_algo;
        krec.r.key.fingerprint_len = fprlen;
        memcpy(krec.r.key.fingerprint, fpr, fprlen );
@@ -1877,34 +2268,143 @@ upd_key_record( PKT_public_key *pk, TRUSTREC *drec, RECNO_LIST *recno_list )
            drec->r.dir.keylist = newrecno;
            drec->dirty = 1;
        }
-       else { /* we already have a key, append it to the list */
+       else { /* we already have a key, append the new one */
+           TRUSTREC save = krec;
            for( ; recno; recno = krec.r.key.next )
                read_record( recno, &krec, RECTYPE_KEY );
            krec.r.key.next = newrecno;
            write_record( &krec );
+           krec = save;
+       }
+    }
+
+    if( !recheck && (krec.r.key.keyflags & KEYF_CHECKED) )
+       return;
+
+    /* check keybindings and revocations */
+    krec.r.key.keyflags = 0;
+    if( keynode->pkt->pkttype == PKT_PUBLIC_KEY ) {
+       /* we assume that a primary key is always valid
+        * and check later whether we have a revocation */
+       krec.r.key.keyflags |= KEYF_CHECKED | KEYF_VALID;
+    }
+
+    for( node=keynode->next; node; node = node->next ) {
+       PKT_signature *sig;
+
+       if( node->pkt->pkttype == PKT_PUBLIC_SUBKEY )
+           break; /* ready */
+       else if( node->pkt->pkttype != PKT_SIGNATURE )
+           continue;
+
+       sig = node->pkt->pkt.signature;
+
+       if( keyid[0] != sig->keyid[0] || keyid[1] != sig->keyid[1] )
+           continue; /* not a self signature */
+       if( sig->sig_class == 0x18 && !keybind_seen ) { /* a keybinding */
+           if( keynode->pkt->pkttype == PKT_PUBLIC_KEY )
+               continue; /* oops, not for a main key */
+           /* we check until we find a valid keybinding */
+           rc = check_key_signature( keyblock, node, NULL );
+           if( !rc ) {
+               if( opt.verbose )
+                   log_info(_(
+                       "key %08lX.%lu: Good subkey binding\n"),
+                        (ulong)keyid_from_pk(pk,NULL), lid );
+               krec.r.key.keyflags |= KEYF_CHECKED | KEYF_VALID;
+           }
+           else {
+               log_info(_(
+                 "key %08lX.%lu: Invalid subkey binding: %s\n"),
+                   (ulong)keyid_from_pk(pk,NULL), lid, g10_errstr(rc) );
+               krec.r.key.keyflags |= KEYF_CHECKED;
+               krec.r.key.keyflags &= ~KEYF_VALID;
+           }
+           keybind_seen = 1;
+       }
+       else if( sig->sig_class == 0x20 && !revoke_seen ) {
+           if( keynode->pkt->pkttype == PKT_PUBLIC_SUBKEY )
+               continue; /* a subkey is not expected here */
+           /* This is a key revocation certificate: check it */
+           rc = check_key_signature( keyblock, node, NULL );
+           if( !rc ) {
+               if( opt.verbose )
+                   log_info(_(
+                       "key %08lX.%lu: Valid key revocation\n"),
+                        (ulong)keyid_from_pk(pk,NULL), lid );
+               krec.r.key.keyflags |= KEYF_REVOKED;
+           }
+           else {
+               log_info(_(
+                 "key %08lX.%lu: Invalid key revocation: %s\n"),
+                 (ulong)keyid_from_pk(pk,NULL), lid, g10_errstr(rc) );
+           }
+           revoke_seen = 1;
+       }
+       else if( sig->sig_class == 0x28 && !revoke_seen ) {
+           if( keynode->pkt->pkttype == PKT_PUBLIC_KEY )
+               continue; /* a mainkey is not expected here */
+           /* This is a subkey revocation certificate: check it */
+           /* fixme: we should also check the revocation
+            * is newer than the key (OpenPGP) */
+           rc = check_key_signature( keyblock, node, NULL );
+           if( !rc ) {
+               if( opt.verbose )
+                   log_info(_(
+                       "key %08lX.%lu: Valid subkey revocation\n"),
+                        (ulong)keyid_from_pk(pk,NULL), lid );
+               krec.r.key.keyflags |= KEYF_REVOKED;
+           }
+           else {
+               log_info(_(
+                 "key %08lX.%lu: Invalid subkey binding: %s\n"),
+                 (ulong)keyid_from_pk(pk,NULL), lid, g10_errstr(rc) );
+           }
+           revoke_seen = 1;
        }
     }
+
+    write_record( &krec );
 }
 
 
+/****************
+ * This function checks the given user ID and inserts or updates
+ * the uid record of the trustdb.  Revocation certificates
+ * are handled here.
+ *
+ * keyblock points to the first node in the keyblock,
+ * uidnode is the node with the user id to check
+ * keyid is the keyid of
+ * the primary key, drec is the directory record and recno_list
+ * is a list used to keep track of visited records.
+ * Existing uidflags are recalculated if recheck is true.
+ */
 static void
-upd_uid_record( PKT_user_id *uid, TRUSTREC *drec, RECNO_LIST *recno_list,
-               u32 *keyid, ulong *uidrecno, byte *uidhash )
+upd_uid_record( KBNODE keyblock, KBNODE uidnode, u32 *keyid,
+               TRUSTREC *drec, RECNO_LIST *recno_list, int recheck )
 {
+    ulong lid = drec->recnum;
+    PKT_user_id *uid = uidnode->pkt->pkt.user_id;
     TRUSTREC urec;
+    PKT_signature *selfsig = NULL;
+    byte uidhash[20];
+    KBNODE node;
     ulong recno, newrecno;
+    int rc;
 
+    /* see whether we already have an uid record */
     rmd160_hash_buffer( uidhash, uid->name, uid->len );
     for( recno=drec->r.dir.uidlist; recno; recno = urec.r.uid.next ) {
        read_record( recno, &urec, RECTYPE_UID );
        if( !memcmp( uidhash, urec.r.uid.namehash, 20 ) )
            break;
     }
-    if( recno ) {
+    if( recno ) { /* we already have this record */
        ins_recno_list( recno_list, recno, RECTYPE_UID );
-       *uidrecno = recno;
     }
     else { /* new user id */
+       recheck = 1;
        memset( &urec, 0 , sizeof(urec) );
        urec.rectype = RECTYPE_UID;
        urec.r.uid.lid = drec->recnum;
@@ -1918,348 +2418,465 @@ upd_uid_record( PKT_user_id *uid, TRUSTREC *drec, RECNO_LIST *recno_list,
            drec->dirty = 1;
        }
        else { /* we already have an uid, append it to the list */
+           TRUSTREC save = urec;
            for( ; recno; recno = urec.r.key.next )
                read_record( recno, &urec, RECTYPE_UID );
            urec.r.uid.next = newrecno;
            write_record( &urec );
+           urec = save;
        }
-       *uidrecno = newrecno;
     }
+
+    if( recheck || !(urec.r.uid.uidflags & UIDF_CHECKED) ) {
+       /* check self signatures */
+       urec.r.uid.uidflags = 0;
+       for( node=uidnode->next; node; node = node->next ) {
+           PKT_signature *sig;
+
+           if( node->pkt->pkttype == PKT_USER_ID )
+               break; /* ready */
+           if( node->pkt->pkttype == PKT_PUBLIC_SUBKEY )
+               break; /* ready */
+           if( node->pkt->pkttype != PKT_SIGNATURE )
+               continue;
+
+           sig = node->pkt->pkt.signature;
+
+           if( keyid[0] != sig->keyid[0] || keyid[1] != sig->keyid[1] )
+               continue; /* not a self signature */
+
+           if( (sig->sig_class&~3) == 0x10 ) { /* regular self signature */
+               rc = check_key_signature( keyblock, node, NULL );
+               if( !rc ) {
+                   if( opt.verbose )
+                       log_info( "uid %08lX.%lu/%02X%02X: %s\n",
+                          (ulong)keyid[1], lid, uidhash[18], uidhash[19],
+                                 _("Good self-signature") );
+                   urec.r.uid.uidflags |= UIDF_CHECKED | UIDF_VALID;
+                   if( !selfsig )
+                       selfsig = sig; /* use the first valid sig */
+                   else if( sig->timestamp > selfsig->timestamp
+                            && sig->sig_class >= selfsig->sig_class )
+                       selfsig = sig; /* but this one is newer */
+               }
+               else {
+                   log_info( "uid %08lX/%02X%02X: %s: %s\n",
+                              (ulong)keyid[1], uidhash[18], uidhash[19],
+                             _("Invalid self-signature"),
+                              g10_errstr(rc) );
+                   urec.r.uid.uidflags |= UIDF_CHECKED;
+               }
+           }
+           else if( sig->sig_class == 0x30 ) { /* cert revocation */
+               rc = check_key_signature( keyblock, node, NULL );
+               if( !rc && selfsig && selfsig->timestamp > sig->timestamp ) {
+                   log_info( "uid %08lX.%lu/%02X%02X: %s\n",
+                          (ulong)keyid[1], lid, uidhash[18], uidhash[19],
+                          _("Valid user ID revocation skipped "
+                            "due to a newer self signature\n") );
+               }
+               else if( !rc ) {
+                   if( opt.verbose )
+                       log_info( "uid %08lX.%lu/%02X%02X: %s\n",
+                          (ulong)keyid[1], lid, uidhash[18], uidhash[19],
+                                _("Valid user ID revocation\n") );
+                   urec.r.uid.uidflags |= UIDF_CHECKED | UIDF_VALID;
+                   urec.r.uid.uidflags |= UIDF_REVOKED;
+               }
+               else {
+                   log_info("uid %08lX/%02X%02X: %s: %s\n",
+                               (ulong)keyid[1], uidhash[18], uidhash[19],
+                              _("Invalid user ID revocation"),
+                                                       g10_errstr(rc) );
+               }
+           }
+       }
+       write_record( &urec );
+    } /* end check self-signatures */
+
+
+    if( (urec.r.uid.uidflags & (UIDF_CHECKED|UIDF_VALID))
+       != (UIDF_CHECKED|UIDF_VALID) )
+       return; /* user ID is not valid, so no need to check more things */
+
+    /* check the preferences */
+    if( selfsig )
+       upd_pref_record( &urec, keyid, selfsig );
+
+    /* check non-self signatures */
+    for( node=uidnode->next; node; node = node->next ) {
+       PKT_signature *sig;
+
+       if( node->pkt->pkttype == PKT_USER_ID )
+           break; /* ready */
+       if( node->pkt->pkttype == PKT_PUBLIC_SUBKEY )
+           break; /* ready */
+       if( node->pkt->pkttype != PKT_SIGNATURE )
+           continue;
+
+       sig = node->pkt->pkt.signature;
+
+       if( keyid[0] == sig->keyid[0] || keyid[1] == sig->keyid[1] )
+           continue; /* skip self signature */
+
+       if( (sig->sig_class&~3) == 0x10 ) { /* regular certification */
+           upd_cert_record( keyblock, node, keyid, drec, recno_list,
+                            recheck, &urec, uidhash, 0 );
+       }
+       else if( sig->sig_class == 0x30 ) { /* cert revocation */
+           upd_cert_record( keyblock, node, keyid, drec, recno_list,
+                            recheck, &urec, uidhash, 1 );
+       }
+    } /* end check certificates */
+
+    write_record( &urec );
 }
 
 
+
+/****************
+ *
+ *
+ */
 static void
-upd_pref_record( PKT_signature *sig, TRUSTREC *drec,
-                u32 *keyid, ulong *uidrecno, byte *uidhash )
+upd_pref_record( TRUSTREC *urec, u32 *keyid, PKT_signature *sig )
 {
     static struct {
        sigsubpkttype_t subpkttype;
        int preftype;
-    } prefs[] = {
+    } ptable[] = {
        { SIGSUBPKT_PREF_SYM,   PREFTYPE_SYM    },
        { SIGSUBPKT_PREF_HASH,  PREFTYPE_HASH   },
        { SIGSUBPKT_PREF_COMPR, PREFTYPE_COMPR  },
        { 0, 0 }
     };
-    TRUSTREC urec, prec;
+    TRUSTREC prec;
+    ulong lid = urec->r.uid.lid ;
+    const byte *uidhash = urec->r.uid.namehash;
     const byte *s;
     size_t n;
     int k, i;
-    ulong recno_tbl[10];
-    int recno_idx = 0;
     ulong recno;
+    byte prefs_sig[200];
+    int n_prefs_sig = 0;
+    byte prefs_rec[200];
+    int n_prefs_rec = 0;
+
+    /* check for changed preferences */
+    for(k=0; ptable[k].subpkttype; k++ ) {
+       s = parse_sig_subpkt2( sig, ptable[k].subpkttype, &n );
+       if( s ) {
+           for( ; n; n--, s++ ) {
+               if( n_prefs_sig >= DIM(prefs_sig)-1 ) {
+                   log_info("uid %08lX.%lu/%02X%02X: %s\n",
+                             (ulong)keyid[1], lid, uidhash[18], uidhash[19],
+                             _("Too many preferences") );
+                   break;
+               }
+               prefs_sig[n_prefs_sig++] = ptable[k].preftype;
+               prefs_sig[n_prefs_sig++] = *s;
+           }
+       }
+    }
+    for( recno=urec->r.uid.prefrec; recno; recno = prec.r.pref.next ) {
+       read_record( recno, &prec, RECTYPE_PREF );
+       for(i = 0; i < ITEMS_PER_PREF_RECORD; i +=2 )  {
+           if( n_prefs_rec >= DIM(prefs_rec)-1 ) {
+               log_info("uid %08lX.%lu/%02X%02X: %s\n",
+                         (ulong)keyid[1], lid, uidhash[18], uidhash[19],
+                         _("Too many preference items") );
+               break;
+           }
+           if( prec.r.pref.data[i] ) {
+               prefs_rec[n_prefs_rec++] = prec.r.pref.data[i];
+               prefs_rec[n_prefs_rec++] = prec.r.pref.data[i+1];
+           }
+       }
+    }
+    if( n_prefs_sig == n_prefs_rec
+       && !memcmp( prefs_sig, prefs_rec, n_prefs_sig ) )
+       return;  /* not changed */
 
-    /* we need the uid record */
-    read_record( *uidrecno, &urec, RECTYPE_UID );
-
-    /* First delete all pref records
+    /* Preferences have changed:  Delete all pref records
      * This is much simpler than checking whether we have to
-     * do update the record at all - the record cache may care about it */
-    for( recno=urec.r.uid.prefrec; recno; recno = prec.r.pref.next ) {
+     * do update the record at all - the record cache may care about it
+     */
+    for( recno=urec->r.uid.prefrec; recno; recno = prec.r.pref.next ) {
        read_record( recno, &prec, RECTYPE_PREF );
        delete_record( recno );
     }
 
-    /* and write the new ones */
-    i = 0;
-    for(k=0; prefs[k].subpkttype; k++ ) {
-       s = parse_sig_subpkt2( sig, prefs[k].subpkttype, &n );
-       if( s ) {
-           while( n ) {
-               if( !i || i >= ITEMS_PER_PREF_RECORD ) {
-                   if( recno_idx >= DIM(recno_tbl)-1 ) {
-                       log_info("too many preferences\n");
-                       break;
-                   }
-                   if( i ) {
-                       recno_tbl[recno_idx]=tdbio_new_recnum();
-                       prec.recnum = recno_tbl[recno_idx++];
-                       write_record( &prec );
-                   }
-                   memset( &prec, 0, sizeof prec );
-                   prec.rectype = RECTYPE_PREF;
-                   prec.r.pref.lid = drec->recnum;
-                   i = 0;
-               }
-               prec.r.pref.data[i++] = prefs[k].preftype;
-               prec.r.pref.data[i++] = *s++;
-               n--;
+    if( n_prefs_sig > ITEMS_PER_PREF_RECORD )
+        log_info(_("WARNING: can't yet handle long pref records\n"));
+
+    memset( &prec, 0, sizeof prec );
+    prec.recnum = tdbio_new_recnum();
+    prec.rectype = RECTYPE_PREF;
+    prec.r.pref.lid = lid;
+    if( n_prefs_sig <= ITEMS_PER_PREF_RECORD )
+       memcpy( prec.r.pref.data, prefs_sig, n_prefs_sig );
+    else { /* need more than one pref record */
+       TRUSTREC tmp;
+       ulong nextrn;
+       int n = n_prefs_sig;
+       byte *pp = prefs_sig;
+
+       memcpy( prec.r.pref.data, pp, ITEMS_PER_PREF_RECORD );
+       n -= ITEMS_PER_PREF_RECORD;
+       pp += ITEMS_PER_PREF_RECORD;
+       nextrn = prec.r.pref.next = tdbio_new_recnum();
+       do {
+           memset( &tmp, 0, sizeof tmp );
+           tmp.recnum = nextrn;
+           tmp.rectype = RECTYPE_PREF;
+           tmp.r.pref.lid = lid;
+           if( n <= ITEMS_PER_PREF_RECORD ) {
+               memcpy( tmp.r.pref.data, pp, n );
+               n = 0;
            }
-       }
-    }
-    if( i ) { /* write the last one */
-       recno_tbl[recno_idx]=tdbio_new_recnum();
-       prec.recnum = recno_tbl[recno_idx++];
-       write_record( &prec );
-    }
-    /* now link them together */
-    for(i=0; i < recno_idx-1; i++ ) {
-       read_record( recno_tbl[i], &prec, RECTYPE_PREF );
-       prec.r.pref.next = recno_tbl[i+1];
-       write_record( &prec );
+           else {
+               memcpy( tmp.r.pref.data, pp, ITEMS_PER_PREF_RECORD );
+               n -= ITEMS_PER_PREF_RECORD;
+               pp += ITEMS_PER_PREF_RECORD;
+               nextrn = tmp.r.pref.next = tdbio_new_recnum();
+           }
+           write_record( &tmp );
+       } while( n );
     }
-    /* don't need to write the last one, but update the uid */
-    urec.r.uid.prefrec = recno_idx? recno_tbl[0] : 0;
-    write_record( &urec );
+    write_record( &prec );
+    urec->r.uid.prefrec = prec.recnum;
+    urec->dirty = 1;
 }
 
 
 
-/****************
- * Note: A signature made with a secondayr key is not considered a
- *      self-signature.
- */
 static void
-upd_sig_record( PKT_signature *sig, TRUSTREC *drec,
-               u32 *keyid, ulong *uidrecno, byte *uidhash,
-               KBNODE keyblock, KBNODE signode )
+upd_cert_record( KBNODE keyblock, KBNODE signode, u32 *keyid,
+                TRUSTREC *drec, RECNO_LIST *recno_list, int recheck,
+                TRUSTREC *urec, const byte *uidhash, int revoke )
 {
-    int rc;
+    /* We simply insert the signature into the sig records but
+     * avoid duplicate ones.  We do not check them here because
+     * there is a big chance, that we import required public keys
+     * later.  The problem with this is that we must somewhere store
+     * the information about this signature (we need a record id).
+     * We do this by using the record type shadow dir, which will
+     * be converted to a dir record as soon as a new public key is
+     * inserted into the trustdb.
+     */
     ulong lid = drec->recnum;
+    PKT_signature *sig = signode->pkt->pkt.signature;
+    TRUSTREC rec;
+    ulong recno;
+    TRUSTREC delrec;
+    int delrecidx=0;
+    int newflag = 0;
+    ulong newlid = 0;
+    PKT_public_key *pk = m_alloc_clear( sizeof *pk );
+    ulong pk_lid = 0;
+    int found_sig = 0;
+    int found_delrec = 0;
+    int rc;
 
-    if( keyid[0] == sig->keyid[0] && keyid[1] == sig->keyid[1] ) {
-       if( (sig->sig_class&~3) == 0x10 ) {
-           /* must verify this selfsignature here, so that we can
-            * build the preference record and validate the uid record
-            */
-           if( !*uidrecno ) {
-               log_error("key %08lX: self-signature without user id\n",
-                         (ulong)keyid[1] );
-           }
-           else if( (rc = check_key_signature( keyblock, signode, NULL ))) {
-               log_error("key %08lX, uid %02X%02X: "
-                         "invalid self-signature: %s\n", (ulong)keyid[1],
-                               uidhash[18], uidhash[19], g10_errstr(rc) );
-           }
-           else { /* valid sig (may be revoked, but that doesn't matter here) */
-               upd_pref_record( sig, drec, keyid, uidrecno, uidhash );
+    delrec.recnum = 0;
+
+    /* get the LID of the pubkey of the signature under verification */
+    rc = get_pubkey( pk, sig->keyid );
+    if( !rc ) {
+       if( pk->local_id )
+           pk_lid = pk->local_id;
+       else {
+           rc = tdbio_search_dir_bypk( pk, &rec );
+           if( !rc )
+               pk_lid = rec.recnum;
+           else if( rc == -1 ) { /* see whether there is a sdir instead */
+               u32 akid[2];
+
+               keyid_from_pk( pk, akid );
+               rc = tdbio_search_sdir( akid, pk->pubkey_algo, &rec );
+               if( !rc )
+                   pk_lid = rec.recnum;
            }
        }
-       else {/* is revocation sig etc */
-           /* FIXME */
-       }
-    }
-    else if( !*uidrecno )
-       ; /* skip record with direct key signatures here */
-    else if( (sig->sig_class&~3) == 0x10 ) {
-       /* We simply insert the signature into the sig records but
-        * avoid duplicate ones.  We do not check them here because
-        * there is a big chance, that we import required public keys
-        * later.  The problem with this is that we must somewhere store
-        * the information about this signature (we need a record id).
-        * We do this by using the record type shadow dir, which will
-        * be converted to a dir record as soon as a new public key is
-        * inserted into the trustdb.
-        */
-       TRUSTREC urec, rec;
-       ulong recno;
-       TRUSTREC delrec;
-       int delrecidx;
-       int newflag = 0;
-       ulong newlid = 0;
-       PKT_public_key *pk = m_alloc_clear( sizeof *pk );
-
-       delrec.recnum = 0;
-       /* we need the uid record */
-       read_record( *uidrecno, &urec, RECTYPE_UID );
+    }
+    free_public_key( pk ); pk = NULL;
 
-       for( recno = urec.r.uid.siglist; recno; recno = rec.r.sig.next ) {
-           int i;
+    /* Loop over all signatures just in case one is not correctly
+     * marked. If we see the correct signature, set a flag.
+     * delete duplicate signatures (should not happen but...) */
+    for( recno = urec->r.uid.siglist; recno; recno = rec.r.sig.next ) {
+       int i;
 
-           read_record( recno, &rec, RECTYPE_SIG );
-           for(i=0; i < SIGS_PER_RECORD; i++ ) {
-               TRUSTREC tmp;
-               if( !rec.r.sig.sig[i].lid ) {
-                   if( !delrec.recnum ) {
-                       delrec = rec;
-                       delrecidx = i;
-                   }
-                   continue; /* skip deleted sigs */
+       read_record( recno, &rec, RECTYPE_SIG );
+       for(i=0; i < SIGS_PER_RECORD; i++ ) {
+           TRUSTREC tmp;
+           if( !rec.r.sig.sig[i].lid ) {
+               if( !found_delrec && !delrec.recnum ) {
+                   delrec = rec;
+                   delrecidx = i;
+                   found_delrec=1;
                }
-               if( rec.r.sig.sig[i].flag & SIGF_CHECKED )
-                   continue; /* we already checked this signature */
-               if( rec.r.sig.sig[i].flag & SIGF_NOPUBKEY )
-                   continue; /* we do not have the public key */
-
-               read_record( rec.r.sig.sig[i].lid, &tmp, 0 );
-               if( tmp.rectype == RECTYPE_DIR ) {
-                   /* In this case we should now be able to check
-                    * the signature: */
-                   rc = check_key_signature( keyblock, signode, NULL );
-                   if( !rc ) { /* valid signature */
-                       rec.r.sig.sig[i].flag = SIGF_CHECKED | SIGF_VALID;
-                   }
-                   else if( rc == G10ERR_NO_PUBKEY ) {
-                       log_info(_("key %08lX.%lu, uid %02X%02X: "
-                          "weird: no public key for signature %08lX\n"),
+               continue; /* skip deleted sigs */
+           }
+           if( rec.r.sig.sig[i].lid == pk_lid ) {
+               if( found_sig ) {
+                   log_info( "sig %08lX.%lu/%02X%02X/%08lX: %s\n",
                              (ulong)keyid[1], lid, uidhash[18],
-                              uidhash[19], (ulong)sig->keyid[1] );
-                       rec.r.sig.sig[i].flag = SIGF_NOPUBKEY;
-                   }
-                   else {
-                       log_info(_("key %08lX.%lu, uid %02X%02X: "
-                                  "invalid signature: %s\n"),
-                                   (ulong)keyid[1], lid,
-                                   uidhash[18], uidhash[19], g10_errstr(rc));
-                       rec.r.sig.sig[i].flag = SIGF_CHECKED;
-                   }
-                   write_record( &rec );
-                   goto ready;
+                              uidhash[19], (ulong)sig->keyid[1],
+                            _("Duplicated certificate - deleted") );
+                   rec.r.sig.sig[i].lid = 0;
+                   rec.dirty = 1;
+                   continue;
                }
-               else if( tmp.rectype == RECTYPE_SDIR ) {
-                   /* must check that it is the right one */
-                   if( tmp.r.sdir.keyid[0] == sig->keyid[0]
-                       && tmp.r.sdir.keyid[1] == sig->keyid[1]
-                       && (!tmp.r.sdir.pubkey_algo
-                            || tmp.r.sdir.pubkey_algo == sig->pubkey_algo )) {
-                       log_info(_("key %08lX.%lu, uid %02X%02X: "
-                                  "has shadow dir %lu but not yet marked.\n"),
-                                   (ulong)keyid[1], lid,
-                                   uidhash[18], uidhash[19], tmp.recnum );
-                       rec.r.sig.sig[i].flag = SIGF_NOPUBKEY;
-                       write_record( &rec );
-                       /* fixme: should we verify that the record is
-                        * in the hintlist? - This case here should anyway
-                        * never occur */
-                       goto ready;
-                   }
+               found_sig = 1;
+           }
+           if( !recheck && !revoke && (rec.r.sig.sig[i].flag & SIGF_CHECKED) )
+               continue; /* we already checked this signature */
+           if( !recheck && (rec.r.sig.sig[i].flag & SIGF_NOPUBKEY) )
+               continue; /* we do not have the public key */
+
+           read_record( rec.r.sig.sig[i].lid, &tmp, 0 );
+           if( tmp.rectype == RECTYPE_DIR ) {
+               /* In this case we should now be able to check the signature */
+               rc = check_key_signature( keyblock, signode, NULL );
+               if( !rc ) { /* valid signature */
+                   if( opt.verbose )
+                       log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
+                               (ulong)keyid[1], lid, uidhash[18],
+                               uidhash[19], (ulong)sig->keyid[1],
+                               revoke? _("Valid certificate revocation")
+                                     : _("Good certificate") );
+                   rec.r.sig.sig[i].flag = SIGF_CHECKED | SIGF_VALID;
+                   if( revoke )
+                       rec.r.sig.sig[i].flag |= SIGF_REVOKED;
+               }
+               else if( rc == G10ERR_NO_PUBKEY ) {
+                 #if 0 /* fixme: For some reason this really happens? */
+                   if( (rec.r.sig.sig[i].flag & SIGF_CHECKED) )
+                       log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
+                                 (ulong)keyid[1], lid, uidhash[18],
+                                uidhash[19], (ulong)sig->keyid[1],
+                                _("Hmmm, public key lost?") );
+                 #endif
+                   rec.r.sig.sig[i].flag = SIGF_NOPUBKEY;
+                   if( revoke )
+                       rec.r.sig.sig[i].flag |= SIGF_REVOKED;
                }
                else {
-                   log_error("sig record %lu[%d] points to wrong record.\n",
-                               rec.r.sig.sig[i].lid, i );
-                   die_invalid_db();
+                   log_info("sig %08lX.%lu/%02X%02X/%08lX: %s: %s\n",
+                               (ulong)keyid[1], lid, uidhash[18],
+                               uidhash[19], (ulong)sig->keyid[1],
+                               revoke? _("Invalid certificate revocation")
+                                     : _("Invalid certificate"),
+                                                   g10_errstr(rc));
+                   rec.r.sig.sig[i].flag = SIGF_CHECKED;
+                   if( revoke )
+                       rec.r.sig.sig[i].flag |= SIGF_REVOKED;
                }
+               rec.dirty = 1;
            }
-       }
-       /* at this point, we have verified, that the signature is not in
-        * our list of signatures.  Add a new record with that signature
-        * and if the public key is there, check the signature. */
-       rc = get_pubkey( pk, sig->keyid );
-       if( !rc ) {
-           /* check that we already have the record in the trustdb;
-            * if not we should not check the signature, because we do
-            * not have the lid of the signature and therefore can't add
-            * a signature record.  We will fake a no_pubkey error, so
-            * that this is handled, like we do not have the publick key at
-            * at all.  The alternative would be a recursive insert of
-            * records - but that is not good.
-            */
-           rc = query_trust_record( pk );
-           if( rc == -1 )
-               rc = G10ERR_NO_PUBKEY;
-           else
-               rc = check_key_signature( keyblock, signode, NULL );
-       }
-       if( !rc ) { /* valid signature */
-           newlid = pk->local_id;  /* this is the pk of the signature */
-           if( !newlid )
-               BUG();
-           newflag = SIGF_CHECKED | SIGF_VALID;
-       }
-       else if( rc == G10ERR_NO_PUBKEY ) {
-           if( opt.verbose )
-               log_info(_("key %08lX.%lu, uid %02X%02X: "
-                          "no public key for signature %08lX\n"),
-                             (ulong)keyid[1], lid, uidhash[18],
-                              uidhash[19], (ulong)sig->keyid[1] );
-           newflag = SIGF_NOPUBKEY;
-       }
-       else {
-           log_info(_("key %08lX.%lu, uid %02X%02X: "
-                      "invalid signature: %s\n"),
-                       (ulong)keyid[1], lid, uidhash[18], uidhash[19],
-                                                      g10_errstr(rc));
-           newflag = SIGF_CHECKED;
-       }
-       free_public_key( pk );
-
-       if( !newlid ) { /* create a shadow dir record */
-           TRUSTREC sdir, hlst, tmphlst;
-           int tmpidx;
-           /* first see whether we already have such a record */
-           rc = tdbio_search_sdir( sig->keyid, sig->pubkey_algo, &sdir );
-           if( rc && rc != -1 ) {
-               log_error("tdbio_search_dir failed: %s\n", g10_errstr(rc));
-               die_invalid_db();
-           }
-           if( rc == -1 ) { /* not found: create */
-               memset( &sdir, 0, sizeof sdir );
-               sdir.recnum = tdbio_new_recnum();
-               sdir.rectype= RECTYPE_SDIR;
-               sdir.r.sdir.lid = sdir.recnum;
-               sdir.r.sdir.keyid[0] = sig->keyid[0];
-               sdir.r.sdir.keyid[1] = sig->keyid[1];
-               sdir.r.sdir.pubkey_algo = sig->pubkey_algo;
-               sdir.r.sdir.hintlist = 0;
-               write_record( &sdir );
-           }
-           newlid = sdir.recnum;
-           /* Put the record number into the hintlist.
-            * (It is easier to use the lid and not the record number of the
-            *  key to save some space (assuming that a signator has
-            *  signed more than one user id - and it is easier to implement.)
-            */
-           tmphlst.recnum = 0;
-           for( recno=sdir.r.sdir.hintlist; recno; recno = hlst.r.hlst.next) {
-               int i;
-               read_record( recno, &hlst, RECTYPE_HLST );
-               for( i=0; i < ITEMS_PER_HLST_RECORD; i++ ) {
-                   if( !hlst.r.hlst.rnum[i] ) {
-                       if( !tmphlst.recnum ) {
-                           tmphlst = hlst;
-                           tmpidx = i;
-                       }
-                   }
-                   else if( hlst.r.hlst.rnum[i] == lid )
-                       goto have_hint;
+           else if( tmp.rectype == RECTYPE_SDIR ) {
+               /* must check that it is the right one */
+               if( tmp.r.sdir.keyid[0] == sig->keyid[0]
+                   && tmp.r.sdir.keyid[1] == sig->keyid[1]
+                   && (!tmp.r.sdir.pubkey_algo
+                        || tmp.r.sdir.pubkey_algo == sig->pubkey_algo )) {
+                   if( !(rec.r.sig.sig[i].flag & SIGF_NOPUBKEY) )
+                       log_info(_("uid %08lX.%lu/%02X%02X: "
+                               "has shadow dir %lu but is not yet marked.\n"),
+                               (ulong)keyid[1], lid,
+                               uidhash[18], uidhash[19], tmp.recnum );
+                   rec.r.sig.sig[i].flag = SIGF_NOPUBKEY;
+                   if( revoke )
+                       rec.r.sig.sig[i].flag |= SIGF_REVOKED;
+                   rec.dirty = 1;
+                   /* fixme: should we verify that the record is
+                    * in the hintlist? - This case here should anyway
+                    * never occur */
                }
            }
-           /* not yet in the hint list, write it */
-           if( tmphlst.recnum ) { /* we have an empty slot */
-               tmphlst.r.hlst.rnum[tmpidx] = lid;
-               write_record( &tmphlst );
-           }
-           else { /* must append a new hlst record */
-               memset( &hlst, 0, sizeof hlst );
-               hlst.recnum = tdbio_new_recnum();
-               hlst.rectype = RECTYPE_HLST;
-               hlst.r.hlst.next = sdir.r.sdir.hintlist;
-               hlst.r.hlst.rnum[0] = lid;
-               write_record( &hlst );
-               sdir.r.sdir.hintlist = hlst.recnum;
-               write_record( &sdir );
+           else {
+               log_error(_("sig record %lu[%d] points to wrong record.\n"),
+                           rec.r.sig.sig[i].lid, i );
+               die_invalid_db();
            }
-         have_hint:  /* "goto considered useful" (don't tell Dijkstra) */
-           ;
        }
-
-       if( delrec.recnum ) { /* we can reuse a deleted slot */
-           delrec.r.sig.sig[delrecidx].lid = newlid;
-           delrec.r.sig.sig[delrecidx].flag= newflag;
-           write_record( &delrec );
+       if( found_delrec && delrec.recnum ) {
+           delrec = rec;
+           found_delrec = 0; /* we only want the first one */
        }
-       else { /* must insert a new sig record */
-           TRUSTREC tmp;
-
-           memset( &tmp, 0, sizeof tmp );
-           tmp.recnum = tdbio_new_recnum();
-           tmp.rectype = RECTYPE_SIG;
-           tmp.r.sig.lid = lid;
-           tmp.r.sig.next = urec.r.uid.siglist;
-           tmp.r.sig.sig[0].lid = newlid;
-           tmp.r.sig.sig[0].flag= newflag;
-           write_record( &tmp );
-           urec.r.uid.siglist = tmp.recnum;
-           write_record( &urec );
+       if( rec.dirty ) {
+           write_record( &rec );
+           rec.dirty = 0;
        }
+    }
+
+    if( found_sig )
+       return;
+
+    /* at this point, we have verified, that the signature is not in
+     * our list of signatures. Add a new record with that signature
+     * and if the public key is there, check the signature. */
 
-      ready:
-       ;
+    if( !pk_lid ) /* we have already seen that there is no pubkey */
+       rc = G10ERR_NO_PUBKEY;
+    else
+       rc = check_key_signature( keyblock, signode, NULL );
+
+    if( !rc ) { /* valid signature */
+       if( opt.verbose )
+           log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
+                         (ulong)keyid[1], lid, uidhash[18],
+                          uidhash[19], (ulong)sig->keyid[1],
+                               revoke? _("Valid certificate revocation")
+                                     : _("Good certificate") );
+       newlid = pk_lid;  /* this is the pk of the signature */
+       newflag = SIGF_CHECKED | SIGF_VALID;
+       if( revoke )
+           newflag |= SIGF_REVOKED;
+    }
+    else if( rc == G10ERR_NO_PUBKEY ) {
+       if( opt.verbose > 1 )
+           log_info("sig %08lX.%lu/%02X%02X/%08lX: %s\n",
+                    (ulong)keyid[1], lid, uidhash[18],
+                     uidhash[19], (ulong)sig->keyid[1], g10_errstr(rc) );
+       newlid = create_shadow_dir( sig, lid );
+       newflag = SIGF_NOPUBKEY;
+       if( revoke )
+           newflag |= SIGF_REVOKED;
     }
     else {
-       /* handle other sig classes */
+       log_info( "sig %08lX.%lu/%02X%02X/%08lX: %s: %s\n",
+                   (ulong)keyid[1], lid, uidhash[18], uidhash[19],
+                             (ulong)sig->keyid[1],
+               revoke? _("Invalid certificate revocation")
+                     : _("Invalid certificate"),
+                                           g10_errstr(rc));
+       newlid = create_shadow_dir( sig, lid );
+       newflag = SIGF_CHECKED;
+       if( revoke )
+           newflag |= SIGF_REVOKED;
+    }
+
+    if( delrec.recnum ) { /* we can reuse a deleted/unused slot */
+       delrec.r.sig.sig[delrecidx].lid = newlid;
+       delrec.r.sig.sig[delrecidx].flag= newflag;
+       write_record( &delrec );
+    }
+    else { /* must insert a new sig record */
+       TRUSTREC tmp;
+
+       memset( &tmp, 0, sizeof tmp );
+       tmp.recnum = tdbio_new_recnum();
+       tmp.rectype = RECTYPE_SIG;
+       tmp.r.sig.lid = lid;
+       tmp.r.sig.next = urec->r.uid.siglist;
+       tmp.r.sig.sig[0].lid = newlid;
+       tmp.r.sig.sig[0].flag= newflag;
+       write_record( &tmp );
+       urec->r.uid.siglist = tmp.recnum;
+       urec->dirty = 1;
     }
-
 }
 
 
@@ -2267,11 +2884,11 @@ upd_sig_record( PKT_signature *sig, TRUSTREC *drec,
  * Update all the info from the public keyblock.
  * The key must already exist in the keydb.
  * This function is responsible for checking the signatures in cases
- * where the public key is already available.  If we no not have the public
+ * where the public key is already available.  If we do not have the public
  * key, the check is done by some special code in insert_trust_record().
  */
 int
-update_trust_record( KBNODE keyblock )
+update_trust_record( KBNODE keyblock, int recheck, int *modified )
 {
     PKT_public_key *primary_pk;
     KBNODE node;
@@ -2283,9 +2900,11 @@ update_trust_record( KBNODE keyblock )
     int rc = 0;
     u32 keyid[2]; /* keyid of primary key */
     ulong recno, lastrecno;
-    ulong uidrecno = 0;
-    byte uidhash[20];
     RECNO_LIST recno_list = NULL; /* list of verified records */
+    /* fixme: replace recno_list by a lookup on node->recno */
+
+    if( modified )
+       *modified = 0;
 
     node = find_kbnode( keyblock, PKT_PUBLIC_KEY );
     primary_pk = node->pkt->pkt.public_key;
@@ -2297,44 +2916,41 @@ update_trust_record( KBNODE keyblock )
 
     keyid_from_pk( primary_pk, keyid );
 
-    /* fixme: start a transaction */
-    /* now update keys and user ids */
-    for( node=keyblock; node; node = node->next ) {
-       switch( node->pkt->pkttype ) {
-         case PKT_PUBLIC_KEY:
-         case PKT_PUBLIC_SUBKEY:
-           uidrecno = 0;
-           upd_key_record( node->pkt->pkt.public_key, &drec, &recno_list );
-           break;
-
-         case PKT_USER_ID:
-           if( drec.dirty ) { /* upd_pref_record may read the drec */
-               write_record( &drec );
-               drec.dirty = 0;
-           }
-           upd_uid_record( node->pkt->pkt.user_id, &drec, &recno_list,
-                           keyid, &uidrecno, uidhash );
-           break;
-
-         case PKT_SIGNATURE:
-           if( drec.dirty ) { /* upd_sig_recrod may read the drec */
-               write_record( &drec );
-               drec.dirty = 0;
-           }
-           upd_sig_record( node->pkt->pkt.signature, &drec,
-                           keyid, &uidrecno, uidhash, keyblock, node );
-           break;
+    /* fixme: check that the keyblock has a valid structure */
 
-         default:
-           break;
-       }
-    } /* end loop over all nodes */
+    rc = tdbio_begin_transaction();
+    if( rc )
+       return rc;
 
+    /* update the keys */
+    for( node=keyblock; node; node = node->next ) {
+       if( node->pkt->pkttype == PKT_PUBLIC_KEY
+           || node->pkt->pkttype == PKT_PUBLIC_SUBKEY )
+           upd_key_record( keyblock, node, keyid,
+                           &drec, &recno_list, recheck );
+    }
+    /* update the user IDs */
+    for( node=keyblock; node; node = node->next ) {
+       if( node->pkt->pkttype == PKT_USER_ID )
+           upd_uid_record( keyblock, node, keyid,
+                           &drec, &recno_list, recheck );
+    }
 
     /* delete keyrecords from the trustdb which are not anymore used */
+    /* should we really do this, or is it better to keep them and */
+    /* mark as unused? */
+    /* And set the revocation flag into the dir record */
+    drec.r.dir.dirflags &= ~DIRF_REVOKED;
     lastrecno = 0;
     for( recno=drec.r.dir.keylist; recno; recno = krec.r.key.next ) {
        read_record( recno, &krec, RECTYPE_KEY );
+       if( recno == drec.r.dir.keylist ) { /* this is the primary key */
+           if( (krec.r.key.keyflags & KEYF_REVOKED) ) {
+               drec.r.dir.dirflags |= DIRF_REVOKED;
+               drec.dirty = 1;
+           }
+       }
+
        if( !qry_recno_list( recno_list, recno, RECTYPE_KEY ) ) {
            /* delete this one */
            if( !lastrecno ) {
@@ -2385,11 +3001,14 @@ update_trust_record( KBNODE keyblock )
 
 
     if( rc )
-       ; /* fixme: cancel transaction */
-    else if( drec.dirty ) {
-       drec.r.dir.dirflags &= ~DIRF_CHECKED; /* reset flag */
+       rc = tdbio_cancel_transaction();
+    else {
+       if( modified && tdbio_is_dirty() )
+           *modified = 1;
+       drec.r.dir.dirflags |= DIRF_CHECKED;
+       drec.r.dir.dirflags &= ~DIRF_VALVALID;
        write_record( &drec );
-       /* fixme: commit_transaction */
+       rc = tdbio_end_transaction();
     }
     rel_recno_list( &recno_list );
     return rc;
@@ -2430,17 +3049,25 @@ insert_trust_record( PKT_public_key *pk )
     /* get the keyblock which has the key */
     rc = get_keyblock_byfprint( &keyblock, fingerprint, fingerlen );
     if( rc ) { /* that should never happen */
-       log_error( "insert_trust_record: keyblock not found: %s\n",
+       log_error( _("insert_trust_record: keyblock not found: %s\n"),
                                                          g10_errstr(rc) );
        goto leave;
     }
 
     /* check that we used the primary key (we are little bit paranoid) */
     {  PKT_public_key *a_pk;
+       u32 akid[2], bkid[2];
+
        node = find_kbnode( keyblock, PKT_PUBLIC_KEY );
        a_pk = node->pkt->pkt.public_key;
-       if( cmp_public_keys( a_pk, pk ) ) {
-           log_error("did not use primary key for insert_trust_record()\n");
+
+       /* we can't use cmp_public_keys here because some parts (expiredate)
+        * might not be set in pk <--- but why (fixme) */
+       keyid_from_pk( a_pk, akid );
+       keyid_from_pk( pk, bkid );
+
+       if( akid[0] != bkid[0] || akid[1] != bkid[1] ) {
+           log_error(_("did not use primary key for insert_trust_record()\n"));
            rc = G10ERR_GENERAL;
            goto leave;
        }
@@ -2452,7 +3079,7 @@ insert_trust_record( PKT_public_key *pk )
      */
     rc = tdbio_search_sdir( pk->keyid, pk->pubkey_algo, &shadow );
     if( rc && rc != -1 ) {
-       log_error("tdbio_search_dir failed: %s\n", g10_errstr(rc));
+       log_error(_("tdbio_search_dir failed: %s\n"), g10_errstr(rc));
        die_invalid_db();
     }
     memset( &dirrec, 0, sizeof dirrec );
@@ -2484,7 +3111,7 @@ insert_trust_record( PKT_public_key *pk )
     }
 
     /* and put all the other stuff into the keydb */
-    rc = update_trust_record( keyblock );
+    rc = update_trust_record( keyblock, 1, NULL );
     if( !rc )
        process_hintlist( hintlist, dirrec.r.dir.lid );
 
@@ -2492,6 +3119,7 @@ insert_trust_record( PKT_public_key *pk )
     if( rc && hintlist )
        ; /* fixme: the hintlist is not anymore anchored */
     release_kbnode( keyblock );
+    do_sync();
     return rc;
 }
 
@@ -2504,6 +3132,7 @@ update_ownertrust( ulong lid, unsigned new_trust )
     read_record( lid, &rec, RECTYPE_DIR );
     rec.r.dir.ownertrust = new_trust;
     write_record( &rec );
+    do_sync();
     return 0;
 }