Fix SIGPIPE ignoring regression.
authorWerner Koch <wk@gnupg.org>
Tue, 18 Nov 2008 11:12:36 +0000 (11:12 +0000)
committerWerner Koch <wk@gnupg.org>
Tue, 18 Nov 2008 11:12:36 +0000 (11:12 +0000)
Fix unsupported algorithm detection.

NEWS
src/ChangeLog
src/decrypt.c
src/version.c
tests/ChangeLog
tests/gpgsm/Makefile.am
tests/gpgsm/cms-decrypt.c [new file with mode: 0644]

diff --git a/NEWS b/NEWS
index 3ae3b99e4277b27c3bdaece9b41540d8b70f57bf..02f3fb92d5fd6c68beb185a5b1434443141ca1e0 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,9 @@
 Noteworthy changes in version 1.1.7 (unreleased)
 ------------------------------------------------
 
+ * SIGPIPE is now again ignored as described in the manual.  Fixes
+   regresion introduced with 1.1.6.
+
  * Interface changes relative to the 1.1.7 release:
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
index 9a8a749549ae2c0986ea12e842b56f987e76f6bf..f899c05586dbe1cf9574f33e32b235521cd64ccd 100644 (file)
@@ -1,3 +1,11 @@
+2008-11-18  Werner Koch  <wk@g10code.com>
+
+       * version.c (do_subsystem_inits): Always initialize I/O
+       subsystem.  Fixes regression from 2007-08-02.
+
+       * decrypt.c (_gpgme_decrypt_status_handler): Use
+       _gpgme_map_gnupg_error to parse the error code for decrypt.algorithm.
+
 2008-10-30  Marcus Brinkmann  <marcus@g10code.de>
 
        * wait-private.c (_gpgme_wait_on_condition): Remove unused
        * w32-io.c (_gpgme_io_spawn): Ditto.
        (_gpgme_io_write): Map ERROR_NO_DATA to EPIPE.
        * debug.c (_gpgme_debug): Enable assuan logging.
-       (_gpgme_debug_subsystem_init): New.  * version.c
-       (do_subsystem_inits): Disable assuan logging and initialize de
-       debug system.
+       (_gpgme_debug_subsystem_init): New.
+       * version.c (do_subsystem_inits): Disable assuan logging and
+       initialize the debug system.
        (gpgme_check_version): Do not trace before the subsystems are
        initialized.
 
