assuan/

2008-10-30  Marcus Brinkmann  <marcus@g10code.de>

	* assuan-pipe-connect.c: Fix prototype for _gpgme_io_spawn.  Cast
	second argument in its invocation to silence gcc warning.

gpgme/
2008-10-30  Marcus Brinkmann  <marcus@g10code.de>

	* wait-private.c (_gpgme_wait_on_condition): Remove unused
	variable IDX.
	* wait-global.c: Include ops.h to silence gcc warning.
	(_gpgme_wait_global_event_cb): Pass error value directly.
	* wait-user.c: Include ops.h to silence gcc warning.

	* posix-io.c (_gpgme_io_spawn): Make ARGV argument const to
	silence gcc warning.  Cast argument to execv to silence warning.
	* w32-io.c (_gpgme_io_spawn): Likewise.
	* priv-io.h (_gpgme_io_spawn): Likewise for prototype.

diff --git a/assuan/ChangeLog b/assuan/ChangeLog
index 2e01943be022f6192a88a418b6cde41dda0feb09..cf15674e8a805f28544edc50d43a9e4582c3dee6 100644 (file)
--- a/assuan/ChangeLog
+++ b/assuan/ChangeLog
@@ -1,3 +1,8 @@
+2008-10-30  Marcus Brinkmann  <marcus@g10code.de>
+
+       * assuan-pipe-connect.c: Fix prototype for _gpgme_io_spawn.  Cast
+       second argument in its invocation to silence gcc warning.
+
 2008-06-25  Marcus Brinkmann  <marcus@g10code.de>
 
        * assuan-pipe-connect.c (struct spawn_fd_item_s): Add new members.

diff --git a/assuan/assuan-pipe-connect.c b/assuan/assuan-pipe-connect.c
index a0091c6a35ca50693e08dce08bae902f405274d3..e35553259d39433730bac35dc8a1ea37da5d8f08 100644 (file)
--- a/assuan/assuan-pipe-connect.c
+++ b/assuan/assuan-pipe-connect.c
@@ -52,7 +52,7 @@ struct spawn_fd_item_s
 
 
 int _gpgme_io_pipe (int filedes[2], int inherit_idx);
