[svn] GnuPG - r4421 - branches/STABLE-BRANCH-1-4/g10

svn author dshaw cvs at cvs.gnupg.org
Wed Jan 31 22:39:27 CET 2007


Author: dshaw
Date: 2007-01-31 22:39:26 +0100 (Wed, 31 Jan 2007)
New Revision: 4421

Modified:
   branches/STABLE-BRANCH-1-4/g10/ChangeLog
   branches/STABLE-BRANCH-1-4/g10/keygen.c
Log:
* keygen.c (do_generate_keypair, proc_parameter_file,
generate_keypair, generate_subkeypair): Pass a timestamp through to
all the gen_xxx functions.


Modified: branches/STABLE-BRANCH-1-4/g10/ChangeLog
===================================================================
--- branches/STABLE-BRANCH-1-4/g10/ChangeLog	2007-01-31 21:17:07 UTC (rev 4420)
+++ branches/STABLE-BRANCH-1-4/g10/ChangeLog	2007-01-31 21:39:26 UTC (rev 4421)
@@ -1,5 +1,9 @@
 2007-01-31  David Shaw  <dshaw at jabberwocky.com>
 
+	* keygen.c (do_generate_keypair, proc_parameter_file,
+	generate_keypair, generate_subkeypair): Pass a timestamp through
+	to all the gen_xxx functions.
+
 	* keyedit.c (sign_uids): Another multiple to single timestamp
 	operation.
 

Modified: branches/STABLE-BRANCH-1-4/g10/keygen.c
===================================================================
--- branches/STABLE-BRANCH-1-4/g10/keygen.c	2007-01-31 21:17:07 UTC (rev 4420)
+++ branches/STABLE-BRANCH-1-4/g10/keygen.c	2007-01-31 21:39:26 UTC (rev 4421)
@@ -60,6 +60,7 @@
   pREVOKER,
   pUSERID,
   pEXPIREDATE,
+  pCREATETIME, /* in n seconds */
   pKEYEXPIRE, /* in n seconds */
   pSUBKEYEXPIRE, /* in n seconds */
   pPASSPHRASE,
