Added "const" to appropriate char pointers.
[rsync/rsync.git] / io.c
diff --git a/io.c b/io.c
index 64c1900..0251176 100644 (file)
--- a/io.c
+++ b/io.c
@@ -1,8 +1,10 @@
-/* -*- c-file-style: "linux" -*-
+/*
+ * Socket and pipe I/O utilities used in rsync.
  *
- * Copyright (C) 1996-2001 by Andrew Tridgell
- * Copyright (C) Paul Mackerras 1996
- * Copyright (C) 2001, 2002 by Martin Pool <mbp@samba.org>
+ * Copyright (C) 1996-2001 Andrew Tridgell
+ * Copyright (C) 1996 Paul Mackerras
+ * Copyright (C) 2001, 2002 Martin Pool <mbp@samba.org>
+ * Copyright (C) 2003, 2004, 2005, 2006 Wayne Davison
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA.
  */
 
-/**
- * @file io.c
- *
- * Socket and pipe I/O utilities used in rsync.
- *
- * rsync provides its own multiplexing system, which is used to send
- * stderr and stdout over a single socket.  We need this because
- * stdout normally carries the binary data stream, and stderr all our
- * error messages.
+/* Rsync provides its own multiplexing system, which is used to send
+ * stderr and stdout over a single socket.
  *
  * For historical reasons this is off during the start of the
  * connection, but it's switched on quite early using
- * io_start_multiplex_out() and io_start_multiplex_in().
- **/
+ * io_start_multiplex_out() and io_start_multiplex_in(). */
 
 #include "rsync.h"
 
 
 extern int bwlimit;
 extern size_t bwlimit_writemax;
-extern int verbose;
 extern int io_timeout;
+extern int allowed_lull;
 extern int am_server;
 extern int am_daemon;
 extern int am_sender;
 extern int am_generator;
 extern int eol_nulls;
+extern int read_batch;
 extern int csum_length;
 extern int checksum_seed;
 extern int protocol_version;
+extern int remove_source_files;
+extern int preserve_hard_links;
 extern char *filesfrom_host;
 extern struct stats stats;
+extern struct file_list *the_file_list;
 
 const char phase_unknown[] = "unknown";
-int select_timeout = SELECT_TIMEOUT;
 int ignore_timeout = 0;
 int batch_fd = -1;
 int batch_gen_fd = -1;
 
-/**
- * The connection might be dropped at some point; perhaps because the
- * remote instance crashed.  Just giving the offset on the stream is
- * not very helpful.  So instead we try to make io_phase_name point to
- * something useful.
- *
- * For buffered/multiplexed I/O these names will be somewhat
- * approximate; perhaps for ease of support we would rather make the
- * buffer always flush when a single application-level I/O finishes.
- *
- * @todo Perhaps we want some simple stack functionality, but there's
- * no need to overdo it.
- **/
-const char *io_write_phase = phase_unknown;
-const char *io_read_phase = phase_unknown;
-
 /* Ignore an EOF error if non-zero. See whine_about_eof(). */
 int kluge_around_eof = 0;
 
 int msg_fd_in = -1;
 int msg_fd_out = -1;
+int sock_f_in = -1;
+int sock_f_out = -1;
 
 static int io_multiplexing_out;
 static int io_multiplexing_in;
-static int sock_f_in = -1;
-static int sock_f_out = -1;
-static time_t last_io;
+static time_t last_io_in;
+static time_t last_io_out;
 static int no_flush;
 
 static int write_batch_monitor_in = -1;
@@ -98,40 +80,67 @@ static char io_filesfrom_buf[2048];
 static char *io_filesfrom_bp;
 static char io_filesfrom_lastchar;
 static int io_filesfrom_buflen;
-static size_t contiguous_write_len = 0;
+static int defer_forwarding_messages = 0;
+static int select_timeout = SELECT_TIMEOUT;
+static int active_filecnt = 0;
+static OFF_T active_bytecnt = 0;
 
 static void read_loop(int fd, char *buf, size_t len);
 
