Moved one fsync() call.
[rsync/rsync-patches.git] / fsync.diff
index 2c8e77f..276c454 100644 (file)
@@ -9,15 +9,15 @@ To use this patch, run these commands for a successful build:
 
 --- old/options.c
 +++ new/options.c
-@@ -45,6 +45,7 @@ int append_mode = 0;
+@@ -44,6 +44,7 @@ int append_mode = 0;
  int keep_dirlinks = 0;
  int copy_dirlinks = 0;
  int copy_links = 0;
 +int do_fsync = 0;
  int preserve_links = 0;
  int preserve_hard_links = 0;
- int preserve_perms = 0;
-@@ -336,6 +337,7 @@ void usage(enum logcode F)
+ int preserve_acls = 0;
+@@ -345,6 +346,7 @@ void usage(enum logcode F)
    rprintf(F,"     --partial-dir=DIR       put a partially transferred file into DIR\n");
    rprintf(F,"     --delay-updates         put all updated files into place at transfer's end\n");
    rprintf(F," -m, --prune-empty-dirs      prune empty directory chains from the file-list\n");
@@ -25,7 +25,7 @@ To use this patch, run these commands for a successful build:
    rprintf(F,"     --numeric-ids           don't map uid/gid values by user/group name\n");
    rprintf(F,"     --timeout=TIME          set I/O timeout in seconds\n");
    rprintf(F," -I, --ignore-times          don't skip files that match in size and mod-time\n");
-@@ -523,6 +525,7 @@ static struct poptOption long_options[] 
+@@ -539,6 +541,7 @@ static struct poptOption long_options[] 
    {"only-write-batch", 0,  POPT_ARG_STRING, &batch_name, OPT_ONLY_WRITE_BATCH, 0, 0 },
    {"files-from",       0,  POPT_ARG_STRING, &files_from, 0, 0, 0 },
    {"from0",           '0', POPT_ARG_NONE,   &eol_nulls, 0, 0, 0},
@@ -33,7 +33,7 @@ To use this patch, run these commands for a successful build:
    {"numeric-ids",      0,  POPT_ARG_NONE,   &numeric_ids, 0, 0, 0 },
    {"timeout",          0,  POPT_ARG_INT,    &io_timeout, 0, 0, 0 },
    {"rsh",             'e', POPT_ARG_STRING, &shell_cmd, 0, 0, 0 },
-@@ -1743,6 +1746,9 @@ void server_options(char **args,int *arg
+@@ -1777,6 +1780,9 @@ void server_options(char **args,int *arg
                args[ac++] = tmpdir;
        }
  
@@ -45,7 +45,7 @@ To use this patch, run these commands for a successful build:
                 *   and it may be an older version that doesn't know this
 --- old/receiver.c
 +++ new/receiver.c
-@@ -38,6 +38,7 @@ extern int protocol_version;
+@@ -37,6 +37,7 @@ extern int protocol_version;
  extern int relative_paths;
  extern int preserve_hard_links;
  extern int preserve_perms;
@@ -53,7 +53,7 @@ To use this patch, run these commands for a successful build:
  extern int basis_dir_cnt;
  extern int make_backups;
  extern int cleanup_got_literal;
-@@ -258,6 +259,12 @@ static int receive_data(int f_in, char *
+@@ -257,6 +258,12 @@ static int receive_data(int f_in, char *
                exit_cleanup(RERR_FILEIO);
        }
  
@@ -63,12 +63,12 @@ To use this patch, run these commands for a successful build:
 +              exit_cleanup(RERR_FILEIO);
 +      }
 +
-       sum_end(file_sum1);
+       sum_len = sum_end(file_sum1);
  
        if (mapbuf)
 --- old/t_stub.c
 +++ new/t_stub.c
-@@ -22,6 +22,7 @@
+@@ -21,6 +21,7 @@
  
  #include "rsync.h"
  
@@ -78,7 +78,7 @@ To use this patch, run these commands for a successful build:
  int relative_paths = 0;
 --- old/util.c
 +++ new/util.c
-@@ -26,6 +26,7 @@
+@@ -25,6 +25,7 @@
  extern int verbose;
  extern int dry_run;
  extern int module_id;
@@ -86,8 +86,8 @@ To use this patch, run these commands for a successful build:
  extern int modify_window;
  extern int relative_paths;
  extern int human_readable;
-@@ -316,6 +317,12 @@ int copy_file(const char *source, const 
-               return -1;
+@@ -309,6 +310,12 @@ int copy_file(const char *source, const 
+                       full_fname(source));
        }
  
 +      if (do_fsync && fsync(ofd) < 0) {
@@ -96,6 +96,6 @@ To use this patch, run these commands for a successful build:
 +              return -1;
 +      }
 +
-       return 0;
- }
+       if (close(ofd) < 0) {
+               rsyserr(FERROR, errno, "close failed on %s",
+                       full_fname(dest));