@@ -78,6 +79,7 @@
     union {
         DEK *dek;
         STRING2KEY *s2k;
+        u32 create;
         u32 expire;
         unsigned int usage;
         struct revocation_key revkey;
@@ -120,7 +122,8 @@
 static int mdc_available,ks_modify;
 
 static void do_generate_keypair( struct para_data_s *para,
-				 struct output_control_s *outctrl, int card );
+				 struct output_control_s *outctrl,
+				 u32 timestamp, int card );
 static int  write_keyblock( IOBUF out, KBNODE node );
 static int gen_card_key (int algo, int keyno, int is_primary,
                          KBNODE pub_root, KBNODE sec_root,
@@ -1008,7 +1011,8 @@
 
 static int
 gen_elg(int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
-	STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 expireval, int is_subkey)
+	STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 timestamp,
+	u32 expireval, int is_subkey)
 {
     int rc;
     PACKET *pkt;
@@ -1037,11 +1041,11 @@
 
     sk = xmalloc_clear( sizeof *sk );
     pk = xmalloc_clear( sizeof *pk );
-    sk->timestamp = pk->timestamp = make_timestamp();
+    sk->timestamp = pk->timestamp = timestamp;
     sk->version = pk->version = 4;
-    if( expireval ) {
-	sk->expiredate = pk->expiredate = sk->timestamp + expireval;
-    }
+    if( expireval )
+      sk->expiredate = pk->expiredate = sk->timestamp + expireval;
+
     sk->pubkey_algo = pk->pubkey_algo = algo;
 		       pk->pkey[0] = mpi_copy( skey[0] );
 		       pk->pkey[1] = mpi_copy( skey[1] );
@@ -1090,7 +1094,8 @@
  */
 static int
 gen_dsa(unsigned int nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
-	STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 expireval, int is_subkey)
+	STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 timestamp,
+	u32 expireval, int is_subkey)
 {
     int rc;
     PACKET *pkt;
@@ -1152,7 +1157,7 @@
 
     sk = xmalloc_clear( sizeof *sk );
     pk = xmalloc_clear( sizeof *pk );
-    sk->timestamp = pk->timestamp = make_timestamp();
+    sk->timestamp = pk->timestamp = timestamp;
     sk->version = pk->version = 4;
     if( expireval )
       sk->expiredate = pk->expiredate = sk->timestamp + expireval;
@@ -1211,7 +1216,8 @@
  */
 static int
 gen_rsa(int algo, unsigned nbits, KBNODE pub_root, KBNODE sec_root, DEK *dek,
-	STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 expireval, int is_subkey)
+	STRING2KEY *s2k, PKT_secret_key **ret_sk, u32 timestamp,
+	u32 expireval, int is_subkey)
 {
     int rc;
     PACKET *pkt;
@@ -1240,11 +1246,11 @@
 
     sk = xmalloc_clear( sizeof *sk );
     pk = xmalloc_clear( sizeof *pk );
-    sk->timestamp = pk->timestamp = make_timestamp();
+    sk->timestamp = pk->timestamp = timestamp;
     sk->version = pk->version = 4;
-    if( expireval ) {
-	sk->expiredate = pk->expiredate = sk->timestamp + expireval;
-    }
+    if( expireval )
+      sk->expiredate = pk->expiredate = sk->timestamp + expireval;
+
     sk->pubkey_algo = pk->pubkey_algo = algo;
 		       pk->pkey[0] = mpi_copy( skey[0] );
 		       pk->pkey[1] = mpi_copy( skey[1] );
@@ -1910,8 +1916,8 @@
 
 static int
 do_create( int algo, unsigned int nbits, KBNODE pub_root, KBNODE sec_root,
-	   DEK *dek, STRING2KEY *s2k, PKT_secret_key **sk, u32 expiredate,
-	   int is_subkey )
+	   DEK *dek, STRING2KEY *s2k, PKT_secret_key **sk, u32 timestamp,
+	   u32 expiredate, int is_subkey )
 {
   int rc=0;
 
@@ -1923,14 +1929,14 @@
 "generator a better chance to gain enough entropy.\n") );
 
   if( algo == PUBKEY_ALGO_ELGAMAL_E )
-    rc = gen_elg(algo, nbits, pub_root, sec_root, dek, s2k, sk, expiredate,
-		 is_subkey);
+    rc = gen_elg(algo, nbits, pub_root, sec_root, dek, s2k, sk, timestamp,
+		 expiredate, is_subkey);
   else if( algo == PUBKEY_ALGO_DSA )
-    rc = gen_dsa(nbits, pub_root, sec_root, dek, s2k, sk, expiredate,
-		 is_subkey);
+    rc = gen_dsa(nbits, pub_root, sec_root, dek, s2k, sk, timestamp,
+		 expiredate, is_subkey);
   else if( algo == PUBKEY_ALGO_RSA )
-    rc = gen_rsa(algo, nbits, pub_root, sec_root, dek, s2k, sk, expiredate,
-		 is_subkey);
+    rc = gen_rsa(algo, nbits, pub_root, sec_root, dek, s2k, sk, timestamp,
+		 expiredate, is_subkey);
   else
     BUG();
 
@@ -2108,6 +2114,8 @@
 	return r->u.expire;
     if( r->key == pKEYUSAGE || r->key == pSUBKEYUSAGE )
 	return r->u.usage;
+    if( r->key == pCREATETIME )
+	return r->u.create;
 
     return (unsigned int)strtoul( r->u.value, NULL, 10 );
 }
@@ -2148,7 +2156,14 @@
   size_t n;
   char *p;
   int have_user_id=0,err,algo;
+  u32 timestamp;
 
+  /* If we were told a creation time from outside, use it.  Otherwise
+     look at the clock. */
+  timestamp=get_parameter_u32( para, pCREATETIME );
+  if(!timestamp)
+    timestamp=make_timestamp();
+
   /* Check that we have all required parameters. */
   r = get_parameter( para, pKEYTYPE );
   if(r)
@@ -2320,7 +2335,7 @@
     return -1;
   }
 
-  do_generate_keypair( para, outctrl, card );
+  do_generate_keypair( para, outctrl, timestamp, card );
   return 0;
 }
 
