Tweaked some whitespace to match the latest version from autoconf.

[rsync/rsync.git] / pipe.c

diff --git a/pipe.c b/pipe.c
index 446afa7..8470608 100644 (file)
--- a/pipe.c
+++ b/pipe.c
@@ -24,12 +24,12 @@
 extern int am_sender;
 extern int am_server;
 extern int blocking_io;
-extern int orig_umask;
-extern int write_batch;
 extern int filesfrom_fd;
+extern mode_t orig_umask;
+extern struct chmod_mode_struct *chmod_modes;
 
 /**
- * Create a child connected to use on stdin/stdout.
+ * Create a child connected to us via its stdin/stdout.
  *
  * This is derived from CVS code
  *
@@ -111,6 +111,9 @@ pid_t local_child(int argc, char **argv, int *f_in, int *f_out,
        int to_child_pipe[2];
        int from_child_pipe[2];
 
+       /* The parent process is always the sender for a local rsync. */
+       assert(am_sender);
+
        if (fd_pair(to_child_pipe) < 0 ||
            fd_pair(from_child_pipe) < 0) {
                rsyserr(FERROR, errno, "pipe");
@@ -124,15 +127,10 @@ pid_t local_child(int argc, char **argv, int *f_in, int *f_out,
        }
 
        if (pid == 0) {
-               am_sender = !am_sender;
+               am_sender = 0;
                am_server = 1;
-
-               /* The server side never writes the batch, even if it
-                * is local (it makes the logic easier elsewhere). */
-               write_batch = 0;
-
-               if (!am_sender)
-                       filesfrom_fd = -1;
+               filesfrom_fd = -1;
+               chmod_modes = NULL; /* Let the sending side handle this. */
 
                if (dup2(to_child_pipe[0], STDIN_FILENO) < 0 ||
                    close(to_child_pipe[1]) < 0 ||
@@ -148,9 +146,6 @@ pid_t local_child(int argc, char **argv, int *f_in, int *f_out,
                child_main(argc, argv);
        }
 
-       if (!am_sender)
-               filesfrom_fd = -1;
-
        if (close(from_child_pipe[1]) < 0 ||
            close(to_child_pipe[0]) < 0) {
                rsyserr(FERROR, errno, "Failed to close");