A lot of cleanups as well as minor API changes.
[libgcrypt.git] / cipher / pubkey.c
index 3584c61..228b0d0 100644 (file)
@@ -1,5 +1,6 @@
 /* pubkey.c  - pubkey dispatcher
- * Copyright (C) 1998,1999,2000,2002,2003 Free Software Foundation, Inc.
+ * Copyright (C) 1998, 1999, 2000, 2002, 2003,
+ *               2005 Free Software Foundation, Inc.
  *
  * This file is part of Libgcrypt.
  *
 #include "cipher.h"
 #include "ath.h"
 
-static gpg_err_code_t pubkey_decrypt (int algo, MPI *result, MPI *data, MPI *skey, int flags);
-static gpg_err_code_t pubkey_sign (int algo, MPI *resarr, MPI hash, MPI *skey);
-static gpg_err_code_t pubkey_verify (int algo, MPI hash, MPI *data, MPI *pkey,
-                                    int (*cmp) (void *, MPI), void *opaque);
+static gcry_err_code_t pubkey_decrypt (int algo, gcry_mpi_t *result,
+                                       gcry_mpi_t *data, gcry_mpi_t *skey,
+                                       int flags);
+static gcry_err_code_t pubkey_sign (int algo, gcry_mpi_t *resarr,
+                                    gcry_mpi_t hash, gcry_mpi_t *skey);
+static gcry_err_code_t pubkey_verify (int algo, gcry_mpi_t hash,
+                                      gcry_mpi_t *data, gcry_mpi_t *pkey,
+                                    int (*cmp) (void *, gcry_mpi_t),
+                                      void *opaque);
 
 /* This is the list of the default public-key ciphers included in
    libgcrypt.  */
-static struct
+static struct pubkey_table_entry
 {
-  GcryPubkeySpec *pubkey;
+  gcry_pk_spec_t *pubkey;
+  unsigned int algorithm;
 } pubkey_table[] =
   {
 #if USE_RSA
-    { &pubkey_spec_rsa },
+    { &_gcry_pubkey_spec_rsa, GCRY_PK_RSA   },
 #endif
 #if USE_ELGAMAL
-    { &pubkey_spec_elg },
+    { &_gcry_pubkey_spec_elg, GCRY_PK_ELG   },
+    { &_gcry_pubkey_spec_elg, GCRY_PK_ELG_E },
 #endif
 #if USE_DSA
-    { &pubkey_spec_dsa },
+    { &_gcry_pubkey_spec_dsa, GCRY_PK_DSA   },
 #endif
-    { NULL },
+    { NULL, 0 },
   };
 
 /* List of registered ciphers.  */
-static GcryModule *pubkeys_registered;
+static gcry_module_t pubkeys_registered;
 
 /* This is the lock protecting PUBKEYS_REGISTERED.  */
-static ath_mutex_t pubkeys_registered_lock;
+static ath_mutex_t pubkeys_registered_lock = ATH_MUTEX_INITIALIZER;;
 
 /* Flag to check wether the default pubkeys have already been
    registered.  */
@@ -71,7 +79,7 @@ static int default_pubkeys_registered;
       ath_mutex_lock (&pubkeys_registered_lock);   \
       if (! default_pubkeys_registered)            \
         {                                          \
-          gcry_pubkey_register_default ();         \
+          gcry_pk_register_default ();         \
           default_pubkeys_registered = 1;          \
         }                                          \
       ath_mutex_unlock (&pubkeys_registered_lock); \
@@ -81,63 +89,89 @@ static int default_pubkeys_registered;
 /* These dummy functions are used in case a cipher implementation
    refuses to provide it's own functions.  */
 
-static gpg_err_code_t
-dummy_generate (int id, unsigned int nbits, unsigned long dummy,
-                MPI *skey, MPI **retfactors)
+static gcry_err_code_t
+dummy_generate (int algorithm, unsigned int nbits, unsigned long dummy,
+                gcry_mpi_t *skey, gcry_mpi_t **retfactors)
 {
-  log_bug ("no generate() for %d\n", id);
-  return GPG_ERR_PUBKEY_ALGO;
+  (void)algorithm;
+  (void)nbits;
+  (void)dummy;
+  (void)skey;
+  (void)retfactors;
+  return GPG_ERR_NOT_IMPLEMENTED;
 }
 
-static gpg_err_code_t
-dummy_check_secret_key (int id, MPI *skey)
+static gcry_err_code_t
+dummy_check_secret_key (int algorithm, gcry_mpi_t *skey)
 {
-  log_bug ("no check_secret_key() for %d\n", id);
-  return GPG_ERR_PUBKEY_ALGO;
+  (void)algorithm;
+  (void)skey;
+  return GPG_ERR_NOT_IMPLEMENTED;
 }
 
-static gpg_err_code_t
-dummy_encrypt (int id, MPI *resarr, MPI data, MPI *pkey, int flags)
+static gcry_err_code_t
+dummy_encrypt (int algorithm, gcry_mpi_t *resarr, gcry_mpi_t data,
+               gcry_mpi_t *pkey, int flags)
 {
-  log_bug ("no encrypt() for %d\n", id);
-  return GPG_ERR_PUBKEY_ALGO;
+  (void)algorithm;
+  (void)resarr;
+  (void)data;
+  (void)pkey;
+  (void)flags;
+  return GPG_ERR_NOT_IMPLEMENTED;
 }
 
-static gpg_err_code_t
-dummy_decrypt (int id, MPI *result, MPI *data, MPI *skey, int flags)
+static gcry_err_code_t
+dummy_decrypt (int algorithm, gcry_mpi_t *result, gcry_mpi_t *data,
+               gcry_mpi_t *skey, int flags)
 {
-  log_bug ("no decrypt() for %d\n", id);
-  return GPG_ERR_PUBKEY_ALGO;
+  (void)algorithm;
+  (void)result;
+  (void)data;
+  (void)skey;
+  (void)flags;
+  return GPG_ERR_NOT_IMPLEMENTED;
 }
 
-static gpg_err_code_t
-dummy_sign (int id, MPI *resarr, MPI data, MPI *skey)
+static gcry_err_code_t
+dummy_sign (int algorithm, gcry_mpi_t *resarr, gcry_mpi_t data,
+            gcry_mpi_t *skey)
 {
-  log_bug ("no sign() for %d\n", id);
-  return GPG_ERR_PUBKEY_ALGO;
+  (void)algorithm;
+  (void)resarr;
+  (void)data;
+  (void)skey;
+  return GPG_ERR_NOT_IMPLEMENTED;
 }
 
-static gpg_err_code_t
-dummy_verify (int id, MPI hash, MPI *data, MPI *pkey,
-             int (*cmp) (void *, MPI), void *opaquev)
+static gcry_err_code_t
+dummy_verify (int algorithm, gcry_mpi_t hash, gcry_mpi_t *data,
+              gcry_mpi_t *pkey,
+             int (*cmp) (void *, gcry_mpi_t), void *opaquev)
 {
-  log_bug ("no verify() for %d\n", id);
-  return GPG_ERR_PUBKEY_ALGO;
+  (void)algorithm;
+  (void)hash;
+  (void)data;
+  (void)pkey;
+  (void)cmp;
+  (void)opaquev;
+  return GPG_ERR_NOT_IMPLEMENTED;
 }
 
 static unsigned
-dummy_get_nbits (int id, MPI *pkey)
+dummy_get_nbits (int algorithm, gcry_mpi_t *pkey)
 {
-  log_bug ("no get_nbits() for %d\n", id);
+  (void)algorithm;
+  (void)pkey;
   return 0;
 }
 
 /* Internal function.  Register all the pubkeys included in
    PUBKEY_TABLE.  Returns zero on success or an error code.  */
 static void
-gcry_pubkey_register_default (void)
+gcry_pk_register_default (void)
 {
-  gpg_err_code_t err = 0;
+  gcry_err_code_t err = 0;
   int i;
   
   for (i = 0; (! err) && pubkey_table[i].pubkey; i++)
@@ -153,8 +187,9 @@ gcry_pubkey_register_default (void)
       pubkey_use_dummy (sign);
       pubkey_use_dummy (verify);
       pubkey_use_dummy (get_nbits);
-
+#undef pubkey_use_dummy
       err = _gcry_module_add (&pubkeys_registered,
+                             pubkey_table[i].algorithm,
                              (void *) pubkey_table[i].pubkey, NULL);
     }
 
@@ -164,99 +199,60 @@ gcry_pubkey_register_default (void)
 
 /* Internal callback function.  Used via _gcry_module_lookup.  */
 static int
-gcry_pubkey_lookup_func_id (void *spec, void *data)
-{
-  GcryPubkeySpec *pubkey = (GcryPubkeySpec *) spec;
-  int id = *((int *) data);
-
-  return (pubkey->id == id);
-}
-
-/* Internal callback function.  Used via _gcry_module_lookup.  */
-static int
-gcry_pubkey_lookup_func_name (void *spec, void *data)
+gcry_pk_lookup_func_name (void *spec, void *data)
 {
-  GcryPubkeySpec *pubkey = (GcryPubkeySpec *) spec;
+  gcry_pk_spec_t *pubkey = (gcry_pk_spec_t *) spec;
   char *name = (char *) data;
-  char **sexp_names = pubkey->sexp_names;
+  const char **aliases = pubkey->aliases;
   int ret = stricmp (name, pubkey->name);
 
-  while (ret && *sexp_names)
-    ret = stricmp (name, *sexp_names++);
+  while (ret && *aliases)
+    ret = stricmp (name, *aliases++);
 
   return ! ret;
 }
 
-/* Internal function.  Lookup a pubkey entry by it's ID.  */
-static GcryModule *
-gcry_pubkey_lookup_id (int id)
-{
-  GcryModule *pubkey;
-
-  pubkey = _gcry_module_lookup (pubkeys_registered, (void *) &id,
-                               gcry_pubkey_lookup_func_id);
-
-  return pubkey;
-}
-
 /* Internal function.  Lookup a pubkey entry by it's name.  */
-static GcryModule *
-gcry_pubkey_lookup_name (const char *name)
+static gcry_module_t 
+gcry_pk_lookup_name (const char *name)
 {
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
 
   pubkey = _gcry_module_lookup (pubkeys_registered, (void *) name,
-                               gcry_pubkey_lookup_func_name);
+                               gcry_pk_lookup_func_name);
 
   return pubkey;
 }
 
-/* Return a new, unused pubkey ID for a user-provided pubkey
-   implementation.  */
-static int
-gcry_pubkey_id_new (void)
-{
-  int id, id_start = 500, id_end = 600;        /* FIXME.  */
-  
-  for (id = id_start; id < id_end; id++)
-    if (! gcry_pubkey_lookup_id (id))
-      return id;
-
-  return 0;
-}
-
-/* Public function.  Register a provided PUBKEY.  Returns zero on
-   success, in which case the chosen pubkey ID has been stored in
-   PUBKEY, or an error code.  */
-gpg_error_t
-gcry_pubkey_register (GcryPubkeySpec *pubkey, GcryModule **module)
+/* Register a new pubkey module whose specification can be found in
+   PUBKEY.  On success, a new algorithm ID is stored in ALGORITHM_ID
+   and a pointer representhing this module is stored in MODULE.  */
+gcry_error_t
+gcry_pk_register (gcry_pk_spec_t *pubkey,
+                 unsigned int *algorithm_id,
+                 gcry_module_t *module)
 {
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
-  int id;
-  GcryModule *mod;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_module_t mod;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  id = gcry_pubkey_id_new ();
-  if (! id)
-    err = GPG_ERR_INTERNAL;    /* FIXME.  */
-  else
-    {
-      pubkey->id = id;
-      err = _gcry_module_add (&pubkeys_registered, (void *) pubkey,
-                             &mod);
-    }
+  err = _gcry_module_add (&pubkeys_registered, 0,
+                         (void *) pubkey, &mod);
   ath_mutex_unlock (&pubkeys_registered_lock);
 
   if (! err)
-    *module = mod;
+    {
+      *module = mod;
+      *algorithm_id = mod->mod_id;
+    }
 
   return err;
 }
 
-/* Public function.  Unregister the pubkey identified by ID, which
-   must have been registered with gcry_pubkey_register.  */
+/* Unregister the pubkey identified by ID, which must have been
+   registered with gcry_pk_register.  */
 void
-gcry_pubkey_unregister (GcryModule *module)
+gcry_pk_unregister (gcry_module_t module)
 {
   ath_mutex_lock (&pubkeys_registered_lock);
   _gcry_module_release (module);
@@ -264,7 +260,7 @@ gcry_pubkey_unregister (GcryModule *module)
 }
 
 static void