-struct redo_list {
-       struct redo_list *next;
-       int num;
+struct flist_ndx_item {
+       struct flist_ndx_item *next;
+       int ndx;
 };
 
-static struct redo_list *redo_list_head;
-static struct redo_list *redo_list_tail;
+struct flist_ndx_list {
+       struct flist_ndx_item *head, *tail;
+};
 
-struct msg_list {
-       struct msg_list *next;
-       char *buf;
+static struct flist_ndx_list redo_list, hlink_list;
+
+struct msg_list_item {
+       struct msg_list_item *next;
        int len;
+       char buf[1];
 };
 
-static struct msg_list *msg_list_head;
-static struct msg_list *msg_list_tail;
+struct msg_list {
+       struct msg_list_item *head, *tail;
+};
+
+static struct msg_list msg2genr, msg2sndr;
 
-static void redo_list_add(int num)
+static void flist_ndx_push(struct flist_ndx_list *lp, int ndx)
 {
-       struct redo_list *rl;
-
-       if (!(rl = new(struct redo_list)))
-               exit_cleanup(RERR_MALLOC);
-       rl->next = NULL;
-       rl->num = num;
-       if (redo_list_tail)
-               redo_list_tail->next = rl;
+       struct flist_ndx_item *item;
+
+       if (!(item = new(struct flist_ndx_item)))
+               out_of_memory("flist_ndx_push");
+       item->next = NULL;
+       item->ndx = ndx;
+       if (lp->tail)
+               lp->tail->next = item;
        else
-               redo_list_head = rl;
-       redo_list_tail = rl;
+               lp->head = item;
+       lp->tail = item;
+}
+
+static int flist_ndx_pop(struct flist_ndx_list *lp)
+{
+       struct flist_ndx_item *next;
+       int ndx;
+
+       if (!lp->head)
+               return -1;
+
+       ndx = lp->head->ndx;
+       next = lp->head->next;
+       free(lp->head);
+       lp->head = next;
+       if (!next)
+               lp->tail = NULL;
+
+       return ndx;
 }
 
 static void check_timeout(void)
@@ -141,17 +150,17 @@ static void check_timeout(void)
        if (!io_timeout || ignore_timeout)
                return;
 
-       if (!last_io) {
-               last_io = time(NULL);
+       if (!last_io_in) {
+               last_io_in = time(NULL);
                return;
        }
 
        t = time(NULL);
 
-       if (t - last_io >= io_timeout) {
+       if (t - last_io_in >= io_timeout) {
                if (!am_server && !am_daemon) {
                        rprintf(FERROR, "io timeout after %d seconds -- exiting\n",
-                               (int)(t-last_io));
+                               (int)(t-last_io_in));
                }
                exit_cleanup(RERR_TIMEOUT);
        }
@@ -165,6 +174,18 @@ void io_set_sock_fds(int f_in, int f_out)
        sock_f_out = f_out;
 }
 
+void set_io_timeout(int secs)
+{
+       io_timeout = secs;
+
+       if (!io_timeout || io_timeout > SELECT_TIMEOUT)
+               select_timeout = SELECT_TIMEOUT;
+       else
+               select_timeout = io_timeout;
+
+       allowed_lull = read_batch ? 0 : (io_timeout + 1) / 2;
+}
+
 /* Setup the fd used to receive MSG_* messages.  Only needed during the
  * early stages of being a local sender (up through the sending of the
  * file list) or when we're the generator (to fetch the messages from
@@ -183,33 +204,22 @@ void set_msg_fd_out(int fd)
 }
 
 /* Add a message to the pending MSG_* list. */
-static void msg_list_add(int code, char *buf, int len)
+static void msg_list_add(struct msg_list *lst, int code, const char *buf, int len)
 {
-       struct msg_list *ml;
-
-       if (!(ml = new(struct msg_list)))
-               exit_cleanup(RERR_MALLOC);
-       ml->next = NULL;
-       if (!(ml->buf = new_array(char, len+4)))
-               exit_cleanup(RERR_MALLOC);
-       SIVAL(ml->buf, 0, ((code+MPLEX_BASE)<<24) | len);
-       memcpy(ml->buf+4, buf, len);
-       ml->len = len+4;
-       if (msg_list_tail)
-               msg_list_tail->next = ml;
+       struct msg_list_item *m;
+       int sz = len + 4 + sizeof m[0] - 1;
+
+       if (!(m = (struct msg_list_item *)new_array(char, sz)))
+               out_of_memory("msg_list_add");
+       m->next = NULL;
+       m->len = len + 4;
+       SIVAL(m->buf, 0, ((code+MPLEX_BASE)<<24) | len);
+       memcpy(m->buf + 4, buf, len);
+       if (lst->tail)
+               lst->tail->next = m;
        else
-               msg_list_head = ml;
-       msg_list_tail = ml;
-}
-
-void send_msg(enum msgcode code, char *buf, int len)
-{
-       if (msg_fd_out < 0) {
-               io_multiplex_write(code, buf, len);
-               return;
-       }
-       msg_list_add(code, buf, len);
-       msg_list_push(NORMAL_FLUSH);
+               lst->head = m;
+       lst->tail = m;
 }
 
 /* Read a message from the MSG_* fd and handle it.  This is called either
@@ -239,7 +249,7 @@ static void read_msg_fd(void)
                        rprintf(FERROR, "invalid message %d:%d\n", tag, len);
                        exit_cleanup(RERR_STREAMIO);
                }
-               redo_list_add(-1);
+               flist_ndx_push(&redo_list, -1);
                break;
        case MSG_REDO:
                if (len != 4 || !am_generator) {
@@ -247,7 +257,9 @@ static void read_msg_fd(void)
                        exit_cleanup(RERR_STREAMIO);
                }
                read_loop(fd, buf, 4);
-               redo_list_add(IVAL(buf,0));
+               if (remove_source_files)
+                       decrement_active_files(IVAL(buf,0));
+               flist_ndx_push(&redo_list, IVAL(buf,0));
                break;
        case MSG_DELETED:
                if (len >= (int)sizeof buf || !am_generator) {
@@ -255,7 +267,7 @@ static void read_msg_fd(void)
                        exit_cleanup(RERR_STREAMIO);
                }
                read_loop(fd, buf, len);
-               io_multiplex_write(MSG_DELETED, buf, len);
+               send_msg(MSG_DELETED, buf, len);
                break;
        case MSG_SUCCESS:
                if (len != 4 || !am_generator) {
@@ -263,8 +275,20 @@ static void read_msg_fd(void)
                        exit_cleanup(RERR_STREAMIO);
                }
                read_loop(fd, buf, len);
-               io_multiplex_write(MSG_SUCCESS, buf, len);
+               if (remove_source_files) {
+                       decrement_active_files(IVAL(buf,0));
+                       send_msg(MSG_SUCCESS, buf, len);
+               }
+               if (preserve_hard_links)
+                       flist_ndx_push(&hlink_list, IVAL(buf,0));
                break;
+       case MSG_SOCKERR:
+               if (!am_generator) {
+                       rprintf(FERROR, "invalid message %d:%d\n", tag, len);
+                       exit_cleanup(RERR_STREAMIO);
+               }
+               close_multiplexing_out();
+               /* FALL THROUGH */
        case MSG_INFO:
        case MSG_ERROR:
        case MSG_LOG:
@@ -273,22 +297,45 @@ static void read_msg_fd(void)
                        if (n >= sizeof buf)
                                n = sizeof buf - 1;
                        read_loop(fd, buf, n);
-                       rwrite((enum logcode)tag, buf, n);
+                       rwrite(tag, buf, n);
                        len -= n;
                }
                break;
        default:
-               rprintf(FERROR, "unknown message %d:%d\n", tag, len);
+               rprintf(FERROR, "unknown message %d:%d [%s]\n",
+                       tag, len, who_am_i());
                exit_cleanup(RERR_STREAMIO);
        }
 
        msg_fd_in = fd;
 }
 
