edit-key is now complete
[gnupg.git] / g10 / ringedit.c
index 28faedd..58eb6d6 100644 (file)
@@ -69,7 +69,6 @@ typedef struct resource_table_struct RESTBL;
 #define MAX_RESOURCES 10
 static RESTBL resource_table[MAX_RESOURCES];
 
-
 static int search( PACKET *pkt, KBPOS *kbpos, int secret );
 
 
@@ -92,7 +91,6 @@ check_pos( KBPOS *kbpos )
 }
 
 
-
 /****************************************************************
  ****************** public functions ****************************
  ****************************************************************/
@@ -119,6 +117,17 @@ add_keyblock_resource( const char *filename, int force, int secret )
     if( !iobuf && !force )
        return G10ERR_OPEN_FILE;
   #endif
+
+    if( !iobuf ) {
+       iobuf = iobuf_create( filename );
+       if( !iobuf ) {
+           log_error("%s: can't create: %s\n", filename, strerror(errno));
+           return G10ERR_OPEN_FILE;
+       }
+       else
+           log_info("%s: keyring created\n", filename );
+    }
+
     resource_table[i].used = 1;
     resource_table[i].secret = !!secret;
     resource_table[i].fname = m_strdup(filename);
@@ -168,7 +177,7 @@ get_keyblock_handle( const char *filename, int secret, KBPOS *kbpos )
  * Search a keyblock which starts with the given packet and puts all
  * information into KBPOS, which can be used later to access this key block.
  * This function looks into all registered keyblock sources.
- * PACKET must be a packet with either a secret_cert or a public_cert
+ * PACKET must be a packet with either a secret_key or a public_key
  *
  * This function is intended to check whether a given certificate
  * is already in a keyring or to prepare it for editing.
@@ -210,20 +219,20 @@ int
 find_keyblock_byname( KBPOS *kbpos, const char *username )
 {
     PACKET pkt;
-    PKT_public_cert *pkc = m_alloc_clear( sizeof *pkc );
+    PKT_public_key *pk = m_alloc_clear( sizeof *pk );
     int rc;
 
-    rc = get_pubkey_byname( pkc, username );
+    rc = get_pubkey_byname( pk, username );
     if( rc ) {
-       free_public_cert(pkc);
+       free_public_key(pk);
        return rc;
     }
 
     init_packet( &pkt );
-    pkt.pkttype = PKT_PUBLIC_CERT;
-    pkt.pkt.public_cert = pkc;
+    pkt.pkttype = PKT_PUBLIC_KEY;
+    pkt.pkt.public_key = pk;
     rc = search( &pkt, kbpos, 0 );
-    free_public_cert(pkc);
+    free_public_key(pk);
     return rc;
 }
 
@@ -233,14 +242,14 @@ find_keyblock_byname( KBPOS *kbpos, const char *username )
  * of the keyblock.
  */
 int
-find_keyblock_bypkc( KBPOS *kbpos, PKT_public_cert *pkc )
+find_keyblock_bypk( KBPOS *kbpos, PKT_public_key *pk )
 {
     PACKET pkt;
     int rc;
 
     init_packet( &pkt );
-    pkt.pkttype = PKT_PUBLIC_CERT;
-    pkt.pkt.public_cert = pkc;
+    pkt.pkttype = PKT_PUBLIC_KEY;
+    pkt.pkt.public_key = pk;
     rc = search( &pkt, kbpos, 0 );
     return rc;
 }
@@ -254,20 +263,20 @@ int
 find_secret_keyblock_byname( KBPOS *kbpos, const char *username )
 {
     PACKET pkt;
-    PKT_secret_cert *skc = m_alloc_clear( sizeof *skc );
+    PKT_secret_key *sk = m_alloc_clear( sizeof *sk );
     int rc;
 
-    rc = get_seckey_byname( skc, username, 0 );
+    rc = get_seckey_byname( sk, username, 0 );
     if( rc ) {
-       free_secret_cert(skc);
+       free_secret_key(sk);
        return rc;
     }
 
     init_packet( &pkt );
-    pkt.pkttype = PKT_SECRET_CERT;
-    pkt.pkt.secret_cert = skc;
+    pkt.pkttype = PKT_SECRET_KEY;
+    pkt.pkt.secret_key = sk;
     rc = search( &pkt, kbpos, 1 );
-    free_secret_cert(skc);
+    free_secret_key(sk);
     return rc;
 }
 
