diff options
author | Pedro F. Giffuni <pfg@FreeBSD.org> | 2016-05-14 04:29:13 +0000 |
---|---|---|
committer | Pedro F. Giffuni <pfg@FreeBSD.org> | 2016-05-14 04:29:13 +0000 |
commit | c8aa42a2da4c939e05d864d85cb3530a6e67f68f (patch) | |
tree | 6a56ad5bae73d7d93ddf3f7ffa700c28112bd673 /usr.sbin/timed | |
parent | 0150b85a1cd3c609a8b971d4296a18c1fa6c5baa (diff) | |
download | src-c8aa42a2da4c939e05d864d85cb3530a6e67f68f.tar.gz src-c8aa42a2da4c939e05d864d85cb3530a6e67f68f.zip |
Undo the bogus gethostname() change from r299709.
The "- 1" is there specifically to enable checking for NUL termination.
I should also admit the rest change was mostly cosmetic and the
overruns can't occur in practice: still I leave them to pacify
static analyzers.
Pointed out by: bde
Notes
Notes:
svn path=/head/; revision=299713
Diffstat (limited to 'usr.sbin/timed')
-rw-r--r-- | usr.sbin/timed/timed/timed.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/usr.sbin/timed/timed/timed.c b/usr.sbin/timed/timed/timed.c index 21c98b0c3b5e..a8efd6dbe66c 100644 --- a/usr.sbin/timed/timed/timed.c +++ b/usr.sbin/timed/timed/timed.c @@ -196,7 +196,7 @@ main(int argc, char *argv[]) if (goodgroup != NULL || goodhosts != NULL) Mflag = 1; - if (gethostname(hostname, sizeof(hostname)) < 0) + if (gethostname(hostname, sizeof(hostname) - 1) < 0) err(1, "gethostname"); self.l_bak = &self; self.l_fwd = &self; |