This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] mips: siginfo.h: add SIGSYS details [BZ #18863]
- From: Markos Chandras <Markos dot Chandras at imgtec dot com>
- To: Joseph Myers <joseph at codesourcery dot com>
- Cc: Mike Frysinger <vapier at gentoo dot org>, <libc-alpha at sourceware dot org>, <linux-mips at linux-mips dot org>
- Date: Thu, 27 Aug 2015 10:56:01 +0100
- Subject: Re: [PATCH] mips: siginfo.h: add SIGSYS details [BZ #18863]
- Authentication-results: sourceware.org; auth=none
- References: <1440563342-5411-1-git-send-email-vapier at gentoo dot org> <alpine dot DEB dot 2 dot 10 dot 1508260918240 dot 26898 at digraph dot polyomino dot org dot uk> <20150826171017 dot GD3116 at vapier> <55DEC3B9 dot 1070105 at imgtec dot com> <alpine dot DEB dot 2 dot 10 dot 1508270940090 dot 29564 at digraph dot polyomino dot org dot uk>
On 08/27/2015 10:41 AM, Joseph Myers wrote:
> On Thu, 27 Aug 2015, Markos Chandras wrote:
>
>>>> OK. CC to linux-mips because I see that the MIPS implementation of
>>>> copy_siginfo_to_user32 doesn't handle __SI_SYS, unlike arm64 at least, so
>>>> I suspect this won't in fact work for n32 or for o32 with a 64-bit kernel.
>>>
>>> i'm getting reports of seccomp misbehavior on mips already which is what
>>> started me down this glibc path. i suspect the original port was tested
>>> against o32 kernels only.
>>> -mike
>>>
>>
>> I have recently tested mips64 n64/n32 with the testsuite from libseccomp
>> and that led me to this fix
>>
>> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=9f161439e4104b641a7bfb9b89581d801159fec8
>>
>> if you are aware of other problems (and perhaps a test to trigger them)
>> that could be kernel related let me know
>
> My observation about copy_siginfo_to_user32 missing a case was purely
> based on the kernel source code. I've no reason to think it has anything
> to do with seccomp problems.
>
Yeah, my email was directed to Mike who said he is getting reports that
seccomp on MIPS has issues and that led him to his glibc patch.
--
markos