2003-01-30 Marcus Brinkmann <marcus@g10code.de>
[gpgme.git] / gpgme / engine-gpgsm.c
index 67f1e7c..475d917 100644 (file)
@@ -1,35 +1,27 @@
-/* engine-gpgsm.c -  GpgSM engine
- *     Copyright (C) 2000 Werner Koch (dd9jn)
- *      Copyright (C) 2001, 2002 g10 Code GmbH
- *
- * This file is part of GPGME.
- *
- * GPGME is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * GPGME is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA
- */
+/* engine-gpgsm.c - GpgSM engine.
+   Copyright (C) 2000 Werner Koch (dd9jn)
+   Copyright (C) 2001, 2002, 2003 g10 Code GmbH
+   This file is part of GPGME.
+
+   GPGME is free software; you can redistribute it and/or modify it
+   under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 2 of the License, or
+   (at your option) any later version.
+   GPGME is distributed in the hope that it will be useful, but
+   WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+   General Public License for more details.
+   You should have received a copy of the GNU General Public License
+   along with GPGME; if not, write to the Free Software Foundation,
+   Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
 
 #if HAVE_CONFIG_H
 #include <config.h>
 #endif
 
-/* FIXME: Correct check?  */
-#ifdef GPGSM_PATH
-#define ENABLE_GPGSM 1
-#endif
-
-#ifdef ENABLE_GPGSM
-
 #include <stdlib.h>
 #include <string.h>
 #include <sys/types.h>
 #include <locale.h>
 #include <fcntl.h> /* FIXME */
 
-#include "rungpg.h"
-#include "status-table.h"
-
 #include "gpgme.h"
 #include "util.h"
-#include "types.h"
 #include "ops.h"
 #include "wait.h"
 #include "io.h"
 #include "key.h"
 #include "sema.h"
 
-#include "engine-gpgsm.h"
-
 #include "assuan.h"
+#include "status-table.h"
+
+#include "engine-backend.h"
+
 
 #define xtoi_1(p)   (*(p) <= '9'? (*(p)- '0'): \
                      *(p) <= 'F'? (*(p)-'A'+10):(*(p)-'a'+10))
 #define xtoi_2(p)   ((xtoi_1(p) * 16) + xtoi_1((p)+1))
 
 
-
 typedef struct
 {
   int fd;      /* FD we talk about.  */
@@ -68,6 +57,7 @@ typedef struct
   void *tag;   /* ID from the user for gpgme_remove_io_callback.  */
 } iocb_data_t;
 
+
 struct gpgsm_object_s
 {
   ASSUAN_CONTEXT assuan_ctx;
@@ -84,17 +74,15 @@ struct gpgsm_object_s
   iocb_data_t message_cb;
   int message_fd_server;
 
-  char *command;
-
   struct
   {
-    GpgStatusHandler fnc;
+    EngineStatusHandler fnc;
     void *fnc_value;
   } status;
 
   struct
   {
-    GpgColonLineHandler fnc;
+    EngineColonLineHandler fnc;
     void *fnc_value;
     struct
     {
@@ -102,14 +90,17 @@ struct gpgsm_object_s
       int linesize;
       int linelen;
     } attic;
+    int any; /* any data line seen */
   } colon; 
 
   struct GpgmeIOCbs io_cbs;
 };
 
+typedef struct gpgsm_object_s *GpgsmObject;
 
-const char *
-_gpgme_gpgsm_get_version (void)
+\f
+static const char *
+gpgsm_get_version (void)
 {
   static const char *gpgsm_version;
   DEFINE_STATIC_LOCK (gpgsm_version_lock);
@@ -123,12 +114,10 @@ _gpgme_gpgsm_get_version (void)
 }
 
 
-GpgmeError
-_gpgme_gpgsm_check_version (void)
+static const char *
+gpgsm_get_req_version (void)
 {
-    return _gpgme_compare_versions (_gpgme_gpgsm_get_version (),
-                                 NEED_GPGSM_VERSION)
-    ? 0 : mk_error (Invalid_Engine);
+  return NEED_GPGSM_VERSION;
 }
 
 
@@ -136,49 +125,32 @@ static void
 close_notify_handler (int fd, void *opaque)
 {
   GpgsmObject gpgsm = opaque;
-  int possibly_done = 0;
 
   assert (fd != -1);
   if (gpgsm->status_cb.fd == fd)
     {
       if (gpgsm->status_cb.tag)
-       {
-         (*gpgsm->io_cbs.remove) (gpgsm->status_cb.tag);
-         possibly_done = 1;
-       }
+       (*gpgsm->io_cbs.remove) (gpgsm->status_cb.tag);
       gpgsm->status_cb.fd = -1;
     }
   else if (gpgsm->input_cb.fd == fd)
     {
       if (gpgsm->input_cb.tag)
-       {
-         (*gpgsm->io_cbs.remove) (gpgsm->input_cb.tag);
-         possibly_done = 1;
-       }
+       (*gpgsm->io_cbs.remove) (gpgsm->input_cb.tag);
       gpgsm->input_cb.fd = -1;
     }
   else if (gpgsm->output_cb.fd == fd)
     {
       if (gpgsm->output_cb.tag)
-       {      
-         (*gpgsm->io_cbs.remove) (gpgsm->output_cb.tag);
-         possibly_done = 1;
-       }
+       (*gpgsm->io_cbs.remove) (gpgsm->output_cb.tag);
       gpgsm->output_cb.fd = -1;
     }
   else if (gpgsm->message_cb.fd == fd)
     {
       if (gpgsm->message_cb.tag)
-       {
-         (*gpgsm->io_cbs.remove) (gpgsm->message_cb.tag);
-         possibly_done = 1;
-       }
+       (*gpgsm->io_cbs.remove) (gpgsm->message_cb.tag);
       gpgsm->message_cb.fd = -1;
     }
-  if (possibly_done && gpgsm->io_cbs.event
-      && gpgsm->status_cb.fd == -1 && gpgsm->input_cb.fd == -1
-      && gpgsm->output_cb.fd == -1 && gpgsm->message_cb.fd == -1)
-    (*gpgsm->io_cbs.event) (gpgsm->io_cbs.event_priv, GPGME_EVENT_DONE, NULL);
 }
 
 
