X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/blobdiff_plain/3eb388185b97920933a1bd17753866815e38539b..d1be231290b3867648ee417fad341fdf1caaa94b:/clientserver.c diff --git a/clientserver.c b/clientserver.c index 955be24f..1cc64373 100644 --- a/clientserver.c +++ b/clientserver.c @@ -24,6 +24,7 @@ extern int module_id; extern int read_only; extern int verbose; extern int rsync_port; +char *auth_user; int start_socket_client(char *host, char *path, int argc, char *argv[]) { @@ -36,6 +37,11 @@ int start_socket_client(char *host, char *path, int argc, char *argv[]) extern int am_client; extern int am_sender; + if (*path == '/') { + rprintf(FERROR,"ERROR: The remote path must start with a module name\n"); + return -1; + } + p = strchr(host, '@'); if (p) { user = host; @@ -50,7 +56,7 @@ int start_socket_client(char *host, char *path, int argc, char *argv[]) fd = open_socket_out(host, rsync_port); if (fd == -1) { - exit_cleanup(1); + exit_cleanup(RERR_SOCKETIO); } server_options(sargs,&sargc); @@ -116,7 +122,7 @@ static int rsync_module(int fd, int i) char *addr = client_addr(fd); char *host = client_name(fd); char *name = lp_name(i); - char *user; + int use_chroot = lp_use_chroot(i); int start_glob=0; int ret; char *request=NULL; @@ -147,9 +153,9 @@ static int rsync_module(int fd, int i) } - user = auth_server(fd, i, addr, "@RSYNCD: AUTHREQD "); + auth_user = auth_server(fd, i, addr, "@RSYNCD: AUTHREQD "); - if (!user) { + if (!auth_user) { rprintf(FERROR,"auth failed on module %s from %s (%s)\n", name, client_name(fd), client_addr(fd)); io_printf(fd,"@ERROR: auth failed on module %s\n",name); @@ -181,6 +187,12 @@ static int rsync_module(int fd, int i) gid = atoi(p); } + p = lp_include_from(i); + add_exclude_file(p, 1, 1); + + p = lp_include(i); + add_include_line(p); + p = lp_exclude_from(i); add_exclude_file(p, 1, 0); @@ -189,28 +201,37 @@ static int rsync_module(int fd, int i) log_open(); - if (chroot(lp_path(i))) { - rprintf(FERROR,"chroot %s failed\n", lp_path(i)); - io_printf(fd,"@ERROR: chroot failed\n"); - return -1; - } + if (use_chroot) { + if (chroot(lp_path(i))) { + rprintf(FERROR,"chroot %s failed\n", lp_path(i)); + io_printf(fd,"@ERROR: chroot failed\n"); + return -1; + } - if (chdir("/")) { - rprintf(FERROR,"chdir %s failed\n", lp_path(i)); - io_printf(fd,"@ERROR: chdir failed\n"); - return -1; - } + if (chdir("/")) { + rprintf(FERROR,"chdir %s failed\n", lp_path(i)); + io_printf(fd,"@ERROR: chdir failed\n"); + return -1; + } - if (setgid(gid) || getgid() != gid) { - rprintf(FERROR,"setgid %d failed\n", gid); - io_printf(fd,"@ERROR: setgid failed\n"); - return -1; - } + if (setgid(gid) || getgid() != gid) { + rprintf(FERROR,"setgid %d failed\n", gid); + io_printf(fd,"@ERROR: setgid failed\n"); + return -1; + } - if (setuid(uid) || getuid() != uid) { - rprintf(FERROR,"setuid %d failed\n", uid); - io_printf(fd,"@ERROR: setuid failed\n"); - return -1; + if (setuid(uid) || getuid() != uid) { + rprintf(FERROR,"setuid %d failed\n", uid); + io_printf(fd,"@ERROR: setuid failed\n"); + return -1; + } + + } else { + if (!push_dir(lp_path(i), 0)) { + rprintf(FERROR,"chdir %s failed\n", lp_path(i)); + io_printf(fd,"@ERROR: chdir failed\n"); + return -1; + } } am_root = (getuid() == 0); @@ -238,7 +259,7 @@ static int rsync_module(int fd, int i) request = strdup(p); start_glob++; } - glob_expand(name, argv, &argc, MAX_ARGS); + glob_expand(name, argv, &argc, MAX_ARGS, !use_chroot); } else { argc++; } @@ -252,13 +273,28 @@ static int rsync_module(int fd, int i) } } + if (!use_chroot) { + /* + * Note that this is applied to all parameters, whether or not + * they are filenames, but no other legal parameters contain + * the forms that need to be sanitized so it doesn't hurt; + * it is not known at this point which parameters are files + * and which aren't. + */ + for (i = 1; i < argc; i++) { + char *copy = sanitize_path(argv[i]); + free((void *)argv[i]); + argv[i] = copy; + } + } + ret = parse_arguments(argc, argv); if (request) { - if (*user) { + if (*auth_user) { rprintf(FINFO,"rsync %s %s from %s@%s (%s)\n", am_sender?"on":"to", - request, user, host, addr); + request, auth_user, host, addr); } else { rprintf(FINFO,"rsync %s %s from %s (%s)\n", am_sender?"on":"to", @@ -280,8 +316,12 @@ static int rsync_module(int fd, int i) io_start_multiplex_out(fd); if (!ret) { - rprintf(FERROR,"Error parsing options (unsupported option?) - aborting\n"); - exit_cleanup(1); + option_error(); + } + + if (lp_timeout(i)) { + extern int io_timeout; + io_timeout = lp_timeout(i); } start_server(fd, fd, argc, argp); @@ -313,7 +353,7 @@ static int start_daemon(int fd) extern int remote_version; if (!lp_load(config_file, 0)) { - exit_cleanup(1); + exit_cleanup(RERR_SYNTAX); } set_socket_options(fd,"SO_KEEPALIVE"); @@ -323,7 +363,7 @@ static int start_daemon(int fd) io_printf(fd,"@RSYNCD: %d\n", PROTOCOL_VERSION); motd = lp_motd_file(); - if (*motd) { + if (motd && *motd) { FILE *f = fopen(motd,"r"); while (f && !feof(f)) { int len = fread(line, 1, sizeof(line)-1, f); @@ -376,6 +416,7 @@ static int start_daemon(int fd) int daemon_main(void) { extern char *config_file; + char *pid_file; /* this ensures that we don't call getcwd after the chroot, which doesn't work on platforms that use popen("pwd","r") @@ -402,13 +443,26 @@ int daemon_main(void) if (!lp_load(config_file, 1)) { fprintf(stderr,"failed to load config file %s\n", config_file); - exit_cleanup(1); + exit_cleanup(RERR_SYNTAX); } log_open(); rprintf(FINFO,"rsyncd version %s starting\n",VERSION); + if (((pid_file = lp_pid_file()) != NULL) && (*pid_file != '\0')) { + FILE *f; + int pid = (int) getpid(); + cleanup_set_pid(pid); + if ((f = fopen(lp_pid_file(), "w")) == NULL) { + cleanup_set_pid(0); + fprintf(stderr,"failed to create pid file %s\n", pid_file); + exit_cleanup(RERR_FILEIO); + } + fprintf(f, "%d\n", pid); + fclose(f); + } + start_accept_loop(rsync_port, start_daemon); return -1; }