Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1507084 > unrolled thread
| Started by | Colin King <colin.king@canonical.com> |
|---|---|
| First post | 2016-10-24 12:50 +0200 |
| Last post | 2016-10-24 18:00 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] ARC: return -EFAULT on failed access_ok for arc_usr_cmpxchg syscall Colin King <colin.king@canonical.com> - 2016-10-24 12:50 +0200
Re: [PATCH] ARC: return -EFAULT on failed access_ok for arc_usr_cmpxchg syscall Colin Ian King <colin.king@canonical.com> - 2016-10-24 18:00 +0200
Re: [PATCH] ARC: return -EFAULT on failed access_ok for arc_usr_cmpxchg syscall Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-10-24 18:00 +0200
| From | Colin King <colin.king@canonical.com> |
|---|---|
| Date | 2016-10-24 12:50 +0200 |
| Subject | [PATCH] ARC: return -EFAULT on failed access_ok for arc_usr_cmpxchg syscall |
| Message-ID | <svKZj-6tF-3@gated-at.bofh.it> |
From: Colin Ian King <colin.king@canonical.com> arc_usr_cmpxchg currently returns an uninitialized value in ret on a failed access_ok call. Instead, return -EFAULT. Signed-off-by: Colin Ian King <colin.king@canonical.com> --- arch/arc/kernel/process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arc/kernel/process.c b/arch/arc/kernel/process.c index a746008..9ae3e1c 100644 --- a/arch/arc/kernel/process.c +++ b/arch/arc/kernel/process.c @@ -47,7 +47,7 @@ SYSCALL_DEFINE3(arc_usr_cmpxchg, int *, uaddr, int, expected, int, new) int ret; if (!access_ok(VERIFY_WRITE, uaddr, sizeof(int))) - return ret; + return -EFAULT; preempt_disable(); -- 2.9.3
[toc] | [next] | [standalone]
| From | Colin Ian King <colin.king@canonical.com> |
|---|---|
| Date | 2016-10-24 18:00 +0200 |
| Subject | Re: [PATCH] ARC: return -EFAULT on failed access_ok for arc_usr_cmpxchg syscall |
| Message-ID | <svPPj-1bs-5@gated-at.bofh.it> |
| In reply to | #1507084 |
On 24/10/16 16:56, Vineet Gupta wrote: > On 10/24/2016 03:47 AM, Colin King wrote: >> From: Colin Ian King <colin.king@canonical.com> >> >> arc_usr_cmpxchg currently returns an uninitialized value in ret >> on a failed access_ok call. Instead, return -EFAULT. >> >> Signed-off-by: Colin Ian King <colin.king@canonical.com> >> --- >> arch/arc/kernel/process.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arc/kernel/process.c b/arch/arc/kernel/process.c >> index a746008..9ae3e1c 100644 >> --- a/arch/arc/kernel/process.c >> +++ b/arch/arc/kernel/process.c >> @@ -47,7 +47,7 @@ SYSCALL_DEFINE3(arc_usr_cmpxchg, int *, uaddr, int, expected, int, new) >> int ret; >> >> if (!access_ok(VERIFY_WRITE, uaddr, sizeof(int))) >> - return ret; >> + return -EFAULT; >> >> preempt_disable(); > > Thx for spotting this. Since this is new code, not yet merged, do you mind if I > squash it with orig patch. I can add a Reviewed-by a SOB as you prefer. Sure, squash it int and add me as a Reviewed-by. Colin > > Thx, > -Vineet > >
[toc] | [prev] | [next] | [standalone]
| From | Vineet Gupta <Vineet.Gupta1@synopsys.com> |
|---|---|
| Date | 2016-10-24 18:00 +0200 |
| Subject | Re: [PATCH] ARC: return -EFAULT on failed access_ok for arc_usr_cmpxchg syscall |
| Message-ID | <svPPj-1bs-7@gated-at.bofh.it> |
| In reply to | #1507084 |
On 10/24/2016 03:47 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > arc_usr_cmpxchg currently returns an uninitialized value in ret > on a failed access_ok call. Instead, return -EFAULT. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > arch/arc/kernel/process.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arc/kernel/process.c b/arch/arc/kernel/process.c > index a746008..9ae3e1c 100644 > --- a/arch/arc/kernel/process.c > +++ b/arch/arc/kernel/process.c > @@ -47,7 +47,7 @@ SYSCALL_DEFINE3(arc_usr_cmpxchg, int *, uaddr, int, expected, int, new) > int ret; > > if (!access_ok(VERIFY_WRITE, uaddr, sizeof(int))) > - return ret; > + return -EFAULT; > > preempt_disable(); Thx for spotting this. Since this is new code, not yet merged, do you mind if I squash it with orig patch. I can add a Reviewed-by a SOB as you prefer. Thx, -Vineet
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web