Remove useless cast, probably motivated by an older and stupider lint.

Update outdated comment (and docs) about which header lines are printed.
I appear to have been carrying this patch around since 2002.

Don't launder const

fix sign-compare issue

Remove the \n and tabs from the __COPYRIGHT() strings.

Clarify how one is supposed to use the pointers returned by getutentries()
(the utmpentry.c code), specifically with respect to who owns them and
when to free them. Now they're owned by utmpentry.c,

Clarify how one is supposed to use the pointers returned by getutentries()
(the utmpentry.c code), specifically with respect to who owns them and
when to free them. Now they're owned by utmpentry.c, only. Abolish the
freeutentries() function, which was the wrong abstraction; add instead
endutentries(), which flushes out the internally managed memory.

Update callers as necessary. Some (e.g. talkd) had been leaking memory;
others (e.g. syslogd) had been accidentally freeing and reloading utmp
more often than necessary. There are a couple untidy bits in users and
rwhod that someone should look after sometime, maybe.

Fixes PR bin/35131, which was about talkd's memory leak.

show more ...

PR/36266: David A. Holland: Don't read and write sig_atomic_t variables in
signal handlers; only write them.

Remove unnecessary ctype.h from libexec/comsat/comsat.c file
From: Slava Semushin <slava.semushin@gmail.com>

Don't do things you are not supposed to inside a signal handler.

PR/30170: Markus W Kilbinger: src/libexec/comsat complains about: '/' in
"/dev/pts/1"

Fix off by one reported by David A. Holland in PR bin/26778.

Whitespace nits.

Use strtoimax rather than strtol to parse a number which represents
an offset in a file. long is insufficient on some architecture.
Problem noticed by Atsushi Onoe while ago.

stat the right file.

No need to pretty print the errno with err() or warn().

Forgot to include utmpx support for this one.

realloc mistake. from openbsd

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22284, verified by myself.

Rename `sigset' locals to avoid symbol shadowing warning.

strncpy() -> strlcpy()

ignore trailing newline which is included in the message from mail.local.

Use strtol() instead of atoi() for reading the number out of the
datagram that we received, which leads to easier support for
(ignoring) the procmail messages that specify the folder to which
the mes

Use strtol() instead of atoi() for reading the number out of the
datagram that we received, which leads to easier support for
(ignoring) the procmail messages that specify the folder to which
the message was delivered.

When reading the mailbox, if we encounter a "From " line, we should
exit(). This can occur if there are a lot of rapidly arriving, yet
short messages.

show more ...

Fix output to a tty that is missing ONLCR (eg emacs, vi, mutt,
etc.). This became broken when strvis(3) started being used instead
of simply scanning for "bad" characters.

convert to use getprogname()

fix nested extern