On Wed, Jun 10, 2020 at 02:34:00PM +0000, Visa Hankala wrote:
> A while ago, finishdup() was changed to release fdplock before calling
> closef() to avoid potential lock order problems in the file close path.
> However, the dup* code can still invoke that path with fdplock held
> through FRELE(). That is fixed by the diff below.
> 
> (In principle, the FRELE(fp, p) could be eliminated by getting fp while
> the file descriptor table is locked. As long as the lock is held, other
> threads are unable to close the file descriptor and hence the temporary
> file reference is unnecessary.)
> 
> OK?

ok anton@

Reply via email to