Re: [PATCH] posix_spawn: skip SIGKILL & SIGSTOP
Corinna Vinschen <[email protected]>
| Newsgroups | gmane.comp.lib.newlib |
|---|---|
| Message-ID | <[email protected]> |
On Mar 11 17:46, Takashi Yano wrote:
> On Mon, 10 Mar 2025 22:55:17 +0800
> Yuyi Wang wrote:
> >
> > sigaction() returns EINVAL on SIGKILL & SIGSTOP. We need to skip them.
> > ---
> > newlib/libc/posix/posix_spawn.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/newlib/libc/posix/posix_spawn.c b/newlib/libc/posix/posix_spawn.c
> > index 6fd6159d0..46e4e5369 100644
> > --- a/newlib/libc/posix/posix_spawn.c
> > +++ b/newlib/libc/posix/posix_spawn.c
> > @@ -199,6 +199,8 @@ process_spawnattr(const posix_spawnattr_t sa)
> >
> > if (sa->sa_flags & POSIX_SPAWN_SETSIGDEF) {
> > for (i = 1; i < NSIG; i++) {
> > + if (i == SIGKILL || i == SIGSTOP)
> > + continue;
> > if (sigismember(&sa->sa_sigdefault, i))
> > if (sigaction(i, &sigact, NULL) != 0)
> > return (errno);
> > --
> > 2.48.1.windows.1-2
> >
>
> Thanks for the patch. LGTM.
>
> Corinna, is this as you intended?
Sure, just go ahead.
Thanks,
Corinna