Call finish_transfer() with a valid partialptr string, when

[rsync/rsync.git] / rsync.c

diff --git a/rsync.c b/rsync.c
index 6d2812c..661f990 100644 (file)
--- a/rsync.c
+++ b/rsync.c
@@ -49,9 +49,8 @@ void free_sums(struct sum_struct *s)
        free(s);
 }
 
-
-int set_perms(char *fname,struct file_struct *file,STRUCT_STAT *st,
-             int flags)
+int set_file_attrs(char *fname, struct file_struct *file, STRUCT_STAT *st,
+                  int flags)
 {
        int updated = 0;
        STRUCT_STAT st2;
@@ -69,8 +68,8 @@ int set_perms(char *fname,struct file_struct *file,STRUCT_STAT *st,
        }
 
        if (!preserve_times || (S_ISDIR(st->st_mode) && omit_dir_times))
-               flags |= PERMS_SKIP_MTIME;
-       if (!(flags & PERMS_SKIP_MTIME)
+               flags |= ATTRS_SKIP_MTIME;
+       if (!(flags & ATTRS_SKIP_MTIME)
            && cmp_modtime(st->st_mtime, file->modtime) != 0) {
                int ret = set_modtime(fname, file->modtime, st->st_mode);
                if (ret < 0) {
@@ -95,13 +94,13 @@ int set_perms(char *fname,struct file_struct *file,STRUCT_STAT *st,
                        if (change_uid) {
                                rprintf(FINFO,
                                        "set uid of %s from %ld to %ld\n",
-                                       safe_fname(fname),
+                                       fname,
                                        (long)st->st_uid, (long)file->uid);
                        }
                        if (change_gid) {
                                rprintf(FINFO,
                                        "set gid of %s from %ld to %ld\n",
-                                       safe_fname(fname),
+                                       fname,
                                        (long)st->st_gid, (long)file->gid);
                        }
                }
@@ -139,13 +138,13 @@ int set_perms(char *fname,struct file_struct *file,STRUCT_STAT *st,
        }
 #endif
 
-       if (verbose > 1 && flags & PERMS_REPORT) {
+       if (verbose > 1 && flags & ATTRS_REPORT) {
                enum logcode code = daemon_log_format_has_i || dry_run
                                  ? FCLIENT : FINFO;
                if (updated)
-                       rprintf(code, "%s\n", safe_fname(fname));
+                       rprintf(code, "%s\n", fname);
                else
-                       rprintf(code, "%s is uptodate\n", safe_fname(fname));
+                       rprintf(code, "%s is uptodate\n", fname);
        }
        return updated;
 }
@@ -175,26 +174,25 @@ void finish_transfer(char *fname, char *fnametmp, struct file_struct *file,
 
        if (inplace) {
                if (verbose > 2)
-                       rprintf(FINFO, "finishing %s\n", safe_fname(fname));
-               goto do_set_perms;
+                       rprintf(FINFO, "finishing %s\n", fname);
+               goto do_set_file_attrs;
        }
 
        if (make_backups && overwriting_basis && !make_backup(fname))
                return;
 
        /* Change permissions before putting the file into place. */
-       set_perms(fnametmp, file, NULL, ok_to_set_time ? 0 : PERMS_SKIP_MTIME);
+       set_file_attrs(fnametmp, file, NULL,
+                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME);
 
        /* move tmp file over real file */
-       if (verbose > 2) {
-               rprintf(FINFO, "renaming %s to %s\n",
-                       safe_fname(fnametmp), safe_fname(fname));
-       }
+       if (verbose > 2)
+               rprintf(FINFO, "renaming %s to %s\n", fnametmp, fname);
        ret = robust_rename(fnametmp, fname, file->mode & INITACCESSPERMS);
        if (ret < 0) {
                rsyserr(FERROR, errno, "%s %s -> \"%s\"",
-                   ret == -2 ? "copy" : "rename",
-                   full_fname(fnametmp), safe_fname(fname));
+                       ret == -2 ? "copy" : "rename",
+                       full_fname(fnametmp), fname);
                do_unlink(fnametmp);
                return;
        }
@@ -202,8 +200,9 @@ void finish_transfer(char *fname, char *fnametmp, struct file_struct *file,
                /* The file was moved into place (not copied), so it's done. */
                return;
        }
-    do_set_perms:
-       set_perms(fname, file, NULL, ok_to_set_time ? 0 : PERMS_SKIP_MTIME);
+  do_set_file_attrs:
+       set_file_attrs(fname, file, NULL,
+                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME);
 }
 
 const char *who_am_i(void)