Date: Sat, 23 Feb 2002 11:47:57 -0500 (EST) From: John Baldwin <jhb@FreeBSD.org> To: Jake Burkholder <jake@locore.ca> Cc: current@FreeBSD.ORG, dillon@FreeBSD.ORG Subject: Re: First (easy) td_ucred patch Message-ID: <XFMail.020223114757.jhb@FreeBSD.org> In-Reply-To: <20020223044503.C27577@locore.ca>
next in thread | previous in thread | raw e-mail | index | archive | help
On 23-Feb-02 Jake Burkholder wrote: > Apparently, On Fri, Feb 22, 2002 at 11:38:07PM -0500, > John Baldwin said words to the effect of; > >> I'm currently testing the following patch whcih is a subset of the td_ucred >> changes. It involves no API changes, but only contains 2 basic changes: >> >> 1) We still need Giant when doing the crhold() to set td_ucred in >> cred_update_thread(). This is an old bug that is my fault. I knew that >> PROC_LOCK was sufficient yet which was my reason for not using td_ucred. >> However, we could still be derferencing a stale p_ucred and doing very >> bad >> things, so this needs to be fixed until p_ucred is fully protected by the >> PROC_LOCK. This also means that td_ucred is now safe to use. As such: >> >> 2) All the "easy" p->p_ucred -> td->td_ucred changes that don't involve the >> changes to API's such as suser() and p_canfoo(). The next patch in this >> series will most likely be the suser() API change. >> >> http://www.FreeBSD.org/~jhb/patches/ucred.patch > > The UGAR changes in sysv_sem.c to not leak Giant are most unreleated and > should probably be committed separately. I wonder who introduced the leaks > in the first place. Yes. The first change will also be a separate commit. > Other than that I don't see anything wrong with this. Commit it. > > Jake -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.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?XFMail.020223114757.jhb>