(parse_signature): Use log_info for messages
authorWerner Koch <wk@gnupg.org>
Sat, 18 Jun 2005 11:49:50 +0000 (11:49 +0000)
committerWerner Koch <wk@gnupg.org>
Sat, 18 Jun 2005 11:49:50 +0000 (11:49 +0000)
about missing timestamp or keyid.  In case we don't use that key
there won't be no further error and thus gpg does not need to
return with an error.

g10/ChangeLog
g10/app-openpgp.c
g10/cardglue.c
g10/parse-packet.c
g10/status.c
g10/status.h

index fe58634..3259eff 100644 (file)
@@ -1,3 +1,10 @@
+2005-06-18  Werner Koch  <wk@g10code.com>
+
+       * parse-packet.c (parse_signature): Use log_info for messages
+       about missing timestamp or keyid.  In case we don't use that key
+       there won't be no further error and thus gpg does not need to
+       return with an error.
+
 2005-06-13  David Shaw  <dshaw@jabberwocky.com>
 
        * keygen.c (save_unprotected_key_to_card): Fix gcc4 warning.
        all of the above.  Export-minimal is the same except it also
        removes all non-selfsigs.  export-unusable-sigs is now a noop.
 
+2005-06-06  Werner Koch  <wk@g10code.com>
+
+       * cardglue.c (open_card): Emit new CARDCTRL status 5 for no reader
+       available.
+
+2005-06-02  Werner Koch  <wk@g10code.com>
+
+       * app-openpgp.c (do_writekey): Typo fix.
+
+       * status.c, status.h: Removed STATUS_BAD_PASSPHRASE_PIN.
+
 2005-06-01  David Shaw  <dshaw@jabberwocky.com>
 
        * signal.c [HAVE_DOSISH_SYSTEM]: Fix unused function warnings on
index f129b16..910dde4 100644 (file)
@@ -1746,7 +1746,7 @@ do_writekey (app_t app, ctrl_t ctrl,
   nbits = rsa_e? count_bits (rsa_e, rsa_e_len) : 0;
   if (nbits < 2 || nbits > 32)
     {
-      log_error (_("RSA public exponent missing or largerr than %d bits\n"),
+      log_error (_("RSA public exponent missing or larger than %d bits\n"),
                  32);
       err = gpg_error (GPG_ERR_BAD_SECKEY);
       goto leave;
index 6ab241d..f9fba68 100644 (file)
@@ -412,6 +412,7 @@ open_card (void)
       slot = apdu_open_reader (default_reader_port);
       if (slot == -1)
         {
+          write_status_text (STATUS_CARDCTRL, "5");
           log_error ("card reader not available\n");
           return NULL;
         }
index 8cc979e..aa349a2 100644 (file)
@@ -1337,7 +1337,7 @@ parse_signature( IOBUF inp, int pkttype, unsigned long pktlen,
        if(p)
          sig->timestamp = buffer_to_u32(p);
        else if(!(sig->pubkey_algo>=100 && sig->pubkey_algo<=110))
-         log_error("signature packet without timestamp\n");
+         log_info ("signature packet without timestamp\n");
 
        p = parse_sig_subpkt2( sig, SIGSUBPKT_ISSUER, NULL );
        if(p)
@@ -1346,7 +1346,7 @@ parse_signature( IOBUF inp, int pkttype, unsigned long pktlen,
            sig->keyid[1] = buffer_to_u32(p+4);
          }
        else if(!(sig->pubkey_algo>=100 && sig->pubkey_algo<=110))
-         log_error("signature packet without keyid\n");
+         log_info ("signature packet without keyid\n");
 
        p=parse_sig_subpkt(sig->hashed,SIGSUBPKT_SIG_EXPIRE,NULL);
        if(p)
index e5eebd2..b14d605 100644 (file)
@@ -113,7 +113,6 @@ get_status_string ( int no )
     case STATUS_ENC_TO  : s = "ENC_TO"; break;
     case STATUS_NODATA  : s = "NODATA"; break;
     case STATUS_BAD_PASSPHRASE : s = "BAD_PASSPHRASE"; break;
-    case STATUS_BAD_PASSPHRASE_PIN: s = "BAD_PASSPHRASE_PIN"; break;
     case STATUS_NO_PUBKEY       : s = "NO_PUBKEY"; break;
     case STATUS_NO_SECKEY       : s = "NO_SECKEY"; break;
     case STATUS_NEED_PASSPHRASE_SYM: s = "NEED_PASSPHRASE_SYM"; break;
index afcfdbc..bf23b34 100644 (file)
 
 /* Extra status codes for certain smartcard operations.  Primary
    useful to double check that change PIN worked as expected.  */
-#define STATUS_BAD_PASSPHRASE_PIN 78
 #define STATUS_SC_OP_FAILURE    79
 #define STATUS_SC_OP_SUCCESS    80