+/* This is used by the generator to limit how many file transfers can
+ * be active at once when --remove-source-files is specified.  Without
+ * this, sender-side deletions were mostly happening at the end. */
+void increment_active_files(int ndx, int itemizing, enum logcode code)
+{
+       /* TODO: tune these limits? */
+       while (active_filecnt >= (active_bytecnt >= 128*1024 ? 10 : 50)) {
+               if (hlink_list.head)
+                       check_for_finished_hlinks(itemizing, code);
+               read_msg_fd();
+       }
+
+       active_filecnt++;
+       active_bytecnt += the_file_list->files[ndx]->length;
+}
+
+void decrement_active_files(int ndx)
+{
+       active_filecnt--;
+       active_bytecnt -= the_file_list->files[ndx]->length;
+}
+
 /* Try to push messages off the list onto the wire.  If we leave with more
  * to do, return 0.  On error, return -1.  If everything flushed, return 1.
  * This is only active in the receiver. */
-int msg_list_push(int flush_it_all)
+static int msg2genr_flush(int flush_it_all)
 {
        static int written = 0;
        struct timeval tv;
@@ -297,9 +344,9 @@ int msg_list_push(int flush_it_all)
        if (msg_fd_out < 0)
                return -1;
 
-       while (msg_list_head) {
-               struct msg_list *ml = msg_list_head;
-               int n = write(msg_fd_out, ml->buf + written, ml->len - written);
+       while (msg2genr.head) {
+               struct msg_list_item *m = msg2genr.head;
+               int n = write(msg_fd_out, m->buf + written, m->len - written);
                if (n < 0) {
                        if (errno == EINTR)
                                continue;
@@ -313,34 +360,48 @@ int msg_list_push(int flush_it_all)
                        tv.tv_usec = 0;
                        if (!select(msg_fd_out+1, NULL, &fds, NULL, &tv))
                                check_timeout();
-               } else if ((written += n) == ml->len) {
-                       free(ml->buf);
-                       msg_list_head = ml->next;
-                       if (!msg_list_head)
-                               msg_list_tail = NULL;
-                       free(ml);
+               } else if ((written += n) == m->len) {
+                       msg2genr.head = m->next;
+                       if (!msg2genr.head)
+                               msg2genr.tail = NULL;
+                       free(m);
                        written = 0;
                }
        }
        return 1;
 }
 
-int get_redo_num(void)
+int send_msg(enum msgcode code, const char *buf, int len)
 {
-       struct redo_list *next;
-       int num;
+       if (msg_fd_out < 0) {
+               if (!defer_forwarding_messages)
+                       return io_multiplex_write(code, buf, len);
+               if (!io_multiplexing_out)
+                       return 0;
+               msg_list_add(&msg2sndr, code, buf, len);
+               return 1;
+       }
+       msg_list_add(&msg2genr, code, buf, len);
+       msg2genr_flush(NORMAL_FLUSH);
+       return 1;
+}
 
-       while (!redo_list_head)
+int get_redo_num(int itemizing, enum logcode code)
+{
+       while (1) {
+               if (hlink_list.head)
+                       check_for_finished_hlinks(itemizing, code);
+               if (redo_list.head)
+                       break;
                read_msg_fd();
+       }
 
-       num = redo_list_head->num;
-       next = redo_list_head->next;
-       free(redo_list_head);
-       redo_list_head = next;
-       if (!next)
-               redo_list_tail = NULL;
+       return flist_ndx_pop(&redo_list);
+}
 
-       return num;
+int get_hlink_num(void)
+{
+       return flist_ndx_pop(&hlink_list);
 }
 
 /**
@@ -392,7 +453,6 @@ static void whine_about_eof(int fd)
        exit_cleanup(RERR_STREAMIO);
 }
 
-
 /**
  * Read from a socket with I/O timeout. return the number of bytes
  * read. If no bytes can be read then exit, never return a number <= 0.
@@ -406,11 +466,11 @@ static void whine_about_eof(int fd)
  */
 static int read_timeout(int fd, char *buf, size_t len)
 {
-       int n, ret = 0;
+       int n, cnt = 0;
 
        io_flush(NORMAL_FLUSH);
 
-       while (ret == 0) {
+       while (cnt == 0) {
                /* until we manage to read *something* */
                fd_set r_fds, w_fds;
                struct timeval tv;
@@ -420,7 +480,7 @@ static int read_timeout(int fd, char *buf, size_t len)
                FD_ZERO(&r_fds);
                FD_ZERO(&w_fds);
                FD_SET(fd, &r_fds);
-               if (msg_list_head) {
+               if (msg2genr.head) {
                        FD_SET(msg_fd_out, &w_fds);
                        if (msg_fd_out > maxfd)
                                maxfd = msg_fd_out;
@@ -457,8 +517,8 @@ static int read_timeout(int fd, char *buf, size_t len)
                        continue;
                }
 
-               if (msg_list_head && FD_ISSET(msg_fd_out, &w_fds))
-                       msg_list_push(NORMAL_FLUSH);
+               if (msg2genr.head && FD_ISSET(msg_fd_out, &w_fds))
+                       msg2genr_flush(NORMAL_FLUSH);
 
                if (io_filesfrom_f_out >= 0) {
                        if (io_filesfrom_buflen) {
@@ -536,21 +596,23 @@ static int read_timeout(int fd, char *buf, size_t len)
                                continue;
 
                        /* Don't write errors on a dead socket. */
-                       if (fd == sock_f_in)
+                       if (fd == sock_f_in) {
                                close_multiplexing_out();
-                       rsyserr(FERROR, errno, "read error");
+                               rsyserr(FSOCKERR, errno, "read error");
+                       } else
+                               rsyserr(FERROR, errno, "read error");
                        exit_cleanup(RERR_STREAMIO);
                }
 
                buf += n;
                len -= n;
-               ret += n;
+               cnt += n;
 
-               if (io_timeout && fd == sock_f_in)
-                       last_io = time(NULL);
+               if (fd == sock_f_in && io_timeout)
+                       last_io_in = time(NULL);
        }
 
-       return ret;
+       return cnt;
 }
 
 /**
@@ -571,13 +633,19 @@ int read_filesfrom_line(int fd, char *fname)
                if (cnt < 0 && (errno == EWOULDBLOCK
                  || errno == EINTR || errno == EAGAIN)) {
                        struct timeval tv;
-                       fd_set fds;
-                       FD_ZERO(&fds);
-                       FD_SET(fd, &fds);
+                       fd_set r_fds, e_fds;
+                       FD_ZERO(&r_fds);
+                       FD_SET(fd, &r_fds);
+                       FD_ZERO(&e_fds);
+                       FD_SET(fd, &e_fds);
                        tv.tv_sec = select_timeout;
                        tv.tv_usec = 0;
-                       if (!select(fd+1, &fds, NULL, NULL, &tv))
+                       if (!select(fd+1, &r_fds, NULL, &e_fds, &tv))
                                check_timeout();
+                       if (FD_ISSET(fd, &e_fds)) {
+                               rsyserr(FINFO, errno,
+                                       "select exception on fd %d", fd);
+                       }
                        continue;
                }
                if (cnt != 1)
@@ -600,7 +668,6 @@ int read_filesfrom_line(int fd, char *fname)
        return s - fname;
 }
 
-
 static char *iobuf_out;
 static int iobuf_out_cnt;
 
@@ -613,7 +680,6 @@ void io_start_buffering_out(void)
        iobuf_out_cnt = 0;
 }
 
-
 static char *iobuf_in;
 static size_t iobuf_in_siz;
 
@@ -626,7 +692,6 @@ void io_start_buffering_in(void)
                out_of_memory("io_start_buffering_in");
 }
 
-
 void io_end_buffering(void)
 {
        io_flush(NORMAL_FLUSH);
@@ -636,14 +701,19 @@ void io_end_buffering(void)
        }
 }
 
+void maybe_flush_socket(void)
+{
+       if (iobuf_out && iobuf_out_cnt && time(NULL) - last_io_out >= 5)
+               io_flush(NORMAL_FLUSH);
+}
 
-void maybe_send_keepalive(int allowed_lull, int ndx)
+void maybe_send_keepalive(void)
 {
-       if (time(NULL) - last_io >= allowed_lull) {
+       if (time(NULL) - last_io_out >= allowed_lull) {
                if (!iobuf_out || !iobuf_out_cnt) {
                        if (protocol_version < 29)
                                return; /* there's nothing we can do */
-                       write_int(sock_f_out, ndx);
+                       write_int(sock_f_out, the_file_list->count);
                        write_shortint(sock_f_out, ITEM_IS_NEW);
                }
                if (iobuf_out)
@@ -651,7 +721,6 @@ void maybe_send_keepalive(int allowed_lull, int ndx)
        }
 }
 
-
 /**
  * Continue trying to read len bytes - don't return until len has been
  * read.
@@ -666,7 +735,6 @@ static void read_loop(int fd, char *buf, size_t len)
        }
 }
 
-
 /**
  * Read from the file descriptor handling multiplexing - return number
  * of bytes read.
@@ -677,8 +745,9 @@ static int readfd_unbuffered(int fd, char *buf, size_t len)
 {
        static size_t remaining;
        static size_t iobuf_in_ndx;
-       int tag, ret = 0;
-       char line[MAXPATHLEN+1];
+       size_t msg_bytes;
+       int tag, cnt = 0;
+       char line[BIGPATHBUFLEN];
 
        if (!iobuf_in || fd != sock_f_in)
                return read_timeout(fd, buf, len);
@@ -688,73 +757,70 @@ static int readfd_unbuffered(int fd, char *buf, size_t len)
                iobuf_in_ndx = 0;
        }
 
-       while (ret == 0) {
+       while (cnt == 0) {
                if (remaining) {
                        len = MIN(len, remaining);
                        memcpy(buf, iobuf_in + iobuf_in_ndx, len);
                        iobuf_in_ndx += len;
                        remaining -= len;
-                       ret = len;
+                       cnt = len;
                        break;
                }
 
                read_loop(fd, line, 4);
                tag = IVAL(line, 0);
 
-               remaining = tag & 0xFFFFFF;
+               msg_bytes = tag & 0xFFFFFF;
                tag = (tag >> 24) - MPLEX_BASE;
 
                switch (tag) {
                case MSG_DATA:
-                       if (remaining > iobuf_in_siz) {
+                       if (msg_bytes > iobuf_in_siz) {
                                if (!(iobuf_in = realloc_array(iobuf_in, char,
-                                                              remaining)))
+                                                              msg_bytes)))
                                        out_of_memory("readfd_unbuffered");
-                               iobuf_in_siz = remaining;
+                               iobuf_in_siz = msg_bytes;
                        }
-                       read_loop(fd, iobuf_in, remaining);
+                       read_loop(fd, iobuf_in, msg_bytes);
+                       remaining = msg_bytes;
                        iobuf_in_ndx = 0;
                        break;
                case MSG_DELETED:
-                       if (remaining >= sizeof line) {
-                               rprintf(FERROR, "invalid multi-message %d:%ld\n",
-                                       tag, (long)remaining);
-                               exit_cleanup(RERR_STREAMIO);
-                       }
-                       read_loop(fd, line, remaining);
-                       line[remaining] = '\0';
+                       if (msg_bytes >= sizeof line)
+                               goto overflow;
+                       read_loop(fd, line, msg_bytes);
                        /* A directory name was sent with the trailing null */
-                       if (remaining > 0 && !line[remaining-1])
+                       if (msg_bytes > 0 && !line[msg_bytes-1])
                                log_delete(line, S_IFDIR);
-                       else
+                       else {
+                               line[msg_bytes] = '\0';
                                log_delete(line, S_IFREG);
-                       remaining = 0;
+                       }
                        break;
                case MSG_SUCCESS:
-                       if (remaining != 4) {
-                               rprintf(FERROR, "invalid multi-message %d:%ld\n",
-                                       tag, (long)remaining);
+                       if (msg_bytes != 4) {
+                               rprintf(FERROR, "invalid multi-message %d:%ld [%s]\n",
+                                       tag, (long)msg_bytes, who_am_i());
                                exit_cleanup(RERR_STREAMIO);
                        }
-                       read_loop(fd, line, remaining);
+                       read_loop(fd, line, msg_bytes);
                        successful_send(IVAL(line, 0));
-                       remaining = 0;
                        break;
                case MSG_INFO:
                case MSG_ERROR:
-                       if (remaining >= sizeof line) {
+                       if (msg_bytes >= sizeof line) {
+                           overflow:
                                rprintf(FERROR,
-                                       "[%s] multiplexing overflow %d:%ld\n\n",
-                                       who_am_i(), tag, (long)remaining);
+                                       "multiplexing overflow %d:%ld [%s]\n",
+                                       tag, (long)msg_bytes, who_am_i());
                                exit_cleanup(RERR_STREAMIO);
                        }
-                       read_loop(fd, line, remaining);
-                       rwrite((enum logcode)tag, line, remaining);
-                       remaining = 0;
+                       read_loop(fd, line, msg_bytes);
+                       rwrite((enum logcode)tag, line, msg_bytes);
                        break;
                default:
-                       rprintf(FERROR, "[%s] unexpected tag %d\n",
-                               who_am_i(), tag);
+                       rprintf(FERROR, "unexpected tag %d [%s]\n",
+                               tag, who_am_i());
                        exit_cleanup(RERR_STREAMIO);
                }
        }
