From 23dbe457bf5f99f80a46e3a2035970149d6cb024 Mon Sep 17 00:00:00 2001 From: dwmalone Date: Wed, 20 Feb 2008 21:54:41 +0000 Subject: Two no-op fixes to improve corretness of syslogd code: 1) Use [AP]F_LOCAL rather than [AP]F_UNIX. 2) When copying a pipe's name, use f->f_un.f_pipe.f_pname, not f->f_un.f_fname. PR: 20889 Submitted by: Damieon Stark PR: 116642 Submitted by: Jim Pirzyk Reviewed by: md5 --- usr.sbin/syslogd/syslogd.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/usr.sbin/syslogd/syslogd.c b/usr.sbin/syslogd/syslogd.c index e06afde..4212ed6 100644 --- a/usr.sbin/syslogd/syslogd.c +++ b/usr.sbin/syslogd/syslogd.c @@ -512,9 +512,9 @@ main(int argc, char *argv[]) STAILQ_FOREACH_SAFE(fx, &funixes, next, fx1) { (void)unlink(fx->name); memset(&sunx, 0, sizeof(sunx)); - sunx.sun_family = AF_UNIX; + sunx.sun_family = AF_LOCAL; (void)strlcpy(sunx.sun_path, fx->name, sizeof(sunx.sun_path)); - fx->s = socket(AF_UNIX, SOCK_DGRAM, 0); + fx->s = socket(PF_LOCAL, SOCK_DGRAM, 0); if (fx->s < 0 || bind(fx->s, (struct sockaddr *)&sunx, SUN_LEN(&sunx)) < 0 || chmod(fx->name, fx->mode) < 0) { @@ -1964,7 +1964,8 @@ cfline(const char *line, struct filed *f, const char *prog, const char *host) case '|': f->f_un.f_pipe.f_pid = 0; - (void)strlcpy(f->f_un.f_fname, p + 1, sizeof(f->f_un.f_fname)); + (void)strlcpy(f->f_un.f_pipe.f_pname, p + 1, + sizeof(f->f_un.f_pipe.f_pname)); f->f_type = F_PIPE; break; -- cgit v1.1