Re: [yocto-patches] [pseudo] [PATCH 0/3] closefrom/close_range: protect every pseudo fd, then drop one

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Wed, 2026-07-29 at 07:19 +0100, Richard Purdie via
lists.yoctoproject.org wrote:
> On Wed, 2026-07-29 at 05:08 +0000, Babanpreet Singh wrote:
> > On Tue, 2026-07-28 at 16:59 +0100, Richard Purdie wrote:
> > > I think the missing piece is this:
> > > 
> > > - if (op != OP_CHROOT && op != OP_CHDIR && op != OP_CLOSE && op
> > > != OP_CLOSEFROM && op != OP_DUP
> > > + if (op != OP_CHROOT && op != OP_CHDIR && op != OP_CLOSE && op
> > > != OP_CLOSEFROM && op != OP_CLOSE_RANGE && op != OP_DUP
> > 
> > Sorry for the breakage, and thanks for chasing it down.
> > 
> > I can reproduce it both ways on master-next at 6d65d74, with a
> > probe that
> > opens an fd and calls close_range(fd, ~0U, 0):
> > 
> >   no PSEUDO_*_PATHS set                        OK
> >   PSEUDO_IGNORE_PATHS covering the fd's path   SIGSEGV
> >   PSEUDO_INCLUDE_PATHS set, path not in it     SIGSEGV
> >   same env, closefrom() instead                OK
> >   all of the above, plus your line             OK
> > 
> > For OP_CLOSE_RANGE the path comes from fd_path(lowfd), the ignore
> > check
> > hits it, and pseudo_client_op() returns early with result still 0,
> > which
> > close_range.c then dereferences.
> > 
> > Two things before I respin. Do you want the three fd-protect
> > patches
> > resent, or will you carry the master-next commits since you have
> > already
> > resolved the conflicts? And the ignore-path line: fold it into the
> > close_range patch with a Suggested-by, or keep it as your own
> > commit ahead
> > of it? Either works for me.
> 
> You can fold the above piece in with a suggested by, it doesn't make
> sense in a commit on its own. I can take the other fd-protect patches
> if you're ok with that change, I need to update the commit messages.
> 
> Whilst most builds are testing cleanly, we are seeing these two
> failures:
> 
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/35/builds/4403
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/23/builds/4466
> 
> which look like a pseudo problem, and that is with the current pseudo
> master-next. I think there is still some issue remaining somewhere
> unfortunately. I'm afraid I don't know what it is.

One of those doesn't look to be pseudo, the arm one definitely does
though. It looks like it may be arch specific.

Cheers,

Richard