Author: ed
Date: Thu Jul 28 16:02:30 2016
New Revision: 303449
URL: https://svnweb.freebsd.org/changeset/base/303449
Log:
Clean up use of basename() and dirname().
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.
Modified:
head/usr.sbin/newsyslog/newsyslog.c
Modified: head/usr.sbin/newsyslog/newsyslog.c
==============================================================================
--- head/usr.sbin/newsyslog/newsyslog.c Thu Jul 28 15:57:01 2016
(r303448)
+++ head/usr.sbin/newsyslog/newsyslog.c Thu Jul 28 16:02:30 2016
(r303449)
@@ -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);
}
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"
