d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Clean up use of basename() and dirname().

Browse Source 
Pull copies of the input pathname string before calling basename() and
dirname() to make this comply to POSIX. Free these copies at the end of
this function. While there, remove the duplication of the 's' ->
'logfname' string. There is no need for this.
This commit is contained in:
Ed Schouten 2016-07-28 16:02:30 +00:00
parent 88ad2d7b47
commit f0d2919243
1 changed files with 17 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
usr.sbin/newsyslog/newsyslog.c
View File

@ -2286,26 +2286,29 @@ mtime_old_timelog(const char *file)
time_t t;
struct dirent *dp;
DIR *dirp;
char *s, *logfname, *dir;
char *logfname, *logfnamebuf, *dir, *dirbuf;
t = -1;
if ((dir = dirname(file)) == NULL) {
warn("dirname() of '%s'", file);
if ((dirbuf = strdup(file)) == NULL) {
warn("strdup() of '%s'", file);
return (t);
}
if ((s = basename(file)) == NULL) {
warn("basename() of '%s'", file);
dir = dirname(dirbuf);
if ((logfnamebuf = strdup(file)) == NULL) {
warn("strdup() of '%s'", file);
free(dirbuf);
return (t);
} else if (s[0] == '/') {
warnx("Invalid log filename '%s'", s);
return (t);
} else if ((logfname = strdup(s)) == NULL)
err(1, "strdup()");
}
logfname = basename(logfnamebuf);
if (logfname[0] == '/') {
warnx("Invalid log filename '%s'", logfname);
goto out;
}
if ((dirp = opendir(dir)) == NULL) {
warn("Cannot open log directory '%s'", dir);
return (t);
goto out;
}
dir_fd = dirfd(dirp);
/* Open the archive dir and find the most recent archive of logfname. */
@ -2322,6 +2325,9 @@ mtime_old_timelog(const char *file)
}
closedir(dirp);
out:
free(dirbuf);
free(logfnamebuf);
return (t);
}