don't reject unknown/future flags in sigaltstack, allow SS_AUTODISARM
authorRich Felker <dalias@aerifal.cx>
Tue, 5 Mar 2019 16:02:15 +0000 (11:02 -0500)
committerRich Felker <dalias@aerifal.cx>
Tue, 5 Mar 2019 16:08:46 +0000 (11:08 -0500)
historically, and likely accidentally, sigaltstack was specified to
fail with EINVAL if any flag bit other than SS_DISABLE was set. the
resolution of Austin Group issue 1187 fixes this so that the
requirement is only to fail for SS_ONSTACK (which cannot be set) or
"invalid" flags.

Linux fails on the kernel side for invalid flags, but historically
accepts SS_ONSTACK as a no-op, so it needs to be rejected in userspace
still.

with this change, the Linux-specific SS_AUTODISARM, provided since
commit 9680e1d03a794b0e0d5815c749478228ed40a36d but unusable due to
rejection at runtime, is now usable.

src/signal/sigaltstack.c

index 62cb81adfe16ea1a112448f1654d55a859718374..cfa3f5c19d0de03384ff3f1a3b884149626db07b 100644 (file)
@@ -9,7 +9,7 @@ int sigaltstack(const stack_t *restrict ss, stack_t *restrict old)
                        errno = ENOMEM;
                        return -1;
                }
-               if (ss->ss_flags & ~SS_DISABLE) {
+               if (ss->ss_flags & SS_ONSTACK) {
                        errno = EINVAL;
                        return -1;
                }