diff -Nuar a/logrotate.c b/logrotate.c --- a/logrotate.c 2013-10-26 18:07:54.809999410 +0200 +++ b/logrotate.c 2013-10-26 18:16:08.539999396 +0200 @@ -304,15 +304,20 @@ int createOutputFile(char *fileName, int flags, struct stat *sb, acl_type acl, int force_mode) { int fd; - struct stat sb_create; - int acl_set = 0; - - fd = open(fileName, (flags | O_EXCL | O_NOFOLLOW), - (S_IRUSR | S_IWUSR) & sb->st_mode); + int acl_set = 0; + struct stat sb_create; + char template[PATH_MAX + 1]; + char *fname; + mode_t umask_value; + snprintf(template, PATH_MAX, "%s/logrotate_temp.XXXXXX", ourDirName(fileName)); + umask_value = umask(0000); + fname = mktemp(template); + fd = open(fname, (flags | O_EXCL | O_NOFOLLOW), (S_IRUSR | S_IWUSR) & sb->st_mode); + umask(umask_value); if (fd < 0) { - message(MESS_ERROR, "error creating output file %s: %s\n", - fileName, strerror(errno)); + message(MESS_ERROR, "error creating unique temp file: %s\n", + strerror(errno)); return -1; } if (fchmod(fd, (S_IRUSR | S_IWUSR) & sb->st_mode)) { @@ -363,6 +368,13 @@ } } + if (rename(template, fileName)) { + message(MESS_ERROR, "error renaming temp file to %s: %s\n", + fileName, strerror(errno)); + close(fd); + return -1; + } + return fd; }