revert rev 1.45:
"PR/54435: Adjust for new kernel behavior of soreceive(9) clearing MSG_OOB"

That change was trying to make rlogin work again after the SIOCATMARK ioctl
was broken, but that kernel

revert rev 1.45:
"PR/54435: Adjust for new kernel behavior of soreceive(9) clearing MSG_OOB"

That change was trying to make rlogin work again after the SIOCATMARK ioctl
was broken, but that kernel bug has now been fixed, so the original rlogin code
now works again. Further, the changed rlogin code actually did the wrong thing,
by treating reception of the MSG_OOB byte as meaning that we are now
"at the mark", but that is not true... we are "at the mark" only when
we have reached the point in the stream where the MSG_OOB byte was originally,
as indicated by SIOCATMARK. So going back to the previous code seems best
all around. ok'd by christos.

show more ...

PR/54435: Adjust for new kernel behavior of soreceive(9) clearing MSG_OOB
when receiving the oob message. This made SIOCATMARK return always 0 since
the oob message was cleared. Instead, use recvmsg(

PR/54435: Adjust for new kernel behavior of soreceive(9) clearing MSG_OOB
when receiving the oob message. This made SIOCATMARK return always 0 since
the oob message was cleared. Instead, use recvmsg(2) to determine if
the message was oob or not. This works with both the old and new kernel
and it is not racy.

show more ...

(foo *) 0 -> NULL
int -> ssize_t/size_t

msg:
avoid passing the same pointer in multiple arguments for restrict
marked arguments:
- sigaction() wants separate in/out
- use memmove() not memcpy() for overlapping regions (this may fix
a rea

msg:
avoid passing the same pointer in multiple arguments for restrict
marked arguments:
- sigaction() wants separate in/out
- use memmove() not memcpy() for overlapping regions (this may fix
a real bug in nvi -- but it seems unlikely)
- select() wants separate read/write/except
- sigprocmask() wants separate set/oset

show more ...

Added missing sa_mask initialization (CID 979636)

OK kamil@ mrg@

Sync usage with man page.

PR/47584: Steffen Daoden: Add option to turn off Nagle to rlogin

static + __dead

Fix sign-compare issues

Remove the \n and tabs from the __COPYRIGHT() strings.
Tweak to use a consistent format.

Remove duplicate #includes
From Slava Semushin <slava.semushin@gmail.com>, via private mail

change (mostly) int to socklen_t. GCC 4 doesn't like that int and
socklen_t are different signness.

Remove some more old kerberos4 code.

Support -4 and -6. Approved by christos.

getuid() returns a uid_t not int, so make uid uid_t.

Dead code clean up:
* The function speed(), imported from 4.4BSD-Lite2, has never been
used by NetBSD. Obtained from FreeBSD.
* Finish removing -L (litout) option. This has not been supported
s

Dead code clean up:
* The function speed(), imported from 4.4BSD-Lite2, has never been
used by NetBSD. Obtained from FreeBSD.
* Finish removing -L (litout) option. This has not been supported
since rev. 1.5 (NetBSD 1.0). The option still appeared in the
usage and man page. Obtained from FreeBSD.
* In function doit(), dead code imported from 4.4BSD-Lite2, dealing
with termios. NetBSD cracked this nut differently long before this
import. Obtained from FreeBSD.

show more ...

Remove stray ; in comment.

- Fix bug in port parsing code; s_port must be in network byte order.
- Factor out port parsing code and add error checking to it.
- WARNS=3

Drop some unnecessary whitespaces from the usage message. Closes PR bin/24993
by Kouichirou Hiratsuka.

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

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

Add -p option to rsh and rlogin to allow specifying a non-standard port
number.

use strlcpy

Remove #ifdef __STDC__. De-__P() and ANSIfy.

convert to use getprogname()

IPv6 support. (I thought I've committed these but was not yet)