diff options
author | Matt McCutchen <matt@mattmccutchen.net> | 2010-02-01 13:29:39 -0500 |
---|---|---|
committer | Wayne Davison <wayned@samba.org> | 2010-04-24 09:28:58 -0700 |
commit | 9a54a640f74dcaeb03ef99799a5320b3eb77f7ec (patch) | |
tree | fb3771462e84532dad9390f98aca2090600f9665 /clientserver.c | |
parent | 58a79f4b44593996e53a9bfc94b56006ea86c2ec (diff) | |
download | android_external_rsync-9a54a640f74dcaeb03ef99799a5320b3eb77f7ec.tar.gz android_external_rsync-9a54a640f74dcaeb03ef99799a5320b3eb77f7ec.tar.bz2 android_external_rsync-9a54a640f74dcaeb03ef99799a5320b3eb77f7ec.zip |
Don't set the umask to 0 any more: it's ugly and pointless.
Diffstat (limited to 'clientserver.c')
-rw-r--r-- | clientserver.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/clientserver.c b/clientserver.c index a7996c7f..878254f3 100644 --- a/clientserver.c +++ b/clientserver.c @@ -48,7 +48,6 @@ extern int write_batch; extern int default_af_hint; extern int logfile_format_has_i; extern int logfile_format_has_o_or_i; -extern mode_t orig_umask; extern char *bind_address; extern char *config_file; extern char *logfile_format; @@ -676,7 +675,6 @@ static int rsync_module(int f_in, int f_out, int i, const char *addr, const char if (*lp_prexfer_exec(i) || *lp_postxfer_exec(i)) { int status; - umask(orig_umask); /* For post-xfer exec, fork a new process to run the rsync * daemon while this process waits for the exit status and * runs the indicated command at that point. */ @@ -745,7 +743,6 @@ static int rsync_module(int f_in, int f_out, int i, const char *addr, const char set_blocking(fds[1]); pre_exec_fd = fds[1]; } - umask(0); } #endif @@ -1076,7 +1073,7 @@ static void create_pid_file(void) return; cleanup_set_pid(pid); - if ((fd = do_open(pid_file, O_WRONLY|O_CREAT|O_EXCL, 0666 & ~orig_umask)) == -1) { + if ((fd = do_open(pid_file, O_WRONLY|O_CREAT|O_EXCL, 0666)) == -1) { failure: cleanup_set_pid(0); fprintf(stderr, "failed to create pid file %s: %s\n", pid_file, strerror(errno)); |