index 66250b129f58f94db8020ebe88cbf148c459b06c..2569f97cdf43892d0bd495669ef44555fec93475 100644 (file)
@@ -180,7 +180,6 @@ _gpgme_decrypt_status_handler (void *priv, gpgme_status_code_t code,
          related to the backend.  */
       {
        const char d_alg[] = "decrypt.algorithm";
-       const char u_alg[] = "Unsupported_Algorithm";
        const char k_alg[] = "decrypt.keyusage";
 
        if (!strncmp (args, d_alg, sizeof (d_alg) - 1))
@@ -189,11 +188,13 @@ _gpgme_decrypt_status_handler (void *priv, gpgme_status_code_t code,
            while (*args == ' ')
              args++;
 
-           if (!strncmp (args, u_alg, sizeof (u_alg) - 1))
+           if (gpg_err_code (_gpgme_map_gnupg_error (args))
+                == GPG_ERR_UNSUPPORTED_ALGORITHM)
              {
                char *end;
 
-               args += sizeof (u_alg) - 1;
+               while (*args && *args != ' ')
+                 args++;
                while (*args == ' ')
                  args++;
 
index dd23ccfc4f3736bf5fba82e150a14f18f3c4347d..084f2a52f8467e8e6cff66f01d846d1eae2af32b 100644 (file)
@@ -60,8 +60,8 @@ do_subsystem_inits (void)
   assuan_set_assuan_err_source (GPG_ERR_SOURCE_GPGME);
 #endif /*HAVE_ASSUAN_H*/
   _gpgme_debug_subsystem_init ();
-#if defined(HAVE_W32_SYSTEM) && defined(HAVE_ASSUAN_H)
   _gpgme_io_subsystem_init ();
+#if defined(HAVE_W32_SYSTEM) && defined(HAVE_ASSUAN_H)
   /* We need to make sure that the sockets are initialized.  */
   {
     WSADATA wsadat;
index 088d5ba3436dbef12c8e903761f559444ce9db6b..5a315240d840f420c1d4988c6609b1c936a28e1a 100644 (file)
@@ -1,9 +1,17 @@
+2008-11-18  Werner Koch  <wk@g10code.com>
+
+       * gpgsm/cms-decrypt.c: New.
+
 2008-11-03  Marcus Brinkmann  <marcus@g10code.com>
 
        * gpgsm/Makefile.am (INCLUDES, LDADD): Replace gpgme path with src.
        * gpg/Makefile.am (INCLUDES, LDADD, t_thread1_LDADD): Likewise.
        * Makefile.am (LDADD): Likewise.
 
+2008-10-30  Werner Koch  <wk@g10code.com>
+
+       * gpgsm/cms-keylist.c: New.
+
 2008-06-19  Werner Koch  <wk@g10code.com>
 
        * gpg/t-gpgconf.c (dump_arg): Add new types.  Print strings in
index 91a8cf733fd28cdbe624b333800ef7cca40cd609..20503303f71ab33eedf2fe1376929e7422835e37 100644 (file)
@@ -36,7 +36,7 @@ LDADD = ../../src/libgpgme.la
 
 # We don't run t-genkey in the test suite, because it takes too long
 # and needs a working pinentry.
-noinst_PROGRAMS = $(TESTS) t-genkey cms-keylist
+noinst_PROGRAMS = $(TESTS) t-genkey cms-keylist cms-decrypt
 
 key_id = 32100C27173EF6E9C4E9A25D3D69F86D37A4F939
 
diff --git a/tests/gpgsm/cms-decrypt.c b/tests/gpgsm/cms-decrypt.c
new file mode 100644 (file)
index 0000000..767f87b
--- /dev/null
@@ -0,0 +1,108 @@
+/* cms-decrypt.c  - Helper to debug the decrupt operation.
+   Copyright (C) 2008 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 Lesser General Public License as
+   published by the Free Software Foundation; either version 2.1 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
+   Lesser General Public License for more details.
+   
+   You should have received a copy of the GNU Lesser General Public
+   License along with this program; if not, see <http://www.gnu.org/licenses/>.
+*/
+
+/* We need to include config.h so that we know whether we are building
+   with large file system (LFS) support. */
+#ifdef HAVE_CONFIG_H
+#include <config.h>
+#endif
+
+#include <stdlib.h>
+#include <stdio.h>
+#include <string.h>
+
+#include <gpgme.h>
+
+#define PGM "cms-decrypt"
+
+#include "t-support.h"
+
+static const char *
+nonnull (const char *s)
+{
+  return s? s :"[none]";
+}
+
+
+int 
+main (int argc, char **argv)
+{
+  gpgme_error_t err;
+  gpgme_ctx_t ctx;
+  gpgme_data_t in, out;
+  gpgme_decrypt_result_t result;
+  gpgme_recipient_t recp;
+
+  if (argc)
+    { argc--; argv++; }
+
+  if (argc != 1)
+    {
+      fputs ("usage: " PGM " FILE\n", stderr);
+      exit (1);
+    }
+
+  init_gpgme (GPGME_PROTOCOL_CMS);
+
+  err = gpgme_new (&ctx);
+  fail_if_err (err);
+  gpgme_set_protocol (ctx, GPGME_PROTOCOL_CMS);
+
+
+  err = gpgme_data_new_from_file (&in, *argv, 1);
+  fail_if_err (err);
+
+  err = gpgme_data_new (&out);
+  fail_if_err (err);
+
+  err = gpgme_op_decrypt (ctx, in, out);
+  printf ("gpgme_op_decrypt: %s <%s> (%u)\n",
+          gpg_strerror (err), gpg_strsource (err), err);
+  result = gpgme_op_decrypt_result (ctx);
+  if (!result)
+    {
+      fputs (PGM ": error: decryption result missing\n", stderr);
+      exit (1);
+    }
+  
+  printf ("unsupported_algorithm: %s\n", 
+          nonnull (result->unsupported_algorithm));
+  printf ("wrong_key_usage: %u\n",  result->wrong_key_usage);
+  printf ("file_name: %s\n", nonnull (result->file_name));
+  for (recp = result->recipients; recp; recp = recp->next)
+    {
+      printf ("recipient.status: %s <%s> (%u)\n",
+              gpg_strerror (recp->status), gpg_strsource (recp->status),
+              recp->status);
+      printf ("recipient.pkalgo: %d\n", recp->pubkey_algo);
+      printf ("recipient.keyid : %s\n", nonnull (recp->keyid));
+    }
+
+  if (!err)
+    {
+      puts ("plaintext:");
+      print_data (out);
+      gpgme_data_release (out);
+    }
+
+  gpgme_data_release (in);
+
+  gpgme_release (ctx);
+  return 0;
+}