From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751709AbcF0Kd0 (ORCPT ); Mon, 27 Jun 2016 06:33:26 -0400 Received: from szxga03-in.huawei.com ([119.145.14.66]:48048 "EHLO szxga03-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750736AbcF0KdX (ORCPT ); Mon, 27 Jun 2016 06:33:23 -0400 Message-ID: <5771005B.3060605@huawei.com> Date: Mon, 27 Jun 2016 18:30:51 +0800 From: zhouchengming User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0) Gecko/20120428 Thunderbird/12.0.1 MIME-Version: 1.0 To: Yury Norov CC: , , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH] ilp32: fix {GET,SET}SIGMASK request for ptrace References: <1466207668-10549-1-git-send-email-ynorov@caviumnetworks.com> <1466207668-10549-13-git-send-email-ynorov@caviumnetworks.com> <5770B041.3040509@huawei.com> <20160627053916.GA305@yury-N73SV> In-Reply-To: <20160627053916.GA305@yury-N73SV> Content-Type: multipart/mixed; boundary="------------000803040600030202030500" X-Originating-IP: [10.177.236.183] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A090204.57710069.007B,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-05-26 15:14:31, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 06023bf1960e5def1aad9a2474dde5f2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --------------000803040600030202030500 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit On 2016/6/27 13:39, Yury Norov wrote: > Hi Zhou, > > Thank you for the patch. The idea is ok, but patch format got broken > for some reason. Could you re-send it? > > Yury. Sorry for the broken patch, maybe my mail guest has some problems. So I send the patch as an attachment. Thanks! > > On Mon, Jun 27, 2016 at 12:49:05PM +0800, zhouchengming wrote: >> atus: RO >> Content-Length: 4732 >> Lines: 181 >> >> The function compat_ptrace_request(used by ilp32) don't handle >> {GET,SET}SIGMASK request, so it will be handled by ptrace_request. >> But it's wrong because the compat_sigset_t of ilp32 differs from >> the sigset_t of aarch64. The patch fixes it. >> >> Signed-off-by: Zhou Chengming >> --- >> arch/arm64/include/asm/signal_ilp32.h | 22 ++++++++++++ >> arch/arm64/kernel/ptrace.c | 62 >> +++++++++++++++++++++++++++++++++ > > Here - unneeded line break > >> arch/arm64/kernel/signal_ilp32.c | 23 +------------ >> 3 files changed, 85 insertions(+), 22 deletions(-) >> >> diff --git a/arch/arm64/include/asm/signal_ilp32.h >> b/arch/arm64/include/asm/signal_ilp32.h > > and here > >> index 30eff23..ed52607 100644 >> --- a/arch/arm64/include/asm/signal_ilp32.h >> +++ b/arch/arm64/include/asm/signal_ilp32.h >> @@ -21,6 +21,28 @@ >> int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, sigset_t *set, >> struct pt_regs *regs); >> >> +static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set) >> +{ >> + compat_sigset_t cset; >> + >> + cset.sig[0] = set->sig[0]& 0xffffffffull; >> + cset.sig[1] = set->sig[0]>> 32; >> + >> + return copy_to_user(uset,&cset, sizeof(*uset)); >> +} >> + >> +static inline int get_sigset_t(sigset_t *set, >> + const compat_sigset_t __user *uset) >> +{ >> + compat_sigset_t s32; >> + >> + if (copy_from_user(&s32, uset, sizeof(*uset))) >> + return -EFAULT; >> + >> + set->sig[0] = s32.sig[0] | (((long)s32.sig[1])<< 32); >> + return 0; >> +} >> + >> #else >> >> static inline int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, >> sigset_t *set, > > and here > >> diff --git a/arch/arm64/kernel/ptrace.c b/arch/arm64/kernel/ptrace.c >> index a861105..8d4cad1 100644 >> --- a/arch/arm64/kernel/ptrace.c >> +++ b/arch/arm64/kernel/ptrace.c >> @@ -44,6 +44,7 @@ >> #include >> #include >> #include >> +#include >> >> #define CREATE_TRACE_POINTS >> #include >> @@ -1231,12 +1232,73 @@ COMPAT_SYSCALL_DEFINE4(aarch32_ptrace, >> compat_long_t, request, compat_long_t, pi > > and later on the patch > >> >> #endif /* CONFIG_AARCH32_EL0 */ >> >> +#ifdef CONFIG_ARM64_ILP32 >> + >> +static int compat_ilp32_ptrace(struct task_struct *child, compat_long_t >> request, >> + compat_ulong_t addr, compat_ulong_t data) >> +{ >> + compat_ulong_t __user *datap = compat_ptr(data); >> + int ret; >> + >> + switch (request) { >> + case PTRACE_GETSIGMASK: >> + if (addr != sizeof(compat_sigset_t)) { >> + ret = -EINVAL; >> + break; >> + } >> + >> + if (put_sigset_t((compat_sigset_t __user *)datap,&child->blocked)) >> + ret = -EFAULT; >> + else >> + ret = 0; >> + break; >> + >> + case PTRACE_SETSIGMASK: { >> + sigset_t new_set; >> + if (addr != sizeof(compat_sigset_t)) { >> + ret = -EINVAL; >> + break; >> + } >> + >> + if (get_sigset_t(&new_set, (compat_sigset_t __user *)datap)) { >> + ret = -EFAULT; >> + break; >> + } >> + >> + sigdelsetmask(&new_set, sigmask(SIGKILL)|sigmask(SIGSTOP)); >> + >> + /* >> + * Every thread does recalc_sigpending() after resume, so >> + * retarget_shared_pending() and recalc_sigpending() are not >> + * called here. >> + */ >> + spin_lock_irq(&child->sighand->siglock); >> + child->blocked = new_set; >> + spin_unlock_irq(&child->sighand->siglock); >> + >> + ret = 0; >> + break; >> + } >> + >> + default: >> + ret = compat_ptrace_request(child, request, addr, data); >> + } >> + >> + return ret; >> +} >> + >> +#endif /* CONFIG_ARM64_ILP32 */ >> + >> #ifdef CONFIG_COMPAT >> >> long compat_arch_ptrace(struct task_struct *child, compat_long_t request, >> compat_ulong_t caddr, compat_ulong_t cdata) >> { >> +#ifdef CONFIG_ARM64_ILP32 >> + return compat_ilp32_ptrace(child, request, caddr, cdata); >> +#else >> return compat_ptrace_request(child, request, caddr, cdata); >> +#endif >> } >> >> #endif /* CONFIG_COMPAT */ >> diff --git a/arch/arm64/kernel/signal_ilp32.c >> b/arch/arm64/kernel/signal_ilp32.c >> index 8ca64b9..3ef2749 100644 >> --- a/arch/arm64/kernel/signal_ilp32.c >> +++ b/arch/arm64/kernel/signal_ilp32.c >> @@ -28,6 +28,7 @@ >> #include >> #include >> #include >> +#include >> #include >> #include >> #include >> @@ -58,28 +59,6 @@ struct ilp32_rt_sigframe { >> struct ilp32_sigframe sig; >> }; >> >> -static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set) >> -{ >> - compat_sigset_t cset; >> - >> - cset.sig[0] = set->sig[0]& 0xffffffffull; >> - cset.sig[1] = set->sig[0]>> 32; >> - >> - return copy_to_user(uset,&cset, sizeof(*uset)); >> -} >> - >> -static inline int get_sigset_t(sigset_t *set, >> - const compat_sigset_t __user *uset) >> -{ >> - compat_sigset_t s32; >> - >> - if (copy_from_user(&s32, uset, sizeof(*uset))) >> - return -EFAULT; >> - >> - set->sig[0] = s32.sig[0] | (((long)s32.sig[1])<< 32); >> - return 0; >> -} >> - >> static int restore_ilp32_sigframe(struct pt_regs *regs, >> struct > > . > --------------000803040600030202030500 Content-Type: text/plain; charset="gb18030"; name="0001-ilp32-fix-GET-SET-SIGMASK-request-for-ptrace.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="0001-ilp32-fix-GET-SET-SIGMASK-request-for-ptrace.patch" >>From 06f5abeca59069cb41b0b4fdfa06240d3f78910d Mon Sep 17 00:00:00 2001 From: Zhou Chengming Date: Mon, 27 Jun 2016 12:57:02 +0800 Subject: [PATCH] ilp32: fix {GET,SET}SIGMASK request for ptrace The function compat_ptrace_request(used by ilp32) don't handle {GET,SET}SIGMASK request, so it will be handled by ptrace_request. But it's wrong because the compat_sigset_t of ilp32 differs from the sigset_t of aarch64. The patch fixes it. Signed-off-by: Zhou Chengming --- arch/arm64/include/asm/signal_ilp32.h | 22 ++++++++++++ arch/arm64/kernel/ptrace.c | 62 +++++++++++++++++++++++++++++++++ arch/arm64/kernel/signal_ilp32.c | 23 +------------ 3 files changed, 85 insertions(+), 22 deletions(-) diff --git a/arch/arm64/include/asm/signal_ilp32.h b/arch/arm64/include/asm/signal_ilp32.h index 30eff23..ed52607 100644 --- a/arch/arm64/include/asm/signal_ilp32.h +++ b/arch/arm64/include/asm/signal_ilp32.h @@ -21,6 +21,28 @@ int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, sigset_t *set, struct pt_regs *regs); +static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set) +{ + compat_sigset_t cset; + + cset.sig[0] = set->sig[0] & 0xffffffffull; + cset.sig[1] = set->sig[0] >> 32; + + return copy_to_user(uset, &cset, sizeof(*uset)); +} + +static inline int get_sigset_t(sigset_t *set, + const compat_sigset_t __user *uset) +{ + compat_sigset_t s32; + + if (copy_from_user(&s32, uset, sizeof(*uset))) + return -EFAULT; + + set->sig[0] = s32.sig[0] | (((long)s32.sig[1]) << 32); + return 0; +} + #else static inline int ilp32_setup_rt_frame(int usig, struct ksignal *ksig, sigset_t *set, diff --git a/arch/arm64/kernel/ptrace.c b/arch/arm64/kernel/ptrace.c index a861105..8d4cad1 100644 --- a/arch/arm64/kernel/ptrace.c +++ b/arch/arm64/kernel/ptrace.c @@ -44,6 +44,7 @@ #include #include #include +#include #define CREATE_TRACE_POINTS #include @@ -1231,12 +1232,73 @@ COMPAT_SYSCALL_DEFINE4(aarch32_ptrace, compat_long_t, request, compat_long_t, pi #endif /* CONFIG_AARCH32_EL0 */ +#ifdef CONFIG_ARM64_ILP32 + +static int compat_ilp32_ptrace(struct task_struct *child, compat_long_t request, + compat_ulong_t addr, compat_ulong_t data) +{ + compat_ulong_t __user *datap = compat_ptr(data); + int ret; + + switch (request) { + case PTRACE_GETSIGMASK: + if (addr != sizeof(compat_sigset_t)) { + ret = -EINVAL; + break; + } + + if (put_sigset_t((compat_sigset_t __user *)datap, &child->blocked)) + ret = -EFAULT; + else + ret = 0; + break; + + case PTRACE_SETSIGMASK: { + sigset_t new_set; + if (addr != sizeof(compat_sigset_t)) { + ret = -EINVAL; + break; + } + + if (get_sigset_t(&new_set, (compat_sigset_t __user *)datap)) { + ret = -EFAULT; + break; + } + + sigdelsetmask(&new_set, sigmask(SIGKILL)|sigmask(SIGSTOP)); + + /* + * Every thread does recalc_sigpending() after resume, so + * retarget_shared_pending() and recalc_sigpending() are not + * called here. + */ + spin_lock_irq(&child->sighand->siglock); + child->blocked = new_set; + spin_unlock_irq(&child->sighand->siglock); + + ret = 0; + break; + } + + default: + ret = compat_ptrace_request(child, request, addr, data); + } + + return ret; +} + +#endif /* CONFIG_ARM64_ILP32 */ + #ifdef CONFIG_COMPAT long compat_arch_ptrace(struct task_struct *child, compat_long_t request, compat_ulong_t caddr, compat_ulong_t cdata) { +#ifdef CONFIG_ARM64_ILP32 + return compat_ilp32_ptrace(child, request, caddr, cdata); +#else return compat_ptrace_request(child, request, caddr, cdata); +#endif } #endif /* CONFIG_COMPAT */ diff --git a/arch/arm64/kernel/signal_ilp32.c b/arch/arm64/kernel/signal_ilp32.c index 8ca64b9..3ef2749 100644 --- a/arch/arm64/kernel/signal_ilp32.c +++ b/arch/arm64/kernel/signal_ilp32.c @@ -28,6 +28,7 @@ #include #include #include +#include #include #include #include @@ -58,28 +59,6 @@ struct ilp32_rt_sigframe { struct ilp32_sigframe sig; }; -static inline int put_sigset_t(compat_sigset_t __user *uset, sigset_t *set) -{ - compat_sigset_t cset; - - cset.sig[0] = set->sig[0] & 0xffffffffull; - cset.sig[1] = set->sig[0] >> 32; - - return copy_to_user(uset, &cset, sizeof(*uset)); -} - -static inline int get_sigset_t(sigset_t *set, - const compat_sigset_t __user *uset) -{ - compat_sigset_t s32; - - if (copy_from_user(&s32, uset, sizeof(*uset))) - return -EFAULT; - - set->sig[0] = s32.sig[0] | (((long)s32.sig[1]) << 32); - return 0; -} - static int restore_ilp32_sigframe(struct pt_regs *regs, struct ilp32_sigframe __user *sf) { -- 1.7.7 --------------000803040600030202030500--