Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.bugs.dist > #967132

Bug#904385: arm64: sigaltstack fails with MINSIGSTKSZ for 32-bit processes

From Aurelien Jarno <aurelien@aurel32.net>
Newsgroups linux.debian.bugs.dist, linux.debian.kernel
Subject Bug#904385: arm64: sigaltstack fails with MINSIGSTKSZ for 32-bit processes
Date 2019-07-29 22:30 +0200
Message-ID <ypkrn-33F-5@gated-at.bofh.it> (permalink)
References (2 earlier) <weP0d-3TQ-3@gated-at.bofh.it> <wftTI-6UN-5@gated-at.bofh.it> <wCfyh-5Vu-1@gated-at.bofh.it> <weOQy-3Py-1@gated-at.bofh.it> <wCfyh-5Vu-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Cross-posted to 2 groups.

Show all headers | View raw


control: notfixed -1 linux/4.18.20-1
control: found -1 4.19.37-6

On 2018-09-26 11:41, Steve McIntyre wrote:
> On Wed, Jul 25, 2018 at 04:25:03PM +0100, Steve McIntyre wrote:
> >On Mon, Jul 23, 2018 at 09:40:33PM +0200, Aurelien Jarno wrote:
> >>control: affects -1 glibc
> >>control: found linux/4.17.6-2
> >>
> >>On 2018-07-23 21:31, Aurelien Jarno wrote:
> >>> Package: src:linux
> >>> Version: 4.9.110-1
> >>> Severity: normal
> >>> 
> >>> Dear Maintainer,
> >>> 
> >>> The arm64 kernel allows one to run aarch32 processes on an aarch64
> >>> processor (if it has support for it), using the standard 32/64-bit
> >>> syscall compatibility. However this compat layer does not correctly
> >>> validate the arguments of the sigaltstack syscall.
> >>> 
> >>
> >>I forgot to say that the problem is reproducible with kernel 4.17.6.
> >
> >Fix proposed in https://lkml.org/lkml/2018/7/25/409 
> 
> At Will's suggestion, I've just tested that patchset locally and it
> definitely fixes this problem so I've added a Tested-by: for him.
> 

The fix is composed of two patches, and only the first one went to the
stable releases. Therefore both our oldstable and stable kernels used on
the build daemons are still buggy.

The following one is still missing in at least 4.9 and 4.19:

| commit 24951465cbd279f60b1fdc2421b3694405bcff42
| Author: Will Deacon <will.deacon@arm.com>
| Date:   Wed Sep 5 15:34:43 2018 +0100
| 
|     arm64: compat: Provide definition for COMPAT_SIGMINSTKSZ
|     
|     arch/arm/ defines a SIGMINSTKSZ of 2k, so we should use the same value
|     for compat tasks.
|     
|     Cc: Arnd Bergmann <arnd@arndb.de>
|     Cc: Dominik Brodowski <linux@dominikbrodowski.net>
|     Cc: "Eric W. Biederman" <ebiederm@xmission.com>
|     Cc: Andrew Morton <akpm@linux-foundation.org>
|     Cc: Al Viro <viro@zeniv.linux.org.uk>
|     Cc: Oleg Nesterov <oleg@redhat.com>
|     Reviewed-by: Dave Martin <Dave.Martin@arm.com>
|     Reported-by: Steve McIntyre <steve.mcintyre@arm.com>
|     Tested-by: Steve McIntyre <93sam@debian.org>
|     Signed-off-by: Will Deacon <will.deacon@arm.com>
|     Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>

-- 
Aurelien Jarno                          GPG: 4096R/1DDD8C9B
aurelien@aurel32.net                 http://www.aurel32.net

Back to linux.debian.bugs.dist | Previous | Next | Find similar | Unroll thread


Thread

Bug#904385: arm64: sigaltstack fails with MINSIGSTKSZ for 32-bit processes Aurelien Jarno <aurelien@aurel32.net> - 2019-07-29 22:30 +0200

csiph-web