2002-11-25 Marcus Brinkmann <marcus@g10code.de>

	* rungpg.c (_gpgme_gpg_spawn): Do not set parent fds to -1.
	* posix-io.c (_gpgme_io_spawn): Call _gpgme_io_close instead close
	for parent fds.
	* w32-io.c (_gpgme_io_spawn): Call _gpgme_io_close instead
	CloseHandle for parent fds.

diff --git a/branches/gpgme-0-3-branch/gpgme/ChangeLog b/branches/gpgme-0-3-branch/gpgme/ChangeLog
index 06e9a323580a771ff16691b6c7cc011b6063ff4f..90d1a5042b026e89fda38137667bf7200996de51 100644 (file)
--- a/branches/gpgme-0-3-branch/gpgme/ChangeLog
+++ b/branches/gpgme-0-3-branch/gpgme/ChangeLog
@@ -1,3 +1,11 @@
+2002-11-25  Marcus Brinkmann  <marcus@g10code.de>
+
+       * rungpg.c (_gpgme_gpg_spawn): Do not set parent fds to -1.
+       * posix-io.c (_gpgme_io_spawn): Call _gpgme_io_close instead close
+       for parent fds.
+       * w32-io.c (_gpgme_io_spawn): Call _gpgme_io_close instead
+       CloseHandle for parent fds.
+       
 2002-11-22  Marcus Brinkmann  <marcus@g10code.de>
 
        * gpgme.h [_MSC_VER]: Define ssize_t as long.

diff --git a/branches/gpgme-0-3-branch/gpgme/posix-io.c b/branches/gpgme-0-3-branch/gpgme/posix-io.c
index da7c106d80437c4d07244626b96ba0c086295591..ecd7a21778bd2c237e78ddc6d4573f20d302550c 100644 (file)
--- a/branches/gpgme-0-3-branch/gpgme/posix-io.c
+++ b/branches/gpgme-0-3-branch/gpgme/posix-io.c
@@ -257,7 +257,7 @@ _gpgme_io_spawn (const char *path, char **argv,
 
   /* .dup_to is not used in the parent list.  */
   for (i = 0; fd_parent_list[i].fd != -1; i++)
-    close (fd_parent_list[i].fd);
+    _gpgme_io_close (fd_parent_list[i].fd);
 
   return 0;
 }

diff --git a/branches/gpgme-0-3-branch/gpgme/rungpg.c b/branches/gpgme-0-3-branch/gpgme/rungpg.c
index 6cffe35e9fa9fc3dcb1a448f941787e083044b10..a3af80d3cdeb08ac5a1e37421a21a81fffe575f7 100644 (file)
--- a/branches/gpgme-0-3-branch/gpgme/rungpg.c
+++ b/branches/gpgme-0-3-branch/gpgme/rungpg.c
@@ -890,21 +890,18 @@ _gpgme_gpg_spawn (GpgObject gpg, void *opaque)
       fd_parent_list[n].fd = gpg->status.fd[1];
       fd_parent_list[n].dup_to = -1;
       n++;
-      gpg->status.fd[1] = -1;
     }
   if (gpg->colon.fd[1] != -1)
     {
       fd_parent_list[n].fd = gpg->colon.fd[1];
       fd_parent_list[n].dup_to = -1;
       n++;
-      gpg->colon.fd[1] = -1;
     }
   for (i = 0; gpg->fd_data_map[i].data; i++)
     {
       fd_parent_list[n].fd = gpg->fd_data_map[i].peer_fd;
       fd_parent_list[n].dup_to = -1;
       n++;
-      gpg->fd_data_map[i].peer_fd = -1;
     }        
   fd_parent_list[n].fd = -1;
   fd_parent_list[n].dup_to = -1;

diff --git a/branches/gpgme-0-3-branch/gpgme/w32-io.c b/branches/gpgme-0-3-branch/gpgme/w32-io.c
index 6d4957fef0d0a3b9fb98b463e5d3fc671eda3e5c..9a1a21261169bb3ca8b19855f7942a6e83899954 100644 (file)
--- a/branches/gpgme-0-3-branch/gpgme/w32-io.c
+++ b/branches/gpgme-0-3-branch/gpgme/w32-io.c
@@ -899,11 +899,8 @@ _gpgme_io_spawn ( const char *path, char **argv,
     }
 
     /* Close the other ends of the pipes */
-    for (i=0; fd_parent_list[i].fd != -1; i++ ) {
-        DEBUG1 ("Closing fd %d\n", fd_parent_list[i].fd );
-        if ( !CloseHandle ( fd_to_handle (fd_parent_list[i].fd) ) )
-            DEBUG1 ("CloseHandle failed: ec=%d", (int)GetLastError());
-    }
+    for (i = 0; fd_parent_list[i].fd != -1; i++)
+      _gpgme_io_close (fd_parent_list[i].fd);
 
     DEBUG4 ("CreateProcess ready\n"
             "-   hProcess=%p  hThread=%p\n"