Marc
/
server
forked from github/server
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #537 from ennorehling/develop 

CID 141509 Unchecked return value from library
This commit is contained in:
Enno Rehling 2016-08-09 18:37:06 +02:00 committed by GitHub
parent 391ef692b9 1735ec9eee
commit 851ae00835
1 changed files with 17 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/util/log.c
View File

@ -93,25 +93,34 @@ 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 err, dst = 1;
int dst = 1;
assert(strlen(filename) < sizeof(buffer[0]) - 4);
sprintf(buffer[dst], "%s.%d", filename, maxindex);
/* make sure we don't overwrite an existing file (hard links) */
remove(buffer[dst]);
if (remove(buffer[dst]) != 0) {
if (errno != ENOENT) {
fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno));
}
errno = 0;
}
while (maxindex > 0) {
int src = 1 - dst;
sprintf(buffer[src], "%s.%d", filename, --maxindex);
err = rename(buffer[src], buffer[dst]);
if ((err != 0) && (err != ENOENT)) {
fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno));
if (rename(buffer[src], buffer[dst]) != 0) {
if (errno != ENOENT) {
fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno));
}
errno = 0;
}
dst = src;
}
err = rename(filename, buffer[dst]);
if ((err != 0) && (err != ENOENT)) {
fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno));
if (rename(filename, buffer[dst]) != 0) {
if (errno != ENOENT) {
fprintf(stderr, "log rotate %s: %d %s", buffer[dst], errno, strerror(errno));
}
errno = 0;
}
}