From 03a8b7383cd41ed499abb67dc305b52d3bb17389 Mon Sep 17 00:00:00 2001 From: Enno Rehling Date: Fri, 5 Aug 2016 20:15:05 +0200 Subject: [PATCH] fix log_rotate warnings (#529) * ignore file-not-found errors when trying to rotate logs. I suspect this is faster and safer than testing existence. * block rename warnings, part deux --- src/util/log.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/src/util/log.c b/src/util/log.c index 1399ce562..ee3909a01 100644 --- a/src/util/log.c +++ b/src/util/log.c @@ -93,7 +93,7 @@ cp_convert(const char *format, char *buffer, size_t length, int codepage) void log_rotate(const char *filename, int maxindex) { char buffer[2][MAX_PATH]; - int dst = 1; + int err, dst = 1; assert(strlen(filename) < sizeof(buffer[0]) - 4); sprintf(buffer[dst], "%s.%d", filename, maxindex); @@ -101,16 +101,17 @@ void log_rotate(const char *filename, int maxindex) remove(buffer[dst]); while (maxindex > 0) { - int err, src = 1 - dst; + int src = 1 - dst; sprintf(buffer[src], "%s.%d", filename, --maxindex); err = rename(buffer[src], buffer[dst]); - if (err != 0) { - fprintf(stderr, "log rotate %s: %s", buffer[dst], strerror(errno)); + if ((err != 0) && (err != ENOENT)) { + fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno)); } dst = src; } - if (rename(filename, buffer[dst]) != 0) { - fprintf(stderr, "log rotate %s: %s", buffer[dst], strerror(errno)); + err = rename(filename, buffer[dst]); + if ((err != 0) && (err != ENOENT)) { + fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno)); } }