Fixed a typo that set stdout_format_has_i instead of logfile_format_has_i.
[rsync/rsync.git] / options.c
index ee7b522..04a5654 100644 (file)
--- a/options.c
+++ b/options.c
@@ -147,7 +147,9 @@ char *partial_dir = NULL;
 char *basis_dir[MAX_BASIS_DIRS+1];
 char *config_file = NULL;
 char *shell_cmd = NULL;
-char *log_format = NULL;
+char *logfile_name = NULL;
+char *logfile_format = NULL;
+char *stdout_format = NULL;
 char *password_file = NULL;
 char *rsync_path = RSYNC_PATH;
 char *backup_dir = NULL;
@@ -163,8 +165,10 @@ char *dest_option = NULL;
 int verbose = 0;
 int quiet = 0;
 int log_before_transfer = 0;
-int log_format_has_i = 0;
-int log_format_has_o_or_i = 0;
+int stdout_format_has_i = 0;
+int stdout_format_has_o_or_i = 0;
+int logfile_format_has_i = 0;
+int logfile_format_has_o_or_i = 0;
 int always_checksum = 0;
 int list_only = 0;
 
@@ -311,15 +315,15 @@ void usage(enum logcode F)
   rprintf(F," -B, --block-size=SIZE       force a fixed checksum block-size\n");
   rprintf(F," -e, --rsh=COMMAND           specify the remote shell to use\n");
   rprintf(F,"     --rsync-path=PROGRAM    specify the rsync to run on the remote machine\n");
-  rprintf(F,"     --existing              ignore non-existing files on receiving side\n");
-  rprintf(F,"     --ignore-existing       ignore files that already exist on receiving side\n");
-  rprintf(F,"     --remove-sent-files     sent files/symlinks are removed from sending side\n");
+  rprintf(F,"     --existing              skip creating new files on receiver\n");
+  rprintf(F,"     --ignore-existing       skip updating files that already exist on receiver\n");
+  rprintf(F,"     --remove-sent-files     sender removes successfully sent files (non-dirs)\n");
   rprintf(F,"     --del                   an alias for --delete-during\n");
-  rprintf(F,"     --delete                delete files that don't exist on the sending side\n");
+  rprintf(F,"     --delete                delete extraneous files from destination dirs\n");
   rprintf(F,"     --delete-before         receiver deletes before transfer (default)\n");
   rprintf(F,"     --delete-during         receiver deletes during transfer, not before\n");
   rprintf(F,"     --delete-after          receiver deletes after transfer, not before\n");
-  rprintf(F,"     --delete-excluded       also delete excluded files on the receiving side\n");
+  rprintf(F,"     --delete-excluded       also delete excluded files from destination dirs\n");
   rprintf(F,"     --ignore-errors         delete even if there are I/O errors\n");
   rprintf(F,"     --force                 force deletion of directories even if not empty\n");
   rprintf(F,"     --max-delete=NUM        don't delete more than NUM files\n");
@@ -361,7 +365,9 @@ void usage(enum logcode F)
   rprintf(F,"     --progress              show progress during transfer\n");
   rprintf(F," -P                          same as --partial --progress\n");
   rprintf(F," -i, --itemize-changes       output a change-summary for all updates\n");
-  rprintf(F,"     --log-format=FORMAT     output filenames using the specified format\n");
+  rprintf(F,"     --out-format=FORMAT     output updates using the specified FORMAT\n");
+  rprintf(F,"     --log-file=FILE         log what we're doing to the specified FILE\n");
+  rprintf(F,"     --log-file-format=FMT   log updates using the specified FMT\n");
   rprintf(F,"     --password-file=FILE    read password from FILE\n");
   rprintf(F,"     --list-only             list the files instead of copying them\n");
   rprintf(F,"     --bwlimit=KBPS          limit I/O bandwidth; KBytes per second\n");
