Fix last change.
[gpgme.git] / gpgme / genkey.c
index 427365e..9353213 100644 (file)
 #endif
 #include <stdlib.h>
 #include <string.h>
+#include <errno.h>
 
-#include "util.h"
+#include "gpgme.h"
 #include "context.h"
 #include "ops.h"
 
-
-struct genkey_result_s
+\f
+typedef struct
 {
-  int created_primary : 1;
-  int created_sub : 1;
-  char *fpr;
-};
+  struct _gpgme_op_genkey_result result;
+
+  /* The key parameters passed to the crypto engine.  */
+  gpgme_data_t key_parameter;
+} *op_data_t;
 
 
-void
-_gpgme_release_genkey_result (GenKeyResult result)
+static void
+release_op_data (void *hook)
 {
-  if (!result)
-    return;
-  if (result->fpr)
-    free (result->fpr);
-  free (result);
+  op_data_t opd = (op_data_t) hook;
+  
+  if (opd->result.fpr)
+    free (opd->result.fpr);
+  if (opd->key_parameter)
+    gpgme_data_release (opd->key_parameter);
 }
 
 
-static GpgmeError
-genkey_status_handler (GpgmeCtx ctx, GpgmeStatusCode code, char *args)
+gpgme_genkey_result_t
+gpgme_op_genkey_result (gpgme_ctx_t ctx)
+{
+  void *hook;
+  op_data_t opd;
+  gpgme_error_t err;
+
+  err = _gpgme_op_data_lookup (ctx, OPDATA_GENKEY, &hook, -1, NULL);
+  opd = hook;
+  if (err || !opd)
+    return NULL;
+
+  return &opd->result;
+}
+
+\f
+static gpgme_error_t
+genkey_status_handler (void *priv, gpgme_status_code_t code, char *args)
 {
-  GpgmeError err = _gpgme_progress_status_handler (ctx, code, args);
+  gpgme_ctx_t ctx = (gpgme_ctx_t) priv;
+  gpgme_error_t err;
+  void *hook;
+  op_data_t opd;
+
+  /* Pipe the status code through the progress status handler.  */
+  err = _gpgme_progress_status_handler (ctx, code, args);
   if (err)
     return err;
 
-  test_and_allocate_result (ctx, genkey);
+  err = _gpgme_op_data_lookup (ctx, OPDATA_GENKEY, &hook, -1, NULL);
+  opd = hook;
+  if (err)
+    return err;
 
   switch (code)
     {
@@ -63,25 +91,24 @@ genkey_status_handler (GpgmeCtx ctx, GpgmeStatusCode code, char *args)
       if (args && *args)
        {
          if (*args == 'B' || *args == 'P')
-           ctx->result.genkey->created_primary = 1;
+           opd->result.primary = 1;
          if (*args == 'B' || *args == 'S')
-           ctx->result.genkey->created_sub = 1;
+           opd->result.sub = 1;
          if (args[1] == ' ')
            {
-             if (ctx->result.genkey->fpr)
-               free (ctx->result.genkey->fpr);
-             ctx->result.genkey->fpr = strdup (&args[2]);
-             if (!ctx->result.genkey->fpr)
-               return GPGME_Out_Of_Core;
+             if (opd->result.fpr)
+               free (opd->result.fpr);
+             opd->result.fpr = strdup (&args[2]);
+             if (!opd->result.fpr)
+               return gpg_error_from_errno (errno);
            }
        }
       break;
 
     case GPGME_STATUS_EOF:
       /* FIXME: Should return some more useful error value.  */
-      if (!ctx->result.genkey->created_primary
-         && !ctx->result.genkey->created_sub)
-       return GPGME_General_Error;
+      if (!opd->result.primary && !opd->result.sub)
+       return gpg_error (GPG_ERR_GENERAL);
       break;
 
     default:
@@ -91,135 +118,87 @@ genkey_status_handler (GpgmeCtx ctx, GpgmeStatusCode code, char *args)
 }
 
 
-static GpgmeError
-_gpgme_op_genkey_start (GpgmeCtx ctx, int synchronous, const char *parms,
-                       GpgmeData pubkey, GpgmeData seckey)
+static gpgme_error_t
+get_key_parameter (const char *parms, gpgme_data_t *key_parameter)
 {
-  int err = 0;
-  const char *s, *s2, *sx;
+  const char *content;
+  const char *attrib;
+  const char *endtag;
+
+  /* Extract the key parameter from the XML structure.  */
+  parms = strstr (parms, "<GnupgKeyParms ");
+  if (!parms)
+    return gpg_error (GPG_ERR_INV_VALUE);
+
+  content = strchr (parms, '>');
+  if (!content)
+    return gpg_error (GPG_ERR_INV_VALUE);
+  content++;
+
+  attrib = strstr (parms, "format=\"internal\"");
+  if (!attrib || attrib >= content)
+    return gpg_error (GPG_ERR_INV_VALUE);
+
+  endtag = strstr (content, "</GnupgKeyParms>");
+  /* FIXME: Check that there are no control statements inside.  */
+  while (*content == '\n')
+    content++;
+
+  return gpgme_data_new_from_mem (key_parameter, content,
+                                 endtag - content, 0);
+}
+
 
+static gpgme_error_t
+genkey_start (gpgme_ctx_t ctx, int synchronous, const char *parms,
+             gpgme_data_t pubkey, gpgme_data_t seckey)
+{
+  gpgme_error_t err;
+  void *hook;
+  op_data_t opd;
   err = _gpgme_op_reset (ctx, synchronous);
   if (err)
-    goto leave;
-
-  gpgme_data_release (ctx->help_data_1);
-  ctx->help_data_1 = NULL;
-
-  if ((parms = strstr (parms, "<GnupgKeyParms ")) 
-      && (s = strchr (parms, '>'))
-      && (sx = strstr (parms, "format=\"internal\""))
-      && sx < s
-      && (s2 = strstr (s+1, "</GnupgKeyParms>")))
-    {
-      /* FIXME: Check that there are no control statements inside.  */
-      s++;  /* Skip '>'.  */
-      while (*s == '\n')
-       s++;
-      err = gpgme_data_new_from_mem (&ctx->help_data_1, s, s2-s, 1);
-    }
-  else 
-    err = GPGME_Invalid_Value;
+    return err;
+  
+  err = _gpgme_op_data_lookup (ctx, OPDATA_GENKEY, &hook,
+                              sizeof (*opd), release_op_data);
+  opd = hook;
+  if (err)
+    return err;
 
+  err = get_key_parameter (parms, &opd->key_parameter);
   if (err)
-    goto leave;
+    return err;
 
   _gpgme_engine_set_status_handler (ctx->engine, genkey_status_handler, ctx);
-  _gpgme_engine_set_verbosity (ctx->engine, ctx->verbosity);
-
-  err = _gpgme_engine_op_genkey (ctx->engine, ctx->help_data_1, ctx->use_armor,
-                                pubkey, seckey);
 
- leave:
-  if (err)
-    {
-      ctx->pending = 0; 
-      _gpgme_engine_release (ctx->engine);
-      ctx->engine = NULL;
-    }
-  return err;
+  return _gpgme_engine_op_genkey (ctx->engine, opd->key_parameter,
+                                 ctx->use_armor, pubkey, seckey);
 }
 
 
-/**
- * gpgme_op_genkey:
- * @c: the context
- * @parms: XML string with the key parameters
- * @pubkey: Returns the public key
- * @seckey: Returns the secret key
- * 
- * Generate a new key and store the key in the default keyrings if
- * both @pubkey and @seckey are NULL.  If @pubkey and @seckey are
- * given, the newly created key will be returned in these data
- * objects.  This function just starts the gheneration and does not
- * wait for completion.
- *
- * Here is an example on how @parms should be formatted; for deatils
- * see the file doc/DETAILS from the GnuPG distribution.
- *
- * <literal>
- * <![CDATA[
- * <GnupgKeyParms format="internal">
- * Key-Type: DSA
- * Key-Length: 1024
- * Subkey-Type: ELG-E
- * Subkey-Length: 1024
- * Name-Real: Joe Tester
- * Name-Comment: with stupid passphrase
- * Name-Email: joe@foo.bar
- * Expire-Date: 0
- * Passphrase: abc
- * </GnupgKeyParms>
- * ]]>
- * </literal> 
- *
- * Strings should be given in UTF-8 encoding.  The format we support
- * for now is only "internal".  The content of the
- * &lt;GnupgKeyParms&gt; container is passed verbatim to GnuPG.
- * Control statements are not allowed.
- * 
- * Return value: 0 for success or an error code
- **/
-GpgmeError
-gpgme_op_genkey_start (GpgmeCtx ctx, const char *parms,
-                      GpgmeData pubkey, GpgmeData seckey)
+/* Generate a new keypair and add it to the keyring.  PUBKEY and
+   SECKEY should be null for now.  PARMS specifies what keys should be
+   generated.  */
+gpgme_error_t
+gpgme_op_genkey_start (gpgme_ctx_t ctx, const char *parms,
+                      gpgme_data_t pubkey, gpgme_data_t seckey)
 {
-  return _gpgme_op_genkey_start (ctx, 0, parms, pubkey, seckey);
+  return genkey_start (ctx, 0, parms, pubkey, seckey);
 }
 
 
-/**
- * gpgme_op_genkey:
- * @c: the context
- * @parms: XML string with the key parameters
- * @pubkey: Returns the public key
- * @seckey: Returns the secret key
- * @fpr: Returns the fingerprint of the key.
- *
- * Generate a new key and store the key in the default keyrings if both
- * @pubkey and @seckey are NULL.  If @pubkey and @seckey are given, the newly
- * created key will be returned in these data objects.
- * See gpgme_op_genkey_start() for a description of @parms.
- * 
- * Return value: 0 for success or an error code
- **/
-GpgmeError
-gpgme_op_genkey (GpgmeCtx ctx, const char *parms,
-                 GpgmeData pubkey, GpgmeData seckey,
-                char **fpr)
+/* Generate a new keypair and add it to the keyring.  PUBKEY and
+   SECKEY should be null for now.  PARMS specifies what keys should be
+   generated.  */
+gpgme_error_t
+gpgme_op_genkey (gpgme_ctx_t ctx, const char *parms, gpgme_data_t pubkey,
+                gpgme_data_t seckey)
 {
-  GpgmeError err = _gpgme_op_genkey_start (ctx, 1, parms, pubkey, seckey);
+  gpgme_error_t err;
+
+  err = genkey_start (ctx, 1, parms, pubkey, seckey);
   if (!err)
     err = _gpgme_wait_one (ctx);
-  if (!err && fpr)
-    {
-      if (ctx->result.genkey->fpr)
-       {
-         *fpr = strdup (ctx->result.genkey->fpr);
-         if (!*fpr)
-           return GPGME_Out_Of_Core;
-       }
-      else
-       *fpr = NULL;
-    }
   return err;
 }