@@ -188,17 +160,17 @@ map_assuan_error (AssuanError err)
   switch (err)
     {
     case ASSUAN_No_Error:
-      return mk_error (No_Error);
+      return GPGME_No_Error;
     case ASSUAN_General_Error:
-      return mk_error (General_Error);
+      return GPGME_General_Error;
     case ASSUAN_Out_Of_Core:
-      return mk_error (Out_Of_Core);
+      return GPGME_Out_Of_Core;
     case ASSUAN_Invalid_Value:
-      return mk_error (Invalid_Value);
+      return GPGME_Invalid_Value;
     case ASSUAN_Read_Error:
-      return mk_error (Read_Error);
+      return GPGME_Read_Error;
     case ASSUAN_Write_Error:
-      return mk_error (Write_Error);
+      return GPGME_Write_Error;
 
     case ASSUAN_Timeout:
     case ASSUAN_Problem_Starting_Server:
@@ -210,18 +182,18 @@ map_assuan_error (AssuanError err)
     case ASSUAN_No_Inquire_Callback:
     case ASSUAN_Connect_Failed:
     case ASSUAN_Accept_Failed:
-      return mk_error (General_Error);
+      return GPGME_General_Error;
 
       /* The following error codes are meant as status codes.  */
     case ASSUAN_Not_Implemented:
-      return mk_error (Not_Implemented);
+      return GPGME_Not_Implemented;
     case ASSUAN_Canceled:
-      return mk_error (Canceled);
+      return GPGME_Canceled;
     case ASSUAN_Unsupported_Algorithm:
-      return mk_error (Not_Implemented);  /* XXX Argh.  */
+      return GPGME_Not_Implemented;  /* XXX Argh.  */
       
     case ASSUAN_No_Data_Available:
-      return mk_error (EOF);
+      return GPGME_EOF;
       
       /* These are errors internal to GPGME.  */
     case ASSUAN_No_Input:
@@ -244,7 +216,7 @@ map_assuan_error (AssuanError err)
     case ASSUAN_Unexpected_Data:
     case ASSUAN_Invalid_Status:
     case ASSUAN_Not_Confirmed:
-      return mk_error (General_Error);
+      return GPGME_General_Error;
 
       /* These are errors in the server.  */
     case ASSUAN_Server_Fault:
@@ -253,7 +225,7 @@ map_assuan_error (AssuanError err)
     case ASSUAN_Server_Bug:
     case ASSUAN_No_Agent:
     case ASSUAN_Agent_Error:
-      return mk_error (Invalid_Engine);  /* XXX:  Need something more useful.  */
+      return GPGME_Invalid_Engine;  /* XXX:  Need something more useful.  */
 
     case ASSUAN_Bad_Certificate:
     case ASSUAN_Bad_Certificate_Path:
@@ -266,25 +238,49 @@ map_assuan_error (AssuanError err)
     case ASSUAN_No_PKCS15_App: /* XXX: Oh well.  */
     case ASSUAN_Card_Not_Present:      /* XXX: Oh well.  */
     case ASSUAN_Invalid_Id:    /* XXX: Oh well.  */
-      return mk_error (Invalid_Key);
+      return GPGME_Invalid_Key;
 
     case ASSUAN_Bad_Signature:
-      return mk_error (Invalid_Key);  /* XXX: This is wrong.  */
+      return GPGME_Invalid_Key;  /* XXX: This is wrong.  */
 
     case ASSUAN_Cert_Revoked:
     case ASSUAN_No_CRL_For_Cert:
     case ASSUAN_CRL_Too_Old:
     case ASSUAN_Not_Trusted:
-      return mk_error (Invalid_Key);  /* XXX Some more details would be good.  */
+      return GPGME_Invalid_Key;  /* XXX Some more details would be good.  */
 
     default:
-      return mk_error (General_Error);
+      return GPGME_General_Error;
     }
 }
 
 
-GpgmeError
-_gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
+static void
+gpgsm_release (void *engine)
+{
+  GpgsmObject gpgsm = engine;
+
+  if (!gpgsm)
+    return;
+
+  if (gpgsm->status_cb.fd != -1)
+    _gpgme_io_close (gpgsm->status_cb.fd);
+  if (gpgsm->input_cb.fd != -1)
+    _gpgme_io_close (gpgsm->input_cb.fd);
+  if (gpgsm->output_cb.fd != -1)
+    _gpgme_io_close (gpgsm->output_cb.fd);
+  if (gpgsm->message_cb.fd != -1)
+    _gpgme_io_close (gpgsm->message_cb.fd);
+
+  assuan_disconnect (gpgsm->assuan_ctx);
+
+  free (gpgsm->colon.attic.line);
+  free (gpgsm);
+}
+
+
+static GpgmeError
+gpgsm_new (void **engine)
 {
   GpgmeError err = 0;
   GpgsmObject gpgsm;
@@ -300,11 +296,10 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
   int fdlist[5];
   int nfds;
 
-  *r_gpgsm = NULL;
-  gpgsm = xtrycalloc (1, sizeof *gpgsm);
+  gpgsm = calloc (1, sizeof *gpgsm);
   if (!gpgsm)
     {
-      err = mk_error (Out_Of_Core);
+      err = GPGME_Out_Of_Core;
       return err;
     }
 
@@ -326,6 +321,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
   gpgsm->colon.attic.line = 0;
   gpgsm->colon.attic.linesize = 0;
   gpgsm->colon.attic.linelen = 0;
+  gpgsm->colon.any = 0;
 
   gpgsm->io_cbs.add = NULL;
   gpgsm->io_cbs.add_priv = NULL;
@@ -335,7 +331,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
 
   if (_gpgme_io_pipe (fds, 0) < 0)
     {
-      err = mk_error (Pipe_Error);
+      err = GPGME_Pipe_Error;
       goto leave;
     }
   gpgsm->input_cb.fd = fds[1];
@@ -344,7 +340,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
 
   if (_gpgme_io_pipe (fds, 1) < 0)
     {
-      err = mk_error (Pipe_Error);
+      err = GPGME_Pipe_Error;
       goto leave;
     }
   gpgsm->output_cb.fd = fds[0];
@@ -353,7 +349,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
 
   if (_gpgme_io_pipe (fds, 0) < 0)
     {
-      err = mk_error (Pipe_Error);
+      err = GPGME_Pipe_Error;
       goto leave;
     }
   gpgsm->message_cb.fd = fds[1];
@@ -380,7 +376,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
                                 fdlist, DIM (fdlist));
   if (nfds < 1)
     {
-      err = mk_error (General_Error);  /* FIXME */
+      err = GPGME_General_Error;  /* FIXME */
       goto leave;
     }
   /* We duplicate the file descriptor, so we can close it without
@@ -391,7 +387,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
   gpgsm->status_cb.fd = dup (fdlist[0]);
   if (gpgsm->status_cb.fd < 0)
     {
-      err = mk_error (General_Error);  /* FIXME */
+      err = GPGME_General_Error;       /* FIXME */
       goto leave;
     }
   gpgsm->status_cb.dir = 1;