@@ -494,7 +500,10 @@ static struct poptOption long_options[] = {
   {"partial-dir",      0,  POPT_ARG_STRING, &partial_dir, 0, 0, 0 },
   {"delay-updates",    0,  POPT_ARG_NONE,   &delay_updates, 0, 0, 0 },
   {"prune-empty-dirs",'m', POPT_ARG_NONE,   &prune_empty_dirs, 0, 0, 0 },
-  {"log-format",       0,  POPT_ARG_STRING, &log_format, 0, 0, 0 },
+  {"log-file",         0,  POPT_ARG_STRING, &logfile_name, 0, 0, 0 },
+  {"log-file-format",  0,  POPT_ARG_STRING, &logfile_format, 0, 0, 0 },
+  {"out-format",       0,  POPT_ARG_STRING, &stdout_format, 0, 0, 0 },
+  {"log-format",       0,  POPT_ARG_STRING, &stdout_format, 0, 0, 0 }, /* DEPRECATED */
   {"itemize-changes", 'i', POPT_ARG_NONE,   0, 'i', 0, 0 },
   {"bwlimit",          0,  POPT_ARG_INT,    &bwlimit, 0, 0, 0 },
   {"backup",          'b', POPT_ARG_NONE,   &make_backups, 0, 0, 0 },
@@ -544,6 +553,8 @@ static void daemon_usage(enum logcode F)
   rprintf(F,"     --config=FILE           specify alternate rsyncd.conf file\n");
   rprintf(F,"     --no-detach             do not detach from the parent\n");
   rprintf(F,"     --port=PORT             listen on alternate port number\n");
+  rprintf(F,"     --log-file=FILE         override the \"log file\" setting\n");
+  rprintf(F,"     --log-file-format=FMT   override the \"log format\" setting\n");
   rprintf(F,"     --sockopts=OPTIONS      specify custom TCP options\n");
   rprintf(F," -v, --verbose               increase verbosity\n");
 #ifdef INET6
@@ -567,6 +578,8 @@ static struct poptOption long_daemon_options[] = {
   {"ipv6",            '6', POPT_ARG_VAL,    &default_af_hint, AF_INET6, 0, 0 },
 #endif
   {"detach",           0,  POPT_ARG_VAL,    &no_detach, 0, 0, 0 },
+  {"log-file",         0,  POPT_ARG_STRING, &logfile_name, 0, 0, 0 },
+  {"log-file-format",  0,  POPT_ARG_STRING, &logfile_format, 0, 0, 0 },
   {"no-detach",        0,  POPT_ARG_VAL,    &no_detach, 1, 0, 0 },
   {"port",             0,  POPT_ARG_INT,    &rsync_port, 0, 0, 0 },
   {"sockopts",         0,  POPT_ARG_STRING, &sockopts, 0, 0, 0 },
@@ -667,17 +680,6 @@ static void set_refuse_options(char *bp)
                *cp = ' ';
                bp = cp + 1;
        }
-
-       for (op = long_options; ; op++) {
-               *shortname = op->shortName;
-               if (!op->longName && !*shortname)
-                       break;
-               if (op->val == OPT_DAEMON) {
-                       if (op->argInfo == POPT_ARG_VAL)
-                               op->argInfo = POPT_ARG_NONE;
-                       op->val = (op - long_options) + OPT_REFUSED_BASE;
-               }
-       }
 }
 
 
@@ -782,6 +784,8 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
 
        if (ref && *ref)
                set_refuse_options(ref);
+       if (am_daemon)
+               set_refuse_options("log-file*");
 
        /* TODO: Call poptReadDefaultConfig; handle errors. */
 
@@ -890,15 +894,20 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
                case OPT_EXCLUDE_FROM:
                case OPT_INCLUDE_FROM:
                        arg = poptGetOptArg(pc);