@@ -2550,7 +2565,7 @@
   int algo;
   unsigned int use;
   int both = 0;
-  u32 expire;
+  u32 timestamp,expire;
   struct para_data_s *para = NULL;
   struct para_data_s *r;
   struct output_control_s outctrl;
@@ -2570,6 +2585,13 @@
       return;
     }
 
+  timestamp=make_timestamp();
+  r = xmalloc_clear( sizeof *r );
+  r->key = pCREATETIME;
+  r->u.create = timestamp;
+  r->next = para;
+  para = r;
+
   if (card_serialno)
     {
 #ifdef ENABLE_CARD_SUPPORT
@@ -2821,8 +2843,8 @@
 }
 
 static void
-do_generate_keypair( struct para_data_s *para,
-		     struct output_control_s *outctrl, int card )
+do_generate_keypair( struct para_data_s *para,struct output_control_s *outctrl,
+		     u32 timestamp,int card )
 {
     KBNODE pub_root = NULL;
     KBNODE sec_root = NULL;
@@ -2925,6 +2947,7 @@
                         get_parameter_dek( para, pPASSPHRASE_DEK ),
                         get_parameter_s2k( para, pPASSPHRASE_S2K ),
                         &pri_sk,
+			timestamp,
                         get_parameter_u32( para, pKEYEXPIRE ), 0 );
       }
     else
@@ -2984,6 +3007,7 @@
                             get_parameter_dek( para, pPASSPHRASE_DEK ),
                             get_parameter_s2k( para, pPASSPHRASE_S2K ),
                             &sub_sk,
+			    timestamp,
                             get_parameter_u32( para, pSUBKEYEXPIRE ), 1 );
           }
         else
@@ -3145,7 +3169,7 @@
     char *passphrase = NULL;
     DEK *dek = NULL;
     STRING2KEY *s2k = NULL;
-    u32 cur_time;
+    u32 timestamp;
     int ask_pass = 0;
 
     /* break out the primary secret key */
@@ -3158,9 +3182,9 @@
     /* make a copy of the sk to keep the protected one in the keyblock */
     pri_sk = copy_secret_key( NULL, node->pkt->pkt.secret_key );
 
-    cur_time = make_timestamp();
-    if( pri_sk->timestamp > cur_time ) {
-	ulong d = pri_sk->timestamp - cur_time;
+    timestamp = make_timestamp();
+    if( pri_sk->timestamp > timestamp ) {
+	ulong d = pri_sk->timestamp - timestamp;
 	log_info( d==1 ? _("key has been created %lu second "
 			   "in future (time warp or clock problem)\n")
 		       : _("key has been created %lu seconds "
@@ -3226,7 +3250,7 @@
     }
 
     rc = do_create( algo, nbits, pub_keyblock, sec_keyblock,
-		    dek, s2k, &sub_sk, expire, 1 );
+		    dek, s2k, &sub_sk, timestamp, expire, 1 );
     if( !rc )
 	rc = write_keybinding(pub_keyblock, pub_keyblock, pri_sk, sub_sk, use);
     if( !rc )
@@ -3263,9 +3287,8 @@
   PKT_secret_key *pri_sk = NULL, *sub_sk;
   int algo;
   unsigned int use;
-  u32 expire;
+  u32 timestamp,expire;
   char *passphrase = NULL;
-  u32 cur_time;
   struct para_data_s *para = NULL;
 
   assert (keyno >= 1 && keyno <= 3);
@@ -3285,10 +3308,10 @@
   /* Make a copy of the sk to keep the protected one in the keyblock */
   pri_sk = copy_secret_key (NULL, node->pkt->pkt.secret_key);
 
-  cur_time = make_timestamp();
-  if (pri_sk->timestamp > cur_time)
+  timestamp = make_timestamp();
+  if (pri_sk->timestamp > timestamp)
     {
-      ulong d = pri_sk->timestamp - cur_time;
+      ulong d = pri_sk->timestamp - timestamp;
       log_info (d==1 ? _("key has been created %lu second "
                          "in future (time warp or clock problem)\n")
                      : _("key has been created %lu seconds "




More information about the Gnupg-commits mailing list