Re: [PATCH] binder: prevent transactions to context manager from its own process.

From: Hridya Valsaraju
Date: Mon Oct 14 2019 - 13:38:10 EST


On Fri, Oct 11, 2019 at 3:11 PM Jann Horn <jannh@xxxxxxxxxx> wrote:
>
> On Fri, Oct 11, 2019 at 11:59 PM Jann Horn <jannh@xxxxxxxxxx> wrote:
> > (I think you could also let A receive a handle
> > to itself and then transact with itself, but I haven't tested that.)
>
> Ignore this sentence, that's obviously wrong because same-binder_proc
> nodes will always show up as a binder, not a handle.

Thank you for the email and steps to reproduce the issue Jann. I need
some time to take a look at the same and I will get back to you once I
understand it and hopefully have a fix. We do want to disallow
same-process transactions. Here is a little bit more of context for
the patch: https://lkml.org/lkml/2018/3/28/173