@@ -762,11 +828,9 @@ static int readfd_unbuffered(int fd, char *buf, size_t len)
        if (remaining == 0)
                io_flush(NORMAL_FLUSH);
 
-       return ret;
+       return cnt;
 }
 
-
-
 /**
  * Do a buffered read from @p fd.  Don't return until all @p n bytes
  * have been read.  If all @p n can't be read then exit with an
@@ -774,12 +838,12 @@ static int readfd_unbuffered(int fd, char *buf, size_t len)
  **/
 static void readfd(int fd, char *buffer, size_t N)
 {
-       int  ret;
+       int  cnt;
        size_t total = 0;
 
        while (total < N) {
-               ret = readfd_unbuffered(fd, buffer + total, N-total);
-               total += ret;
+               cnt = readfd_unbuffered(fd, buffer + total, N-total);
+               total += cnt;
        }
 
        if (fd == write_batch_monitor_in) {
@@ -791,7 +855,6 @@ static void readfd(int fd, char *buffer, size_t N)
                stats.total_read += total;
 }
 
-
 int read_shortint(int f)
 {
        uchar b[2];
@@ -799,45 +862,44 @@ int read_shortint(int f)
        return (b[1] << 8) + b[0];
 }
 
-
 int32 read_int(int f)
 {
        char b[4];
-       int32 ret;
+       int32 num;
 
        readfd(f,b,4);
-       ret = IVAL(b,0);
-       if (ret == (int32)0xffffffff)
+       num = IVAL(b,0);
+       if (num == (int32)0xffffffff)
                return -1;
-       return ret;
+       return num;
 }
 
 int64 read_longint(int f)
 {
-       int64 ret;
+       int64 num;
        char b[8];
-       ret = read_int(f);
+       num = read_int(f);
 
-       if ((int32)ret != (int32)0xffffffff)
-               return ret;
+       if ((int32)num != (int32)0xffffffff)
+               return num;
 
 #if SIZEOF_INT64 < 8
        rprintf(FERROR, "Integer overflow: attempted 64-bit offset\n");
        exit_cleanup(RERR_UNSUPPORTED);
 #else
        readfd(f,b,8);
-       ret = IVAL(b,0) | (((int64)IVAL(b,4))<<32);
+       num = IVAL(b,0) | (((int64)IVAL(b,4))<<32);
 #endif
 
-       return ret;
+       return num;
 }
 
-void read_buf(int f,char *buf,size_t len)
+void read_buf(int f, char *buf, size_t len)
 {
        readfd(f,buf,len);
 }
 
-void read_sbuf(int f,char *buf,size_t len)
+void read_sbuf(int f, char *buf, size_t len)
 {
        readfd(f, buf, len);
        buf[len] = '\0';
@@ -860,7 +922,7 @@ int read_vstring(int f, char *buf, int bufsize)
        if (len >= bufsize) {
                rprintf(FERROR, "over-long vstring received (%d > %d)\n",
                        len, bufsize - 1);
-               exit_cleanup(RERR_PROTOCOL);
+               return -1;
        }
 
        if (len)
@@ -874,22 +936,27 @@ int read_vstring(int f, char *buf, int bufsize)
 void read_sum_head(int f, struct sum_struct *sum)
 {
        sum->count = read_int(f);
+       if (sum->count < 0) {
+               rprintf(FERROR, "Invalid checksum count %ld [%s]\n",
+                       (long)sum->count, who_am_i());
+               exit_cleanup(RERR_PROTOCOL);
+       }
        sum->blength = read_int(f);
        if (sum->blength < 0 || sum->blength > MAX_BLOCK_SIZE) {
-               rprintf(FERROR, "[%s] Invalid block length %ld\n",
-                       who_am_i(), (long)sum->blength);
+               rprintf(FERROR, "Invalid block length %ld [%s]\n",
+                       (long)sum->blength, who_am_i());
                exit_cleanup(RERR_PROTOCOL);
        }
        sum->s2length = protocol_version < 27 ? csum_length : (int)read_int(f);
        if (sum->s2length < 0 || sum->s2length > MD4_SUM_LENGTH) {
-               rprintf(FERROR, "[%s] Invalid checksum length %d\n",
-                       who_am_i(), sum->s2length);
+               rprintf(FERROR, "Invalid checksum length %d [%s]\n",
+                       sum->s2length, who_am_i());
                exit_cleanup(RERR_PROTOCOL);
        }
        sum->remainder = read_int(f);
        if (sum->remainder < 0 || sum->remainder > sum->blength) {
-               rprintf(FERROR, "[%s] Invalid remainder length %ld\n",
-                       who_am_i(), (long)sum->remainder);
+               rprintf(FERROR, "Invalid remainder length %ld [%s]\n",
+                       (long)sum->remainder, who_am_i());
                exit_cleanup(RERR_PROTOCOL);
        }
 }
@@ -911,7 +978,6 @@ void write_sum_head(int f, struct sum_struct *sum)
        write_int(f, sum->remainder);
 }
 
-
 /**
  * Sleep after writing to limit I/O bandwidth usage.
  *
@@ -931,16 +997,16 @@ void write_sum_head(int f, struct sum_struct *sum)
 static void sleep_for_bwlimit(int bytes_written)
 {
        static struct timeval prior_tv;
-       static long total_written = 0; 
+       static long total_written = 0;
        struct timeval tv, start_tv;
        long elapsed_usec, sleep_usec;
 
 #define ONE_SEC        1000000L /* # of microseconds in a second */
 
-       if (!bwlimit)
+       if (!bwlimit_writemax)
                return;
 
-       total_written += bytes_written; 
+       total_written += bytes_written;
 
        gettimeofday(&start_tv, NULL);
        if (prior_tv.tv_sec) {
@@ -967,45 +1033,44 @@ static void sleep_for_bwlimit(int bytes_written)
        total_written = (sleep_usec - elapsed_usec) * bwlimit / (ONE_SEC/1024);
 }
 
-
 /* Write len bytes to the file descriptor fd, looping as necessary to get
- * the job done and also (in certain circumstnces) reading any data on
+ * the job done and also (in certain circumstances) reading any data on
  * msg_fd_in to avoid deadlock.
  *
  * This function underlies the multiplexing system.  The body of the
  * application never calls this function directly. */
-static void writefd_unbuffered(int fd,char *buf,size_t len)
+static void writefd_unbuffered(int fd, const char *buf, size_t len)
 {
        size_t n, total = 0;
-       fd_set w_fds, r_fds;
-       int maxfd, count, ret;
+       fd_set w_fds, r_fds, e_fds;
+       int maxfd, count, cnt, using_r_fds;
+       int defer_save = defer_forwarding_messages;
        struct timeval tv;
 
        no_flush++;
 
        while (total < len) {
                FD_ZERO(&w_fds);
-               FD_SET(fd,&w_fds);
+               FD_SET(fd, &w_fds);
+               FD_ZERO(&e_fds);
+               FD_SET(fd, &e_fds);
                maxfd = fd;
 
-               if (msg_fd_in >= 0 && len-total >= contiguous_write_len) {
+               if (msg_fd_in >= 0) {
                        FD_ZERO(&r_fds);
-                       FD_SET(msg_fd_in,&r_fds);
+                       FD_SET(msg_fd_in, &r_fds);
                        if (msg_fd_in > maxfd)
                                maxfd = msg_fd_in;
-               }
-               if (fd != sock_f_out && iobuf_out_cnt && no_flush == 1) {
-                       FD_SET(sock_f_out, &w_fds);
-                       if (sock_f_out > maxfd)
-                               maxfd = sock_f_out;
-               }
+                       using_r_fds = 1;
+               } else
+                       using_r_fds = 0;
 
                tv.tv_sec = select_timeout;
                tv.tv_usec = 0;
 
                errno = 0;
-               count = select(maxfd + 1, msg_fd_in >= 0 ? &r_fds : NULL,
-                              &w_fds, NULL, &tv);
+               count = select(maxfd + 1, using_r_fds ? &r_fds : NULL,
+                              &w_fds, &e_fds, &tv);
 
                if (count <= 0) {
                        if (count < 0 && errno == EBADF)
@@ -1014,25 +1079,24 @@ static void writefd_unbuffered(int fd,char *buf,size_t len)
                        continue;
                }
 
-               if (msg_fd_in >= 0 && FD_ISSET(msg_fd_in, &r_fds))
+               if (FD_ISSET(fd, &e_fds)) {
+                       rsyserr(FINFO, errno,
+                               "select exception on fd %d", fd);
+               }
+
+               if (using_r_fds && FD_ISSET(msg_fd_in, &r_fds))
                        read_msg_fd();
 
-               if (!FD_ISSET(fd, &w_fds)) {
-                       if (fd != sock_f_out && iobuf_out_cnt) {
-                               no_flush--;
-                               io_flush(NORMAL_FLUSH);
-                               no_flush++;
-                       }
+               if (!FD_ISSET(fd, &w_fds))
                        continue;
-               }
 
                n = len - total;
-               if (bwlimit && n > bwlimit_writemax)
+               if (bwlimit_writemax && n > bwlimit_writemax)
                        n = bwlimit_writemax;
-               ret = write(fd, buf + total, n);
+               cnt = write(fd, buf + total, n);
 
-               if (ret <= 0) {
-                       if (ret < 0) {
+               if (cnt <= 0) {
+                       if (cnt < 0) {
                                if (errno == EINTR)
                                        continue;
                                if (errno == EWOULDBLOCK || errno == EAGAIN) {
@@ -1045,12 +1109,12 @@ static void writefd_unbuffered(int fd,char *buf,size_t len)
                        if (fd == sock_f_out)
                                close_multiplexing_out();
                        rsyserr(FERROR, errno,
-                               "writefd_unbuffered failed to write %ld bytes: phase \"%s\" [%s]",
-                               (long)len, io_write_phase, who_am_i());
+                               "writefd_unbuffered failed to write %ld bytes [%s]",
+                               (long)len, who_am_i());
                        /* If the other side is sending us error messages, try
                         * to grab any messages they sent before they died. */
                        while (fd == sock_f_out && io_multiplexing_in) {
-                               io_timeout = select_timeout = 30;
+                               set_io_timeout(30);
                                ignore_timeout = 0;
                                readfd_unbuffered(sock_f_in, io_filesfrom_buf,
                                                  sizeof io_filesfrom_buf);
@@ -1058,57 +1122,70 @@ static void writefd_unbuffered(int fd,char *buf,size_t len)
                        exit_cleanup(RERR_STREAMIO);
                }
 
-               total += ret;
+               total += cnt;
+               defer_forwarding_messages = 1;
 
                if (fd == sock_f_out) {
-                       if (io_timeout)
-                               last_io = time(NULL);
-                       sleep_for_bwlimit(ret);
+                       if (io_timeout || am_generator)
+                               last_io_out = time(NULL);
+                       sleep_for_bwlimit(cnt);
                }
        }
 
+       defer_forwarding_messages = defer_save;
        no_flush--;
 }
 
+static void msg2sndr_flush(void)
+{
+       if (defer_forwarding_messages)
+               return;
+
+       while (msg2sndr.head && io_multiplexing_out) {
+               struct msg_list_item *m = msg2sndr.head;
+               if (!(msg2sndr.head = m->next))
+                       msg2sndr.tail = NULL;
+               stats.total_written += m->len;
+               defer_forwarding_messages = 1;
+               writefd_unbuffered(sock_f_out, m->buf, m->len);
+               defer_forwarding_messages = 0;
+               free(m);
+       }
+}
 
 /**
  * Write an message to a multiplexed stream. If this fails then rsync
  * exits.
  **/
-static void mplex_write(enum msgcode code, char *buf, size_t len)
+static void mplex_write(enum msgcode code, const char *buf, size_t len)
 {
-       char buffer[4096];
+       char buffer[1024];
        size_t n = len;
 
        SIVAL(buffer, 0, ((MPLEX_BASE + (int)code)<<24) + len);
 
-       /* When the generator reads messages from the msg_fd_in pipe, it can
-        * cause output to occur down the socket.  Setting contiguous_write_len
-        * prevents the reading of msg_fd_in once we actually start to write
-        * this sequence of data (though we might read it before the start). */
-       if (am_generator && msg_fd_in >= 0)
-               contiguous_write_len = len + 4;
-
        if (n > sizeof buffer - 4)
-               n = sizeof buffer - 4;
+               n = 0;
+       else
+               memcpy(buffer + 4, buf, n);
 
-       memcpy(&buffer[4], buf, n);
        writefd_unbuffered(sock_f_out, buffer, n+4);
 
        len -= n;
        buf += n;
 
-       if (len)
+       if (len) {
+               defer_forwarding_messages = 1;
                writefd_unbuffered(sock_f_out, buf, len);
-
-       if (am_generator && msg_fd_in >= 0)
-               contiguous_write_len = 0;
+               defer_forwarding_messages = 0;
+               msg2sndr_flush();
+       }
 }
 
-
 void io_flush(int flush_it_all)
 {
-       msg_list_push(flush_it_all);
+       msg2genr_flush(flush_it_all);
+       msg2sndr_flush();
 
        if (!iobuf_out_cnt || no_flush)
                return;
@@ -1120,8 +1197,7 @@ void io_flush(int flush_it_all)
        iobuf_out_cnt = 0;
 }
 
-
-static void writefd(int fd,char *buf,size_t len)
+static void writefd(int fd, const char *buf, size_t len)
 {
        if (fd == msg_fd_out) {
                rprintf(FERROR, "Internal error: wrong write used in receiver.\n");
@@ -1155,7 +1231,6 @@ static void writefd(int fd,char *buf,size_t len)
        }
 }
 
-
 void write_shortint(int f, int x)
 {
        uchar b[2];
@@ -1164,7 +1239,6 @@ void write_shortint(int f, int x)
        writefd(f, (char *)b, 2);
 }
 
-
 void write_int(int f,int32 x)
 {
        char b[4];
@@ -1172,15 +1246,6 @@ void write_int(int f,int32 x)
        writefd(f,b,4);
 }
 
-
-void write_int_named(int f, int32 x, const char *phase)
-{
-       io_write_phase = phase;
-       write_int(f, x);
-       io_write_phase = phase_unknown;
-}
-
-
 /*
  * Note: int64 may actually be a 32-bit type if ./configure couldn't find any
  * 64-bit types on this platform.
@@ -1206,13 +1271,13 @@ void write_longint(int f, int64 x)
 #endif
 }
 
-void write_buf(int f,char *buf,size_t len)
+void write_buf(int f, const char *buf, size_t len)
 {
        writefd(f,buf,len);
 }
 
 /** Write a string to the connection */
-void write_sbuf(int f, char *buf)
+void write_sbuf(int f, const char *buf)
 {
        writefd(f, buf, strlen(buf));
 }
@@ -1222,7 +1287,7 @@ void write_byte(int f, uchar c)
        writefd(f, (char *)&c, 1);
 }
 
-void write_vstring(int f, char *str, int len)
+void write_vstring(int f, const char *str, int len)
 {
        uchar lenbuf[3], *lb = lenbuf;
 
@@ -1242,7 +1307,6 @@ void write_vstring(int f, char *str, int len)
                writefd(f, str, len);
 }
 
-
 /**
  * Read a line of up to @p maxlen characters into @p buf (not counting
  * the trailing null).  Strips the (required) trailing newline and all
@@ -1268,11 +1332,10 @@ int read_line(int f, char *buf, size_t maxlen)
        return maxlen > 0;
 }
 
-
 void io_printf(int fd, const char *format, ...)
 {
        va_list ap;
-       char buf[1024];
+       char buf[BIGPATHBUFLEN];
        int len;
 
        va_start(ap, format);
@@ -1282,10 +1345,14 @@ void io_printf(int fd, const char *format, ...)
        if (len < 0)
                exit_cleanup(RERR_STREAMIO);
 
+       if (len > (int)sizeof buf) {
+               rprintf(FERROR, "io_printf() was too long for the buffer.\n");
+               exit_cleanup(RERR_STREAMIO);
+       }
+
        write_sbuf(fd, buf);
 }
 
-
 /** Setup for multiplexing a MSG_* stream with the data stream. */
 void io_start_multiplex_out(void)
 {
@@ -1303,7 +1370,7 @@ void io_start_multiplex_in(void)
 }
 
 /** Write an message to the multiplexed data stream. */
-int io_multiplex_write(enum msgcode code, char *buf, size_t len)
+int io_multiplex_write(enum msgcode code, const char *buf, size_t len)
 {
        if (!io_multiplexing_out)
                return 0;