doc/ChangeLog:
authorMoritz Schulte <mo@g10code.com>
Sat, 8 Oct 2005 16:16:47 +0000 (16:16 +0000)
committerMoritz Schulte <mo@g10code.com>
Sat, 8 Oct 2005 16:16:47 +0000 (16:16 +0000)
2005-10-08  Moritz Schulte  <moritz@g10code.com>

* poldi.texi: Improve documentation, add application-specific
information.

src/ctrl/ChangeLog:

2005-10-08  Moritz Schulte  <moritz@g10code.com>

* poldi-ctrl.c: Remove `fake-wait-for-card'-feature.

src/pam/ChangeLog:

2005-10-08  Moritz Schulte  <moritz@g10code.com>

* pam_poldi.c: Remove `fake-wait-for-card'-feature.

doc/ChangeLog
doc/poldi.texi
src/ctrl/ChangeLog
src/ctrl/poldi-ctrl.c
src/pam/ChangeLog
src/pam/pam_poldi.c

index e69de29..2ca6020 100644 (file)
@@ -0,0 +1,4 @@
+2005-10-08  Moritz Schulte  <moritz@g10code.com>
+
+       * poldi.texi: Improve documentation, add application-specific
+       information.
index e006e7d..0b3e748 100644 (file)
@@ -106,7 +106,7 @@ Introduction
 
 Poldi is package consisting of a PAM module (pam_poldi) and a CLI
 program (poldi-ctrl) , which implement authentication via OpenPGP
-smartcards (http://www.g10code.de/p-card.html). FIXME footnote.
+smartcards (http://www.g10code.de/p-card.html).
 
 Poldi expects a 1:1 mapping between Unix accounts and smartcards; this
 mapping can be established with the help of poldi-ctrl.
@@ -124,8 +124,8 @@ Details on the build procedure can be found in the file ``INSTALL''.
 
 FIXME (installing).
 
-Poldi requires some directories beneath `${sysconfdir}/poldi', which is
-usually equal to `${prefix}/etc/poldi'.
+Poldi requires some directories beneath `SYSCONFDIR/poldi', which is
+usually equal to `PREFIX/etc/poldi'.
 
 The directories can be created manually or through the
 `install-conf-skeleton' make target.
@@ -162,7 +162,6 @@ Poldi supports the following options:
 @item disable-ccid
 @item disable-opensc
 @item debug-ccid-driver
-@item fake-wait-for-card
 @item logfile
 @end table
 
index 6af2d62..76bf490 100644 (file)
@@ -1,3 +1,7 @@
+2005-10-08  Moritz Schulte  <moritz@g10code.com>
+
+       * poldi-ctrl.c: Remove `fake-wait-for-card'-feature.
+
 2005-10-01  Moritz Schulte  <moritz@g10code.com>
 
        * poldi-ctrl.c (struct poldi_ctrl_opt): New member: WAIT_TIMEOUT.
index 3d66aaa..b47e109 100644 (file)
@@ -53,7 +53,6 @@ struct poldi_ctrl_opt
   char *config_file;
   char *account;
   char *serialno;
-  int fake_wait_for_card;
   int require_card_switch;
   int cmd_test;
   int cmd_dump;
@@ -89,7 +88,6 @@ struct poldi_ctrl_opt poldi_ctrl_opt =
     0,
     0,
     0,
-    0,
     0
   };
 
@@ -113,7 +111,6 @@ enum arg_opt_ids
     arg_disable_opensc,
     arg_debug,
     arg_debug_ccid_driver,
-    arg_fake_wait_for_card,
     arg_require_card_switch,
     arg_wait_timeout
   };
@@ -160,8 +157,6 @@ static ARGPARSE_OPTS arg_opts[] =
     { arg_disable_opensc,
       "disable-opensc", 0, "do not use the OpenSC layer"       },
 #endif