-int _gpgme_io_spawn (const char *path, char **argv,
+int _gpgme_io_spawn (const char *path, char *const argv[],
                     struct spawn_fd_item_s *fd_list, pid_t *r_pid);
 #endif
 
@@ -665,7 +665,7 @@ pipe_connect_gpgme (assuan_context_t *ctx,
   child_fds[nr].dup_to = -1;
 
   /* Start the process.  */
-  res = _gpgme_io_spawn (name, argv, child_fds, NULL);
+  res = _gpgme_io_spawn (name, (char *const *) argv, child_fds, NULL);
   if (res == -1)
     {
       _assuan_log_printf ("CreateProcess failed: %s\n", strerror (errno));

diff --git a/gpgme/ChangeLog b/gpgme/ChangeLog
index f848d18130f9d096815491c64f58e05a086dcadc..9a8a749549ae2c0986ea12e842b56f987e76f6bf 100644 (file)
--- a/gpgme/ChangeLog
+++ b/gpgme/ChangeLog
@@ -1,3 +1,16 @@
+2008-10-30  Marcus Brinkmann  <marcus@g10code.de>
+
+       * wait-private.c (_gpgme_wait_on_condition): Remove unused
+       variable IDX.
+       * wait-global.c: Include ops.h to silence gcc warning.
+       (_gpgme_wait_global_event_cb): Pass error value directly.
+       * wait-user.c: Include ops.h to silence gcc warning.
+
+       * posix-io.c (_gpgme_io_spawn): Make ARGV argument const to
+       silence gcc warning.  Cast argument to execv to silence warning.
+       * w32-io.c (_gpgme_io_spawn): Likewise.
+       * priv-io.h (_gpgme_io_spawn): Likewise for prototype.
+
 2008-10-24  Werner Koch  <wk@g10code.com>
 
        * rungpg.c (gpg_keylist_preprocess): Escape backslashes too.

diff --git a/gpgme/posix-io.c b/gpgme/posix-io.c
index 8c6888296ca78a5ad58f47071c2c969ddbb44447..a7047bd280a4a9e73e4258833ff6560a0edeb8cf 100644 (file)
--- a/gpgme/posix-io.c
+++ b/gpgme/posix-io.c
@@ -304,7 +304,7 @@ _gpgme_io_waitpid (int pid, int hang, int *r_status, int *r_signal)
 
 /* Returns 0 on success, -1 on error.  */
 int
-_gpgme_io_spawn (const char *path, char **argv,
+_gpgme_io_spawn (const char *path, char *const argv[],
                 struct spawn_fd_item_s *fd_list, pid_t *r_pid)
 {
   pid_t pid;
@@ -427,7 +427,7 @@ _gpgme_io_spawn (const char *path, char **argv,
                close (fd);
            }
     
-         execv (path, argv);
+         execv (path, (char *const *) argv);
          /* Hmm: in that case we could write a special status code to the
             status-pipe.  */
 #if 0

diff --git a/gpgme/priv-io.h b/gpgme/priv-io.h
index 90a385dabc3f1f437e4b3f7fc4b971f885e0a629..4a87f650b463e1727021cda4307f657ebc610431 100644 (file)
--- a/gpgme/priv-io.h
+++ b/gpgme/priv-io.h
@@ -63,7 +63,7 @@ int _gpgme_io_set_nonblocking (int fd);
    close all fds except for those in FD_LIST in the child, then
    optionally dup() the child fds.  Finally, all fds in the list are
    closed in the parent.  */
-int _gpgme_io_spawn (const char *path, char **argv,
+int _gpgme_io_spawn (const char *path, char *const argv[],
                     struct spawn_fd_item_s *fd_list, pid_t *r_pid);
 
 int _gpgme_io_select (struct io_select_fd_s *fds, size_t nfds, int nonblock);

diff --git a/gpgme/w32-io.c b/gpgme/w32-io.c
index e90f77968b0afd8db342a13ee46fa1846e299b9b..1f62a6f1ab4ed64a5dde355264c2d06af1453169 100644 (file)
--- a/gpgme/w32-io.c
+++ b/gpgme/w32-io.c
@@ -1009,7 +1009,7 @@ build_commandline (char **argv)
 
 
 int
-_gpgme_io_spawn (const char *path, char **argv,
+_gpgme_io_spawn (const char *path, char *const argv[],
                 struct spawn_fd_item_s *fd_list, pid_t *r_pid)
 {
   SECURITY_ATTRIBUTES sec_attr;

diff --git a/gpgme/wait-global.c b/gpgme/wait-global.c
index 1ab44d167be88d54c372f5716443f63abcbc506d..ececc1bc8b6d77bd8e985866e232d9a7f8df2241 100644 (file)
--- a/gpgme/wait-global.c
+++ b/gpgme/wait-global.c
@@ -33,6 +33,7 @@
 #include "context.h"
 #include "wait.h"
 #include "priv-io.h"
+#include "ops.h"
 
 /* The global event loop is used for all asynchronous operations
    (except key listing) for which no user I/O callbacks are specified.
@@ -202,7 +203,7 @@ _gpgme_wait_global_event_cb (void *data, gpgme_event_io_t type,
        if (err)
          /* An error occured.  Close all fds in this context, and
             send the error in a done event.  */
-         _gpgme_cancel_with_err (ctx, &err);
+         _gpgme_cancel_with_err (ctx, err);
       }
       break;
 

diff --git a/gpgme/wait-private.c b/gpgme/wait-private.c
index 0366057401f0be6c212b7ffe7c3361ddb343cf0c..2dee1a93952c97fc6894440555c10bdf1a613708 100644 (file)
--- a/gpgme/wait-private.c
+++ b/gpgme/wait-private.c
@@ -86,8 +86,6 @@ _gpgme_wait_on_condition (gpgme_ctx_t ctx, volatile int *cond)
        {
          /* An error occured.  Close all fds in this context, and
             signal it.  */
-         unsigned int idx;
-
          err = gpg_error_from_errno (errno);
           _gpgme_cancel_with_err (ctx, err);
 

diff --git a/gpgme/wait-user.c b/gpgme/wait-user.c
index 05750e2a444ad03c095342fb7c599e3fdee30c28..995657fa19be7e02121d9daf0617c8abe7efee7a 100644 (file)
--- a/gpgme/wait-user.c
+++ b/gpgme/wait-user.c
@@ -28,6 +28,7 @@
 #include "context.h"
 #include "priv-io.h"
 #include "wait.h"
+#include "ops.h"
 
 \f
 /* The user event loops are used for all asynchronous operations for