@@ -448,6 +457,35 @@ update_keyblock( KBPOS *kbpos, KBNODE root )
  ********** Functions which operates on regular keyrings ********
  ****************************************************************/
 
+static int
+cmp_seckey( PKT_secret_key *req_sk, PKT_secret_key *sk )
+{
+    int n,i;
+
+    assert( req_sk->pubkey_algo == sk->pubkey_algo );
+
+    n = pubkey_get_nskey( req_sk->pubkey_algo );
+    for(i=0; i < n; i++ ) {
+       if( mpi_cmp( req_sk->skey[i], sk->skey[i] ) )
+           return -1;
+    }
+    return 0;
+}
+
+static int
+cmp_pubkey( PKT_public_key *req_pk, PKT_public_key *pk )
+{
+    int n, i;
+
+    assert( req_pk->pubkey_algo == pk->pubkey_algo );
+
+    n = pubkey_get_npkey( req_pk->pubkey_algo );
+    for(i=0; i < n; i++ ) {
+       if( mpi_cmp( req_pk->pkey[i], pk->pkey[i] )  )
+           return -1;
+    }
+    return 0;
+}
 
 /****************
  * search one keyring, return 0 if found, -1 if not found or an errorcode.
@@ -460,8 +498,8 @@ keyring_search( PACKET *req, KBPOS *kbpos, IOBUF iobuf, const char *fname )
     int save_mode;
     ulong offset;
     int pkttype = req->pkttype;
-    PKT_public_cert *req_pkc = req->pkt.public_cert;
-    PKT_secret_cert *req_skc = req->pkt.secret_cert;
+    PKT_public_key *req_pk = req->pkt.public_key;
+    PKT_secret_key *req_sk = req->pkt.secret_key;
 
     init_packet(&pkt);
     save_mode = set_packet_list_mode(0);
@@ -483,57 +521,22 @@ keyring_search( PACKET *req, KBPOS *kbpos, IOBUF iobuf, const char *fname )
   #endif
 
     while( !(rc=search_packet(iobuf, &pkt, pkttype, &offset)) ) {
-       if( pkt.pkttype == PKT_SECRET_CERT ) {
-           PKT_secret_cert *skc = pkt.pkt.secret_cert;
-
-           if(   req_skc->timestamp == skc->timestamp
-              && req_skc->valid_days == skc->valid_days
-              && req_skc->pubkey_algo == skc->pubkey_algo
-              && (   ( is_ELGAMAL(skc->pubkey_algo)
-                       && !mpi_cmp( req_skc->d.elg.p, skc->d.elg.p )
-                       && !mpi_cmp( req_skc->d.elg.g, skc->d.elg.g )
-                       && !mpi_cmp( req_skc->d.elg.y, skc->d.elg.y )
-                       && !mpi_cmp( req_skc->d.elg.x, skc->d.elg.x )
-                     )
-                  || ( skc->pubkey_algo == PUBKEY_ALGO_DSA
-                       && !mpi_cmp( req_skc->d.dsa.p, skc->d.dsa.p )
-                       && !mpi_cmp( req_skc->d.dsa.q, skc->d.dsa.q )
-                       && !mpi_cmp( req_skc->d.dsa.g, skc->d.dsa.g )
-                       && !mpi_cmp( req_skc->d.dsa.y, skc->d.dsa.y )
-                       && !mpi_cmp( req_skc->d.dsa.x, skc->d.dsa.x )
-                     )
-                  || ( is_RSA(skc->pubkey_algo)
-                       && !mpi_cmp( req_skc->d.rsa.n, skc->d.rsa.n )
-                       && !mpi_cmp( req_skc->d.rsa.e, skc->d.rsa.e )
-                       && !mpi_cmp( req_skc->d.rsa.d, skc->d.rsa.d )
-                     )
-                 )
-             )
+       if( pkt.pkttype == PKT_SECRET_KEY ) {
+           PKT_secret_key *sk = pkt.pkt.secret_key;
+
+           if(   req_sk->timestamp == sk->timestamp
+              && req_sk->valid_days == sk->valid_days
+              && req_sk->pubkey_algo == sk->pubkey_algo
+              && !cmp_seckey( req_sk, sk) )
                break; /* found */
        }
