faraphel/chrony
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

reference: terminate string returned by gethostname()

Browse source 
POSIX doesn't require the string to be terminated if it didn't fit in
the buffer.

This issue was found in a Frama-C analysis.
This commit is contained in:
Miroslav Lichvar 2018-06-20 12:27:59 +02:00
parent 63fe34e890
commit 49cdd6bf09
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
reference.c
View file

@ -545,6 +545,8 @@ maybe_log_offset(double offset, time_t now)
if (gethostname(host, sizeof(host)) < 0) { if (gethostname(host, sizeof(host)) < 0) {
strcpy(host, "<UNKNOWN>"); strcpy(host, "<UNKNOWN>");
} }
host[sizeof (host) - 1] = '\0';
fprintf(p, "Subject: chronyd reports change to system clock on node [%s]\n", host); fprintf(p, "Subject: chronyd reports change to system clock on node [%s]\n", host);
fputs("\n", p); fputs("\n", p);