* [PATCH v2] ANDROID: binder: prevent transactions into own process.
@ 2018-03-28 9:14 Martijn Coenen
2018-03-28 11:28 ` Greg KH
0 siblings, 1 reply; 5+ messages in thread
From: Martijn Coenen @ 2018-03-28 9:14 UTC (permalink / raw)
To: gregkh, john.stultz, tkjos, arve, amit.pundir
Cc: devel, maco, Martijn Coenen, linux-kernel
This can't happen with normal nodes (because you can't get a ref
to a node you own), but it could happen with the context manager;
to make the behavior consistent with regular nodes, reject
transactions into the context manager by the process owning it.
Reported-by: syzbot+09e05aba06723a94d43d@syzkaller.appspotmail.com
Signed-off-by: Martijn Coenen <maco@android.com>
Cc: stable <stable@vger.kernel.org>
---
drivers/android/binder.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/android/binder.c b/drivers/android/binder.c
index 764b63a5aade..e578eee31589 100644
--- a/drivers/android/binder.c
+++ b/drivers/android/binder.c
@@ -2839,6 +2839,14 @@ static void binder_transaction(struct binder_proc *proc,
else
return_error = BR_DEAD_REPLY;
mutex_unlock(&context->context_mgr_node_lock);
+ if (target_node && target_proc == proc) {
+ binder_user_error("%d:%d got transaction to context manager from process owning it\n",
+ proc->pid, thread->pid);
+ return_error = BR_FAILED_REPLY;
+ return_error_param = -EINVAL;
+ return_error_line = __LINE__;
+ goto err_invalid_target_handle;
+ }
}
if (!target_node) {
/*
--
2.17.0.rc1.321.gba9d0f2565-goog
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.
2018-03-28 9:14 [PATCH v2] ANDROID: binder: prevent transactions into own process Martijn Coenen
@ 2018-03-28 11:28 ` Greg KH
2018-03-28 11:34 ` Martijn Coenen
0 siblings, 1 reply; 5+ messages in thread
From: Greg KH @ 2018-03-28 11:28 UTC (permalink / raw)
To: Martijn Coenen
Cc: john.stultz, tkjos, arve, amit.pundir, devel, maco, linux-kernel
On Wed, Mar 28, 2018 at 11:14:50AM +0200, Martijn Coenen wrote:
> This can't happen with normal nodes (because you can't get a ref
> to a node you own), but it could happen with the context manager;
> to make the behavior consistent with regular nodes, reject
> transactions into the context manager by the process owning it.
>
> Reported-by: syzbot+09e05aba06723a94d43d@syzkaller.appspotmail.com
> Signed-off-by: Martijn Coenen <maco@android.com>
> ---
> drivers/android/binder.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
What is different from "v2" you sent before this? No change information
from v1?
I'm totally confused as to which is the "latest" patch here :(
thanks,
greg k-h-
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.
2018-03-28 11:28 ` Greg KH
@ 2018-03-28 11:34 ` Martijn Coenen
2018-04-23 8:07 ` Martijn Coenen
0 siblings, 1 reply; 5+ messages in thread
From: Martijn Coenen @ 2018-03-28 11:34 UTC (permalink / raw)
To: Greg KH
Cc: John Stultz, Todd Kjos, Arve Hjønnevåg, Amit Pundir,
open list:ANDROID DRIVERS, Martijn Coenen, LKML
On Wed, Mar 28, 2018 at 1:28 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> What is different from "v2" you sent before this? No change information
> from v1?
Sorry I messed this up - the first resend did not have "v2" in the
subject but did contain v2 change information, the second resend had
the v2 subject and did not contain the change information :( Either of
the last two patches is fine - it just used "target_proc" instead of
"target_node->proc".
>
> I'm totally confused as to which is the "latest" patch here :(
>
> thanks,
>
> greg k-h-
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.
2018-03-28 11:34 ` Martijn Coenen
@ 2018-04-23 8:07 ` Martijn Coenen
2018-04-23 8:24 ` Greg KH
0 siblings, 1 reply; 5+ messages in thread
From: Martijn Coenen @ 2018-04-23 8:07 UTC (permalink / raw)
To: Greg KH
Cc: Amit Pundir, open list:ANDROID DRIVERS, LKML,
Arve Hjønnevåg, Martijn Coenen, John Stultz, Todd Kjos
On Wed, Mar 28, 2018 at 1:34 PM, Martijn Coenen <maco@android.com> wrote:
> On Wed, Mar 28, 2018 at 1:28 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> What is different from "v2" you sent before this? No change information
>> from v1?
Greg, is this in your queue, or should I just send a v3 to clean this
up? See below for my original reply.
Thanks,
Martijn
>
> Sorry I messed this up - the first resend did not have "v2" in the
> subject but did contain v2 change information, the second resend had
> the v2 subject and did not contain the change information :( Either of
> the last two patches is fine - it just used "target_proc" instead of
> "target_node->proc".
>
>>
>> I'm totally confused as to which is the "latest" patch here :(
>>
>> thanks,
>>
>> greg k-h-
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.
2018-04-23 8:07 ` Martijn Coenen
@ 2018-04-23 8:24 ` Greg KH
0 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2018-04-23 8:24 UTC (permalink / raw)
To: Martijn Coenen
Cc: Amit Pundir, open list:ANDROID DRIVERS, LKML,
Arve Hjønnevåg, Martijn Coenen, John Stultz, Todd Kjos
On Mon, Apr 23, 2018 at 10:07:43AM +0200, Martijn Coenen wrote:
> On Wed, Mar 28, 2018 at 1:34 PM, Martijn Coenen <maco@android.com> wrote:
> > On Wed, Mar 28, 2018 at 1:28 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> >> What is different from "v2" you sent before this? No change information
> >> from v1?
>
> Greg, is this in your queue, or should I just send a v3 to clean this
> up? See below for my original reply.
It's in my queue, I couldn't do anything during the merge window and I
was gone all last week, so am catching up on stuff now. Should get to
it by the end of this week, sorry for the delay.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-04-23 8:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-28 9:14 [PATCH v2] ANDROID: binder: prevent transactions into own process Martijn Coenen
2018-03-28 11:28 ` Greg KH
2018-03-28 11:34 ` Martijn Coenen
2018-04-23 8:07 ` Martijn Coenen
2018-04-23 8:24 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).