@@ -402,7 +398,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
     {
       if (asprintf (&optstr, "OPTION display=%s", dft_display) < 0)
         {
-         err = mk_error (Out_Of_Core);
+         err = GPGME_Out_Of_Core;
          goto leave;
        }
       err = assuan_transact (gpgsm->assuan_ctx, optstr, NULL, NULL, NULL,
@@ -419,7 +415,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
     {
       if (asprintf (&optstr, "OPTION ttyname=%s", dft_ttyname) < 0)
         {
-         err = mk_error (Out_Of_Core);
+         err = GPGME_Out_Of_Core;
          goto leave;
        }
       err = assuan_transact (gpgsm->assuan_ctx, optstr, NULL, NULL, NULL, NULL, NULL,
@@ -436,7 +432,7 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
        {
          if (asprintf (&optstr, "OPTION ttytype=%s", dft_ttytype) < 0)
            {
-             err = mk_error (Out_Of_Core);
+             err = GPGME_Out_Of_Core;
              goto leave;
            }
          err = assuan_transact (gpgsm->assuan_ctx, optstr, NULL, NULL, NULL, NULL, NULL,
@@ -448,32 +444,55 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
              goto leave;
            }
        }
+
       old_lc = setlocale (LC_CTYPE, NULL);
+      if (old_lc)
+        {
+          old_lc = strdup (old_lc);
+          if (!old_lc)
+            {
+              err = GPGME_Out_Of_Core;
+              goto leave;
+            }
+        }
       dft_lc = setlocale (LC_CTYPE, "");
       if (dft_lc)
        {
          if (asprintf (&optstr, "OPTION lc-ctype=%s", dft_lc) < 0)
-           err = mk_error (Out_Of_Core);
+           err = GPGME_Out_Of_Core;
          else
            {
-             err = assuan_transact (gpgsm->assuan_ctx, optstr, NULL, NULL, NULL, NULL, NULL,
-                                    NULL);
+             err = assuan_transact (gpgsm->assuan_ctx, optstr, NULL, NULL,
+                                     NULL, NULL, NULL, NULL);
              free (optstr);
              if (err)
                err = map_assuan_error (err);
            }
        }
       if (old_lc)
-       setlocale (LC_CTYPE, old_lc);
+        {
+          setlocale (LC_CTYPE, old_lc);
+          free (old_lc);
+        }
       if (err)
        goto leave;
 
+
       old_lc = setlocale (LC_MESSAGES, NULL);
+      if (old_lc)
+        {
+          old_lc = strdup (old_lc);
+          if (!old_lc)
+            {
+              err = GPGME_Out_Of_Core;
+              goto leave;
+            }
+        }
       dft_lc = setlocale (LC_MESSAGES, "");
       if (dft_lc)
        {
          if (asprintf (&optstr, "OPTION lc-messages=%s", dft_lc) < 0)
-           err = mk_error (Out_Of_Core);
+           err = GPGME_Out_Of_Core;
          else
            {
              err = assuan_transact (gpgsm->assuan_ctx, optstr, NULL, NULL, NULL, NULL, NULL,
@@ -484,7 +503,10 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
            }
        }
       if (old_lc)
-       setlocale (LC_MESSAGES, old_lc);
+        {
+          setlocale (LC_MESSAGES, old_lc);
+          free (old_lc);
+        }
       if (err)
        goto leave;
     }
@@ -499,13 +521,13 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
        || _gpgme_io_set_close_notify (gpgsm->message_cb.fd,
                                      close_notify_handler, gpgsm)))
     {
-      err = mk_error (General_Error);
+      err = GPGME_General_Error;
       goto leave;
     }
       
  leave:
   /* Close the server ends of the pipes.  Our ends are closed in
-     _gpgme_gpgsm_release.  */
+     gpgsm_release().  */
   if (gpgsm->input_fd_server != -1)
     _gpgme_io_close (gpgsm->input_fd_server);
   if (gpgsm->output_fd_server != -1)
@@ -514,41 +536,19 @@ _gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
     _gpgme_io_close (gpgsm->message_fd_server);
 
   if (err)
-    _gpgme_gpgsm_release (gpgsm);
+    gpgsm_release (gpgsm);
   else
-    *r_gpgsm = gpgsm;
+    *engine = gpgsm;
 
   return err;
 }
 
 
-void
-_gpgme_gpgsm_release (GpgsmObject gpgsm)
-{
-  if (!gpgsm)
-    return;
-
-  if (gpgsm->status_cb.fd != -1)
-    _gpgme_io_close (gpgsm->status_cb.fd);
-  if (gpgsm->input_cb.fd != -1)
-    _gpgme_io_close (gpgsm->input_cb.fd);
-  if (gpgsm->output_cb.fd != -1)
-    _gpgme_io_close (gpgsm->output_cb.fd);
-  if (gpgsm->message_cb.fd != -1)
-    _gpgme_io_close (gpgsm->message_cb.fd);
-
-  assuan_disconnect (gpgsm->assuan_ctx);
-
-  xfree (gpgsm->colon.attic.line);
-  xfree (gpgsm->command);
-  xfree (gpgsm);
-}
-
 /* Forward declaration.  */
-static GpgStatusCode parse_status (const char *name);
+static GpgmeStatusCode parse_status (const char *name);
 
 static GpgmeError
-gpgsm_assuan_simple_command (ASSUAN_CONTEXT ctx, char *cmd, GpgStatusHandler status_fnc,
+gpgsm_assuan_simple_command (ASSUAN_CONTEXT ctx, char *cmd, EngineStatusHandler status_fnc,
                             void *status_fnc_value)
 {
   AssuanError err;
@@ -580,7 +580,7 @@ gpgsm_assuan_simple_command (ASSUAN_CONTEXT ctx, char *cmd, GpgStatusHandler sta
               && line[0] == 'S' && line[1] == ' ')
        {
          char *rest;
-         GpgStatusCode r;
+         GpgmeStatusCode r;
 
          rest = strchr (line + 2, ' ');
          if (!rest)
@@ -593,10 +593,10 @@ gpgsm_assuan_simple_command (ASSUAN_CONTEXT ctx, char *cmd, GpgStatusHandler sta
          if (r >= 0 && status_fnc)
            status_fnc (status_fnc_value, r, rest);
          else
-           err = mk_error (General_Error);
+           err = GPGME_General_Error;
        }
       else
-       err = mk_error (General_Error);
+       err = GPGME_General_Error;
     }
   while (!err);
 
@@ -639,43 +639,309 @@ map_input_enc (GpgmeData d)
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_decrypt (GpgsmObject gpgsm, GpgmeData ciph, GpgmeData plain)
+static int
+status_cmp (const void *ap, const void *bp)
+{
+  const struct status_table_s *a = ap;
+  const struct status_table_s *b = bp;
+
+  return strcmp (a->name, b->name);
+}
+
+
+static GpgmeStatusCode
+parse_status (const char *name)
+{
+  struct status_table_s t, *r;
+  t.name = name;
+  r = bsearch (&t, status_table, DIM(status_table) - 1,
+              sizeof t, status_cmp);
+  return r ? r->code : -1;
+}
+
+
+static GpgmeError
+status_handler (void *opaque, int fd)
+{
+  AssuanError assuan_err;
+  GpgmeError err = 0;
+  GpgsmObject gpgsm = opaque;
+  char *line;
+  size_t linelen;
+
+  do
+    {
+      assuan_err = assuan_read_line (gpgsm->assuan_ctx, &line, &linelen);
+      if (assuan_err)
+       {
+         /* Try our best to terminate the connection friendly.  */
+         assuan_write_line (gpgsm->assuan_ctx, "BYE");
+         err = map_assuan_error (assuan_err);
+       }
+      else if (linelen >= 3
+              && line[0] == 'E' && line[1] == 'R' && line[2] == 'R'
+              && (line[3] == '\0' || line[3] == ' '))
+       {
+         if (line[3] == ' ')
+           err = map_assuan_error (atoi (&line[4]));
+         else
+           err = GPGME_General_Error;
+       }
+      else if (linelen >= 2
+              && line[0] == 'O' && line[1] == 'K'
+              && (line[2] == '\0' || line[2] == ' '))
+       {
+         if (gpgsm->status.fnc)
+           err = gpgsm->status.fnc (gpgsm->status.fnc_value,
+                                    GPGME_STATUS_EOF, "");
+         
+         if (!err && gpgsm->colon.fnc && gpgsm->colon.any )
+            {
+              /* We must tell a colon function about the EOF. We do
+                 this only when we have seen any data lines.  Note
+                 that this inlined use of colon data lines will
+                 eventually be changed into using a regular data
+                 channel. */
+              gpgsm->colon.any = 0;
+              err = gpgsm->colon.fnc (gpgsm->colon.fnc_value, NULL);
+            }
+         _gpgme_io_close (gpgsm->status_cb.fd);
+         return err;
+       }
+      else if (linelen > 2
+              && line[0] == 'D' && line[1] == ' '
+              && gpgsm->colon.fnc)
+        {
+         /* We are using the colon handler even for plain inline data
+             - strange name for that function but for historic reasons
+             we keep it.  */
+          /* FIXME We can't use this for binary data because we
+             assume this is a string.  For the current usage of colon
+             output it is correct.  */
+          unsigned char *src = line + 2;
+         unsigned char *end = line + linelen;
+         unsigned char *dst;
+          unsigned char **aline = &gpgsm->colon.attic.line;
+         int *alinelen = &gpgsm->colon.attic.linelen;
+
+         if (gpgsm->colon.attic.linesize
+             < *alinelen + linelen + 1)
+           {
+             unsigned char *newline = realloc (*aline,
+                                               *alinelen + linelen + 1);
+             if (!newline)
+               err = GPGME_Out_Of_Core;
+             else
+               {
+                 *aline = newline;
+                 gpgsm->colon.attic.linesize += linelen + 1;
+               }
+           }
+         if (!err)
+           {
+             dst = *aline + *alinelen;
+
+             while (!err && src < end)
+               {
+                 if (*src == '%' && src + 2 < end)
+                   {
+                     /* Handle escaped characters.  */
+                     ++src;
+                     *dst = xtoi_2 (src);
+                     (*alinelen)++;
+                     src += 2;
+                   }
+                 else
+                   {
+                     *dst = *src++;
+                     (*alinelen)++;
+                   }
+                 
+                 if (*dst == '\n')
+                   {
+                     /* Terminate the pending line, pass it to the colon
+                        handler and reset it.  */
+                     
+                     gpgsm->colon.any = 1;
+                     if (*alinelen > 1 && *(dst - 1) == '\r')
+                       dst--;
+                     *dst = '\0';
+
+                     /* FIXME How should we handle the return code?  */
+                     err = gpgsm->colon.fnc (gpgsm->colon.fnc_value, *aline);
+                     if (!err)
+                       {
+                         dst = *aline;
+                         *alinelen = 0;
+                       }
+                   }
+                 else
+                   dst++;
+               }
+           }
+        }
+      else if (linelen > 2
+              && line[0] == 'S' && line[1] == ' ')
+       {
+         char *rest;
+         GpgmeStatusCode r;
+         
+         rest = strchr (line + 2, ' ');
+         if (!rest)
+           rest = line + linelen; /* set to an empty string */
+         else
+           *(rest++) = 0;
+
+         r = parse_status (line + 2);
+
+         if (r >= 0)
+           {
+             if (gpgsm->status.fnc)
+               err = gpgsm->status.fnc (gpgsm->status.fnc_value, r, rest);
+           }
+         else
+           fprintf (stderr, "[UNKNOWN STATUS]%s %s", line + 2, rest);
+       }
+    }
+  while (!err && assuan_pending_line (gpgsm->assuan_ctx));
+         
+  return err;
+}
+
+
+static GpgmeError
+add_io_cb (GpgsmObject gpgsm, iocb_data_t *iocbd, GpgmeIOCb handler)
 {
   GpgmeError err;
 
-  if (!gpgsm)
-    return mk_error (Invalid_Value);
+  err = (*gpgsm->io_cbs.add) (gpgsm->io_cbs.add_priv,
+                             iocbd->fd, iocbd->dir,
+                             handler, iocbd->data, &iocbd->tag);
+  if (err)
+    return err;
+  if (!iocbd->dir)
+    /* FIXME Kludge around poll() problem.  */
+    err = _gpgme_io_set_nonblocking (iocbd->fd);
+  return err;
+}
+
 
-  gpgsm->command = xtrystrdup ("DECRYPT");
-  if (!gpgsm->command)
-    return mk_error (Out_Of_Core);
+static GpgmeError
+start (GpgsmObject gpgsm, const char *command)
+{
+  GpgmeError err = 0;
+
+  err = add_io_cb (gpgsm, &gpgsm->status_cb, status_handler);
+  if (gpgsm->input_cb.fd != -1)
+    err = add_io_cb (gpgsm, &gpgsm->input_cb, _gpgme_data_outbound_handler);
+  if (!err && gpgsm->output_cb.fd != -1)
+    err = add_io_cb (gpgsm, &gpgsm->output_cb, _gpgme_data_inbound_handler);
+  if (!err && gpgsm->message_cb.fd != -1)
+    err = add_io_cb (gpgsm, &gpgsm->message_cb, _gpgme_data_outbound_handler);
+
+  if (!err)
+    err = assuan_write_line (gpgsm->assuan_ctx, command);
+
+  (*gpgsm->io_cbs.event) (gpgsm->io_cbs.event_priv, GPGME_EVENT_START, NULL);
+
+  return err;
+}
+
+
+static GpgmeError
+gpgsm_decrypt (void *engine, GpgmeData ciph, GpgmeData plain)
+{
+  GpgsmObject gpgsm = engine;
+  GpgmeError err;
+
+  if (!gpgsm)
+    return GPGME_Invalid_Value;
 
   gpgsm->input_cb.data = ciph;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "INPUT", gpgsm->input_fd_server, 
                       map_input_enc (gpgsm->input_cb.data));
   if (err)
-    return mk_error (General_Error);   /* FIXME */
+    return GPGME_General_Error;        /* FIXME */
   gpgsm->output_cb.data = plain;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "OUTPUT", gpgsm->output_fd_server, 0);
   if (err)
-    return mk_error (General_Error);   /* FIXME */
+    return GPGME_General_Error;        /* FIXME */
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  return 0;
+  err = start (engine, "DECRYPT");
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_delete (GpgsmObject gpgsm, GpgmeKey key, int allow_secret)
+static GpgmeError
+gpgsm_delete (void *engine, GpgmeKey key, int allow_secret)
 {
-  /* FIXME */
-  return mk_error (Not_Implemented);
+  GpgsmObject gpgsm = engine;
+  GpgmeError err;
+  char *fpr = (char *) gpgme_key_get_string_attr (key, GPGME_ATTR_FPR, NULL, 0);
+  char *linep = fpr;
+  char *line;
+  int length = 8;      /* "DELKEYS " */
+
+  if (!fpr)
+    return GPGME_Invalid_Key;
+
+  while (*linep)
+    {
+      length++;
+      if (*linep == '%' || *linep == ' ' || *linep == '+')
+       length += 2;
+      linep++;
+    }
+  length++;
+
+  line = malloc (length);
+  if (!line)
+    return GPGME_Out_Of_Core;
+
+  strcpy (line, "DELKEYS ");
+  linep = &line[8];
+
+  while (*fpr)
+    {
+      switch (*fpr)
+       {
+       case '%':
+         *(linep++) = '%';
+         *(linep++) = '2';
+         *(linep++) = '5';
+         break;
+       case ' ':
+         *(linep++) = '%';
+         *(linep++) = '2';
+         *(linep++) = '0';
+         break;
+       case '+':
+         *(linep++) = '%';
+         *(linep++) = '2';
+         *(linep++) = 'B';
+         break;
+       default:
+         *(linep++) = *fpr;
+         break;
+       }
+      fpr++;
+    }
+  *linep = '\0';
+
+  _gpgme_io_close (gpgsm->output_cb.fd);
+  _gpgme_io_close (gpgsm->input_cb.fd);
+  _gpgme_io_close (gpgsm->message_cb.fd);
+
+  err = start (gpgsm, line);
+  free (line);
+
+  return err;
 }
 
 
 static GpgmeError
-gpgsm_set_recipients (GpgsmObject gpgsm, GpgmeRecipients recp)
+set_recipients (GpgsmObject gpgsm, GpgmeRecipients recp)
 {
   GpgmeError err;
   ASSUAN_CONTEXT ctx = gpgsm->assuan_ctx;
@@ -685,20 +951,20 @@ gpgsm_set_recipients (GpgsmObject gpgsm, GpgmeRecipients recp)
   int valid_recipients = 0;
 
   linelen = 10 + 40 + 1;       /* "RECIPIENT " + guess + '\0'.  */
-  line = xtrymalloc (10 + 40 + 1);
+  line = malloc (10 + 40 + 1);
   if (!line)
-    return mk_error (Out_Of_Core);
+    return GPGME_Out_Of_Core;
   strcpy (line, "RECIPIENT ");
   for (r = recp->list; r; r = r->next)
     {
       int newlen = 11 + strlen (r->name);
       if (linelen < newlen)
        {
-         char *newline = xtryrealloc (line, newlen);
+         char *newline = realloc (line, newlen);
          if (! newline)
            {
-             xfree (line);
-             return mk_error (Out_Of_Core);
+             free (line);
+             return GPGME_Out_Of_Core;
            }
          line = newline;
          linelen = newlen;
@@ -711,31 +977,28 @@ gpgsm_set_recipients (GpgsmObject gpgsm, GpgmeRecipients recp)
        valid_recipients = 1;
       else if (err != GPGME_Invalid_Key)
        {
-         xfree (line);
+         free (line);
          return err;
        }
     }
-  xfree (line);
+  free (line);
   if (!valid_recipients && gpgsm->status.fnc)
-    gpgsm->status.fnc (gpgsm->status.fnc_value, STATUS_NO_RECP, "");
+    gpgsm->status.fnc (gpgsm->status.fnc_value, GPGME_STATUS_NO_RECP, "");
   return 0;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_encrypt (GpgsmObject gpgsm, GpgmeRecipients recp,
-                        GpgmeData plain, GpgmeData ciph, int use_armor)
+static GpgmeError
+gpgsm_encrypt (void *engine, GpgmeRecipients recp, GpgmeData plain,
+              GpgmeData ciph, int use_armor)
 {
+  GpgsmObject gpgsm = engine;
   GpgmeError err;
 
   if (!gpgsm)
-    return mk_error (Invalid_Value);
+    return GPGME_Invalid_Value;
   if (!recp)
-    return mk_error (Not_Implemented);
-
-  gpgsm->command = xtrystrdup ("ENCRYPT");
-  if (!gpgsm->command)
-    return mk_error (Out_Of_Core);
+    return GPGME_Not_Implemented;
 
   gpgsm->input_cb.data = plain;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "INPUT", gpgsm->input_fd_server,
@@ -749,29 +1012,31 @@ _gpgme_gpgsm_op_encrypt (GpgsmObject gpgsm, GpgmeRecipients recp,
     return err;
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  err = gpgsm_set_recipients (gpgsm, recp);
-  if (err)
-    return err;
+  err = set_recipients (gpgsm, recp);
 
-  return 0;
+  if (!err)
+    err = start (gpgsm, "ENCRYPT");
+
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_export (GpgsmObject gpgsm, GpgmeRecipients recp,
-                       GpgmeData keydata, int use_armor)
+static GpgmeError
+gpgsm_export (void *engine, GpgmeRecipients recp, GpgmeData keydata,
+             int use_armor)
 {
+  GpgsmObject gpgsm = engine;
   GpgmeError err = 0;
   char *cmd = NULL;
   int cmdi;
   int cmdlen = 32;
 
   if (!gpgsm)
-    return mk_error (Invalid_Value);
+    return GPGME_Invalid_Value;
 
   cmd = malloc (cmdlen);
   if (!cmd)
-    return mk_error (Out_Of_Core);
+    return GPGME_Out_Of_Core;
   strcpy (cmd, "EXPORT");
   cmdi = 6;
 
@@ -787,11 +1052,11 @@ _gpgme_gpgsm_op_export (GpgsmObject gpgsm, GpgmeRecipients recp,
          /* New string is old string + ' ' + s + '\0'.  */
          if (cmdlen < cmdi + 1 + slen + 1)
            {
-             char *newcmd = xtryrealloc (cmd, cmdlen * 2);
+             char *newcmd = realloc (cmd, cmdlen * 2);
              if (!newcmd)
                {
-                 xfree (cmd);
-                 return mk_error (Out_Of_Core);
+                 free (cmd);
+                 return GPGME_Out_Of_Core;
                }
              cmd = newcmd;
              cmdlen *= 2;
@@ -806,8 +1071,6 @@ _gpgme_gpgsm_op_export (GpgsmObject gpgsm, GpgmeRecipients recp,
        return err;
     }
 
-  gpgsm->command = cmd;
-
   gpgsm->output_cb.data = keydata;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "OUTPUT", gpgsm->output_fd_server,
                      use_armor ? "--armor" : 0);
@@ -816,22 +1079,21 @@ _gpgme_gpgsm_op_export (GpgsmObject gpgsm, GpgmeRecipients recp,
   _gpgme_io_close (gpgsm->input_cb.fd);
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  return 0;
+  err = start (gpgsm, cmd);
+  free (cmd);
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_genkey (GpgsmObject gpgsm, GpgmeData help_data, int use_armor,
-                       GpgmeData pubkey, GpgmeData seckey)
+static GpgmeError
+gpgsm_genkey (void *engine, GpgmeData help_data, int use_armor,
+             GpgmeData pubkey, GpgmeData seckey)
 {
+  GpgsmObject gpgsm = engine;
   GpgmeError err;
 
   if (!gpgsm || !pubkey || seckey)
-    return mk_error (Invalid_Value);
-
-  gpgsm->command = xtrystrdup ("GENKEY");
-  if (!gpgsm->command)
-    return mk_error (Out_Of_Core);
+    return GPGME_Invalid_Value;
 
   gpgsm->input_cb.data = help_data;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "INPUT", gpgsm->input_fd_server,
@@ -845,21 +1107,19 @@ _gpgme_gpgsm_op_genkey (GpgsmObject gpgsm, GpgmeData help_data, int use_armor,
     return err;
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  return 0;
+  err = start (gpgsm, "GENKEY");
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_import (GpgsmObject gpgsm, GpgmeData keydata)
+static GpgmeError
+gpgsm_import (void *engine, GpgmeData keydata)
 {
+  GpgsmObject gpgsm = engine;
   GpgmeError err;
 
   if (!gpgsm)
-    return mk_error (Invalid_Value);
-
-  gpgsm->command = xtrystrdup ("IMPORT");
-  if (!gpgsm->command)
-    return mk_error (Out_Of_Core);
+    return GPGME_Invalid_Value;
 
   gpgsm->input_cb.data = keydata;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "INPUT", gpgsm->input_fd_server,
@@ -869,14 +1129,16 @@ _gpgme_gpgsm_op_import (GpgsmObject gpgsm, GpgmeData keydata)
   _gpgme_io_close (gpgsm->output_cb.fd);
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  return 0;
+  err = start (gpgsm, "IMPORT");
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_keylist (GpgsmObject gpgsm, const char *pattern,
-                        int secret_only, int keylist_mode)
+static GpgmeError
+gpgsm_keylist (void *engine, const char *pattern, int secret_only,
+              int keylist_mode)
 {
+  GpgsmObject gpgsm = engine;
   char *line;
   GpgmeError err;
 
@@ -884,16 +1146,16 @@ _gpgme_gpgsm_op_keylist (GpgsmObject gpgsm, const char *pattern,
     pattern = "";
 
   if (asprintf (&line, "OPTION list-mode=%d", (keylist_mode & 3)) < 0)
-    return mk_error (Out_Of_Core);
+    return GPGME_Out_Of_Core;
   err = gpgsm_assuan_simple_command (gpgsm->assuan_ctx, line, NULL, NULL);
   free (line);
   if (err)
     return err;
 
   /* Length is "LISTSECRETKEYS " + p + '\0'.  */
-  line = xtrymalloc (15 + strlen (pattern) + 1);
+  line = malloc (15 + strlen (pattern) + 1);
   if (!line)
-    return mk_error (Out_Of_Core);
+    return GPGME_Out_Of_Core;
   if (secret_only)
     {
       strcpy (line, "LISTSECRETKEYS ");
@@ -909,15 +1171,17 @@ _gpgme_gpgsm_op_keylist (GpgsmObject gpgsm, const char *pattern,
   _gpgme_io_close (gpgsm->output_cb.fd);
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  gpgsm->command = line;
-  return 0;
+  err = start (gpgsm, line);
+  free (line);
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_keylist_ext (GpgsmObject gpgsm, const char *pattern[],
-                            int secret_only, int reserved, int keylist_mode)
+static GpgmeError
+gpgsm_keylist_ext (void *engine, const char *pattern[], int secret_only,
+                  int reserved, int keylist_mode)
 {
+  GpgsmObject gpgsm = engine;
   char *line;
   GpgmeError err;
   /* Length is "LISTSECRETKEYS " + p + '\0'.  */
@@ -925,10 +1189,10 @@ _gpgme_gpgsm_op_keylist_ext (GpgsmObject gpgsm, const char *pattern[],
   char *linep;
   
   if (reserved)
-    return mk_error (Invalid_Value);
+    return GPGME_Invalid_Value;
 
   if (asprintf (&line, "OPTION list-mode=%d", (keylist_mode & 3)) < 0)
-    return mk_error (Out_Of_Core);
+    return GPGME_Out_Of_Core;
   err = gpgsm_assuan_simple_command (gpgsm->assuan_ctx, line, NULL, NULL);
   free (line);
   if (err)
@@ -954,9 +1218,9 @@ _gpgme_gpgsm_op_keylist_ext (GpgsmObject gpgsm, const char *pattern[],
          length++;
        }
     }
-  line = xtrymalloc (length);
+  line = malloc (length);
   if (!line)
-    return mk_error (Out_Of_Core);
+    return GPGME_Out_Of_Core;
   if (secret_only)
     {
       strcpy (line, "LISTSECRETKEYS ");
@@ -1008,35 +1272,58 @@ _gpgme_gpgsm_op_keylist_ext (GpgsmObject gpgsm, const char *pattern[],
   _gpgme_io_close (gpgsm->output_cb.fd);
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  gpgsm->command = line;
-  return 0;
+  err = start (gpgsm, line);
+  free (line);
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_sign (GpgsmObject gpgsm, GpgmeData in, GpgmeData out,
-                     GpgmeSigMode mode, int use_armor,
-                     int use_textmode, int include_certs,
-                     GpgmeCtx ctx /* FIXME */)
+static GpgmeError
+gpgsm_sign (void *engine, GpgmeData in, GpgmeData out, GpgmeSigMode mode,
+           int use_armor, int use_textmode, int include_certs,
+           GpgmeCtx ctx /* FIXME */)
 {
+  GpgsmObject gpgsm = engine;
   GpgmeError err;
   char *assuan_cmd;
+  int i;
+  GpgmeKey key;
 
   if (!gpgsm)
-    return mk_error (Invalid_Value);
-
-  gpgsm->command = xtrystrdup (mode == GPGME_SIG_MODE_DETACH
-                              ? "SIGN --detached" : "SIGN");
-  if (!gpgsm->command)
-    return mk_error (Out_Of_Core);
+    return GPGME_Invalid_Value;
 
   if (asprintf (&assuan_cmd, "OPTION include-certs %i", include_certs) < 0)
-    return mk_error (Out_Of_Core);
-  err = gpgsm_assuan_simple_command (gpgsm->assuan_ctx, assuan_cmd, NULL, NULL);
+    return GPGME_Out_Of_Core;
+  err = gpgsm_assuan_simple_command (gpgsm->assuan_ctx, assuan_cmd, NULL,NULL);
   free (assuan_cmd);
   if (err)
     return err;
 
+  /* We must do a reset becuase we need to reset the list of signers.  Note
+     that RESET does not reset OPTION commands. */
+  err = gpgsm_assuan_simple_command (gpgsm->assuan_ctx, "RESET", NULL, NULL);
+  if (err)
+    return err;
+
+  for (i = 0; (key = gpgme_signers_enum (ctx, i)); i++)
+    {
+      const char *s = gpgme_key_get_string_attr (key, GPGME_ATTR_FPR,
+                                                NULL, 0);
+      if (s && strlen (s) < 80)
+       {
+          char buf[100];
+
+          strcpy (stpcpy (buf, "SIGNER "), s);
+          err = gpgsm_assuan_simple_command (gpgsm->assuan_ctx, buf,
+                                             NULL, NULL);
+       }
+      else
+        err = GPGME_Invalid_Key;
+      gpgme_key_unref (key);
+      if (err) 
+        return err;
+    }
+
   gpgsm->input_cb.data = in;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "INPUT", gpgsm->input_fd_server,
                       map_input_enc (gpgsm->input_cb.data));
@@ -1049,39 +1336,39 @@ _gpgme_gpgsm_op_sign (GpgsmObject gpgsm, GpgmeData in, GpgmeData out,
     return err;
   _gpgme_io_close (gpgsm->message_cb.fd);
 
-  return 0;
+  err = start (gpgsm, mode == GPGME_SIG_MODE_DETACH
+              ? "SIGN --detached" : "SIGN");
+  return err;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_trustlist (GpgsmObject gpgsm, const char *pattern)
+static GpgmeError
+gpgsm_trustlist (void *engine, const char *pattern)
 {
   /* FIXME */
-  return mk_error (Not_Implemented);
+  return GPGME_Not_Implemented;
 }
 
 
-GpgmeError
-_gpgme_gpgsm_op_verify (GpgsmObject gpgsm, GpgmeData sig, GpgmeData text)
+static GpgmeError
+gpgsm_verify (void *engine, GpgmeData sig, GpgmeData signed_text,
+             GpgmeData plaintext)
 {
+  GpgsmObject gpgsm = engine;
   GpgmeError err;
 
   if (!gpgsm)
-    return mk_error (Invalid_Value);
-
-  gpgsm->command = xtrystrdup ("VERIFY");
-  if (!gpgsm->command)
-    return mk_error (Out_Of_Core);
+    return GPGME_Invalid_Value;
 
   gpgsm->input_cb.data = sig;
   err = gpgsm_set_fd (gpgsm->assuan_ctx, "INPUT", gpgsm->input_fd_server,
                       map_input_enc (gpgsm->input_cb.data));
   if (err)
     return err;
-  if (_gpgme_data_get_mode (text) == GPGME_DATA_MODE_IN)
+  if (plaintext)
     {
       /* Normal or cleartext signature.  */
-      gpgsm->output_cb.data = text;
+      gpgsm->output_cb.data = plaintext;
       err = gpgsm_set_fd (gpgsm->assuan_ctx, "OUTPUT", gpgsm->output_fd_server,
                          0);
       _gpgme_io_close (gpgsm->message_cb.fd);
@@ -1089,401 +1376,88 @@ _gpgme_gpgsm_op_verify (GpgsmObject gpgsm, GpgmeData sig, GpgmeData text)
   else
     {
       /* Detached signature.  */
-      gpgsm->message_cb.data = text;
+      gpgsm->message_cb.data = signed_text;
       err = gpgsm_set_fd (gpgsm->assuan_ctx, "MESSAGE",
                          gpgsm->message_fd_server, 0);
       _gpgme_io_close (gpgsm->output_cb.fd);
     }
-  if (err)
-    return err;
-
-  return 0;
-}
-
-
-static int
-status_cmp (const void *ap, const void *bp)
-{
-  const struct status_table_s *a = ap;
-  const struct status_table_s *b = bp;
-
-  return strcmp (a->name, b->name);
-}
 
+  if (!err)
+    err = start (gpgsm, "VERIFY");
 
-static GpgStatusCode
-parse_status (const char *name)
-{
-  struct status_table_s t, *r;
-  t.name = name;
-  r = bsearch (&t, status_table, DIM(status_table) - 1,
-              sizeof t, status_cmp);
-  return r ? r->code : -1;
+  return err;
 }
 
 
 static void
-gpgsm_status_handler (void *opaque, int fd)
+gpgsm_set_status_handler (void *engine, EngineStatusHandler fnc,
+                         void *fnc_value) 
 {
-  AssuanError err;
-  GpgsmObject gpgsm = opaque;
-  char *line;
-  size_t linelen;
-
-  do
-    {
-      err = assuan_read_line (gpgsm->assuan_ctx, &line, &linelen);
-
-      if (err
-          || (linelen >= 2
-              && line[0] == 'O' && line[1] == 'K'
-              && (line[2] == '\0' || line[2] == ' '))
-         || (linelen >= 3
-             && line[0] == 'E' && line[1] == 'R' && line[2] == 'R'
-             && (line[3] == '\0' || line[3] == ' ')))
-       {
-         /* XXX: If an error occured, find out what happened, then
-            save the error value before running the status handler
-            (so it takes precedence).  */
-         if (!err && line[0] == 'E' && line[3] == ' ')
-           {
-             err = map_assuan_error (atoi (&line[4]));
-             if (!err)
-               err = mk_error (General_Error);
-           }
-         if (err)
-           {
-             /* XXX Kludge ahead.  We really, really, really must not
-                make use of status.fnc_value.  */
-             GpgmeCtx ctx = (GpgmeCtx) gpgsm->status.fnc_value;
-             if (!ctx->error)
-               ctx->error = err;
-           }
-
-         if (gpgsm->status.fnc)
-           gpgsm->status.fnc (gpgsm->status.fnc_value, STATUS_EOF, "");
-
-         /* XXX: Try our best to terminate the connection.  */
-         if (err)
-           assuan_write_line (gpgsm->assuan_ctx, "BYE");
-
-         _gpgme_io_close (gpgsm->status_cb.fd);
-         return;
-       }
-
-      if (linelen > 2
-         && line[0] == 'D' && line[1] == ' '
-          && gpgsm->colon.fnc)
-        {
-         /* We are using the colon handler even for plain inline data
-             - strange name for that function but for historic reasons
-             we keep it.  */
-          /* FIXME We can't use this for binary data because we
-             assume this is a string.  For the current usage of colon
-             output it is correct.  */
-          unsigned char *src = line + 2;
-         unsigned char *end = line + linelen;
-         unsigned char *dst;
-          unsigned char **aline = &gpgsm->colon.attic.line;
-         int *alinelen = &gpgsm->colon.attic.linelen;
-
-         if (gpgsm->colon.attic.linesize
-             < *alinelen + linelen + 1)
-           {
-             unsigned char *newline = xtryrealloc (*aline,
-                                                   *alinelen + linelen + 1);
-             if (!newline)
-               {
-                 _gpgme_io_close (gpgsm->status_cb.fd);
-                 return;
-               }
-             *aline = newline;
-             gpgsm->colon.attic.linesize += linelen + 1;
-           }
-
-         dst = *aline + *alinelen;
-
-          while (src < end)
-            {
-              if (*src == '%' && src + 2 < end)
-                {
-                 /* Handle escaped characters.  */
-                 ++src;
-                  *dst = xtoi_2 (src);
-                 (*alinelen)++;
-                  src += 2;
-                }
-              else
-               {
-                 *dst = *src++;
-                 (*alinelen)++;
-               }
-
-             if (*dst == '\n')
-               {
-                 /* Terminate the pending line, pass it to the colon
-                    handler and reset it.  */
-
-                 if (*alinelen > 1 && *(dst - 1) == '\r')
-                   dst--;
-                 *dst = '\0';
-
-                 /* FIXME How should we handle the return code? */
-                 gpgsm->colon.fnc (gpgsm->colon.fnc_value, *aline);
-                 dst = *aline;
-                 *alinelen = 0;
-               }
-             else
-               dst++;
-            }
-        }
-      else if (linelen > 2
-         && line[0] == 'S' && line[1] == ' ')
-       {
-         char *rest;
-         GpgStatusCode r;
-
-         rest = strchr (line + 2, ' ');
-         if (!rest)
-           rest = line + linelen; /* set to an empty string */
-         else
-           *(rest++) = 0;
-
-         r = parse_status (line + 2);
-
-         if (r >= 0)
-           {
-             if (gpgsm->status.fnc)
-               gpgsm->status.fnc (gpgsm->status.fnc_value, r, rest);
-           }
-         else
-           fprintf (stderr, "[UNKNOWN STATUS]%s %s", line + 2, rest);
-       }
-    }
-  while (assuan_pending_line (gpgsm->assuan_ctx));
-}
-
-
-void
-_gpgme_gpgsm_set_status_handler (GpgsmObject gpgsm,
-                                GpgStatusHandler fnc, void *fnc_value) 
-{
-  assert (gpgsm);
+  GpgsmObject gpgsm = engine;
 
   gpgsm->status.fnc = fnc;
   gpgsm->status.fnc_value = fnc_value;
 }
 
 
-void
-_gpgme_gpgsm_set_colon_line_handler (GpgsmObject gpgsm,
-                                     GpgColonLineHandler fnc, void *fnc_value) 
+static GpgmeError
+gpgsm_set_colon_line_handler (void *engine, EngineColonLineHandler fnc,
+                             void *fnc_value) 
 {
-  assert (gpgsm);
+  GpgsmObject gpgsm = engine;
 
   gpgsm->colon.fnc = fnc;
   gpgsm->colon.fnc_value = fnc_value;
+  gpgsm->colon.any = 0;
+  return 0;
 }
 
 
-static GpgmeError
-_gpgme_gpgsm_add_io_cb (GpgsmObject gpgsm, iocb_data_t *iocbd,
-                       GpgmeIOCb handler)
-{
-  GpgmeError err = 0;
-
-  iocbd->tag = (*gpgsm->io_cbs.add) (gpgsm->io_cbs.add_priv,
-                                    iocbd->fd, iocbd->dir,
-                                    handler, iocbd->data);
-  if (!iocbd->tag)
-    err = mk_error (General_Error);
-  if (!err && !iocbd->dir)
-    /* FIXME Kludge around poll() problem.  */
-    err = _gpgme_io_set_nonblocking (iocbd->fd);
-  return err;
-}
-
-GpgmeError
-_gpgme_gpgsm_start (GpgsmObject gpgsm, void *opaque)
-{
-  GpgmeError err = 0;
-  pid_t pid;
-
-  if (!gpgsm)
-    return mk_error (Invalid_Value);
-
-  pid = assuan_get_pid (gpgsm->assuan_ctx);
-
-  err = _gpgme_gpgsm_add_io_cb (gpgsm, &gpgsm->status_cb,
-                               gpgsm_status_handler);
-  if (gpgsm->input_cb.fd != -1)
-    err = _gpgme_gpgsm_add_io_cb (gpgsm, &gpgsm->input_cb,
-                                 _gpgme_data_outbound_handler);
-  if (!err && gpgsm->output_cb.fd != -1)
-    err = _gpgme_gpgsm_add_io_cb (gpgsm, &gpgsm->output_cb,
-                                 _gpgme_data_inbound_handler);
-  if (!err && gpgsm->message_cb.fd != -1)
-    err = _gpgme_gpgsm_add_io_cb (gpgsm, &gpgsm->message_cb,
-                                 _gpgme_data_outbound_handler);
-
-  if (!err)
-    err = assuan_write_line (gpgsm->assuan_ctx, gpgsm->command);
-
-  return err;
-}
-
-void
-_gpgme_gpgsm_set_io_cbs (GpgsmObject gpgsm, struct GpgmeIOCbs *io_cbs)
+static void
+gpgsm_set_io_cbs (void *engine, struct GpgmeIOCbs *io_cbs)
 {
+  GpgsmObject gpgsm = engine;
   gpgsm->io_cbs = *io_cbs;
 }
 
 
-#else  /* ENABLE_GPGSM */
-
-
-#include <stddef.h>
-#include "util.h"
-
-#include "engine-gpgsm.h"
-
-
-const char *
-_gpgme_gpgsm_get_version (void)
-{
-  return NULL;
-}
-
-
-GpgmeError
-_gpgme_gpgsm_check_version (void)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_new (GpgsmObject *r_gpgsm)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-void
-_gpgme_gpgsm_release (GpgsmObject gpgsm)
-{
-  return;
-}
-
-
-void
-_gpgme_gpgsm_set_status_handler (GpgsmObject gpgsm,
-                                GpgStatusHandler fnc, void *fnc_value) 
-{
-  return;
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_decrypt (GpgsmObject gpgsm, GpgmeData ciph, GpgmeData plain)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_delete (GpgsmObject gpgsm, GpgmeKey key, int allow_secret)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_encrypt (GpgsmObject gpgsm, GpgmeRecipients recp,
-                        GpgmeData plain, GpgmeData ciph, int use_armor)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_export (GpgsmObject gpgsm, GpgmeRecipients recp,
-                       GpgmeData keydata, int use_armor)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_genkey (GpgsmObject gpgsm, GpgmeData help_data, int use_armor,
-                       GpgmeData pubkey, GpgmeData seckey)
-{
-  return mk_error (Invalid_Engine);
-}
-  
-
-GpgmeError
-_gpgme_gpgsm_op_import (GpgsmObject gpgsm, GpgmeData keydata)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_keylist (GpgsmObject gpgsm, const char *pattern,
-                        int secret_only, int keylist_mode)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_keylist_ext (GpgsmObject gpgsm, const char *pattern[],
-                            int secret_only, int reserved, int keylist_mode)
-{
-  return mk_error (Invalid_Engine);
-}
-
-GpgmeError
-_gpgme_gpgsm_op_sign (GpgsmObject gpgsm, GpgmeData in, GpgmeData out,
-                     GpgmeSigMode mode, int use_armor,
-                     int use_textmode, int include_certs,
-                     GpgmeCtx ctx /* FIXME */)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_trustlist (GpgsmObject gpgsm, const char *pattern)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-GpgmeError
-_gpgme_gpgsm_op_verify (GpgsmObject gpgsm, GpgmeData sig, GpgmeData text)
-{
-  return mk_error (Invalid_Engine);
-}
-
-
-void
-_gpgme_gpgsm_set_colon_line_handler (GpgsmObject gpgsm,
-                                     GpgColonLineHandler fnc, void *fnc_value) 
+static void
+gpgsm_io_event (void *engine, GpgmeEventIO type, void *type_data)
 {
-}
+  GpgsmObject gpgsm = engine;
 
-
-GpgmeError
-_gpgme_gpgsm_start (GpgsmObject gpgsm, void *opaque)
-{
-  return mk_error (Invalid_Engine);
+  if (gpgsm->io_cbs.event)
+    (*gpgsm->io_cbs.event) (gpgsm->io_cbs.event_priv, type, type_data);
 }
 
-void
-_gpgme_gpgsm_set_io_cbs (GpgsmObject gpgsm, struct GpgmeIOCbs *io_cbs)
-{
-}
 
-#endif /* ! ENABLE_GPGSM */
+struct engine_ops _gpgme_engine_ops_gpgsm =
+  {
+    /* Static functions.  */
+    _gpgme_get_gpgsm_path,
+    gpgsm_get_version,
+    gpgsm_get_req_version,
+    gpgsm_new,
+
+    /* Member functions.  */
+    gpgsm_release,
+    gpgsm_set_status_handler,
+    NULL,              /* set_command_handler */
+    gpgsm_set_colon_line_handler,
+    NULL,              /* set_verbosity */
+    gpgsm_decrypt,
+    gpgsm_delete,
+    NULL,              /* edit */
+    gpgsm_encrypt,
+    NULL,
+    gpgsm_export,
+    gpgsm_genkey,
+    gpgsm_import,
+    gpgsm_keylist,
+    gpgsm_keylist_ext,
+    gpgsm_sign,
+    gpgsm_trustlist,
+    gpgsm_verify,
+    gpgsm_set_io_cbs,
+    gpgsm_io_event
+  };