Date: Wed, 29 Sep 1999 20:59:37 +0200 From: Marcel Moolenaar <marcel@scc.nl> To: John Polstra <jdp@polstra.com> Cc: current@freebsd.org Subject: Re: HEADS UP: sigset_t changes committed Message-ID: <37F26199.56F69F79@scc.nl> References: <37F23064.98EEBC67@scc.nl> <199909291848.LAA22988@vashon.polstra.com>
next in thread | previous in thread | raw e-mail | index | archive | help
John Polstra wrote:
>
> Following up on my previous mail regarding the panic on the Alpha,
> I've been looking at the diff for the code in question, in
> "src/sys/nfs/nfs_socket.c":
>
> @@ -1501,14 +1502,16 @@
> struct nfsreq *rep;
> register struct proc *p;
> {
> + sigset_t tmpset;
>
> + tmpset = p->p_siglist;
> + SIGSETNAND(tmpset, p->p_sigmask);
> + SIGSETNAND(tmpset, p->p_sigignore);
> if (rep && (rep->r_flags & R_SOFTTERM))
> return (EINTR);
> if (!(nmp->nm_flag & NFSMNT_INT))
> return (0);
> - if (p && p->p_siglist &&
> - (((p->p_siglist & ~p->p_sigmask) & ~p->p_sigignore) &
> - NFSINT_SIGMASK))
> + if (p && SIGNOTEMPTY(p->p_siglist) && NFSINT_SIGMASK(tmpset))
> return (EINTR);
> return (0);
> }
>
> It looks like the old code was prepared for "p" to be NULL, but the
> new code assumes it is non-NULL.
Thanks, I overlooked that one. I'll fix it right away.
--
Marcel Moolenaar mailto:marcel@scc.nl
SCC Internetworking & Databases http://www.scc.nl/
The FreeBSD project mailto:marcel@FreeBSD.org
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?37F26199.56F69F79>