PR/54435 CVS commit: src/usr.bin/rlogin

["Chuck Silvers" <chs%netbsd.org@localhost>]

The following reply was made to PR kern/54435; it has been noted by GNATS.

From: "Chuck Silvers" <chs%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/54435 CVS commit: src/usr.bin/rlogin
Date: Tue, 3 Aug 2021 23:21:08 +0000

 Module Name:	src
 Committed By:	chs
 Date:		Tue Aug  3 23:21:07 UTC 2021
 
 Modified Files:
 	src/usr.bin/rlogin: rlogin.c
 
 Log Message:
 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.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.47 -r1.48 src/usr.bin/rlogin/rlogin.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.