diff options
| author |   Dmitry V. Levin <ldv@altlinux.org> | 2018-04-13 17:19:02 +0300 |
|---|---|---|
| committer |   Eric W. Biederman <ebiederm@xmission.com> | 2018-04-19 16:39:39 -0500 |
| commit | 32772c9e2e8b0f9bf4ae0d70eb3c9d8c40d5de22 (patch) | |
| tree | a980a4fb674320919ded78d43d2277a74aa62db8 /arch/sparc/include/uapi/asm | |
| parent | signal/alpha: Document a conflict with SI_USER for SIGFPE (diff) | |
| download | linux-dev-32772c9e2e8b0f9bf4ae0d70eb3c9d8c40d5de22.tar.xz linux-dev-32772c9e2e8b0f9bf4ae0d70eb3c9d8c40d5de22.zip | |
sparc: fix compat siginfo ABI regression
Starting with commit v4.14-rc1~60^2^2~1, a SIGFPE signal sent via kill
results to wrong values in si_pid and si_uid fields of compat siginfo_t.
This happens due to FPE_FIXME being defined to 0 for sparc, and at the
same time siginfo_layout() introduced by the same commit returns
SIL_FAULT for SIGFPE if si_code == SI_USER and FPE_FIXME is defined to 0.
Fix this regression by removing FPE_FIXME macro and changing all its users
to assign FPE_FLTUNK to si_code instead of FPE_FIXME.
Note that FPE_FLTUNK is a new macro introduced by commit
266da65e9156d93e1126e185259a4aae68188d0e.
Tested with commit v4.16-11958-g16e205cf42da.
This bug was found by strace test suite.
In the discussion about FPE_FLTUNK on sparc David Miller said:
> Eric, feel free to do something similar on Sparc.
Link: https://github.com/strace/strace/issues/21
Fixes: cc731525f26a ("signal: Remove kernel interal si_code magic")
Fixes: 2.3.41
Cc: David Miller <davem@davemloft.net>
Cc: sparclinux@vger.kernel.org
Conceptually-Acked-By: David Miller <davem@davemloft.net>
Thanks-to: Anatoly Pugachev <matorola@gmail.com>
Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Diffstat (limited to 'arch/sparc/include/uapi/asm')
| -rw-r--r-- | arch/sparc/include/uapi/asm/siginfo.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/arch/sparc/include/uapi/asm/siginfo.h b/arch/sparc/include/uapi/asm/siginfo.h index 896ce447d16a..e7049550ac82 100644 --- a/arch/sparc/include/uapi/asm/siginfo.h +++ b/arch/sparc/include/uapi/asm/siginfo.h @@ -18,13 +18,6 @@ #define SI_NOINFO 32767 /* no information in siginfo_t */ /* - * SIGFPE si_codes - */ -#ifdef __KERNEL__ -#define FPE_FIXME 0 /* Broken dup of SI_USER */ -#endif /* __KERNEL__ */ - -/* * SIGEMT si_codes */ #define EMT_TAGOVF 1 /* tag overflow */ |