gpgme/

2005-06-03  Marcus Brinkmann  <marcus@g10code.de>

	* wait-global.c (gpgme_wait): Break out of the fd processing loop
	after an error.
	Reported by Igor Belyi <gpgme@katehok.ac93.org>.

tests/
2005-06-03  Marcus Brinkmann  <marcus@g10code.de>

	* gpg/Makefile.am (TESTS): Add t-wait.
	* gpg/t-wait.c (main): New test.

diff --git a/trunk/gpgme/ChangeLog b/trunk/gpgme/ChangeLog
index 82f5fd64e6c1a7590fd1a037989aba437b521370..a28ece04eab2ceab998597e5f7f58230f4058d8d 100644 (file)
--- a/trunk/gpgme/ChangeLog
+++ b/trunk/gpgme/ChangeLog
@@ -1,3 +1,9 @@
+2005-06-03  Marcus Brinkmann  <marcus@g10code.de>
+
+       * wait-global.c (gpgme_wait): Break out of the fd processing loop
+       after an error.
+       Reported by Igor Belyi <gpgme@katehok.ac93.org>.
+
 2005-06-02  Marcus Brinkmann  <marcus@g10code.de>
 
        * wait.h (_gpgme_run_io_cb): New prototype.

diff --git a/trunk/gpgme/wait-global.c b/trunk/gpgme/wait-global.c
index d9f8727202e5b7294e401d9ebc23586f573fdf8c..a6da116e6d5684ca052a54deaf21126f52259b9c 100644 (file)
--- a/trunk/gpgme/wait-global.c
+++ b/trunk/gpgme/wait-global.c
@@ -322,6 +322,11 @@ gpgme_wait (gpgme_ctx_t ctx, gpgme_error_t *status, int hang)
                      _gpgme_io_close (ictx->fdt.fds[idx].fd);
                  _gpgme_engine_io_event (ictx->engine, GPGME_EVENT_DONE,
                                          &err);
+
+                 /* Break out of the loop, and retry the select()
+                    from scratch, because now all fds should be
+                    gone.  */
+                 break;
                }
            }
        }

diff --git a/trunk/tests/ChangeLog b/trunk/tests/ChangeLog
index d7bc24244e2d6df1c3458ed01cb978461f3b5ac1..690e4585b82cdeff65dbd7c763087e15a7693951 100644 (file)
--- a/trunk/tests/ChangeLog
+++ b/trunk/tests/ChangeLog
@@ -1,3 +1,8 @@
+2005-06-03  Marcus Brinkmann  <marcus@g10code.de>
+
+       * gpg/Makefile.am (TESTS): Add t-wait.
+       * gpg/t-wait.c (main): New test.
+
 2004-12-07  Marcus Brinkmann  <marcus@g10code.de>
 
        * gpg/mkdemodirs: Add copyright notice.

diff --git a/trunk/tests/gpg/Makefile.am b/trunk/tests/gpg/Makefile.am
index ceda8698cc585c85269fadfebd8d9bb3484d2448..73009a98fb18a6f54b42a2e29c8fc12ee3a991b3 100644 (file)
--- a/trunk/tests/gpg/Makefile.am
+++ b/trunk/tests/gpg/Makefile.am
@@ -1,5 +1,5 @@
 # Copyright (C) 2000 Werner Koch (dd9jn)
-# Copyright (C) 2001, 2004 g10 Code GmbH
+# Copyright (C) 2001, 2004, 2005 g10 Code GmbH
 # 
 # This file is part of GPGME.
 # 
@@ -28,7 +28,7 @@ noinst_HEADERS = t-support.h
 TESTS = t-encrypt t-encrypt-sym t-encrypt-sign t-sign t-signers        \
        t-decrypt t-verify t-decrypt-verify \
        t-export t-import t-trustlist t-eventloop t-edit \
-       t-keylist t-keylist-sig t-thread1
+       t-keylist t-keylist-sig t-thread1 t-wait
 
 CLEANFILES = secring.gpg pubring.gpg trustdb.gpg
 DISTCLEANFILES = pubring.gpg~ random_seed

diff --git a/trunk/tests/gpg/t-wait.c b/trunk/tests/gpg/t-wait.c
new file mode 100644 (file)
index 0000000..c12835d
--- /dev/null
+++ b/trunk/tests/gpg/t-wait.c
@@ -0,0 +1,72 @@
+/* t-wait.c - Regression test.
+   Copyright (C) 2000 Werner Koch (dd9jn)
+   Copyright (C) 2001, 2002, 2003, 2004, 2005 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, write to the Free Software
+   Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
+   02111-1307, USA.  */
+
+/* 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>
+
+#include "t-support.h"
+
+\f
+int 
+main (int argc, char *argv[])
+{
+  gpgme_ctx_t ctx;
+  gpgme_error_t err;
+  gpgme_data_t sig, text;
+
+  init_gpgme (GPGME_PROTOCOL_OpenPGP);
+
+  err = gpgme_new (&ctx);
+  fail_if_err (err);
+
+  /* Checking a message without a signature.  */
+  err = gpgme_data_new_from_mem (&sig, "foo\n", 4, 0);
+  fail_if_err (err);
+  err = gpgme_data_new (&text);
+  fail_if_err (err);
+  err = gpgme_op_verify_start (ctx, sig, NULL, text);
+  fail_if_err (err);
+
+  while (gpgme_wait (ctx, &err, 0) == NULL && err == 0)
+    sleep(1);
+
+  if (gpg_err_code (err) != GPG_ERR_NO_DATA)
+    {
+      fprintf (stderr, "%s:%d: %s: %s\n",
+              __FILE__, __LINE__, gpgme_strsource (err),
+              gpgme_strerror (err));
+      exit (1);
+    }
+
+  gpgme_data_release (sig);
+  gpgme_data_release (text);
+  gpgme_release (ctx);
+  return 0;
+}