Split code out into separate files and remove some global variables to

[rsync/rsync.git] / log.c

diff --git a/log.c b/log.c
index f36b105..e79bc22 100644 (file)
--- a/log.c
+++ b/log.c
@@ -30,6 +30,7 @@
 static char *logfname;
 static FILE *logfile;
 static int log_error_fd = -1;
+struct stats stats;
 
 int log_got_error=0;
 
@@ -41,6 +42,7 @@ struct {
        { RERR_PROTOCOL   , "protocol incompatibility" }, 
        { RERR_FILESELECT , "errors selecting input/output files, dirs" }, 
        { RERR_UNSUPPORTED, "requested action not supported" }, 
+       { RERR_STARTCLIENT, "error starting client-server protocol" }, 
        { RERR_SOCKETIO   , "error in socket IO" }, 
        { RERR_FILEIO     , "error in file IO" }, 
        { RERR_STREAMIO   , "error in rsync protocol data stream" }, 
@@ -236,7 +238,11 @@ void rwrite(enum logcode code, char *buf, int len)
                return;
        }
 
-       /* if that fails, try to pass it to the other end */
+       /* If that fails, try to pass it to the other end.
+        *
+        * io_multiplex_write can fail if we do not have a multiplexed
+        * connection at the moment, in which case we fall through and
+        * log locally instead. */
        if (am_server && io_multiplex_write(code, buf, len)) {
                return;
        }
@@ -341,6 +347,8 @@ void rsyserr(enum logcode code, int errcode, const char *format, ...)
        len = vsnprintf(buf, sizeof(buf), format, ap);
        va_end(ap);
 
+       /* TODO: Put in RSYNC_NAME at the start. */
+
        if ((size_t) len > sizeof(buf)-1)
                exit_cleanup(RERR_MESSAGEIO);
 
@@ -409,6 +417,12 @@ static void log_formatted(enum logcode code,
        extern int am_daemon;
        int64 b;
 
+       /* We expand % codes one by one in place in buf.  We don't
+        * copy in the terminating nul of the inserted strings, but
+        * rather keep going until we reach the nul of the format.
+        * Just to make sure we don't clobber that nul and therefore
+        * accidentally keep going, we zero the buffer now. */
+       memset(buf, 0, sizeof buf);
        strlcpy(buf, format, sizeof(buf));
        
        for (s=&buf[0]; 
@@ -466,7 +480,11 @@ static void log_formatted(enum logcode code,
                        break;
                }
 
-               if (!n) continue;
+               /* n is the string to be inserted in place of this %
+                * code; l is its length not including the trailing
+                * NUL */
+               if (!n)
+                       continue;
 
                l = strlen(n);
 
@@ -476,11 +494,16 @@ static void log_formatted(enum logcode code,
                        exit_cleanup(RERR_MESSAGEIO);
                }
 
+               /* Shuffle the rest of the string along to make space for n */
                if (l != 2) {
                        memmove(s+(l-1), s+1, strlen(s+1)+1);
                }
+
+               /* Copy in n but NOT its nul, because the format sting
+                * probably continues after this. */
                memcpy(p, n, l);
 
+               /* Skip over inserted string; continue looking */
                s = p+l;
        }
 
@@ -544,20 +567,16 @@ void log_exit(int code, const char *file, int line)
        }
 }
 
-
-
-
-/* log the incoming transfer of a file for interactive use, this
-   will be called at the end where the client was run 
-   
-   it i called when a file starts to be transferred
-*/
+/*
+ * Log the incoming transfer of a file for interactive use,
+ * this will be called at the end where the client was run.
+ * Called when a file starts to be transferred.
+ */
 void log_transfer(struct file_struct *file, const char *fname)
 {
        extern int verbose;
 
        if (!verbose) return;
 
-       rprintf(FINFO,"%s\n", fname);
+       rprintf(FINFO, "%s\n", fname);
 }
-