Date: Tue, 17 Mar 1998 15:00:23 +0000 (GMT) From: Robin Carey <r.carey@dcs.napier.ac.uk> To: freebsd-bugs@FreeBSD.ORG Subject: rlogind Message-ID: <Pine.SOL.3.91.980317143952.1515A-100000@artemis>
next in thread | raw e-mail | index | archive | help
Hi, heres a couple of things I noticed the other day - Sorry for no send-pr. This is relative to: FreeBSD-stable/src/libexec/rlogind/rlogind.c of todays date. line 203: "char line[MAXPATHLEN];" should surely be: "char line[MAXPATHLEN+1];" to allow space for terminating NUL. doit(): line 314 (just after the forkpty() -- in the childs portion of code): Should there not be a "closelog()" before execl(3), as is the case for telnetd(8) ? doit(): line 306 (just after the forkpty()): I would have thought that the resgistration of the "cleanup" handler for SIGCHLD (done on line 352) should have come BEFORE the forkpty() on line 306. However if there is some reason for it to come after the forkpty() that I don't know about, surely it would make more sense to do this earlier than on line 352 ? local_domain(): line 758: "char localhost[MAXHOSTNAMELEN];" should surely be: "char localhost[MAXHOSTNAMELEN+1];", to allow space for terminating NUL. And if you're feeling really Keen, you could always update the bcopy(3) on line 377 to memcpy(3), and replace those 1's and 0's in the various calls to exit(3) with EXIT_FAILURE and EXIT_SUCCESS, respectively :) Cheers, Rob C To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.SOL.3.91.980317143952.1515A-100000>