summaryrefslogtreecommitdiffstats
path: root/usr.bin/rwho
diff options
context:
space:
mode:
authorache <ache@FreeBSD.org>1996-12-05 20:19:33 +0000
committerache <ache@FreeBSD.org>1996-12-05 20:19:33 +0000
commit17cc02ba938335a8fe09df5e682cf5aa00547774 (patch)
tree40fcfa3e4ec55867d9273b26e75c5667101d6cb3 /usr.bin/rwho
parente1ff7f23f852545fc959bfc5f5a158938af54ac5 (diff)
downloadFreeBSD-src-17cc02ba938335a8fe09df5e682cf5aa00547774.zip
FreeBSD-src-17cc02ba938335a8fe09df5e682cf5aa00547774.tar.gz
Fix my error from previous commit with mixing rwhod protocol
and utmp sizes. Replace hardcoded constants by sizeofs or symbolic constants
Diffstat (limited to 'usr.bin/rwho')
-rw-r--r--usr.bin/rwho/rwho.c15
1 files changed, 8 insertions, 7 deletions
diff --git a/usr.bin/rwho/rwho.c b/usr.bin/rwho/rwho.c
index 762a34f..b617af5 100644
--- a/usr.bin/rwho/rwho.c
+++ b/usr.bin/rwho/rwho.c
@@ -57,7 +57,7 @@ struct whod wd;
int utmpcmp();
#define NUSERS 1000
struct myutmp {
- char myhost[MAXHOSTNAMELEN];
+ char myhost[sizeof(wd.wd_hostname)];
int myidle;
struct outmp myutmp;
} myutmp[NUSERS];
@@ -141,7 +141,7 @@ main(argc, argv)
width = 0;
for (i = 0; i < nusers; i++) {
/* append one for the blank and use 8 for the out_line */
- int j = strlen(mp->myhost) + 1 + 8;
+ int j = strlen(mp->myhost) + 1 + sizeof(mp->myutmp.out_line);
if (j > width)
width = j;
mp++;
@@ -150,9 +150,10 @@ main(argc, argv)
for (i = 0; i < nusers; i++) {
char buf[BUFSIZ], cbuf[80];
strftime(cbuf, sizeof(cbuf), "%c", localtime((time_t *)&mp->myutmp.out_time));
- (void)sprintf(buf, "%s:%-.*s", mp->myhost, UT_LINESIZE, mp->myutmp.out_line);
+ (void)sprintf(buf, "%s:%-.*s", mp->myhost,
+ sizeof(mp->myutmp.out_line), mp->myutmp.out_line);
printf("%-*.*s %-*s %.12s",
- UT_NAMESIZE, UT_NAMESIZE,
+ UT_NAMESIZE, sizeof(mp->myutmp.out_name),
mp->myutmp.out_name,
width,
buf,
@@ -181,11 +182,11 @@ utmpcmp(u1, u2)
{
int rc;
- rc = strncmp(u1->myutmp.out_name, u2->myutmp.out_name, UT_NAMESIZE);
+ rc = strncmp(u1->myutmp.out_name, u2->myutmp.out_name, sizeof(u2->myutmp.out_name));
if (rc)
return (rc);
- rc = strncmp(u1->myhost, u2->myhost, UT_HOSTSIZE);
+ rc = strcmp(u1->myhost, u2->myhost);
if (rc)
return (rc);
- return (strncmp(u1->myutmp.out_line, u2->myutmp.out_line, UT_LINESIZE));
+ return (strncmp(u1->myutmp.out_line, u2->myutmp.out_line, sizeof(u2->myutmp.out_line)));
}
OpenPOWER on IntegriCloud