Date: Wed, 22 Aug 2001 02:18:48 -0700 From: "Crist J. Clark" <cristjc@earthlink.net> To: Peter Pentchev <roam@ringlet.net> Cc: freebsd-audit@FreeBSD.ORG Subject: Re: syslogd(8) Hostname Upgrade Message-ID: <20010822021848.M313@blossom.cjclark.org> In-Reply-To: <20010822114241.C8189@ringworld.oblivion.bg>; from roam@ringlet.net on Wed, Aug 22, 2001 at 11:42:41AM %2B0300 References: <20010822013841.A76483@blossom.cjclark.org> <20010822114241.C8189@ringworld.oblivion.bg>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Aug 22, 2001 at 11:42:41AM +0300, Peter Pentchev wrote: > On Wed, Aug 22, 2001 at 01:38:41AM -0700, Crist J. Clark wrote: > > I brought this up as a PR (bin/24444) some months ago. I figured I'd > > finally close the PR and add this. > > > > This patch makes syslogd(8) update the hostname when the daemon is > > reinitialized. Before anyone points it out, this is not meant to be a > > security feature. It allows syslogd(8) to update the hostname without > > terminating and restarting the daemon and generates a message logging > > the change, nothing more, nothing less. > > > > I've been running with it for months on -CURRENT and -STABLE with no > > problems. Nit picks anyone? > > Just one, and not a major one at that :) > > [snip] > > /* > > + * Load hostname (may have changed). > > + */ > > + if (signo) > > + (void)strlcpy(oldLocalHostName, LocalHostName, > > + sizeof(oldLocalHostName)); > > Do you really need the (void) here? If you want to make lint and friends > happy, you should put the (void) in a couple of other places, too - like > snprintf(), logmsg(), dprintf().. IMHO, just drop it. No, don't _need_ it. I'll add one to snprintf(3), good catch. However, logmsg() is already a void and dprintf is a macro. -- Crist J. Clark cjclark@alum.mit.edu To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-audit" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010822021848.M313>