-                       if (sanitize_paths)
-                               arg = sanitize_path(NULL, arg, NULL, 0);
+                       if (sanitize_paths) {
+                               arg = sanitize_path(NULL, arg, NULL, 0, NULL);
+                               die_on_unsafe_path((char*)arg, 0);
+                       }
                        if (server_filter_list.head) {
-                               char *cp = (char *)arg;
+                               char *cp = strdup(arg);
+                               if (!cp)
+                                       out_of_memory("parse_arguments");
                                if (!*cp)
                                        goto options_rejected;
                                clean_fname(cp, 1);
                                if (check_filter(&server_filter_list, cp, 0) < 0)
                                        goto options_rejected;
+                               free(cp);
                        }
                        parse_filter_file(&filter_list, arg,
                                opt == OPT_INCLUDE_FROM ? MATCHFLG_INCLUDE : 0,
@@ -1022,7 +1031,7 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
                        break;
 
                case OPT_LINK_DEST:
-#ifdef HAVE_LINK
+#ifdef SUPPORT_HARD_LINKS
                        link_dest = 1;
                        dest_option = "--link-dest";
                        goto set_dest_dir;
@@ -1205,17 +1214,18 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
        if (sanitize_paths) {
                int i;
                for (i = *argc; i-- > 0; )
-                       (*argv)[i] = sanitize_path(NULL, (*argv)[i], "", 0);
-               if (tmpdir)
-                       tmpdir = sanitize_path(NULL, tmpdir, NULL, 0);
-               if (partial_dir)
-                       partial_dir = sanitize_path(NULL, partial_dir, NULL, 0);
-               if (backup_dir)
-                       backup_dir = sanitize_path(NULL, backup_dir, NULL, 0);
+                       (*argv)[i] = sanitize_path(NULL, (*argv)[i], "", 0, NULL);
+               if (tmpdir) {
+                       tmpdir = sanitize_path(NULL, tmpdir, NULL, 0, NULL);
+                       die_on_unsafe_path(tmpdir, 0);
+               }
+               if (backup_dir) {
+                       backup_dir = sanitize_path(NULL, backup_dir, NULL, 0, NULL);
+                       die_on_unsafe_path(backup_dir, 0);
+               }
        }
        if (server_filter_list.head && !am_sender) {
                struct filter_list_struct *elp = &server_filter_list;
-               int i;
                if (tmpdir) {
                        if (!*tmpdir)
                                goto options_rejected;
@@ -1223,18 +1233,6 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
                        if (check_filter(elp, tmpdir, 1) < 0)
                                goto options_rejected;
                }
-               if (partial_dir && *partial_dir) {
-                       clean_fname(partial_dir, 1);
-                       if (check_filter(elp, partial_dir, 1) < 0)
-                               goto options_rejected;
-               }
-               for (i = 0; i < basis_dir_cnt; i++) {
-                       if (!*basis_dir[i])
-                               goto options_rejected;
-                       clean_fname(basis_dir[i], 1);
-                       if (check_filter(elp, basis_dir[i], 1) < 0)
-                               goto options_rejected;
-               }
                if (backup_dir) {
                        if (!*backup_dir)
                                goto options_rejected;
@@ -1283,17 +1281,17 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
        if (make_backups && !backup_dir)
                omit_dir_times = 1;
 
-       if (log_format) {
-               if (am_server && log_format_has(log_format, 'I'))
-                       log_format_has_i = 2;
-               else if (log_format_has(log_format, 'i'))
-                       log_format_has_i = itemize_changes | 1;
-               if (!log_format_has(log_format, 'b')
-                && !log_format_has(log_format, 'c'))
+       if (stdout_format) {
+               if (am_server && log_format_has(stdout_format, 'I'))
+                       stdout_format_has_i = 2;
+               else if (log_format_has(stdout_format, 'i'))
+                       stdout_format_has_i = itemize_changes | 1;
+               if (!log_format_has(stdout_format, 'b')
+                && !log_format_has(stdout_format, 'c'))
                        log_before_transfer = !am_server;
        } else if (itemize_changes) {
-               log_format = "%i %n%L";
-               log_format_has_i = itemize_changes;
+               stdout_format = "%i %n%L";
+               stdout_format_has_i = itemize_changes;
                log_before_transfer = !am_server;
        }
 
@@ -1305,12 +1303,26 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
 
        set_io_timeout(io_timeout);
 
-       if (verbose && !log_format) {
-               log_format = "%n%L";
+       if (verbose && !stdout_format) {
+               stdout_format = "%n%L";
                log_before_transfer = !am_server;
        }
-       if (log_format_has_i || log_format_has(log_format, 'o'))
-               log_format_has_o_or_i = 1;
+       if (stdout_format_has_i || log_format_has(stdout_format, 'o'))
+               stdout_format_has_o_or_i = 1;
+
+       if (logfile_name && !am_daemon) {
+               if (!logfile_format) {
+                       logfile_format = "%i %n%L";
+                       logfile_format_has_i = logfile_format_has_o_or_i = 1;
+               } else {
+                       if (log_format_has(logfile_format, 'i'))
+                               logfile_format_has_i = 1;
+                       if (logfile_format_has_i || log_format_has(logfile_format, 'o'))
+                               logfile_format_has_o_or_i = 1;
+               }
+               log_init();
+       } else if (!am_daemon)
+               logfile_format = NULL;
 
        if (daemon_bwlimit && (!bwlimit || bwlimit > daemon_bwlimit))
                bwlimit = daemon_bwlimit;
@@ -1415,7 +1427,7 @@ int parse_arguments(int *argc, const char ***argv, int frommain)
                        }
                } else {
                        if (sanitize_paths)
-                               files_from = sanitize_path(NULL, files_from, NULL, 0);
+                               files_from = sanitize_path(NULL, files_from, NULL, 0, NULL);
                        if (server_filter_list.head) {
                                if (!*files_from)
                                        goto options_rejected;
@@ -1575,12 +1587,13 @@ void server_options(char **args,int *argc)
 
        /* The server side doesn't use our log-format, but in certain
         * circumstances they need to know a little about the option. */
-       if (log_format && am_sender) {
-               if (log_format_has_i > 1)
+       if (stdout_format && am_sender) {
+               /* Use --log-format, not --out-format, for compatibility. */
+               if (stdout_format_has_i > 1)
                        args[ac++] = "--log-format=%i%I";
-               else if (log_format_has_i)
+               else if (stdout_format_has_i)
                        args[ac++] = "--log-format=%i";
-               else if (log_format_has_o_or_i)
+               else if (stdout_format_has_o_or_i)
                        args[ac++] = "--log-format=%o";
                else if (!verbose)
                        args[ac++] = "--log-format=X";