X-Git-Url: https://mattmccutchen.net/rsync/rsync.git/blobdiff_plain/9baed7609ca497814e6ad3ea6e4b315caca2bb4e..624d6be2a529114604da511dd1f3eadf509b4b9b:/log.c diff --git a/log.c b/log.c index 148d6b15..c8a433db 100644 --- a/log.c +++ b/log.c @@ -364,13 +364,15 @@ static void log_formatted(enum logcode code, char *format, char *op, * rather keep going until we reach the nul of the format. */ total = strlcpy(buf, format, sizeof buf); - for (p = buf; (p = strchr(p, '%')) != NULL && p[1]; ) { + for (p = buf; (p = strchr(p, '%')) != NULL; ) { s = p++; n = fmt + 1; if (*p == '-') *n++ = *p++; - while (isdigit(*(uchar*)p) && n - fmt < 16) + while (isdigit(*(uchar*)p) && n - fmt < (int)(sizeof fmt) - 8) *n++ = *p++; + if (!*p) + break; *n = '\0'; n = NULL; @@ -397,7 +399,10 @@ static void log_formatted(enum logcode code, char *format, char *op, file->dir.root, n); /* The buffer from safe_fname() has more * room than MAXPATHLEN, so this is safe. */ - strcpy(n, buf2); + if (fmt[1]) + strcpy(n, buf2); + else + n = buf2; } clean_fname(n, 0); if (*n == '/') @@ -456,7 +461,7 @@ static void log_formatted(enum logcode code, char *format, char *op, n = "*deleting"; break; } - n = buf2; + n = buf2 + MAXPATHLEN - 32; n[0] = iflags & ITEM_LOCAL_CHANGE ? iflags & ITEM_XNAME_FOLLOWS ? 'h' : 'c' : !(iflags & ITEM_TRANSFER) ? '.' @@ -494,9 +499,6 @@ static void log_formatted(enum logcode code, char *format, char *op, break; } - /* Subtract the length of the escape from the string's size. */ - total -= p - s; - /* "n" is the string to be inserted in place of this % code. */ if (!n) continue; @@ -507,6 +509,9 @@ static void log_formatted(enum logcode code, char *format, char *op, } len = strlen(n); + /* Subtract the length of the escape from the string's size. */ + total -= p - s; + if (len + total >= sizeof buf) { rprintf(FERROR, "buffer overflow expanding %%%c -- exiting\n", @@ -530,6 +535,26 @@ static void log_formatted(enum logcode code, char *format, char *op, rprintf(code, "%s\n", buf); } +int log_format_has(const char *format, char esc) +{ + const char *p; + + if (!format) + return 0; + + for (p = format; (p = strchr(p, '%')) != NULL; ) { + if (*++p == '-') + p++; + while (isdigit(*(uchar*)p)) + p++; + if (!*p) + break; + if (*p == esc) + return 1; + } + return 0; +} + /* log the transfer of a file */ void log_item(struct file_struct *file, struct stats *initial_stats, int iflags, char *hlink)