ready to release 0.4.3
[gnupg.git] / g10 / keygen.c
index d9f8579..a17d479 100644 (file)
@@ -32,6 +32,7 @@
 #include "ttyio.h"
 #include "options.h"
 #include "keydb.h"
+#include "status.h"
 #include "i18n.h"
 
 
@@ -50,15 +51,15 @@ write_uid( KBNODE root, const char *s )
 
 
 
-static int
-add_key_expire( PKT_signature *sig, void *opaque )
+int
+keygen_add_key_expire( PKT_signature *sig, void *opaque )
 {
     PKT_secret_key *sk = opaque;
     byte buf[8];
     u32  u;
 
-    if( sk->valid_days ) {
-       u = sk->valid_days * 86400L;
+    if( sk->expiredate ) {
+       u = sk->expiredate;
        buf[0] = (u >> 24) & 0xff;
        buf[1] = (u >> 16) & 0xff;
        buf[2] = (u >>  8) & 0xff;
@@ -74,12 +75,12 @@ add_key_expire( PKT_signature *sig, void *opaque )
  * Add preference to the self signature packet.
  * This is only called for packets with version > 3.
  */
-static int
-add_prefs( PKT_signature *sig, void *opaque )
+int
+keygen_add_std_prefs( PKT_signature *sig, void *opaque )
 {
     byte buf[8];
 
-    add_key_expire( sig, opaque );
+    keygen_add_key_expire( sig, opaque );
 
     buf[0] = CIPHER_ALGO_BLOWFISH;
     buf[1] = CIPHER_ALGO_CAST5;
@@ -134,7 +135,7 @@ write_selfsig( KBNODE root, KBNODE pub_root, PKT_secret_key *sk )
 
     /* and make the signature */
     rc = make_keysig_packet( &sig, pk, uid, NULL, sk, 0x13, 0,
-                            add_prefs, sk );
+                            keygen_add_std_prefs, sk );
     if( rc ) {
        log_error("make_keysig_packet failed: %s\n", g10_errstr(rc) );
        return rc;
@@ -175,7 +176,7 @@ write_keybinding( KBNODE root, KBNODE pub_root, PKT_secret_key *sk )
 
     /* and make the signature */
     rc = make_keysig_packet( &sig, pk, NULL, subpk, sk, 0x18, 0,
-                                   add_key_expire, sk );
+                                   keygen_add_key_expire, sk );
     if( rc ) {
        log_error("make_keysig_packet failed: %s\n", g10_errstr(rc) );
        return rc;
@@ -191,7 +192,7 @@ write_keybinding( KBNODE root, KBNODE pub_root, PKT_secret_key *sk )
 
 static int
 gen_elg(int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
-       STRING2KEY *s2k, PKT_secret_key **ret_sk, u16 valid_days,
+       STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 expiredate,
                                                        int version )
 {
     int rc;
@@ -213,7 +214,9 @@ gen_elg(int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
     pk = m_alloc_clear( sizeof *pk );
     sk->timestamp = pk->timestamp = make_timestamp();
     sk->version = pk->version = version;
-    sk->valid_days = pk->valid_days = valid_days;
+    if( expiredate && expiredate < sk->timestamp )
+       expiredate = sk->timestamp; /* key generatio may take long */
+    sk->expiredate = pk->expiredate = expiredate;
     sk->pubkey_algo = pk->pubkey_algo = algo;
                       pk->pkey[0] = mpi_copy( skey[0] );
                       pk->pkey[1] = mpi_copy( skey[1] );
@@ -265,7 +268,7 @@ gen_elg(int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
  */
 static int
 gen_dsa(unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
-           STRING2KEY *s2k, PKT_secret_key **ret_sk, u16 valid_days )
+           STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 expiredate )
 {
     int rc;
     int i;
@@ -288,7 +291,9 @@ gen_dsa(unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
     pk = m_alloc_clear( sizeof *pk );
     sk->timestamp = pk->timestamp = make_timestamp();
     sk->version = pk->version = 4;
-    sk->valid_days = pk->valid_days = valid_days;
+    if( expiredate && expiredate < pk->timestamp )
+       expiredate = pk->timestamp; /* key generation may take long */
+    sk->expiredate = pk->expiredate = expiredate;
     sk->pubkey_algo = pk->pubkey_algo = PUBKEY_ALGO_DSA;
                       pk->pkey[0] = mpi_copy( skey[0] );
                       pk->pkey[1] = mpi_copy( skey[1] );
@@ -371,7 +376,7 @@ check_valid_days( const char *s )
 
 
 /****************
- * Returns o to create both a DSA and a ElGamal key.
+ * Returns: 0 to create both a DSA and a ElGamal key.
  */
 static int
 ask_algo( int *ret_v4, int addmode )
@@ -383,14 +388,15 @@ ask_algo( int *ret_v4, int addmode )
     if( !addmode )
        tty_printf(_("   (%d) DSA and ElGamal (default)\n"), 1 );
     tty_printf(    _("   (%d) ElGamal (sign and encrypt)\n"), 2 );
-    tty_printf(    _("   (%d) ElGamal (encrypt only)\n"), 3 );
+    if( addmode )
+       tty_printf(    _("   (%d) ElGamal (encrypt only)\n"), 3 );
     tty_printf(    _("   (%d) DSA (sign only)\n"), 4 );
     tty_printf(    _("   (%d) ElGamal in a v3 packet\n"), 5 );
 
     *ret_v4 = 1;
     for(;;) {
-       answer = tty_get(_("Your selection? "));
-       tty_kill_prompt();
+       answer = cpr_get(N_("keygen.algo"),_("Your selection? "));
+       cpr_kill_prompt();
        algo = *answer? atoi(answer): 1;
        m_free(answer);
        if( algo == 1 && !addmode ) {
@@ -401,7 +407,7 @@ ask_algo( int *ret_v4, int addmode )
            algo = PUBKEY_ALGO_ELGAMAL;
            break;
        }
-       else if( algo == 3 ) {
+       else if( algo == 3 && addmode ) {
            algo = PUBKEY_ALGO_ELGAMAL_E;
            break;
        }
@@ -433,36 +439,31 @@ ask_keysize( int algo )
                 "    highest suggested keysize is 2048 bits\n"),
                                        pubkey_algo_to_string(algo) );
     for(;;) {
-       answer = tty_get(_("What keysize do you want? (1024) "));
-       tty_kill_prompt();
+       answer = cpr_get(N_("keygen.size"),
+                         _("What keysize do you want? (1024) "));
+       cpr_kill_prompt();
        nbits = *answer? atoi(answer): 1024;
        m_free(answer);
        if( algo == PUBKEY_ALGO_DSA && (nbits < 512 || nbits > 1024) )
            tty_printf(_("DSA only allows keysizes from 512 to 1024\n"));
        else if( nbits < 768 )
            tty_printf(_("keysize too small; 768 is smallest value allowed.\n"));
-       else if( nbits > 2048 ) {
-           tty_printf(_("Keysizes larger than 2048 are not suggested, because "
-                        "computations take REALLY long!\n"));
-           answer = tty_get(_("Are you sure, that you want this keysize? "));
-           tty_kill_prompt();
-           if( answer_is_yes(answer) ) {
-               m_free(answer);
+       else if( nbits > 2048 && !cpr_enabled() ) {
+           tty_printf(
+               _("Keysizes larger than 2048 are not suggested because\n"
+                 "computations take REALLY long!\n"));
+           if( cpr_get_answer_is_yes(N_("keygen.size.huge.okay"),_(
+                       "Are you sure that you want this keysize? ")) ) {
                tty_printf(_("Okay, but keep in mind that your monitor "
                             "and keyboard radiation is also very vulnerable "
                             "to attacks!\n"));
                break;
            }
-           m_free(answer);
        }
-       else if( nbits > 1536 ) {
-           answer = tty_get(_("Do you really need such a large keysize? "));
-           tty_kill_prompt();
-           if( answer_is_yes(answer) ) {
-               m_free(answer);
+       else if( nbits > 1536 && !cpr_enabled() ) {
+           if( cpr_get_answer_is_yes(N_("keygen.size.large.okay"),_(
+                   "Do you really need such a large keysize? ")) )
                break;
-           }
-           m_free(answer);
        }
        else
            break;
@@ -480,11 +481,12 @@ ask_keysize( int algo )
 }
 
 
-static int
-ask_valid_days()
+u32
+ask_expiredate()
 {
     char *answer;
     int valid_days=0;
+    u32 expiredate = 0;
 
     tty_printf(_("Please specify how long the key should be valid.\n"
                 "         0 = key does not expire\n"
@@ -493,7 +495,7 @@ ask_valid_days()
                 "      <n>m = key expires in n months\n"
                 "      <n>y = key expires in n years\n"));
     /* Note: The elgamal subkey for DSA has no exiration date because
-     * is must be signed with the DSA key and this one has the expiration
+     * it must be signed with the DSA key and this one has the expiration
      * date */
 
     answer = NULL;
@@ -501,8 +503,8 @@ ask_valid_days()
        int mult;
 
        m_free(answer);
-       answer = tty_get(_("Key is valid for? (0) "));
-       tty_kill_prompt();
+       answer = cpr_get(N_("keygen.valid"),_("Key is valid for? (0) "));
+       cpr_kill_prompt();
        trim_spaces(answer);
        if( !*answer )
            valid_days = 0;
@@ -516,32 +518,41 @@ ask_valid_days()
            continue;
        }
 
-       if( !valid_days )
+       if( !valid_days ) {
            tty_printf(_("Key does not expire at all\n"));
+           expiredate = 0;
+       }
        else {
+           expiredate = make_timestamp() + valid_days * 86400L;
            /* print the date when the key expires */
-           tty_printf(_("Key expires at %s\n"), strtimestamp(
-                      add_days_to_timestamp( make_timestamp(), valid_days )));
+           tty_printf(_("Key expires at %s\n"), asctimestamp(expiredate) );
        }
 
-       m_free(answer);
-       answer = tty_get(_("Is this correct (y/n)? "));
-       tty_kill_prompt();
-       if( answer_is_yes(answer) )
+       if( !cpr_enabled()
+            && cpr_get_answer_is_yes(N_("keygen.valid.okay"),
+                                      _("Is this correct (y/n)? ")) )
            break;
     }
     m_free(answer);
-    return valid_days;
+    return expiredate;
 }
 
 
 static int
 has_invalid_email_chars( const char *s )
 {
+    int at_seen=0;
+
     for( ; *s; s++ ) {
        if( *s & 0x80 )
            return 1;
-       if( !strchr("01234567890abcdefghijklmnopqrstuvwxyz_-.@", *s ) )
+       if( *s == '@' )
+           at_seen=1;
+       else if( !at_seen
+                && !strchr("01234567890abcdefghijklmnopqrstuvwxyz_-.+", *s ))
+           return 1;
+       else if( at_seen
+                && !strchr("01234567890abcdefghijklmnopqrstuvwxyz_-.", *s ) )
            return 1;
     }
     return 0;
@@ -549,12 +560,13 @@ has_invalid_email_chars( const char *s )
 
 
 static char *
-ask_user_id()
+ask_user_id( int mode )
 {
     char *answer;
     char *aname, *acomment, *amail, *uid;
 
-    tty_printf( _("\n"
+    if( !mode )
+       tty_printf( _("\n"
 "You need a User-ID to identify your key; the software constructs the user id\n"
 "from Real Name, Comment and Email Address in this form:\n"
 "    \"Heinrich Heine (Der Dichter) <heinrichh@duesseldorf.de>\"\n\n") );
@@ -565,9 +577,9 @@ ask_user_id()
        if( !aname ) {
            for(;;) {
                m_free(aname);
-               aname = tty_get(_("Real name: "));
+               aname = cpr_get(N_("keygen.name"),_("Real name: "));
                trim_spaces(aname);
-               tty_kill_prompt();
+               cpr_kill_prompt();
                if( strpbrk( aname, "<([])>" ) )
                    tty_printf(_("Invalid character in name\n"));
                else if( isdigit(*aname) )
@@ -581,10 +593,10 @@ ask_user_id()
        if( !amail ) {
            for(;;) {
                m_free(amail);
-               amail = tty_get(_("Email address: "));
+               amail = cpr_get(N_("keygen.email"),_("Email address: "));
                trim_spaces(amail);
                strlwr(amail);
-               tty_kill_prompt();
+               cpr_kill_prompt();
                if( !*amail )
                    break;   /* no email address is okay */
                else if( has_invalid_email_chars(amail)
@@ -601,9 +613,9 @@ ask_user_id()
        if( !acomment ) {
            for(;;) {
                m_free(acomment);
-               acomment = tty_get(_("Comment: "));
+               acomment = cpr_get(N_("keygen.comment"),_("Comment: "));
                trim_spaces(acomment);
-               tty_kill_prompt();
+               cpr_kill_prompt();
                if( !*acomment )
                    break;   /* no comment is okay */
                else if( strpbrk( acomment, "()" ) )
@@ -630,28 +642,46 @@ ask_user_id()
        tty_printf(_("You selected this USER-ID:\n    \"%s\"\n\n"), uid);
        /* fixme: add a warning if this user-id already exists */
        for(;;) {
-           answer = tty_get(_("Edit (N)ame, (C)omment, (E)mail or (O)kay? "));
-           tty_kill_prompt();
+           char *ansstr = _("NnCcEeOoQq");
+
+           if( strlen(ansstr) != 10 )
+               BUG();
+           if( cpr_enabled() ) {
+               answer = m_strdup(ansstr+6);
+               answer[1] = 0;
+           }
+           else {
+               answer = cpr_get(N_("keygen.userid.cmd"),_(
+                   "Change (N)ame, (C)omment, (E)mail or (O)kay/(Q)uit? "));
+               cpr_kill_prompt();
+           }
            if( strlen(answer) > 1 )
                ;
-           else if( *answer == 'N' || *answer == 'n' ) {
+           else if( *answer == ansstr[0] || *answer == ansstr[1] ) {
                m_free(aname); aname = NULL;
                break;
            }
-           else if( *answer == 'C' || *answer == 'c' ) {
+           else if( *answer == ansstr[2] || *answer == ansstr[3] ) {
                m_free(acomment); acomment = NULL;
                break;
            }
-           else if( *answer == 'E' || *answer == 'e' ) {
+           else if( *answer == ansstr[4] || *answer == ansstr[5] ) {
                m_free(amail); amail = NULL;
                break;
            }
-           else if( *answer == 'O' || *answer == 'o' ) {
+           else if( *answer == ansstr[6] || *answer == ansstr[7] ) {
                m_free(aname); aname = NULL;
                m_free(acomment); acomment = NULL;
                m_free(amail); amail = NULL;
                break;
            }
+           else if( *answer == ansstr[8] || *answer == ansstr[9] ) {
+               m_free(aname); aname = NULL;
+               m_free(acomment); acomment = NULL;
+               m_free(amail); amail = NULL;
+               m_free(uid); uid = NULL;
+               break;
+           }
            m_free(answer);
        }
        m_free(answer);
@@ -659,6 +689,11 @@ ask_user_id()
            break;
        m_free(uid); uid = NULL;
     }
+    if( uid ) {
+       char *p = native_to_utf8( uid );
+       m_free( uid );
+       uid = p;
+    }
     return uid;
 }
 
@@ -673,9 +708,9 @@ ask_passphrase( STRING2KEY **ret_s2k )
 
     s2k = m_alloc_secure( sizeof *s2k );
     for(;;) {
-       s2k->mode = 1;
-       s2k->hash_algo = DIGEST_ALGO_RMD160;
-       dek = passphrase_to_dek( NULL, CIPHER_ALGO_BLOWFISH, s2k, 2 );
+       s2k->mode = opt.s2k_mode;
+       s2k->hash_algo = opt.s2k_digest_algo;
+       dek = passphrase_to_dek( NULL, opt.s2k_cipher_algo, s2k, 2 );
        if( !dek ) {
            tty_printf(_("passphrase not correctly repeated; try again.\n"));
        }
@@ -685,7 +720,7 @@ ask_passphrase( STRING2KEY **ret_s2k )
            tty_printf(_(
            "You don't want a passphrase - this is probably a *bad* idea!\n"
            "I will do it anyway.  You can change your passphrase at any time,\n"
-           "using this program with the option \"--change-passphrase\"\n\n"));
+           "using this program with the option \"--edit-key\".\n\n"));
            break;
        }
        else
@@ -698,7 +733,7 @@ ask_passphrase( STRING2KEY **ret_s2k )
 
 static int
 do_create( int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root,
-          DEK *dek, STRING2KEY *s2k, PKT_secret_key **sk, int valid_days,
+          DEK *dek, STRING2KEY *s2k, PKT_secret_key **sk, u32 expiredate,
                                                             int v4_packet )
 {
     int rc=0;
@@ -711,11 +746,13 @@ do_create( int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root,
 
     if( algo == PUBKEY_ALGO_ELGAMAL || algo == PUBKEY_ALGO_ELGAMAL_E )
        rc = gen_elg(algo, nbits, pub_root, sec_root, dek, s2k,
-                          sk, valid_days, v4_packet? 4:3 );
+                          sk, expiredate, v4_packet? 4:3 );
     else if( algo == PUBKEY_ALGO_DSA )
-       rc = gen_dsa(nbits, pub_root, sec_root, dek, s2k, sk, valid_days);
+       rc = gen_dsa(nbits, pub_root, sec_root, dek, s2k, sk, expiredate);
     else
        BUG();
+
+  #ifdef ENABLE_COMMENT_PACKETS
     if( !rc ) {
        add_kbnode( pub_root,
                make_comment_node("#created by GNUPG v" VERSION " ("
@@ -724,11 +761,33 @@ do_create( int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root,
                make_comment_node("#created by GNUPG v" VERSION " ("
                                            PRINTABLE_OS_NAME ")"));
     }
+  #endif
     return rc;
 }
 
 
 /****************
+ * Generate a new user id packet, or return NULL if cancelled
+ */
+PKT_user_id *
+generate_user_id()
+{
+    PKT_user_id *uid;
+    char *p;
+    size_t n;
+
+    p = ask_user_id( 1 );
+    if( !p )
+       return NULL;
+    n = strlen(p);
+    uid = m_alloc( sizeof *uid + n - 1 );
+    uid->len = n;
+    strcpy(uid->name, p);
+    return uid;
+}
+
+
+/****************
  * Generate a keypair
  */
 void
@@ -745,7 +804,7 @@ generate_keypair()
     STRING2KEY *s2k;
     int rc;
     int algo;
-    int ndays;
+    u32 expiredate;
     int v4;
     int both = 0;
 
@@ -756,13 +815,17 @@ generate_keypair()
 
     algo = ask_algo( &v4, 0 );
     if( !algo ) {
-       algo = PUBKEY_ALGO_ELGAMAL;
+       algo = PUBKEY_ALGO_ELGAMAL_E;
        both = 1;
        tty_printf(_("DSA keypair will have 1024 bits.\n"));
     }
     nbits = ask_keysize( algo );
-    ndays = ask_valid_days();
-    uid = ask_user_id();
+    expiredate = ask_expiredate();
+    uid = ask_user_id(0);
+    if( !uid ) {
+       log_error(_("Key generation cancelled.\n"));
+       return;
+    }
     dek = ask_passphrase( &s2k );
 
 
@@ -784,10 +847,10 @@ generate_keypair()
 
     if( both )
        rc = do_create( PUBKEY_ALGO_DSA, 1024, pub_root, sec_root,
-                                              dek, s2k, &sk, ndays, 1);
+                                              dek, s2k, &sk, expiredate, 1);
     else
        rc = do_create( algo,           nbits, pub_root, sec_root,
-                                              dek, s2k, &sk, ndays, v4);
+                                              dek, s2k, &sk, expiredate, v4);
     if( !rc )
        write_uid(pub_root, uid );
     if( !rc )
@@ -799,7 +862,7 @@ generate_keypair()
 
     if( both ) {
        rc = do_create( algo, nbits, pub_root, sec_root,
-                                         dek, s2k, NULL, ndays, 1 );
+                                         dek, s2k, NULL, expiredate, 1 );
        if( !rc )
            rc = write_keybinding(pub_root, pub_root, sk);
        if( !rc )
@@ -879,88 +942,30 @@ generate_keypair()
 
 /****************
  * add a new subkey to an existing key.
+ * Returns true if a new key has been generated and put into the keyblocks.
  */
-void
-generate_subkeypair( const char *username )
+int
+generate_subkeypair( KBNODE pub_keyblock, KBNODE sec_keyblock )
 {
-    int rc=0;
-    KBPOS pub_kbpos, sec_kbpos;
-    KBNODE pub_keyblock = NULL;
-    KBNODE sec_keyblock = NULL;
+    int okay=0, rc=0;
     KBNODE node;
     PKT_secret_key *sk = NULL; /* this is the primary sk */
-    u32 keyid[2];
-    int v4, algo, ndays;
+    int v4, algo;
+    u32 expiredate;
     unsigned nbits;
     char *passphrase = NULL;
     DEK *dek = NULL;
     STRING2KEY *s2k = NULL;
 
-    if( opt.batch || opt.answer_yes || opt.answer_no ) {
-       log_error(_("Key generation can only be used in interactive mode\n"));
-       return;
-    }
-
-    /* search the userid */
-    rc = find_secret_keyblock_byname( &sec_kbpos, username );
-    if( rc ) {
-       log_error("user '%s' not found\n", username );
-       goto leave;
-    }
-    rc = read_keyblock( &sec_kbpos, &sec_keyblock );
-    if( rc ) {
-       log_error("error reading the secret key: %s\n", g10_errstr(rc) );
-       goto leave;
-    }
-    /* and the public key */
-    rc = find_keyblock_byname( &pub_kbpos, username );
-    if( rc ) {
-       log_error("user '%s' not found in public ring\n", username );
-       goto leave;
-    }
-    rc = read_keyblock( &pub_kbpos, &pub_keyblock );
-    if( rc ) {
-       log_error("error reading the public key: %s\n", g10_errstr(rc) );
-       goto leave;
-    }
-
-    /* break out the primary key */
+    /* break out the primary secret key */
     node = find_kbnode( sec_keyblock, PKT_SECRET_KEY );
     if( !node ) {
        log_error("Oops; secret key not found anymore!\n");
-       rc = G10ERR_GENERAL;
        goto leave;
     }
 
     /* make a copy of the sk to keep the protected one in the keyblock */
     sk = copy_secret_key( NULL, node->pkt->pkt.secret_key );
-    keyid_from_sk( sk, keyid );
-    /* display primary and all secondary keys */
-    tty_printf("sec  %4u%c/%08lX %s   ",
-             nbits_from_sk( sk ),
-             pubkey_letter( sk->pubkey_algo ),
-             keyid[1], datestr_from_sk(sk) );
-    {
-       size_t n;
-       char *p = get_user_id( keyid, &n );
-       tty_print_string( p, n );
-       m_free(p);
-       tty_printf("\n");
-    }
-    for(node=sec_keyblock; node; node = node->next ) {
-       if( node->pkt->pkttype == PKT_SECRET_SUBKEY ) {
-           PKT_secret_key *subsk = node->pkt->pkt.secret_key;
-           keyid_from_sk( subsk, keyid );
-           tty_printf("sub  %4u%c/%08lX %s\n",
-                     nbits_from_sk( subsk ),
-                     pubkey_letter( subsk->pubkey_algo ),
-                     keyid[1], datestr_from_sk(subsk) );
-       }
-    }
-    tty_printf("\n");
-
-
-
     /* unprotect to get the passphrase */
     switch( is_secret_key_protected( sk ) ) {
       case -1:
@@ -971,7 +976,7 @@ generate_subkeypair( const char *username )
        break;
       default:
        tty_printf("Key is protected.\n");
-       rc = check_secret_key( sk );
+       rc = check_secret_key( sk, 0 );
        if( !rc )
            passphrase = get_last_passphrase();
        break;
@@ -983,47 +988,37 @@ generate_subkeypair( const char *username )
     algo = ask_algo( &v4, 1 );
     assert(algo);
     nbits = ask_keysize( algo );
-    ndays = ask_valid_days();
+    expiredate = ask_expiredate();
+    if( !cpr_enabled() && !cpr_get_answer_is_yes(N_("keygen.sub.okay"),
+                                                 _("Really create? ") ) )
+       goto leave;
 
     if( passphrase ) {
        s2k = m_alloc_secure( sizeof *s2k );
-       s2k->mode = 1;
-       s2k->hash_algo = DIGEST_ALGO_RMD160;
+       s2k->mode = opt.s2k_mode;
+       s2k->hash_algo = opt.s2k_digest_algo;
        set_next_passphrase( passphrase );
-       dek = passphrase_to_dek( NULL, CIPHER_ALGO_BLOWFISH, s2k, 2 );
+       dek = passphrase_to_dek( NULL, opt.s2k_cipher_algo, s2k, 2 );
     }
 
     rc = do_create( algo, nbits, pub_keyblock, sec_keyblock,
-                                     dek, s2k, NULL, ndays, v4 );
+                                     dek, s2k, NULL, expiredate, v4 );
     if( !rc )
        rc = write_keybinding(pub_keyblock, pub_keyblock, sk);
     if( !rc )
        rc = write_keybinding(sec_keyblock, pub_keyblock, sk);
-    /* write back */
-    if( !rc ) {
-       rc = update_keyblock( &pub_kbpos, pub_keyblock );
-       if( rc )
-           log_error("update_public_keyblock failed\n" );
-    }
-    if( !rc ) {
-       rc = update_keyblock( &sec_kbpos, sec_keyblock );
-       if( rc )
-           log_error("update_secret_keyblock failed\n" );
-    }
     if( !rc )
-       tty_printf(_("public and secret subkey created.\n") );
-
+       okay = 1;
 
   leave:
     if( rc )
-       tty_printf(_("Key generation failed: %s\n"), g10_errstr(rc) );
+       log_error(_("Key generation failed: %s\n"), g10_errstr(rc) );
     m_free( passphrase );
     m_free( dek );
     m_free( s2k );
     if( sk ) /* release the copy of the (now unprotected) secret key */
        free_secret_key(sk);
-    release_kbnode( sec_keyblock );
-    release_kbnode( pub_keyblock );
     set_next_passphrase( NULL );
+    return okay;
 }