if (SCARG(uap, nd) > p->p_fd->fd_nfiles) {
   			/* forgiving; slightly wrong */
   			SCARG(uap, nd) = p->p_fd->fd_nfiles;
   		}


while this is marked as "slightly wrong" it is done so *because*
it was actually chosen to be that way.  why, exactly, do you want
to "fix" this (and break backwards compatibility for old programs,
both binary & source.)  IMO it should be left alone.


.mrg.