-       else if( pkt.pkttype == PKT_PUBLIC_CERT ) {
-           PKT_public_cert *pkc = pkt.pkt.public_cert;
-
-           if(   req_pkc->timestamp == pkc->timestamp
-              && req_pkc->valid_days == pkc->valid_days
-              && req_pkc->pubkey_algo == pkc->pubkey_algo
-              && (   ( is_ELGAMAL(pkc->pubkey_algo)
-                       && !mpi_cmp( req_pkc->d.elg.p, pkc->d.elg.p )
-                       && !mpi_cmp( req_pkc->d.elg.g, pkc->d.elg.g )
-                       && !mpi_cmp( req_pkc->d.elg.y, pkc->d.elg.y )
-                     )
-                  || ( pkc->pubkey_algo == PUBKEY_ALGO_DSA
-                       && !mpi_cmp( req_pkc->d.dsa.p, pkc->d.dsa.p )
-                       && !mpi_cmp( req_pkc->d.dsa.q, pkc->d.dsa.q )
-                       && !mpi_cmp( req_pkc->d.dsa.g, pkc->d.dsa.g )
-                       && !mpi_cmp( req_pkc->d.dsa.y, pkc->d.dsa.y )
-                     )
-                  || ( is_RSA(pkc->pubkey_algo)
-                       && !mpi_cmp( req_pkc->d.rsa.n, pkc->d.rsa.n )
-                       && !mpi_cmp( req_pkc->d.rsa.e, pkc->d.rsa.e )
-                     )
-                 )
-             )
+       else if( pkt.pkttype == PKT_PUBLIC_KEY ) {
+           PKT_public_key *pk = pkt.pkt.public_key;
+
+           if(   req_pk->timestamp == pk->timestamp
+              && req_pk->valid_days == pk->valid_days
+              && req_pk->pubkey_algo == pk->pubkey_algo
+              && !cmp_pubkey( req_pk, pk ) )
                break; /* found */
        }
        else
@@ -588,7 +591,6 @@ keyring_read( KBPOS *kbpos, KBNODE *ret_root )
                rc = G10ERR_INV_KEYRING;
                goto ready;
            }
-               log_info("read_keyblock: read error: %s\n", g10_errstr(rc) );
            kbpos->count++;
            free_packet( pkt );
            init_packet( pkt );
@@ -596,8 +598,8 @@ keyring_read( KBPOS *kbpos, KBNODE *ret_root )
        }
        /* make a linked list of all packets */
        switch( pkt->pkttype ) {
-         case PKT_PUBLIC_CERT:
-         case PKT_SECRET_CERT:
+         case PKT_PUBLIC_KEY:
+         case PKT_SECRET_KEY:
            if( in_cert )
                goto ready;
            in_cert = 1;
@@ -658,8 +660,8 @@ keyring_enum( KBPOS *kbpos, KBNODE *ret_root, int skipsigs )
        }
        /* make a linked list of all packets */
        switch( pkt->pkttype ) {
-         case PKT_PUBLIC_CERT:
-         case PKT_SECRET_CERT:
+         case PKT_PUBLIC_KEY:
+         case PKT_SECRET_KEY:
            if( root ) { /* store this packet */
                kbpos->pkt = pkt;
                pkt = NULL;
@@ -735,6 +737,8 @@ keyring_copy( KBPOS *kbpos, int mode, KBNODE root )
            log_error("%s: can't create: %s\n", rentry->fname, strerror(errno));
            return G10ERR_OPEN_FILE;
        }
+       else
+           log_info("%s: keyring created\n", rentry->fname );
 
        kbctx=NULL;
        while( (node = walk_kbnode( root, &kbctx, 0 )) ) {