-    { arg_fake_wait_for_card,
-      "fake-wait-for-card", 0, "Fake wait-for-card feature"    },
     { arg_require_card_switch,
       "require-card-switch", 0, "Require re-insertion of card" },
     { arg_wait_timeout,
@@ -304,11 +299,6 @@ poldi_ctrl_options_cb (ARGPARSE_ARGS *parg, void *opaque)
        poldi_ctrl_opt.debug_ccid_driver = 1;
       break;
 
-    case arg_fake_wait_for_card:
-      if (parsing_stage)
-       poldi_ctrl_opt.fake_wait_for_card = 1;
-      break;
-
     case arg_require_card_switch:
       if (parsing_stage)
        poldi_ctrl_opt.require_card_switch = 1;
@@ -364,16 +354,8 @@ cmd_test (void)
   if (err)
     goto out;
 
-  if (poldi_ctrl_opt.fake_wait_for_card)
-    {
-      printf ("Press ENTER when card is available...\n");
-      getchar ();
-    }
-  else
-    printf ("Waiting for card...\n");
-  err = card_init (slot,
-                  !poldi_ctrl_opt.fake_wait_for_card,
-                  poldi_ctrl_opt.wait_timeout,
+  printf ("Waiting for card...\n");
+  err = card_init (slot, 1, poldi_ctrl_opt.wait_timeout,
                   poldi_ctrl_opt.require_card_switch);
   if (err)
     goto out;
index e38ffa4..bb3cf50 100644 (file)
@@ -1,5 +1,7 @@
 2005-10-08  Moritz Schulte  <moritz@g10code.com>
 
+       * pam_poldi.c: Remove `fake-wait-for-card'-feature.
+
        * Makefile.am: Drop ugly install-pam-module mechanism, implement
        install-exec-local and uninstall-local target.
 
index 62bb079..432910c 100644 (file)
@@ -63,7 +63,6 @@ struct pam_poldi_opt
   int disable_opensc;  /* Disable the use of the OpenSC framework. */
   int disable_ccid;    /* Disable the use of the internal CCID driver. */
   int debug_ccid_driver;       /* Debug the internal CCID driver.  */
-  int fake_wait_for_card;
   int require_card_switch;
   const char *logfile;
   unsigned int wait_timeout;
@@ -82,7 +81,6 @@ struct pam_poldi_opt pam_poldi_opt =
     0,
     0,
     0,
-    0,
     NULL,
     0
   };
@@ -100,7 +98,6 @@ enum arg_opt_ids
     arg_disable_opensc,
     arg_disable_ccid,
     arg_debug_ccid_driver,
-    arg_fake_wait_for_card,
     arg_require_card_switch,
     arg_logfile,
     arg_wait_timeout
@@ -129,8 +126,6 @@ static ARGPARSE_OPTS arg_opts[] =
     { arg_disable_opensc,
       "disable-opensc", 0, "do not use the OpenSC layer" },
 #endif
-    { arg_fake_wait_for_card,
-      "fake-wait-for-card", 0, "fake wait-for-card-feature" },
     { arg_require_card_switch,
       "require-card-switch", 0, "Require re-insertion of card" },
     { arg_logfile,
@@ -179,10 +174,6 @@ pam_poldi_options_cb (ARGPARSE_ARGS *parg, void *opaque)
       pam_poldi_opt.debug_ccid_driver = 1;
       break;
 
-    case arg_fake_wait_for_card:
-      pam_poldi_opt.fake_wait_for_card = 1;
-      break;
-
     case arg_require_card_switch:
       pam_poldi_opt.require_card_switch = 1;
       break;
@@ -365,30 +356,17 @@ lookup_key (const char *username, gcry_sexp_t *key)
 }
 
 static gpg_error_t
-wait_for_card (int slot, int fake, int require_card_switch,
+wait_for_card (int slot, int require_card_switch,
               const struct pam_conv *conv, char **serialno)
 {
   char *serialno_new;
-  unsigned int timeout;
   gpg_error_t err;
-  int wait;
 
-  if (fake)
-    {
-      timeout = 0;
-      wait = 0;
-      err = ask_user (conv, "Press ENTER when card is available ...", NULL);
-    }
-  else
-    {
-      timeout = pam_poldi_opt.wait_timeout;
-      wait = 1; 
-      err = tell_user (conv, "Insert card ...");
-    }
+  err = tell_user (conv, "Insert card ...");
   if (err)
     goto out;
 
-  err = card_init (slot, wait, timeout, require_card_switch);
+  err = card_init (slot, 1, pam_poldi_opt.wait_timeout, require_card_switch);
   if (err)
     {
       if (gpg_err_code (err) == GPG_ERR_CARD_NOT_PRESENT)
@@ -519,8 +497,8 @@ pam_sm_authenticate (pam_handle_t *pam_handle, int flags, int argc, const char *
 
       /* Got key.  */
 
-      err = wait_for_card (slot, pam_poldi_opt.fake_wait_for_card,
-                          pam_poldi_opt.require_card_switch, conv, &serialno);
+      err = wait_for_card (slot, pam_poldi_opt.require_card_switch,
+                          conv, &serialno);
       if (err)
        goto out;
 
@@ -545,8 +523,8 @@ pam_sm_authenticate (pam_handle_t *pam_handle, int flags, int argc, const char *
     }
   else
     {
-      err = wait_for_card (slot, pam_poldi_opt.fake_wait_for_card,
-                          pam_poldi_opt.require_card_switch, conv, &serialno);
+      err = wait_for_card (slot, pam_poldi_opt.require_card_switch,
+                          conv, &serialno);
       if (err)
        goto out;