extern int preserve_times;
extern int stdout_format_has_i;
extern int stdout_format_has_o_or_i;
+extern int logfile_format_has_i;
extern int logfile_format_has_o_or_i;
extern mode_t orig_umask;
extern char *auth_user;
#if defined HAVE_ICONV_OPEN && defined HAVE_ICONV_H
extern iconv_t ic_chck;
#endif
+extern char curr_dir[];
+extern unsigned int module_dirlen;
static int log_initialised;
static int logfile_was_closed;
{ 0, NULL }
};
-
/*
* Map from rsync error code to name, or return NULL.
*/
#endif
#ifdef LOG_DAEMON
- openlog("rsyncd", options, lp_syslog_facility());
+ openlog("rsyncd", options, lp_syslog_facility(module_id));
#else
openlog("rsyncd", options);
#endif
}
}
-void log_init(void)
+void log_init(int restart)
{
- if (log_initialised)
- return;
- log_initialised = 1;
+ if (log_initialised) {
+ if (!restart)
+ return;
+ if (strcmp(logfile_name, lp_log_file(module_id)) != 0) {
+ if (logfile_fp) {
+ fclose(logfile_fp);
+ logfile_fp = NULL;
+ } else
+ closelog();
+ logfile_name = NULL;
+ } else if (*logfile_name)
+ return; /* unchanged, non-empty "log file" names */
+ else if (lp_syslog_facility(-1) != lp_syslog_facility(module_id))
+ closelog();
+ else
+ return; /* unchanged syslog settings */
+ } else
+ log_initialised = 1;
/* This looks pointless, but it is needed in order for the
* C library on some systems to fetch the timezone info
/* Optionally use a log file instead of syslog. (Non-daemon
* rsyncs will have already set logfile_name, as needed.) */
if (am_daemon && !logfile_name)
- logfile_name = lp_log_file();
+ logfile_name = lp_log_file(module_id);
if (logfile_name && *logfile_name)
logfile_open();
else
for (s = buf; s < end; s++) {
if ((s < end - 4
&& *s == '\\' && s[1] == '#'
- && isdigit(*(uchar*)(s+2))
- && isdigit(*(uchar*)(s+3))
- && isdigit(*(uchar*)(s+4)))
+ && isDigit(s + 2)
+ && isDigit(s + 3)
+ && isDigit(s + 4))
|| (*s != '\t'
- && ((use_isprint && !isprint(*(uchar*)s))
+ && ((use_isprint && !isPrint(s))
|| *(uchar*)s < ' '))) {
if (s != buf && fwrite(buf, s - buf, 1, f) != 1)
exit_cleanup(RERR_MESSAGEIO);
if (code == FCLIENT)
code = FINFO;
- else if (code == FNAME) {
- if (am_server)
- code = FINFO;
- } else if (am_daemon || logfile_name) {
+ else if (am_daemon || logfile_name) {
static int in_block;
- char msg[2048], *s;
+ char msg[2048];
int priority = code == FERROR ? LOG_WARNING : LOG_INFO;
if (in_block)
return;
in_block = 1;
if (!log_initialised)
- log_init();
+ log_init(0);
strlcpy(msg, buf, MIN((int)sizeof msg, len + 1));
- for (s = msg; *s == '\n' && s[1]; s++) {}
- logit(priority, s);
+ logit(priority, msg);
in_block = 0;
if (code == FLOG || (am_daemon && !am_server))
case FERROR:
log_got_error = 1;
f = stderr;
- goto pre_scan;
- case FINFO:
- f = am_server ? stderr : stdout;
- pre_scan:
- while (len > 1 && *buf == '\n') {
- fputc(*buf, f);
- buf++;
- len--;
- }
break;
- case FNAME:
+ case FINFO:
f = am_server ? stderr : stdout;
break;
default:
char buf[BIGPATHBUFLEN];
size_t len;
- strcpy(buf, RSYNC_NAME ": ");
+ strlcpy(buf, RSYNC_NAME ": ", sizeof buf);
len = (sizeof RSYNC_NAME ": ") - 1;
va_start(ap, format);
n = fmt + 1;
if (*p == '-')
*n++ = *p++;
- while (isdigit(*(uchar*)p) && n - fmt < (int)(sizeof fmt) - 8)
+ while (isDigit(p) && n - fmt < (int)(sizeof fmt) - 8)
*n++ = *p++;
if (!*p)
break;
strlcpy(n, buf2, MAXPATHLEN);
else
n = buf2;
+ } else if (*n != '/') {
+ pathjoin(buf2, sizeof buf2,
+ curr_dir + module_dirlen, n);
+ clean_fname(buf2, 0);
+ if (fmt[1])
+ strlcpy(n, buf2, MAXPATHLEN);
+ else
+ n = buf2;
} else
clean_fname(n, 0);
if (*n == '/')
case 'L':
if (hlink && *hlink) {
n = hlink;
- strcpy(buf2, " => ");
+ strlcpy(buf2, " => ", sizeof buf2);
} else if (S_ISLNK(file->mode) && file->u.link) {
n = file->u.link;
- strcpy(buf2, " -> ");
+ strlcpy(buf2, " -> ", sizeof buf2);
} else {
n = "";
if (!fmt[1])
break;
- strcpy(buf2, " ");
+ strlcpy(buf2, " ", sizeof buf2);
}
strlcat(fmt, "s", sizeof fmt);
snprintf(buf2 + 4, sizeof buf2 - 4, fmt, n);
int i;
for (i = 2; n[i]; i++)
n[i] = ch;
- } else if (n[0] == '.' || n[0] == 'h'
- || (n[0] == 'c' && n[1] == 'f')) {
+ } else if (n[0] == '.' || n[0] == 'h' || n[0] == 'c') {
int i;
for (i = 2; n[i]; i++) {
if (n[i] != '.')
for (p = format; (p = strchr(p, '%')) != NULL; ) {
if (*++p == '-')
p++;
- while (isdigit(*(uchar*)p))
+ while (isDigit(p))
p++;
if (!*p)
break;
return 0;
}
-/* Log the transfer of a file. If the code is FNAME, the output just goes
+/* Log the transfer of a file. If the code is FCLIENT, the output just goes
* to stdout. If it is FLOG, it just goes to the log file. Otherwise we
* output to both. */
void log_item(enum logcode code, struct file_struct *file,
char *s_or_r = am_sender ? "send" : "recv";
if (code != FLOG && stdout_format && !am_server) {
- log_formatted(FNAME, stdout_format, s_or_r,
+ log_formatted(FCLIENT, stdout_format, s_or_r,
file, initial_stats, iflags, hlink);
}
- if (code != FNAME && logfile_format && *logfile_format) {
+ if (code != FCLIENT && logfile_format && *logfile_format) {
log_formatted(FLOG, logfile_format, s_or_r,
file, initial_stats, iflags, hlink);
}
|| stdout_format_has_i > 1 || (verbose > 1 && stdout_format_has_i));
int local_change = iflags & ITEM_LOCAL_CHANGE && significant_flags;
if (am_server) {
- if (logfile_name && !dry_run && see_item)
+ if (logfile_name && !dry_run && see_item
+ && (significant_flags || logfile_format_has_i))
log_item(FLOG, file, &stats, iflags, buf);
} else if (see_item || local_change || *buf
- || (S_ISDIR(file->mode) && significant_flags))
- log_item(FINFO, file, &stats, iflags, buf);
+ || (S_ISDIR(file->mode) && significant_flags)) {
+ enum logcode code = significant_flags || logfile_format_has_i ? FINFO : FCLIENT;
+ log_item(code, file, &stats, iflags, buf);
+ }
}
void log_delete(char *fname, int mode)