-release_mpi_array (MPI *array)
+release_mpi_array (gcry_mpi_t *array)
 {
   for (; *array; array++)
     {
@@ -279,42 +275,75 @@ release_mpi_array (MPI *array)
 int
 gcry_pk_map_name (const char *string)
 {
-  GcryModule *pubkey;
-  int id = 0;
+  gcry_module_t pubkey;
+  int algorithm = 0;
+
+  if (!string)
+    return 0;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_name (string);
+  pubkey = gcry_pk_lookup_name (string);
   if (pubkey)
     {
-      id = ((GcryPubkeySpec *) pubkey->spec)->id;
+      algorithm = pubkey->mod_id;
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
 
-  return id;
+  return algorithm;
 }
 
 
-/****************
- * Map a pubkey algo to a string
- */
+/* Map the public key algorithm whose ID is contained in ALGORITHM to
+   a string representation of the algorithm name.  For unknown
  algorithm IDs this functions returns "?". */
 const char *
-gcry_pk_algo_name (int id)
+gcry_pk_algo_name (int algorithm)
 {
-  const char *name = NULL;
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
+  const char *name;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      name = ((GcryPubkeySpec *) pubkey->spec)->name;
+      name = ((gcry_pk_spec_t *) pubkey->spec)->name;
       _gcry_module_release (pubkey);
     }
+  else
+    name = "?";
+  ath_mutex_unlock (&pubkeys_registered_lock);
+
+  return name;
+}
+
+
+/* A special version of gcry_pk_algo name to return the first aliased
+   name of the algorithm.  This is required to adhere to the spki
+   specs where the algorithm names are lowercase. */
+const char *
+_gcry_pk_aliased_algo_name (int algorithm)
+{
+  const char *name = NULL;
+  gcry_module_t module;
+
+  REGISTER_DEFAULT_PUBKEYS;
+
+  ath_mutex_lock (&pubkeys_registered_lock);
+  module = _gcry_module_lookup_id (pubkeys_registered, algorithm);
+  if (module)
+    {
+      gcry_pk_spec_t *pubkey = (gcry_pk_spec_t *) module->spec;
+
+      name = pubkey->aliases? *pubkey->aliases : NULL;
+      if (!name || !*name)
+        name = pubkey->name;
+      _gcry_module_release (module);
+    }
   ath_mutex_unlock (&pubkeys_registered_lock);
 
   return name;
@@ -322,12 +351,12 @@ gcry_pk_algo_name (int id)
 
 
 static void
-disable_pubkey_algo (int id)
+disable_pubkey_algo (int algorithm)
 {
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
       if (! (pubkey-> flags & FLAG_MODULE_DISABLED))
@@ -339,22 +368,22 @@ disable_pubkey_algo (int id)
 
 
 /****************
- * a use of 0 means: don't care
+ * A USE of 0 means: don't care.
  */
-static gpg_err_code_t
-check_pubkey_algo (int id, unsigned use)
+static gcry_err_code_t
+check_pubkey_algo (int algorithm, unsigned use)
 {
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
-  GcryPubkeySpec *pubkey;
-  GcryModule *module;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_pk_spec_t *pubkey;
+  gcry_module_t module;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  module = gcry_pubkey_lookup_id (id);
+  module = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (module)
     {
-      pubkey = (GcryPubkeySpec *) module->spec;
+      pubkey = (gcry_pk_spec_t *) module->spec;
 
       if (((use & GCRY_PK_USAGE_SIGN)
           && (! (pubkey->use & GCRY_PK_USAGE_SIGN)))
@@ -377,18 +406,18 @@ check_pubkey_algo (int id, unsigned use)
  * Return the number of public key material numbers
  */
 static int
-pubkey_get_npkey (int id)
+pubkey_get_npkey (int algorithm)
 {
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
   int npkey = 0;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      npkey = strlen (((GcryPubkeySpec *) pubkey->spec)->elements_pkey);
+      npkey = strlen (((gcry_pk_spec_t *) pubkey->spec)->elements_pkey);
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
@@ -400,18 +429,18 @@ pubkey_get_npkey (int id)
  * Return the number of secret key material numbers
  */
 static int
-pubkey_get_nskey (int id)
+pubkey_get_nskey (int algorithm)
 {
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
   int nskey = 0;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      nskey = strlen (((GcryPubkeySpec *) pubkey->spec)->elements_skey);
+      nskey = strlen (((gcry_pk_spec_t *) pubkey->spec)->elements_skey);
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
@@ -423,18 +452,18 @@ pubkey_get_nskey (int id)
  * Return the number of signature material numbers
  */
 static int
-pubkey_get_nsig (int id)
+pubkey_get_nsig (int algorithm)
 {
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
   int nsig = 0;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      nsig = strlen (((GcryPubkeySpec *) pubkey->spec)->elements_sig);
+      nsig = strlen (((gcry_pk_spec_t *) pubkey->spec)->elements_sig);
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
@@ -446,18 +475,18 @@ pubkey_get_nsig (int id)
  * Return the number of encryption material numbers
  */
 static int
-pubkey_get_nenc (int id)
+pubkey_get_nenc (int algorithm)
 {
-  GcryModule *pubkey;
+  gcry_module_t pubkey;
   int nenc = 0;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      nenc = strlen (((GcryPubkeySpec *) pubkey->spec)->elements_enc);
+      nenc = strlen (((gcry_pk_spec_t *) pubkey->spec)->elements_enc);
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
@@ -466,21 +495,27 @@ pubkey_get_nenc (int id)
 }
 
 
-static gpg_err_code_t
-pubkey_generate (int id, unsigned int nbits, unsigned long use_e,
-                 MPI *skey, MPI **retfactors)
+static gcry_err_code_t
+pubkey_generate (int algorithm, unsigned int nbits, unsigned int qbits,
+                 unsigned long use_e,
+                 gcry_mpi_t *skey, gcry_mpi_t **retfactors)
 {
-  gpg_err_code_t err = GPG_ERR_PUBKEY_ALGO;
-  GcryModule *pubkey;
+  gcry_err_code_t err = GPG_ERR_PUBKEY_ALGO;
+  gcry_module_t pubkey;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      err = (*((GcryPubkeySpec *) pubkey->spec)->generate) (id, nbits, use_e, skey,
-                                                           retfactors);
+      /* Hack to pass QBITS to the DSA generation.  */
+      if (qbits && pubkey->spec == &_gcry_pubkey_spec_dsa)
+        err = _gcry_dsa_generate2
+          (algorithm, nbits, qbits, 0, skey, retfactors);
+      else
+        err = ((gcry_pk_spec_t *) pubkey->spec)->generate 
+          (algorithm, nbits, use_e, skey, retfactors);
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
@@ -488,19 +523,20 @@ pubkey_generate (int id, unsigned int nbits, unsigned long use_e,
   return err;
 }
 
-static gpg_err_code_t
-pubkey_check_secret_key (int id, MPI *skey)
+static gcry_err_code_t
+pubkey_check_secret_key (int algorithm, gcry_mpi_t *skey)
 {
-  gpg_err_code_t err = GPG_ERR_PUBKEY_ALGO;
-  GcryModule *pubkey;
+  gcry_err_code_t err = GPG_ERR_PUBKEY_ALGO;
+  gcry_module_t pubkey;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  pubkey = gcry_pubkey_lookup_id (id);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (pubkey)
     {
-      err = (*((GcryPubkeySpec *) pubkey->spec)->check_secret_key) (id, skey);
+      err = ((gcry_pk_spec_t *) pubkey->spec)->check_secret_key
+        (algorithm, skey);
       _gcry_module_release (pubkey);
     }
   ath_mutex_unlock (&pubkeys_registered_lock);
@@ -510,34 +546,34 @@ pubkey_check_secret_key (int id, MPI *skey)
 
 
 /****************
- * This is the interface to the public key encryption.
- * Encrypt DATA with PKEY and put it into RESARR which
- * should be an array of MPIs of size PUBKEY_MAX_NENC (or less if the
- * algorithm allows this - check with pubkey_get_nenc() )
+ * This is the interface to the public key encryption.  Encrypt DATA
+ * with PKEY and put it into RESARR which should be an array of MPIs
+ * of size PUBKEY_MAX_NENC (or less if the algorithm allows this -
+ * check with pubkey_get_nenc() )
  */
-static gpg_err_code_t
-pubkey_encrypt (int id, MPI *resarr, MPI data, MPI *pkey,
-               int flags)
+static gcry_err_code_t
+pubkey_encrypt (int algorithm, gcry_mpi_t *resarr, gcry_mpi_t data,
+                gcry_mpi_t *pkey, int flags)
 {
-  GcryPubkeySpec *pubkey;
-  GcryModule *module;
-  gpg_err_code_t rc;
+  gcry_pk_spec_t *pubkey;
+  gcry_module_t module;
+  gcry_err_code_t rc;
   int i;
 
   if (DBG_CIPHER)
     {
-      log_debug ("pubkey_encrypt: algo=%d\n", id);
-      for(i = 0; i < pubkey_get_npkey (id); i++)
+      log_debug ("pubkey_encrypt: algo=%d\n", algorithm);
+      for(i = 0; i < pubkey_get_npkey (algorithm); i++)
        log_mpidump ("  pkey:", pkey[i]);
       log_mpidump ("  data:", data);
     }
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  module = gcry_pubkey_lookup_id (id);
+  module = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (module)
     {
-      pubkey = (GcryPubkeySpec *) module->spec;
-      rc = (*pubkey->encrypt) (id, resarr, data, pkey, flags);
+      pubkey = (gcry_pk_spec_t *) module->spec;
+      rc = pubkey->encrypt (algorithm, resarr, data, pkey, flags);
       _gcry_module_release (module);
       goto ready;
     }
@@ -548,7 +584,7 @@ pubkey_encrypt (int id, MPI *resarr, MPI data, MPI *pkey,
 
   if (!rc && DBG_CIPHER)
     {
-      for(i = 0; i < pubkey_get_nenc (id); i++)
+      for(i = 0; i < pubkey_get_nenc (algorithm); i++)
        log_mpidump("  encr:", resarr[i] );
     }
   return rc;
@@ -562,31 +598,31 @@ pubkey_encrypt (int id, MPI *resarr, MPI data, MPI *pkey,
  * result is a pointer to a mpi variable which will receive a
  * newly allocated mpi or NULL in case of an error.
  */
-static gpg_err_code_t
-pubkey_decrypt (int id, MPI *result, MPI *data, MPI *skey,
-               int flags)
+static gcry_err_code_t
+pubkey_decrypt (int algorithm, gcry_mpi_t *result, gcry_mpi_t *data,
+                gcry_mpi_t *skey, int flags)
 {
-  GcryPubkeySpec *pubkey;
-  GcryModule *module;
-  gpg_err_code_t rc;
+  gcry_pk_spec_t *pubkey;
+  gcry_module_t module;
+  gcry_err_code_t rc;
   int i;
 
   *result = NULL; /* so the caller can always do a mpi_free */
   if (DBG_CIPHER)
     {
-      log_debug ("pubkey_decrypt: algo=%d\n", id);
-      for(i = 0; i < pubkey_get_nskey (id); i++)
+      log_debug ("pubkey_decrypt: algo=%d\n", algorithm);
+      for(i = 0; i < pubkey_get_nskey (algorithm); i++)
        log_mpidump ("  skey:", skey[i]);
-      for(i = 0; i < pubkey_get_nenc (id); i++)
+      for(i = 0; i < pubkey_get_nenc (algorithm); i++)
        log_mpidump ("  data:", data[i]);
     }
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  module = gcry_pubkey_lookup_id (id);
+  module = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (module)
     {
-      pubkey = (GcryPubkeySpec *) module->spec;
-      rc = (*pubkey->decrypt) (id, result, data, skey, flags);
+      pubkey = (gcry_pk_spec_t *) module->spec;
+      rc = pubkey->decrypt (algorithm, result, data, skey, flags);
       _gcry_module_release (module);
       goto ready;
     }
@@ -609,28 +645,29 @@ pubkey_decrypt (int id, MPI *result, MPI *data, MPI *skey,
  * should be an array of MPIs of size PUBKEY_MAX_NSIG (or less if the
  * algorithm allows this - check with pubkey_get_nsig() )
  */
-static gpg_err_code_t
-pubkey_sign (int id, MPI *resarr, MPI data, MPI *skey)
+static gcry_err_code_t
+pubkey_sign (int algorithm, gcry_mpi_t *resarr, gcry_mpi_t data,
+             gcry_mpi_t *skey)
 {
-  GcryPubkeySpec *pubkey;
-  GcryModule *module;
-  gpg_err_code_t rc;
+  gcry_pk_spec_t *pubkey;
+  gcry_module_t module;
+  gcry_err_code_t rc;
   int i;
 
   if (DBG_CIPHER)
     {
-      log_debug ("pubkey_sign: algo=%d\n", id);
-      for(i = 0; i < pubkey_get_nskey (id); i++)
+      log_debug ("pubkey_sign: algo=%d\n", algorithm);
+      for(i = 0; i < pubkey_get_nskey (algorithm); i++)
        log_mpidump ("  skey:", skey[i]);
       log_mpidump("  data:", data );
     }
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  module = gcry_pubkey_lookup_id (id);
+  module = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (module)
     {
-      pubkey = (GcryPubkeySpec *) module->spec;
-      rc = (*pubkey->sign) (id, resarr, data, skey);
+      pubkey = (gcry_pk_spec_t *) module->spec;
+      rc = pubkey->sign (algorithm, resarr, data, skey);
       _gcry_module_release (module);
       goto ready;
     }
@@ -641,7 +678,7 @@ pubkey_sign (int id, MPI *resarr, MPI data, MPI *skey)
   ath_mutex_unlock (&pubkeys_registered_lock);
 
   if (! rc && DBG_CIPHER)
-    for (i = 0; i < pubkey_get_nsig (id); i++)
+    for (i = 0; i < pubkey_get_nsig (algorithm); i++)
       log_mpidump ("   sig:", resarr[i]);
 
   return rc;
@@ -651,31 +688,32 @@ pubkey_sign (int id, MPI *resarr, MPI data, MPI *skey)
  * Verify a public key signature.
  * Return 0 if the signature is good
  */
-static gpg_err_code_t
-pubkey_verify (int id, MPI hash, MPI *data, MPI *pkey,
-              int (*cmp)(void *, MPI), void *opaquev)
+static gcry_err_code_t
+pubkey_verify (int algorithm, gcry_mpi_t hash, gcry_mpi_t *data,
+               gcry_mpi_t *pkey,
+              int (*cmp)(void *, gcry_mpi_t), void *opaquev)
 {
-  GcryPubkeySpec *pubkey;
-  GcryModule *module;
-  gpg_err_code_t rc;
+  gcry_pk_spec_t *pubkey;
+  gcry_module_t module;
+  gcry_err_code_t rc;
   int i;
 
   if (DBG_CIPHER)
     {
-      log_debug ("pubkey_verify: algo=%d\n", id);
-      for (i = 0; i < pubkey_get_npkey (id); i++)
+      log_debug ("pubkey_verify: algo=%d\n", algorithm);
+      for (i = 0; i < pubkey_get_npkey (algorithm); i++)
        log_mpidump ("  pkey:", pkey[i]);
-      for (i = 0; i < pubkey_get_nsig (id); i++)
+      for (i = 0; i < pubkey_get_nsig (algorithm); i++)
        log_mpidump ("   sig:", data[i]);
       log_mpidump ("  hash:", hash);
     }
 
   ath_mutex_lock (&pubkeys_registered_lock);
-  module = gcry_pubkey_lookup_id (id);
+  module = _gcry_module_lookup_id (pubkeys_registered, algorithm);
   if (module)
     {
-      pubkey = (GcryPubkeySpec *) module->spec;
-      rc = (*pubkey->verify) (id, hash, data, pkey, cmp, opaquev);
+      pubkey = (gcry_pk_spec_t *) module->spec;
+      rc = pubkey->verify (algorithm, hash, data, pkey, cmp, opaquev);
       _gcry_module_release (module);
       goto ready;
     }
@@ -687,35 +725,37 @@ pubkey_verify (int id, MPI hash, MPI *data, MPI *pkey,
   return rc;
 }
 
+
 /* Internal function.   */
-static gpg_err_code_t
+static gcry_err_code_t
 sexp_elements_extract (gcry_sexp_t key_sexp, const char *element_names,
                       gcry_mpi_t *elements)
 {
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
-  int i, index;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
+  int i, idx;
   const char *name;
   gcry_sexp_t list;
 
-  for (name = element_names, index = 0; *name && (! err); name++, index++)
+  for (name = element_names, idx = 0; *name && !err; name++, idx++)
     {
       list = gcry_sexp_find_token (key_sexp, name, 1);
       if (! list)
        err = GPG_ERR_NO_OBJ;
       else
        {
-         elements[index] = gcry_sexp_nth_mpi (list, 1, GCRYMPI_FMT_USG);
+         elements[idx] = gcry_sexp_nth_mpi (list, 1, GCRYMPI_FMT_USG);
          gcry_sexp_release (list);
-         if (! elements[index])
+         if (! elements[idx])
            err = GPG_ERR_INV_OBJ;
        }
     }
 
   if (err)
-    for (i = 0; i < index; i++)
-      if (elements[i])
-       gcry_free (elements[i]);
-
+    {
+      for (i = 0; i < idx; i++)
+        if (elements[i])
+          gcry_free (elements[i]);
+    }
   return err;
 }
 
@@ -730,8 +770,8 @@ sexp_elements_extract (gcry_sexp_t key_sexp, const char *element_names,
  *    openpgp-elg
  *    openpgp-elg-sig
  * Provide a SE with the first element be either "private-key" or
- * or "public-key". the followed by a list with its first element
- * be one of the above algorithm identifiers and the following
+ * or "public-key". It is followed by a list with its first element
+ * be one of the above algorithm identifiers and the remaning
  * elements are pairs with parameter-id and value.
  * NOTE: we look through the list to find a list beginning with
  * "private-key" or "public-key" - the first one found is used.
@@ -752,25 +792,25 @@ sexp_elements_extract (gcry_sexp_t key_sexp, const char *element_names,
  *  )
  * The <mpi> are expected to be in GCRYMPI_FMT_USG
  */
-static gpg_err_code_t
-sexp_to_key (gcry_sexp_t sexp, int want_private, MPI **retarray,
-             GcryModule **retalgo)
+static gcry_err_code_t
+sexp_to_key (gcry_sexp_t sexp, int want_private, gcry_mpi_t **retarray,
+             gcry_module_t *retalgo)
 {
     gcry_sexp_t list, l2;
     const char *name;
     size_t n;
-    int algo;
     const char *elems;
     gcry_mpi_t *array;
-    gpg_err_code_t err = GPG_ERR_NO_ERROR;
-    GcryModule *module;
-    GcryPubkeySpec *pubkey;
+    gcry_err_code_t err = GPG_ERR_NO_ERROR;
+    gcry_module_t module;
+    gcry_pk_spec_t *pubkey;
 
     /* check that the first element is valid */
     list = gcry_sexp_find_token( sexp, want_private? "private-key"
                                                    :"public-key", 0 );
     if( !list )
-       return GPG_ERR_INV_OBJ; /* Does not contain a public- or private-key object */
+       return GPG_ERR_INV_OBJ; /* Does not contain a public-
+                                   or private-key object */
     l2 = gcry_sexp_cadr( list );
     gcry_sexp_release ( list );
     list = l2;
@@ -781,15 +821,23 @@ sexp_to_key (gcry_sexp_t sexp, int want_private, MPI **retarray,
     }
 
     {
-      char *name_terminated = gcry_xmalloc (n + 1);
-      strncpy (name_terminated, name, n);
+      char *name_terminated;
+
+      name_terminated = gcry_malloc (n + 1);
+      if (!name_terminated)
+        {
+          err = gpg_err_code_from_errno (errno);
+          gcry_sexp_release (list);
+          return err;
+        }
+      memcpy (name_terminated, name, n);
       name_terminated[n] = 0;
 
       ath_mutex_lock (&pubkeys_registered_lock);
-      module = gcry_pubkey_lookup_name (name_terminated);
+      module = gcry_pk_lookup_name (name_terminated);
       ath_mutex_unlock (&pubkeys_registered_lock);
 
-      free (name_terminated);
+      gcry_free (name_terminated);
     }
 
     if (! module)
@@ -798,9 +846,8 @@ sexp_to_key (gcry_sexp_t sexp, int want_private, MPI **retarray,
        return GPG_ERR_PUBKEY_ALGO; /* unknown algorithm */
       }
     else
-      pubkey = (GcryPubkeySpec *) module->spec;
+      pubkey = (gcry_pk_spec_t *) module->spec;
 
-    algo = pubkey->id;
     elems = want_private ? pubkey->elements_skey : pubkey->elements_pkey;
     array = gcry_calloc (strlen (elems) + 1, sizeof (*array));
     if (! array)
@@ -829,56 +876,79 @@ sexp_to_key (gcry_sexp_t sexp, int want_private, MPI **retarray,
     return err;
 }
 
-static gpg_err_code_t
-sexp_to_sig (gcry_sexp_t sexp, MPI **retarray,
-            GcryModule **retalgo)
+static gcry_err_code_t
+sexp_to_sig (gcry_sexp_t sexp, gcry_mpi_t **retarray,
+            gcry_module_t *retalgo)
 {
     gcry_sexp_t list, l2;
     const char *name;
     size_t n;
-    int algo;
     const char *elems;
     gcry_mpi_t *array;
-    gpg_err_code_t err = GPG_ERR_NO_ERROR;
-    GcryModule *module;
-    GcryPubkeySpec *pubkey;
+    gcry_err_code_t err = GPG_ERR_NO_ERROR;
+    gcry_module_t module;
+    gcry_pk_spec_t *pubkey;
 
     /* check that the first element is valid */
     list = gcry_sexp_find_token( sexp, "sig-val" , 0 );
     if( !list )
        return GPG_ERR_INV_OBJ; /* Does not contain a signature value object */
-    l2 = gcry_sexp_cadr( list );
-    gcry_sexp_release ( list );
-    list = l2;
-    if( !list )
+    l2 = gcry_sexp_nth (list, 1);
+    if(! l2)
+      {
+       gcry_sexp_release (list);
        return GPG_ERR_NO_OBJ; /* no cadr for the sig object */
-    name = gcry_sexp_nth_data( list, 0, &n );
+      }
+    name = gcry_sexp_nth_data( l2, 0, &n );
     if( !name ) {
-       gcry_sexp_release ( list );
+       gcry_sexp_release ( list );
+       gcry_sexp_release ( l2 );
        return GPG_ERR_INV_OBJ; /* invalid structure of object */
     }
-
+    else if (n == 5 && (! memcmp (name, "flags", 5))) {
+      /* Skip flags, since they are not used but just here for the
+        sake of consistent S-expressions.  */
+      gcry_sexp_release (l2);
+      l2 = gcry_sexp_nth (list, 2);
+      if (! l2)
+       {
+         gcry_sexp_release (list);
+         return GPG_ERR_INV_OBJ;
+       }
+      name = gcry_sexp_nth_data (l2, 0, &n);
+    }
+      
     {
-      char *name_terminated = gcry_xmalloc (n + 1);
-      strncpy (name_terminated, name, n);
+      char *name_terminated;
+
+      name_terminated = gcry_malloc (n + 1);
+      if (!name_terminated)
+        {
+          err = gcry_err_code_from_errno (errno);
+         gcry_sexp_release (l2);
+         gcry_sexp_release (list);
+          return err;
+        }
+          
+      memcpy (name_terminated, name, n);
       name_terminated[n] = 0;
       
       ath_mutex_lock (&pubkeys_registered_lock);
-      module = gcry_pubkey_lookup_name (name_terminated);
+      module = gcry_pk_lookup_name (name_terminated);
       ath_mutex_unlock (&pubkeys_registered_lock);
 
-      free (name_terminated);
+      gcry_free (name_terminated);
     }
 
     if (! module)
       {
+       gcry_sexp_release (l2);
        gcry_sexp_release (list);
        return GPG_ERR_PUBKEY_ALGO; /* unknown algorithm */
       }
     else
-      pubkey = (GcryPubkeySpec *) module->spec;
+      pubkey = (gcry_pk_spec_t *) module->spec;
 
-    algo = pubkey->id;
     elems = pubkey->elements_sig;
     array = gcry_calloc (strlen (elems) + 1 , sizeof (*array));
     if (! array)
@@ -887,8 +957,8 @@ sexp_to_sig (gcry_sexp_t sexp, MPI **retarray,
     if (! err)
       err = sexp_elements_extract (list, elems, array);
 
-    if (list)
-      gcry_sexp_release (list);
+    gcry_sexp_release (l2);
+    gcry_sexp_release (list);
 
     if (err)
       {
@@ -921,19 +991,19 @@ sexp_to_sig (gcry_sexp_t sexp, MPI **retarray,
  *           ))
  * RET_MODERN is set to true when at least an empty flags list has been found.
  */
-static gpg_err_code_t
-sexp_to_enc (gcry_sexp_t sexp, MPI **retarray, GcryModule **retalgo,
+static gcry_err_code_t
+sexp_to_enc (gcry_sexp_t sexp, gcry_mpi_t **retarray, gcry_module_t *retalgo,
              int *ret_modern, int *ret_want_pkcs1, int *flags)
 {
   gcry_sexp_t list = NULL, l2 = NULL;
-  GcryPubkeySpec *pubkey = NULL;
-  GcryModule *module = NULL;
+  gcry_pk_spec_t *pubkey = NULL;
+  gcry_module_t module = NULL;
   const char *name;
   size_t n;
   int parsed_flags = 0;
   const char *elems;
   gcry_mpi_t *array = NULL;
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
 
   *ret_want_pkcs1 = 0;
   *ret_modern = 0;
@@ -941,31 +1011,34 @@ sexp_to_enc (gcry_sexp_t sexp, MPI **retarray, GcryModule **retalgo,
   /* check that the first element is valid */
   list = gcry_sexp_find_token (sexp, "enc-val" , 0);
   if (! list)
-    err = GPG_ERR_INV_OBJ; /* Does not contain an encrypted value object */
-
-  if (! err)
     {
-      l2 = gcry_sexp_nth (list, 1);
-      if (! l2)
-       err = GPG_ERR_NO_OBJ; /* no cdr for the data object */
+      err = GPG_ERR_INV_OBJ; /* Does not contain an encrypted value object */
+      goto leave;
     }
 
-  if (! err)
+  l2 = gcry_sexp_nth (list, 1);
+  if (! l2)
     {
-      /* Extract the name of the algorithm.  */
-      name = gcry_sexp_nth_data (l2, 0, &n);
-      if (! name)
-       err = GPG_ERR_INV_OBJ; /* invalid structure of object */
+      err = GPG_ERR_NO_OBJ; /* no cdr for the data object */
+      goto leave;
     }
 
-  if ((! err) && (n == 5) && (! memcmp (name, "flags", 5)))
+  /* Extract identifier of sublist.  */
+  name = gcry_sexp_nth_data (l2, 0, &n);
+  if (! name)
+    {
+      err = GPG_ERR_INV_OBJ; /* invalid structure of object */
+      goto leave;
+    }
+  
+  if ((n == 5) && (! memcmp (name, "flags", 5)))
     {
       /* There is a flags element - process it */
       const char *s;
       int i;
-
+      
       *ret_modern = 1;
-      for (i = gcry_sexp_length (l2) - 1; i > 0 && (! err); i--)
+      for (i = gcry_sexp_length (l2) - 1; i > 0; i--)
         {
           s = gcry_sexp_nth_data (l2, i, &n);
           if (! s)
@@ -974,64 +1047,74 @@ sexp_to_enc (gcry_sexp_t sexp, MPI **retarray, GcryModule **retalgo,
             ; /* just a dummy because it is the default */
           else if (n == 5 && ! memcmp (s, "pkcs1", 5))
             *ret_want_pkcs1 = 1;
-         else if (n == 11 && ! memcmp (s, "no-blinding", 11))
-           parsed_flags |= PUBKEY_FLAG_NO_BLINDING;
+          else if (n == 11 && ! memcmp (s, "no-blinding", 11))
+            parsed_flags |= PUBKEY_FLAG_NO_BLINDING;
           else
-           err = GPG_ERR_INV_FLAG;
-       }
-    }
-
-  if (! err)
-    {
+            {
+              err = GPG_ERR_INV_FLAG;
+              goto leave;
+            }
+        }
+      
       /* Get the next which has the actual data */
       gcry_sexp_release (l2);
       l2 = gcry_sexp_nth (list, 2);
       if (! l2)
-       err = GPG_ERR_NO_OBJ; /* no cdr for the data object */
-    }
+        {
+          err = GPG_ERR_NO_OBJ; /* no cdr for the data object */
+          goto leave;
+        }
 
-  if (! err)
-    {
+      /* Extract sublist identifier.  */
       name = gcry_sexp_nth_data (l2, 0, &n);
       if (! name)
-       err = GPG_ERR_INV_OBJ; /* invalid structure of object */
-      else
-       {
-         gcry_sexp_release (list);
-         list = l2;
-         l2 = NULL;
-       }
-    }
-
-  if (! err)
-    {
-      char *name_terminated = gcry_xmalloc (n + 1);
-      strncpy (name_terminated, name, n);
-      name_terminated[n] = 0;
+        {
+          err = GPG_ERR_INV_OBJ; /* invalid structure of object */
+          goto leave;
+        }
 
-      ath_mutex_lock (&pubkeys_registered_lock);
-      module = gcry_pubkey_lookup_name (name_terminated);
-      ath_mutex_unlock (&pubkeys_registered_lock);
+      gcry_sexp_release (list);
+      list = l2;
+      l2 = NULL;
+    }
 
-      free (name_terminated);
+  {
+    char *name_terminated;
 
-      if (! module)
-       err = GPG_ERR_PUBKEY_ALGO; /* unknown algorithm */
-      else
-       pubkey = (GcryPubkeySpec *) module->spec;
-    }
+    name_terminated = gcry_malloc (n + 1);
+    if (!name_terminated)
+      {
+        err = gcry_err_code_from_errno (errno);
+        goto leave;
+      }
+    memcpy (name_terminated, name, n);
+    name_terminated[n] = 0;
+    
+    ath_mutex_lock (&pubkeys_registered_lock);
+    module = gcry_pk_lookup_name (name_terminated);
+    ath_mutex_unlock (&pubkeys_registered_lock);
+    
+    gcry_free (name_terminated);
+    
+    if (! module)
+      {
+        err = GPG_ERR_PUBKEY_ALGO; /* unknown algorithm */
+        goto leave;
+      }
+    pubkey = (gcry_pk_spec_t *) module->spec;
+  }
 
-  if (! err)
+  elems = pubkey->elements_enc;
+  array = gcry_calloc (strlen (elems) + 1, sizeof (*array));
+  if (! array)
     {
-      elems = pubkey->elements_enc;
-      array = gcry_calloc (strlen (elems) + 1, sizeof (*array));
-      if (! array)
-       err = gpg_err_code_from_errno (errno);
+      err = gpg_err_code_from_errno (errno);
+      goto leave;
     }
 
-  if (! err)
-    err = sexp_elements_extract (list, elems, array);
+  err = sexp_elements_extract (list, elems, array);
 
+ leave:
   if (list)
     gcry_sexp_release (list);
   if (l2)
@@ -1075,11 +1158,11 @@ sexp_to_enc (gcry_sexp_t sexp, MPI **retarray, GcryModule **retalgo,
    NBITS is the length of the key in bits. 
 
 */
-static gpg_err_code_t
+static gcry_err_code_t
 sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
                   int for_encryption, int *flags)
 {
-  gpg_err_code_t rc = 0;
+  gcry_err_code_t rc = 0;
   gcry_sexp_t ldata, lhash, lvalue;
   int i;
   size_t n;
@@ -1141,7 +1224,7 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
         rc = GPG_ERR_INV_OBJ;
     }
   else if (is_pkcs1 && lvalue && for_encryption)
-    { /* create pkcs#1 block type 2 padding */
+    { /* Create pkcs#1 block type 2 padding. */
       unsigned char *frame = NULL;
       size_t nframe = (nbits+7) / 8;
       const void * value;
@@ -1165,27 +1248,29 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
           i = nframe - 3 - valuelen;
           assert (i > 0);
           p = gcry_random_bytes_secure (i, GCRY_STRONG_RANDOM);
-          /* replace zero bytes by new values*/
+          /* Replace zero bytes by new values. */
           for (;;)
             {
               int j, k;
               unsigned char *pp;
               
-              /* count the zero bytes */
+              /* Count the zero bytes. */
               for (j=k=0; j < i; j++)
                 {
                   if (!p[j])
                     k++;
                 }
               if (!k)
-                break; /* okay: no (more) zero bytes */
+                break; /* Okay: no (more) zero bytes. */
               
-              k += k/128; /* better get some more */
+              k += k/128 + 3; /* Better get some more. */
               pp = gcry_random_bytes_secure (k, GCRY_STRONG_RANDOM);
-              for (j=0; j < i && k; j++)
+              for (j=0; j < i && k; )
                 {
                   if (!p[j])
                     p[j] = pp[--k];
+                  if (p[j])
+                    j++;
                 }
               gcry_free (pp);
             }
@@ -1199,13 +1284,13 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
           assert (n == nframe);
 
          /* FIXME, error checking?  */
-          gcry_mpi_scan (ret_mpi, GCRYMPI_FMT_USG, frame, &nframe);
+          gcry_mpi_scan (ret_mpi, GCRYMPI_FMT_USG, frame, n, &nframe);
         }
 
       gcry_free(frame);
     }
   else if (is_pkcs1 && lhash && !for_encryption)
-    { /* create pkcs#1 block type 1 padding */
+    { /* Create pkcs#1 block type 1 padding. */
       if (gcry_sexp_length (lhash) != 3)
         rc = GPG_ERR_INV_OBJ;
       else if ( !(s=gcry_sexp_nth_data (lhash, 1, &n)) || !n )
@@ -1215,15 +1300,16 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
           static struct { const char *name; int algo; } hashnames[] = 
           { { "sha1",   GCRY_MD_SHA1 },
             { "md5",    GCRY_MD_MD5 },
-            { "rmd160", GCRY_MD_RMD160 },
             { "sha256", GCRY_MD_SHA256 },
+            { "ripemd160", GCRY_MD_RMD160 },
+            { "rmd160", GCRY_MD_RMD160 },
             { "sha384", GCRY_MD_SHA384 },
             { "sha512", GCRY_MD_SHA512 },
             { "md2",    GCRY_MD_MD2 },
             { "md4",    GCRY_MD_MD4 },
             { "tiger",  GCRY_MD_TIGER },
             { "haval",  GCRY_MD_HAVAL },
-            { NULL }
+            { NULL, 0 }
           };
           int algo;
           byte asn[100];
@@ -1250,22 +1336,26 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
                     || !valuelen )
             rc = GPG_ERR_INV_OBJ;
           else if (gcry_md_algo_info (algo, GCRYCTL_GET_ASNOID, asn, &asnlen))
-            rc = GPG_ERR_NOT_IMPLEMENTED; /* we don't have all of the above algos */
+            {
+              /* We don't have yet all of the above algorithms.  */
+              rc = GPG_ERR_NOT_IMPLEMENTED;
+            }
           else if ( valuelen != dlen )
             {
-              /* hash value does not match the length of digest for
-                 the given algo */
+              /* Hash value does not match the length of digest for
+                 the given algorithm. */
               rc = GPG_ERR_CONFLICT;
             }
           else if( !dlen || dlen + asnlen + 4 > nframe)
             {
-              /* can't encode an DLEN byte digest MD into a NFRAME byte frame */
+              /* Can't encode an DLEN byte digest MD into a NFRAME
+                 byte frame. */
               rc = GPG_ERR_TOO_SHORT;
             }
           else if ( !(frame = gcry_malloc (nframe)) )
             rc = gpg_err_code_from_errno (errno);
           else
-            { /* assemble the pkcs#1 block type 1 */
+            { /* Assemble the pkcs#1 block type 1. */
               n = 0;
               frame[n++] = 0;
               frame[n++] = 1; /* block type */
@@ -1281,7 +1371,7 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
               assert (n == nframe);
       
               /* convert it into an MPI, FIXME: error checking?  */
-              gcry_mpi_scan (ret_mpi, GCRYMPI_FMT_USG, frame, &nframe);
+              gcry_mpi_scan (ret_mpi, GCRYMPI_FMT_USG, frame, n, &nframe);
             }
           
           gcry_free (frame);
@@ -1294,7 +1384,7 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
   gcry_sexp_release (lhash);
   gcry_sexp_release (lvalue);
 
-  if (! rc)
+  if (!rc)
     *flags = parsed_flags;
 
   return rc;
@@ -1323,89 +1413,108 @@ sexp_data_to_mpi (gcry_sexp_t input, unsigned int nbits, gcry_mpi_t *ret_mpi,
                ))
 
 */
-gpg_error_t
+gcry_error_t
 gcry_pk_encrypt (gcry_sexp_t *r_ciph, gcry_sexp_t s_data, gcry_sexp_t s_pkey)
 {
-  MPI *pkey = NULL, data = NULL, *ciph = NULL;
+  gcry_mpi_t *pkey = NULL, data = NULL, *ciph = NULL;
   const char *algo_name, *algo_elems;
   int flags;
-  gpg_err_code_t rc;
-  GcryPubkeySpec *pubkey = NULL;
-  GcryModule *module = NULL;
+  gcry_err_code_t rc;
+  gcry_pk_spec_t *pubkey = NULL;
+  gcry_module_t module = NULL;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   *r_ciph = NULL;
   /* get the key */
   rc = sexp_to_key (s_pkey, 0, &pkey, &module);
-  if (! rc)
-    {
-      assert (module);
-      pubkey = (GcryPubkeySpec *) module->spec;
-      algo_name = pubkey->name;
-      algo_elems = pubkey->elements_enc;
-      
-      /* get the stuff we want to encrypt */
-      rc = sexp_data_to_mpi (s_data, gcry_pk_get_nbits (s_pkey), &data, 1,
-                            &flags);
-    }
+  if (rc)
+    goto leave;
+
+  assert (module);
+  pubkey = (gcry_pk_spec_t *) module->spec;
+
+  /* If aliases for the algorithm name exists, take the first one
+     instead of the regular name to adhere to SPKI conventions.  We
+     assume that the first alias name is the lowercase version of the
+     regular one.  This change is required for compatibility with
+     1.1.12 generated S-expressions. */
+  algo_name = pubkey->aliases? *pubkey->aliases : NULL;
+  if (!algo_name || !*algo_name)
+    algo_name = pubkey->name;
+  
+  algo_elems = pubkey->elements_enc;
+  
+  /* Get the stuff we want to encrypt. */
+  rc = sexp_data_to_mpi (s_data, gcry_pk_get_nbits (s_pkey), &data, 1,
+                         &flags);
+  if (rc)
+    goto leave;
 
-  if (! rc)
+  /* Now we can encrypt DATA to CIPH. */
+  ciph = gcry_calloc (strlen (algo_elems) + 1, sizeof (*ciph));
+  if (!ciph)
     {
-      /* Now we can encrypt data to ciph */
-      ciph = gcry_xcalloc (strlen (algo_elems) + 1, sizeof (*ciph));
-      rc = pubkey_encrypt (pubkey->id, ciph, data, pkey, flags);
-      mpi_free (data);
+      rc = gpg_err_code_from_errno (errno);
+      goto leave;
     }
+  rc = pubkey_encrypt (module->mod_id, ciph, data, pkey, flags);
+  mpi_free (data);
+  data = NULL;
+  if (rc)
+    goto leave;
 
   /* We did it.  Now build the return list */
-  if (! rc)
-    {
-      char *string, *p;
-      int i;
-      size_t nelem = strlen (algo_elems);
-      size_t needed = 19 + strlen (algo_name) + (nelem * 5);
-
-      if (flags & PUBKEY_FLAG_NO_BLINDING)
-       needed += 12;
-
-      /* Build the string.  */
-      string = p = gcry_xmalloc (needed);
-      p = stpcpy ( p, "(enc-val(flags" );
-      if (flags & PUBKEY_FLAG_NO_BLINDING)
-       p = stpcpy (p, " no-blinding");
-      p = stpcpy (p, ")(");
-      p = stpcpy ( p, algo_name );
-      for(i=0; algo_elems[i]; i++ ) {
-       *p++ = '(';
-       *p++ = algo_elems[i];
-       p = stpcpy ( p, "%m)" );
+  {
+    char *string, *p;
+    int i;
+    size_t nelem = strlen (algo_elems);
+    size_t needed = 19 + strlen (algo_name) + (nelem * 5);
+    void **arg_list;
+    
+    /* Build the string.  */
+    string = p = gcry_malloc (needed);
+    if (!string)
+      {
+        rc = gpg_err_code_from_errno (errno);
+        goto leave;
       }
-      strcpy ( p, "))" );
-
-      /* and now the ugly part:  we don't have a function to
-       * pass an array to a format string, so we have to do it this way :-(
-       */
-
+    p = stpcpy ( p, "(enc-val(" );
+    p = stpcpy ( p, algo_name );
+    for (i=0; algo_elems[i]; i++ )
       {
-       int i;
-       void **arg_list = malloc (sizeof (void *) * nelem);
-       if (arg_list)
-         {
-           for (i = 0; i < nelem; i++)
-             arg_list[i] = &ciph[i];
-
-           rc = gcry_sexp_build_array (r_ciph, NULL, string, arg_list);
-
-           free (arg_list);
-         }
-       else
-         rc = gpg_err_code_from_errno (errno);
+        *p++ = '(';
+        *p++ = algo_elems[i];
+        p = stpcpy ( p, "%m)" );
+      }
+    strcpy ( p, "))" );
+    
+    /* And now the ugly part: We don't have a function to pass an
+     * array to a format string, so we have to do it this way :-(.  */
+    /* FIXME: There is now such a format spefier, so we can could
+       change the code to be more clear. */
+    arg_list = malloc (nelem * sizeof *arg_list);
+    if (!arg_list)
+      {
+        rc = gpg_err_code_from_errno (errno);
+        goto leave;
       }
 
-      if (rc)
-       BUG ();
-      gcry_free (string);
+    for (i = 0; i < nelem; i++)
+      arg_list[i] = ciph + i;
+    
+    rc = gcry_sexp_build_array (r_ciph, NULL, string, arg_list);
+    free (arg_list);
+    if (rc)
+      BUG ();
+    gcry_free (string);
+  }
+
+ leave:
+  if (pkey)
+    {
+      release_mpi_array (pkey);
+      gcry_free (pkey);
     }
 
   if (ciph)
@@ -1421,75 +1530,69 @@ gcry_pk_encrypt (gcry_sexp_t *r_ciph, gcry_sexp_t s_data, gcry_sexp_t s_pkey)
       ath_mutex_unlock (&pubkeys_registered_lock);
     }
 
-  return gpg_error (rc);
+  return gcry_error (rc);
 }
 
-/****************
- * Do a PK decrypt operation
- *
- * Caller has to provide a secret key as the SEXP skey and data in a
- * format as created by gcry_pk_encrypt.  For historic reasons the
- * function returns simply an MPI as an S-expression part; this is
- * deprecated and the new method should be used which returns a real
- * S-expressionl this is selected by adding at least an empty flags
- * list to S_DATA.
- * 
- * Returns: 0 or an errorcode.
- *
- * s_data = (enc-val
- *            [(flags)]
- *           (<algo>
- *             (<param_name1> <mpi>)
- *             ...
- *             (<param_namen> <mpi>)
- *           ))
- * s_skey = <key-as-defined-in-sexp_to_key>
- * r_plain= Either an incomplete S-expression without the parentheses
- *          or if the flags list is used (even if empty) a real S-expression:
- *          (value PLAIN).  */
-gpg_error_t
+/* 
+   Do a PK decrypt operation
+  
+   Caller has to provide a secret key as the SEXP skey and data in a
+   format as created by gcry_pk_encrypt.  For historic reasons the
+   function returns simply an MPI as an S-expression part; this is
+   deprecated and the new method should be used which returns a real
+   S-expressionl this is selected by adding at least an empty flags
+   list to S_DATA.
+   
+   Returns: 0 or an errorcode.
+  
+   s_data = (enc-val
+              [(flags)]
+              (<algo>
+                (<param_name1> <mpi>)
+                ...
+                (<param_namen> <mpi>)
+              ))
+   s_skey = <key-as-defined-in-sexp_to_key>
+   r_plain= Either an incomplete S-expression without the parentheses
+            or if the flags list is used (even if empty) a real S-expression:
+            (value PLAIN). 
+ */
+gcry_error_t
 gcry_pk_decrypt (gcry_sexp_t *r_plain, gcry_sexp_t s_data, gcry_sexp_t s_skey)
 {
-  MPI *skey = NULL, *data = NULL, plain = NULL;
+  gcry_mpi_t *skey = NULL, *data = NULL, plain = NULL;
   int modern, want_pkcs1, flags;
-  gpg_err_code_t rc;
-  GcryModule *module_enc = NULL, *module_key = NULL;
-  GcryPubkeySpec *pubkey = NULL;
+  gcry_err_code_t rc;
+  gcry_module_t module_enc = NULL, module_key = NULL;
+  gcry_pk_spec_t *pubkey = NULL;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   *r_plain = NULL;
   rc = sexp_to_key (s_skey, 1, &skey, &module_key);
+  if (rc)
+    goto leave;
 
-  if (! rc)
-    rc = sexp_to_enc (s_data, &data, &module_enc, &modern, &want_pkcs1, &flags);
-
-  if (! rc)
+  rc = sexp_to_enc (s_data, &data, &module_enc, &modern, &want_pkcs1, &flags);
+  if (rc)
+    goto leave;
+  
+  if (module_key->mod_id != module_enc->mod_id)
     {
-      if (((GcryPubkeySpec *) module_key->spec)->id
-         != ((GcryPubkeySpec *) module_enc->spec)->id)
-       rc = GPG_ERR_CONFLICT; /* key algo does not match data algo */
-      else
-       pubkey = (GcryPubkeySpec *) module_key->spec;
+      rc = GPG_ERR_CONFLICT; /* Key algo does not match data algo. */
+      goto leave;
     }
 
-  if (! rc)
-    rc = pubkey_decrypt (pubkey->id, &plain, data, skey, flags);
+  pubkey = (gcry_pk_spec_t *) module_key->spec;
 
-  if (! rc)
-    {
-      if (! modern)
-       {
-         if (gcry_sexp_build (r_plain, NULL, "%m", plain))
-           BUG ();
-       }
-      else
-       {
-         if (gcry_sexp_build (r_plain, NULL, "(value %m)", plain))
-           BUG ();
-       }
-    }
+  rc = pubkey_decrypt (module_key->mod_id, &plain, data, skey, flags);
+  if (rc)
+    goto leave;
 
+  if (gcry_sexp_build (r_plain, NULL, modern? "(value %m)" : "%m", plain))
+    BUG ();
+  
+ leave:
   if (skey)
     {
       release_mpi_array (skey);
@@ -1515,114 +1618,123 @@ gcry_pk_decrypt (gcry_sexp_t *r_plain, gcry_sexp_t s_data, gcry_sexp_t s_skey)
       ath_mutex_unlock (&pubkeys_registered_lock);
     }
 
-  return gpg_error (rc);
+  return gcry_error (rc);
 }
 
 
 
-/****************
* Create a signature.
- *
* Caller has to provide a secret key as the SEXP skey and data
* expressed as a SEXP list hash with only one element which should
* instantly be available as a MPI. Alternatively the structure given
* below may be used for S_HASH, it provides the abiliy to pass flags
* to the operation; the only flag defined by now is "pkcs1" which
* does PKCS#1 block type 1 style padding.
- *
* Returns: 0 or an errorcode.
*         In case of 0 the function returns a new SEXP with the
*         signature value; the structure of this signature depends on the
*         other arguments but is always suitable to be passed to
*         gcry_pk_verify
- *
* s_hash = See comment for sexp_data_to_mpi
- *             
* s_skey = <key-as-defined-in-sexp_to_key>
* r_sig  = (sig-val
*           (<algo>
*             (<param_name1> <mpi>)
*             ...
- *             (<param_namen> <mpi>)
- * )) */
-gpg_error_t
+/*
  Create a signature.
+  
  Caller has to provide a secret key as the SEXP skey and data
  expressed as a SEXP list hash with only one element which should
  instantly be available as a MPI. Alternatively the structure given
  below may be used for S_HASH, it provides the abiliy to pass flags
  to the operation; the only flag defined by now is "pkcs1" which
  does PKCS#1 block type 1 style padding.
+  
  Returns: 0 or an errorcode.
           In case of 0 the function returns a new SEXP with the
           signature value; the structure of this signature depends on the
           other arguments but is always suitable to be passed to
           gcry_pk_verify
+  
  s_hash = See comment for sexp_data_to_mpi
+               
  s_skey = <key-as-defined-in-sexp_to_key>
  r_sig  = (sig-val
             (<algo>
               (<param_name1> <mpi>)
               ...
+                (<param_namen> <mpi>))) 
+*/
+gcry_error_t
 gcry_pk_sign (gcry_sexp_t *r_sig, gcry_sexp_t s_hash, gcry_sexp_t s_skey)
 {
-  MPI *skey = NULL, hash = NULL, *result = NULL;
-  GcryPubkeySpec *pubkey = NULL;
-  GcryModule *module = NULL;
-  const char *key_algo_name, *algo_name, *algo_elems;
+  gcry_mpi_t *skey = NULL, hash = NULL, *result = NULL;
+  gcry_pk_spec_t *pubkey = NULL;
+  gcry_module_t module = NULL;
+  const char *algo_name, *algo_elems;
   int i;
-  gpg_err_code_t rc;
-
+  gcry_err_code_t rc;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   *r_sig = NULL;
   rc = sexp_to_key (s_skey, 1, &skey, &module);
+  if (rc)
+    goto leave;
 
-  if (! rc)
-    {
-      assert (module);
-      pubkey = (GcryPubkeySpec *) module->spec;
-      algo_name = key_algo_name = pubkey->name;
-
-      algo_elems = pubkey->elements_sig;
+  assert (module);
+  pubkey = (gcry_pk_spec_t *) module->spec;
+  algo_name = pubkey->aliases? *pubkey->aliases : NULL;
+  if (!algo_name || !*algo_name)
+    algo_name = pubkey->name;
+  
+  algo_elems = pubkey->elements_sig;
 
-      /* get the stuff we want to sign */
-      /* Note that pk_get_nbits does also work on a private key */
-      rc = sexp_data_to_mpi (s_hash, gcry_pk_get_nbits (s_skey), &hash, 0, NULL);
-    }
+  /* Get the stuff we want to sign.  Note that pk_get_nbits does also
+      work on a private key. */
+  rc = sexp_data_to_mpi (s_hash, gcry_pk_get_nbits (s_skey),
+                             &hash, 0, NULL);
+  if (rc)
+    goto leave;
 
-  if (! rc)
+  result = gcry_calloc (strlen (algo_elems) + 1, sizeof (*result));
+  if (!result)
     {
-      result = gcry_xcalloc (strlen (algo_elems) + 1, sizeof (*result));
-      rc = pubkey_sign (pubkey->id, result, hash, skey);
+      rc = gpg_err_code_from_errno (errno);
+      goto leave;
     }
+  rc = pubkey_sign (module->mod_id, result, hash, skey);
+  if (rc)
+    goto leave;
 
-  if (! rc)
-    {
-      char *string, *p;
-      size_t nelem, needed = strlen (algo_name) + 20;
-
-      nelem = strlen (algo_elems);
-
-      /* count elements, so that we can allocate enough space */
-      needed += 10 * nelem;
+  {
+    char *string, *p;
+    size_t nelem, needed = strlen (algo_name) + 20;
+    void **arg_list;
 
-      /* build the string */
-      string = p = gcry_xmalloc (needed);
-      p = stpcpy (p, "(sig-val(");
-      p = stpcpy (p, algo_name);
-      for(i = 0; algo_elems[i]; i++)
-       {
-         *p++ = '(';
-         *p++ = algo_elems[i];
-         p = stpcpy (p, "%m)");
-       }
-      strcpy (p, "))");
+    nelem = strlen (algo_elems);
+    
+    /* Count elements, so that we can allocate enough space. */
+    needed += 10 * nelem;
 
+    /* Build the string. */
+    string = p = gcry_malloc (needed);
+    if (!string)
       {
-       int i;
-       void **arg_list = malloc (sizeof (void *) * nelem);
-       if (arg_list)
-         {
-           for (i = 0; i < nelem; i++)
-             arg_list[i] = &result[i];
-
-           rc = gcry_sexp_build_array (r_sig, NULL, string, arg_list);
+        rc = gpg_err_code_from_errno (errno);
+        goto leave;
+      }
+    p = stpcpy (p, "(sig-val(");
+    p = stpcpy (p, algo_name);
+    for (i = 0; algo_elems[i]; i++)
+      {
+        *p++ = '(';
+        *p++ = algo_elems[i];
+        p = stpcpy (p, "%m)");
+      }
+    strcpy (p, "))");
 
-           free (arg_list);
-         }
-       else
-         rc = gpg_err_code_from_errno (errno);
+    arg_list = malloc (nelem * sizeof *arg_list);
+    if (!arg_list)
+      {
+        rc = gpg_err_code_from_errno (errno);
+        goto leave;
       }
 
-      if (rc)
-       BUG ();
-      gcry_free (string);
-    }
+    for (i = 0; i < nelem; i++)
+      arg_list[i] = result + i;
+
+    rc = gcry_sexp_build_array (r_sig, NULL, string, arg_list);
+    free (arg_list);
+    if (rc)
+      BUG ();
+    gcry_free (string);
+  }
 
+ leave:
   if (skey)
     {
       release_mpi_array (skey);
@@ -1633,44 +1745,52 @@ gcry_pk_sign (gcry_sexp_t *r_sig, gcry_sexp_t s_hash, gcry_sexp_t s_skey)
     mpi_free (hash);
 
   if (result)
-    gcry_free (result);
+    {
+      release_mpi_array (result);
+      gcry_free (result);
+    }
 
-  return gpg_error (rc);
+  return gcry_error (rc);
 }
 
 
-/****************
- * Verify a signature.  Caller has to supply the public key pkey, the
- * signature sig and his hashvalue data.  Public key has to be a
- * standard public key given as an S-Exp, sig is a S-Exp as returned
* from gcry_pk_sign and data must be an S-Exp like the one in sign
- * too.
- */
-gpg_error_t
+/*
+   Verify a signature.
+
+   Caller has to supply the public key pkey, the signature sig and his
  hashvalue data.  Public key has to be a standard public key given
+   as an S-Exp, sig is a S-Exp as returned from gcry_pk_sign and data
  must be an S-Exp like the one in sign too.  */
+gcry_error_t
 gcry_pk_verify (gcry_sexp_t s_sig, gcry_sexp_t s_hash, gcry_sexp_t s_pkey)
 {
-  GcryModule *module_key = NULL, *module_sig = NULL;
-  MPI *pkey = NULL, hash = NULL, *sig = NULL;
-  gpg_err_code_t rc;
+  gcry_module_t module_key = NULL, module_sig = NULL;
+  gcry_mpi_t *pkey = NULL, hash = NULL, *sig = NULL;
+  gcry_err_code_t rc;
 
   REGISTER_DEFAULT_PUBKEYS;
  
   rc = sexp_to_key (s_pkey, 0, &pkey, &module_key);
-  if (rc)
-    rc = sexp_to_sig (s_sig, &sig, &module_sig);
+  if (rc)
+    goto leave;
 
-  if ((! rc)
-      && (((GcryPubkeySpec *) module_key->spec)->id
-         != ((GcryPubkeySpec *) module_sig->spec)->id))
-    rc = GPG_ERR_CONFLICT;
+  rc = sexp_to_sig (s_sig, &sig, &module_sig);
+  if (rc)
+    goto leave;
 
-  if (! rc)
-    rc = sexp_data_to_mpi (s_hash, gcry_pk_get_nbits (s_pkey), &hash, 0, 0);
+  if (module_key->mod_id != module_sig->mod_id)
+    {
+      rc = GPG_ERR_CONFLICT;
+      goto leave;
+    }
 
-  if (! rc)
-    rc = pubkey_verify (((GcryPubkeySpec *) module_key->spec)->id,
-                       hash, sig, pkey, NULL, NULL);
+  rc = sexp_data_to_mpi (s_hash, gcry_pk_get_nbits (s_pkey), &hash, 0, 0);
+  if (rc)
+    goto leave;
 
+  rc = pubkey_verify (module_key->mod_id, hash, sig, pkey, NULL, NULL);
+
+ leave:
   if (pkey)
     {
       release_mpi_array (pkey);
@@ -1694,281 +1814,324 @@ gcry_pk_verify (gcry_sexp_t s_sig, gcry_sexp_t s_hash, gcry_sexp_t s_pkey)
       ath_mutex_unlock (&pubkeys_registered_lock);
     }
 
-  return gpg_error (rc);
+  return gcry_error (rc);
 }
 
 
-/****************
- * Test a key. This may be used either for a public or a secret key
- * to see whether internal structre is valid.
- *
- * Returns: 0 or an errorcode.
- *
- * s_key = <key-as-defined-in-sexp_to_key>
- */
-gpg_error_t
+/*
+   Test a key.
+
+   This may be used either for a public or a secret key to see whether
+   internal structre is valid.
+  
+   Returns: 0 or an errorcode.
+  
+   s_key = <key-as-defined-in-sexp_to_key> */
+gcry_error_t
 gcry_pk_testkey (gcry_sexp_t s_key)
 {
-  GcryModule *module = NULL;
-  MPI *key = NULL;
-  gpg_err_code_t rc;
+  gcry_module_t module = NULL;
+  gcry_mpi_t *key = NULL;
+  gcry_err_code_t rc;
   
   REGISTER_DEFAULT_PUBKEYS;
 
-  /* Note we currently support only secret key checking */
+  /* Note we currently support only secret key checking. */
   rc = sexp_to_key (s_key, 1, &key, &module);
   if (! rc)
     {
-      rc = pubkey_check_secret_key (((GcryPubkeySpec *) module->spec)->id, key);
+      rc = pubkey_check_secret_key (module->mod_id, key);
       release_mpi_array (key);
       gcry_free (key);
     }
-  return gpg_error (rc);
+  return gcry_error (rc);
 }
 
 
-/****************
- * Create a public key pair and return it in r_key.
- * How the key is created depends on s_parms:
- * (genkey
- *  (algo
- *    (parameter_name_1 ....)
- *     ....
- *    (parameter_name_n ....)
- * ))
- * The key is returned in a format depending on the
- * algorithm. Both, private and secret keys are returned
- * and optionally some additional informatin.
- * For elgamal we return this structure:
- * (key-data
- *  (public-key
- *    (elg
- *     (p <mpi>)
- *     (g <mpi>)
- *     (y <mpi>)
- *    )
- *  )
- *  (private-key
- *    (elg
- *     (p <mpi>)
- *     (g <mpi>)
- *     (y <mpi>)
- *     (x <mpi>)
- *    )
- *  )
- *  (misc-key-info
- *     (pm1-factors n1 n2 ... nn)
- *  )
- * )
+/*
+  Create a public key pair and return it in r_key.
+  How the key is created depends on s_parms:
+  (genkey
+   (algo
+     (parameter_name_1 ....)
+      ....
+     (parameter_name_n ....)
+  ))
+  The key is returned in a format depending on the
+  algorithm. Both, private and secret keys are returned
+  and optionally some additional informatin.
+  For elgamal we return this structure:
+  (key-data
+   (public-key
+     (elg
+       (p <mpi>)
+       (g <mpi>)
+       (y <mpi>)
+     )
+   )
+   (private-key
+     (elg
+       (p <mpi>)
+       (g <mpi>)
+       (y <mpi>)
+       (x <mpi>)
+     )
+   )
+   (misc-key-info
+      (pm1-factors n1 n2 ... nn)
+   ))
  */
-gpg_error_t
+gcry_error_t
 gcry_pk_genkey (gcry_sexp_t *r_key, gcry_sexp_t s_parms)
 {
-  GcryPubkeySpec *pubkey = NULL;
-  GcryModule *module = NULL;
+  gcry_pk_spec_t *pubkey = NULL;
+  gcry_module_t module = NULL;
   gcry_sexp_t list = NULL, l2 = NULL;
   const char *name;
   size_t n;
-  gpg_err_code_t rc = GPG_ERR_NO_ERROR;
+  gcry_err_code_t rc = GPG_ERR_NO_ERROR;
   int i;
   const char *algo_name = NULL;
   int algo;
   const char *sec_elems = NULL, *pub_elems = NULL;
-  gcry_mpi_t skey[10] = { NULL }, *factors = NULL;
+  gcry_mpi_t skey[10], *factors = NULL;
   unsigned int nbits = 0;
   unsigned long use_e = 0;
+  unsigned int qbits;
+  char *name_terminated;
 
   REGISTER_DEFAULT_PUBKEYS;
 
+  skey[0] = NULL;
   *r_key = NULL;
-  list = gcry_sexp_find_token (s_parms, "genkey", 0);
-  if (! list)
-    rc = GPG_ERR_INV_OBJ; /* Does not contain genkey data */
 
-  if (! rc)
+  list = gcry_sexp_find_token (s_parms, "genkey", 0);
+  if (!list)
     {
-      l2 = gcry_sexp_cadr (list);
-      gcry_sexp_release (list);
-      list = l2;
-      l2 = NULL;
-      if (! list)
-       rc = GPG_ERR_NO_OBJ; /* no cdr for the genkey */
+      rc = GPG_ERR_INV_OBJ; /* Does not contain genkey data. */
+      goto leave;
     }
 
-  if (! rc)
+  l2 = gcry_sexp_cadr (list);
+  gcry_sexp_release (list);
+  list = l2;
+  l2 = NULL;
+  if (! list)
     {
-      name = gcry_sexp_nth_data (list, 0, &n);
-      if (! name)
-       rc = GPG_ERR_INV_OBJ; /* algo string missing */
+      rc = GPG_ERR_NO_OBJ; /* No cdr for the genkey. */
+      goto leave;
     }
 
-  if (! rc)
+  name = gcry_sexp_nth_data (list, 0, &n);
+  if (! name)
     {
-      char *name_terminated = gcry_xmalloc (n + 1);
-      strncpy (name_terminated, name, n);
-      name_terminated[n] = 0;
-
-      ath_mutex_lock (&pubkeys_registered_lock);
-      module = gcry_pubkey_lookup_name (name_terminated);
-      ath_mutex_unlock (&pubkeys_registered_lock);
-
-      free (name_terminated);
-
-      if (! module)
-       rc = GPG_ERR_PUBKEY_ALGO; /* unknown algorithm */
-      else
-       {
-         pubkey = (GcryPubkeySpec *) module->spec;
-         algo = pubkey->id;
-         algo_name = pubkey->name;
-         pub_elems = pubkey->elements_pkey;
-         sec_elems = pubkey->elements_skey;
-       }
+      rc = GPG_ERR_INV_OBJ; /* Algo string missing. */
+      goto leave;
     }
 
-  if (! rc)
+  name_terminated = gcry_malloc (n + 1);
+  if (!name_terminated)
     {
-      l2 = gcry_sexp_find_token (list, "rsa-use-e", 0);
-      if (l2)
-       {
-         char buf[50];
-
-         name = gcry_sexp_nth_data (l2, 1, &n);
-         if ((! name) || (n >= DIM (buf) - 1))
-           rc = GPG_ERR_INV_OBJ; /* no value or value too large */
-         else
-           {
-             memcpy (buf, name, n);
-             buf[n] = 0;
-             use_e = strtoul (buf, NULL, 0);
-           }
-         gcry_sexp_release (l2);
-         l2 = NULL;
-       }
-      else
-       use_e = 65537; /* not given, use the value generated by old versions. */
+      rc = gpg_err_code_from_errno (errno);
+      goto leave;
     }
+  memcpy (name_terminated, name, n);
+  name_terminated[n] = 0;
+  ath_mutex_lock (&pubkeys_registered_lock);
+  module = gcry_pk_lookup_name (name_terminated);
+  ath_mutex_unlock (&pubkeys_registered_lock);
+  gcry_free (name_terminated);
 
-  if (! rc)
+  if (! module)
     {
-      l2 = gcry_sexp_find_token (list, "nbits", 0);
-      gcry_sexp_release (list);
-      list = l2;
-      l2 = NULL;
-      if (! list)
-       rc = GPG_ERR_NO_OBJ; /* no nbits parameter */
-      else
-       {
-         name = gcry_sexp_nth_data (list, 1, &n);
-         if (! name)
-           rc = GPG_ERR_INV_OBJ; /* nbits without a cdr */
-         else
-           {
-             char *p = gcry_xmalloc (n + 1);
-             memcpy (p, name, n);
-             p[n] = 0;
-             nbits = (unsigned int) strtol (p, NULL, 0);
-             gcry_free (p);
-           }
-       }
+      rc = GPG_ERR_PUBKEY_ALGO; /* Unknown algorithm. */
+      goto leave;
     }
+  
+  pubkey = (gcry_pk_spec_t *) module->spec;
+  algo = module->mod_id;
+  algo_name = pubkey->aliases? *pubkey->aliases : NULL;
+  if (!algo_name || !*algo_name)
+    algo_name = pubkey->name;
+  pub_elems = pubkey->elements_pkey;
+  sec_elems = pubkey->elements_skey;
+
+  /* Handle the optional rsa-use-e element. */
+  l2 = gcry_sexp_find_token (list, "rsa-use-e", 0);
+  if (l2)
+    {
+      char buf[50];
 
-  if (! rc)
-    rc = pubkey_generate (pubkey->id, nbits, use_e, skey, &factors);
+      name = gcry_sexp_nth_data (l2, 1, &n);
+      if ((! name) || (n >= DIM (buf) - 1))
+        {
+          rc = GPG_ERR_INV_OBJ; /* No value or value too large. */
+          goto leave;
+        }
+      memcpy (buf, name, n);
+      buf[n] = 0;
+      use_e = strtoul (buf, NULL, 0);
+      gcry_sexp_release (l2);
+      l2 = NULL;
+    }
+  else
+    use_e = 65537; /* Not given, use the value generated by old versions. */
 
-  if (! rc)
+  /* Handle the optional qbits element. */
+  l2 = gcry_sexp_find_token (list, "qbits", 0);
+  if (l2)
     {
-      char *string, *p;
-      size_t nelem=0, nelem_cp = 0, needed=0;
-      gcry_mpi_t mpis[30];
+      char buf[50];
 
-      nelem = strlen (pub_elems) + strlen (sec_elems);
-      for (i = 0; factors[i]; i++)
-       nelem++;
-      nelem_cp = nelem;
+      name = gcry_sexp_nth_data (l2, 1, &n);
+      if ((! name) || (n >= DIM (buf) - 1))
+        {
+          rc = GPG_ERR_INV_OBJ; /* No value or value too large. */
+          goto leave;
+        }
+      memcpy (buf, name, n);
+      buf[n] = 0;
+      qbits = (unsigned int)strtoul (buf, NULL, 0);
+      gcry_sexp_release (l2);
+      l2 = NULL;
+    }
+  else
+    qbits = 0;
 
-      needed += nelem * 10;
-      needed += 2 * strlen (algo_name) + 300;
-      if (nelem > DIM (mpis))
-       BUG ();
+  /* Now parse the required nbits element. */
+  l2 = gcry_sexp_find_token (list, "nbits", 0);
+  gcry_sexp_release (list);
+  list = l2;
+  l2 = NULL;
+  
+  if (! list)
+    {
+      rc = GPG_ERR_NO_OBJ; /* No nbits parameter. */
+      goto leave;
+    }
 
-      /* build the string */
-      nelem = 0;
-      string = p = gcry_xmalloc (needed);
-      p = stpcpy (p, "(key-data");
-      p = stpcpy (p, "(public-key(");
-      p = stpcpy (p, algo_name);
-      for(i = 0; pub_elems[i]; i++)
-       {
-         *p++ = '(';
-         *p++ = pub_elems[i];
-         p = stpcpy (p, "%m)");
-         mpis[nelem++] = skey[i];
-       }
-      p = stpcpy (p, "))");
-      p = stpcpy (p, "(private-key(");
-      p = stpcpy (p, algo_name);
-      for (i = 0; sec_elems[i]; i++)
-       {
-         *p++ = '(';
-         *p++ = sec_elems[i];
-         p = stpcpy (p, "%m)");
-         mpis[nelem++] = skey[i];
-       }
-      p = stpcpy (p, "))");
+  name = gcry_sexp_nth_data (list, 1, &n);
+  if (! name)
+    {
+      rc = GPG_ERR_INV_OBJ; /* nbits without a cdr. */
+      goto leave;
+    }
+  
+  name_terminated = gcry_malloc (n + 1);
+  if (!name_terminated)
+    {
+      rc = gpg_err_code_from_errno (errno);
+      goto leave;
+    }
+  memcpy (name_terminated, name, n);
+  name_terminated[n] = 0;
+  nbits = (unsigned int) strtoul (name_terminated, NULL, 0);
+  gcry_free (name_terminated);
 
-      /* Very ugly hack to make release_mpi_array() work FIXME */
-      skey[i] = NULL;
+  rc = pubkey_generate (module->mod_id, nbits, qbits, use_e, skey, &factors);
+  if (rc)
+    goto leave;
 
-      p = stpcpy (p, "(misc-key-info(pm1-factors");
-      for(i = 0; factors[i]; i++)
-       {
-         p = stpcpy (p, "%m");
-         mpis[nelem++] = factors[i];
-       }
-      strcpy (p, ")))");
+  {
+    char *string, *p;
+    size_t nelem=0, nelem_cp = 0, needed=0;
+    gcry_mpi_t mpis[30];
+    
+    nelem = strlen (pub_elems) + strlen (sec_elems);
+    for (i = 0; factors[i]; i++)
+      nelem++;
+    nelem_cp = nelem;
+
+    needed += nelem * 10;
+    needed += 2 * strlen (algo_name) + 300;
+    if (nelem > DIM (mpis))
+      BUG ();
+
+    /* Build the string. */
+    nelem = 0;
+    string = p = gcry_malloc (needed);
+    if (!string)
+      {
+        rc = gpg_err_code_from_errno (errno);
+        goto leave;
+      }
+    p = stpcpy (p, "(key-data");
+    p = stpcpy (p, "(public-key(");
+    p = stpcpy (p, algo_name);
+    for(i = 0; pub_elems[i]; i++)
+      {
+        *p++ = '(';
+        *p++ = pub_elems[i];
+        p = stpcpy (p, "%m)");
+        mpis[nelem++] = skey[i];
+      }
+    p = stpcpy (p, "))");
+    p = stpcpy (p, "(private-key(");
+    p = stpcpy (p, algo_name);
+    for (i = 0; sec_elems[i]; i++)
+      {
+        *p++ = '(';
+        *p++ = sec_elems[i];
+        p = stpcpy (p, "%m)");
+        mpis[nelem++] = skey[i];
+      }
+    p = stpcpy (p, "))");
 
-      while (nelem < DIM (mpis))
-       mpis[nelem++] = NULL;
+    /* Very ugly hack to make release_mpi_array() work FIXME */
+    skey[i] = NULL;
 
+    p = stpcpy (p, "(misc-key-info(pm1-factors");
+    for(i = 0; factors[i]; i++)
       {
-       int elem_n = strlen (pub_elems) + strlen (sec_elems), i;
-       void **arg_list = malloc (sizeof (void *) * nelem_cp);
-       if (arg_list)
-         {
-           for (i = 0; i < elem_n; i++)
-             arg_list[i] = &mpis[i];
-           for (; i < nelem_cp; i++)
-             arg_list[i] = &factors[i - elem_n];
+        p = stpcpy (p, "%m");
+        mpis[nelem++] = factors[i];
+      }
+    strcpy (p, ")))");
 
-           rc = gcry_sexp_build_array (r_key, NULL, string, arg_list);
+    while (nelem < DIM (mpis))
+      mpis[nelem++] = NULL;
 
-           free (arg_list);
-         }
-       else
-         rc = gpg_err_code_from_errno (errno);
-      }
+    {
+      int elem_n = strlen (pub_elems) + strlen (sec_elems);
+      void **arg_list;
 
+      arg_list = malloc (nelem_cp * sizeof *arg_list);
+      if (!arg_list)
+        {
+          rc = gpg_err_code_from_errno (errno);
+          goto leave;
+        }
+      for (i = 0; i < elem_n; i++)
+        arg_list[i] = mpis + i;
+      for (; i < nelem_cp; i++)
+        arg_list[i] = factors + i - elem_n;
+      
+      rc = gcry_sexp_build_array (r_key, NULL, string, arg_list);
+      free (arg_list);
       if (rc)
        BUG ();
-      assert (DIM (mpis) == 30);       /* ? */
-      gcry_free (string);
+      assert (DIM (mpis) == 30); /* Reminder to make sure that the
+                                    array gets increased if new
+                                    parameters are added. */
     }
+    gcry_free (string);
+  }
 
+ leave:
   release_mpi_array (skey);
-  /* no free:  skey is a static array */
-
+  /* Don't free SKEY itself, it is a static array. */
+    
   if (factors)
     {
       release_mpi_array ( factors );
       gcry_free (factors);
     }
-
+  
   if (l2)
     gcry_sexp_release (l2);
   if (list)
     gcry_sexp_release (list);
-
+  
   if (module)
     {
       ath_mutex_lock (&pubkeys_registered_lock);
@@ -1976,23 +2139,23 @@ gcry_pk_genkey (gcry_sexp_t *r_key, gcry_sexp_t s_parms)
       ath_mutex_unlock (&pubkeys_registered_lock);
     }
 
-  return gpg_error (rc);
+  return gcry_error (rc);
 }
 
-/****************
- * Get the number of nbits from the public key
- * Hmmm: Should we have really this function or is it
- * better to have a more general function to retrieve
- * different propoerties of the key?
- */
+
+/* 
+   Get the number of nbits from the public key.
+
+   Hmmm: Should we have really this function or is it better to have a
  more general function to retrieve different propoerties of the key?  */
 unsigned int
 gcry_pk_get_nbits (gcry_sexp_t key)
 {
-  GcryModule *module = NULL;
-  GcryPubkeySpec *pubkey;
-  MPI *keyarr = NULL;
+  gcry_module_t module = NULL;
+  gcry_pk_spec_t *pubkey;
+  gcry_mpi_t *keyarr = NULL;
   unsigned int nbits = 0;
-  gpg_err_code_t rc;
+  gcry_err_code_t rc;
 
   REGISTER_DEFAULT_PUBKEYS;
 
@@ -2000,16 +2163,14 @@ gcry_pk_get_nbits (gcry_sexp_t key)
   if (rc == GPG_ERR_INV_OBJ)
     rc = sexp_to_key (key, 1, &keyarr, &module);
   if (rc)
-    return 0;
-  else
-    {
-      pubkey = (GcryPubkeySpec *) module->spec;
-      nbits = (*pubkey->get_nbits) (pubkey->id, keyarr);
+    return 0; /* Error - 0 is a suitable indication for that. */
 
-      ath_mutex_lock (&pubkeys_registered_lock);
-      _gcry_module_release (module);
-      ath_mutex_unlock (&pubkeys_registered_lock);
-    }
+  pubkey = (gcry_pk_spec_t *) module->spec;
+  nbits = (*pubkey->get_nbits) (module->mod_id, keyarr);
+  
+  ath_mutex_lock (&pubkeys_registered_lock);
+  _gcry_module_release (module);
+  ath_mutex_unlock (&pubkeys_registered_lock);
 
   release_mpi_array (keyarr);
   gcry_free (keyarr);
@@ -2029,8 +2190,8 @@ unsigned char *
 gcry_pk_get_keygrip (gcry_sexp_t key, unsigned char *array)
 {
   gcry_sexp_t list = NULL, l2 = NULL;
-  GcryPubkeySpec *pubkey = NULL;
-  GcryModule *module = NULL;
+  gcry_pk_spec_t *pubkey = NULL;
+  gcry_module_t module = NULL;
   const char *s, *name;
   size_t n;
   int idx;
@@ -2040,14 +2201,16 @@ gcry_pk_get_keygrip (gcry_sexp_t key, unsigned char *array)
 
   REGISTER_DEFAULT_PUBKEYS;
 
-  /* check that the first element is valid */
+  /* Check that the first element is valid. */
   list = gcry_sexp_find_token (key, "public-key", 0);
   if (! list)
     list = gcry_sexp_find_token (key, "private-key", 0);
   if (! list)
     list = gcry_sexp_find_token (key, "protected-private-key", 0);
   if (! list)
-    return NULL; /* no public- or private-key object */
+    list = gcry_sexp_find_token (key, "shadowed-private-key", 0);
+  if (! list)
+    return NULL; /* No public- or private-key object. */
 
   l2 = gcry_sexp_cadr (list);
   gcry_sexp_release (list);
@@ -2056,29 +2219,28 @@ gcry_pk_get_keygrip (gcry_sexp_t key, unsigned char *array)
 
   name = gcry_sexp_nth_data (list, 0, &n);
   if (! name)
-    goto fail; /* invalid structure of object */
+    goto fail; /* Invalid structure of object. */
 
-  
   {
-    char *name_terminated = gcry_xmalloc (n + 1);
-    strncpy (name_terminated, name, n);
+    char *name_terminated = gcry_malloc (n + 1);
+    if (!name_terminated)
+      goto fail;
+    memcpy (name_terminated, name, n);
     name_terminated[n] = 0;
-
     ath_mutex_lock (&pubkeys_registered_lock);
-    module = gcry_pubkey_lookup_name (name_terminated);
+    module = gcry_pk_lookup_name (name_terminated);
     ath_mutex_unlock (&pubkeys_registered_lock);
-
-    free (name_terminated);
+    gcry_free (name_terminated);
   }
 
   if (! module)
     goto fail; /* unknown algorithm */
-  else
-    pubkey = (GcryPubkeySpec *) module->spec;
+
+  pubkey = (gcry_pk_spec_t *) module->spec;
 
   /* FIXME, special handling should be implemented by the algorithms,
      not by the libgcrypt core.  */
-  is_rsa = pubkey->id == GCRY_PK_RSA;
+  is_rsa = module->mod_id == GCRY_PK_RSA;
   elems = pubkey->elements_grip;
   if (! elems)
     goto fail; /* no grip parameter */
@@ -2104,10 +2266,11 @@ gcry_pk_get_keygrip (gcry_sexp_t key, unsigned char *array)
           sprintf (buf, "(1:%c%u:", *s, (unsigned int)datalen);
           gcry_md_write (md, buf, strlen (buf));
         }
-      /* pkcs-15 says that for RSA only the modulus should be hashed -
+  
+      /* PKCS-15 says that for RSA only the modulus should be hashed -
          however, it is not clear wether this is meant to has the raw
          bytes assuming this is an unsigned integer or whether the DER
-         required 0 should be prefixed. We hash th raw bytes.  For
+         required 0 should be prefixed. We hash the raw bytes.  For
          non-RSA we hash S-expressions. */
       gcry_md_write (md, data, datalen);
       gcry_sexp_release (l2);
@@ -2121,10 +2284,11 @@ gcry_pk_get_keygrip (gcry_sexp_t key, unsigned char *array)
       if (! array)
         goto fail;
     }
+
   memcpy (array, gcry_md_read (md, GCRY_MD_SHA1), 20);
   gcry_md_close (md);
   gcry_sexp_release (list);
-   return array;
+  return array;
 
  fail:
   if (l2)
@@ -2136,21 +2300,20 @@ gcry_pk_get_keygrip (gcry_sexp_t key, unsigned char *array)
 }
 
 
-gpg_error_t
+gcry_error_t
 gcry_pk_ctl (int cmd, void *buffer, size_t buflen)
 {
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
 
   REGISTER_DEFAULT_PUBKEYS;
 
   switch (cmd)
     {
     case GCRYCTL_DISABLE_ALGO:
-      /* this one expects a buffer pointing to an integer with the
-       * algo number.
-       */
+      /* This one expects a buffer pointing to an integer with the
+         algo number.  */
       if ((! buffer) || (buflen != sizeof (int)))
-       err = GPG_ERR_CIPHER_ALGO;  /* FIXME?  */
+       err = GPG_ERR_INV_ARG;
       else
        disable_pubkey_algo (*((int *) buffer));
       break;
@@ -2159,34 +2322,33 @@ gcry_pk_ctl (int cmd, void *buffer, size_t buflen)
       err = GPG_ERR_INV_OP;
     }
 
-  return gpg_error (err);
+  return gcry_error (err);
 }
 
 
-/****************
- * Return information about the given algorithm
- * WHAT select the kind of information returned:
- *  GCRYCTL_TEST_ALGO:
- *     Returns 0 when the specified algorithm is available for use.
- *     Buffer must be NULL, nbytes  may have the address of a variable
- *     with the required usage of the algorithm. It may be 0 for don't
- *     care or a combination of the GCRY_PK_USAGE_xxx flags;
- *  GCRYCTL_GET_ALGO_USAGE:
- *      Return the usage glafs for the give algo.  An invalid alog
- *      does return 0.  Disabled algos are ignored here becuase we
- *      only want to know whether the algo is at all capable of
- *      the usage.
- *
- * Note:  Because this function is in most cases used to return an
- * integer value, we can make it easier for the caller to just look at
- * the return value.  The caller will in all cases consult the value
- * and thereby detecting whether a error occured or not (i.e. while checking
- * the block size)
- */
-gpg_error_t
-gcry_pk_algo_info (int id, int what, void *buffer, size_t *nbytes)
+/*
+   Return information about the given algorithm
+   WHAT select the kind of information returned:
+    GCRYCTL_TEST_ALGO:
+        Returns 0 when the specified algorithm is available for use.
+        Buffer must be NULL, nbytes  may have the address of a variable
+        with the required usage of the algorithm. It may be 0 for don't
+        care or a combination of the GCRY_PK_USAGE_xxx flags;
+    GCRYCTL_GET_ALGO_USAGE:
+        Return the usage glafs for the give algo.  An invalid alog
+        does return 0.  Disabled algos are ignored here becuase we
+        only want to know whether the algo is at all capable of
+        the usage.
+  
+   Note: Because this function is in most cases used to return an
+   integer value, we can make it easier for the caller to just look at
+   the return value.  The caller will in all cases consult the value
+   and thereby detecting whether a error occured or not (i.e. while
+   checking the block size) */
+gcry_error_t
+gcry_pk_algo_info (int algorithm, int what, void *buffer, size_t *nbytes)
 {
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
 
   switch (what)
     {
@@ -2195,56 +2357,58 @@ gcry_pk_algo_info (int id, int what, void *buffer, size_t *nbytes)
        int use = nbytes ? *nbytes : 0;
        if (buffer)
          err = GPG_ERR_INV_ARG;
-       else if (check_pubkey_algo (id, use))
+       else if (check_pubkey_algo (algorithm, use))
          err = GPG_ERR_PUBKEY_ALGO;
        break;
       }
 
     case GCRYCTL_GET_ALGO_USAGE:
       {
-       GcryModule *pubkey;
+       gcry_module_t pubkey;
        int use = 0;
 
        REGISTER_DEFAULT_PUBKEYS;
 
        ath_mutex_lock (&pubkeys_registered_lock);
-       pubkey = gcry_pubkey_lookup_id (id);
+       pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
        if (pubkey)
          {
-           use = ((GcryPubkeySpec *) pubkey->spec)->use;
+           use = ((gcry_pk_spec_t *) pubkey->spec)->use;
            _gcry_module_release (pubkey);
          }
        ath_mutex_unlock (&pubkeys_registered_lock);
 
        /* FIXME? */
        *nbytes = use;
+
+       break;
       }
 
     case GCRYCTL_GET_ALGO_NPKEY:
       {
        /* FIXME?  */
-       int npkey = pubkey_get_npkey (id);
+       int npkey = pubkey_get_npkey (algorithm);
        *nbytes = npkey;
        break;
       }
     case GCRYCTL_GET_ALGO_NSKEY:
       {
        /* FIXME?  */
-       int nskey = pubkey_get_nskey (id);
+       int nskey = pubkey_get_nskey (algorithm);
        *nbytes = nskey;
        break;
       }
     case GCRYCTL_GET_ALGO_NSIGN:
       {
        /* FIXME?  */
-       int nsign = pubkey_get_nsig (id);
+       int nsign = pubkey_get_nsig (algorithm);
        *nbytes = nsign;
        break;
       }
     case GCRYCTL_GET_ALGO_NENCR:
       {
        /* FIXME?  */
-       int nencr = pubkey_get_nenc (id);
+       int nencr = pubkey_get_nenc (algorithm);
        *nbytes = nencr;
        break;
       }
@@ -2253,15 +2417,124 @@ gcry_pk_algo_info (int id, int what, void *buffer, size_t *nbytes)
       err = GPG_ERR_INV_OP;
     }
 
-  return gpg_error (err);
+  return gcry_error (err);
 }
 
-gpg_err_code_t
+
+gcry_err_code_t
 _gcry_pk_init (void)
 {
-  gpg_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
+
+  REGISTER_DEFAULT_PUBKEYS;
+
+  return err;
+}
+
+
+gcry_err_code_t
+_gcry_pk_module_lookup (int algorithm, gcry_module_t *module)
+{
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
+  gcry_module_t pubkey;
 
   REGISTER_DEFAULT_PUBKEYS;
 
+  ath_mutex_lock (&pubkeys_registered_lock);
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algorithm);
+  if (pubkey)
+    *module = pubkey;
+  else
+    err = GPG_ERR_PUBKEY_ALGO;
+  ath_mutex_unlock (&pubkeys_registered_lock);
+
+  return err;
+}
+
+
+void
+_gcry_pk_module_release (gcry_module_t module)
+{
+  ath_mutex_lock (&pubkeys_registered_lock);
+  _gcry_module_release (module);
+  ath_mutex_unlock (&pubkeys_registered_lock);
+}
+
+/* Get a list consisting of the IDs of the loaded pubkey modules.  If
+   LIST is zero, write the number of loaded pubkey modules to
+   LIST_LENGTH and return.  If LIST is non-zero, the first
+   *LIST_LENGTH algorithm IDs are stored in LIST, which must be of
+   according size.  In case there are less pubkey modules than
+   *LIST_LENGTH, *LIST_LENGTH is updated to the correct number.  */
+gcry_error_t
+gcry_pk_list (int *list, int *list_length)
+{
+  gcry_err_code_t err = GPG_ERR_NO_ERROR;
+
+  ath_mutex_lock (&pubkeys_registered_lock);
+  err = _gcry_module_list (pubkeys_registered, list, list_length);
+  ath_mutex_unlock (&pubkeys_registered_lock);
+
+  return err;
+}
+
+gcry_err_code_t
+_gcry_pk_get_elements (int algo, char **enc, char **sig)
+{
+  gcry_module_t pubkey;
+  gcry_pk_spec_t *spec;
+  gcry_err_code_t err;
+  char *enc_cp;
+  char *sig_cp;
+
+  REGISTER_DEFAULT_PUBKEYS;
+
+  enc_cp = NULL;
+  sig_cp = NULL;
+  spec = NULL;
+
+  pubkey = _gcry_module_lookup_id (pubkeys_registered, algo);
+  if (! pubkey)
+    {
+      err = GPG_ERR_INTERNAL;
+      goto out;
+    }
+  spec = pubkey->spec;
+
+  if (enc)
+    {
+      enc_cp = strdup (spec->elements_enc);
+      if (! enc_cp)
+       {
+         err = gpg_err_code_from_errno (errno);
+         goto out;
+       }
+    }
+  
+  if (sig)
+    {
+      sig_cp = strdup (spec->elements_sig);
+      if (! sig_cp)
+       {
+         err = gpg_err_code_from_errno (errno);
+         goto out;
+       }
+    }
+
+  if (enc)
+    *enc = enc_cp;
+  if (sig)
+    *sig = sig_cp;
+  err = 0;
+
+ out:
+
+  _gcry_module_release (pubkey);
+  if (err)
+    {
+      free (enc_cp);
+      free (sig_cp);
+    }
+
   return err;
 }