Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-04-23 Thread Greg KH
On Mon, Apr 23, 2018 at 10:07:43AM +0200, Martijn Coenen wrote: > On Wed, Mar 28, 2018 at 1:34 PM, Martijn Coenen wrote: > > On Wed, Mar 28, 2018 at 1:28 PM, Greg KH wrote: > >> What is different from "v2" you sent before this? No change information

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-04-23 Thread Greg KH
On Mon, Apr 23, 2018 at 10:07:43AM +0200, Martijn Coenen wrote: > On Wed, Mar 28, 2018 at 1:34 PM, Martijn Coenen wrote: > > On Wed, Mar 28, 2018 at 1:28 PM, Greg KH wrote: > >> What is different from "v2" you sent before this? No change information > >> from v1? > > Greg, is this in your

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-04-23 Thread Martijn Coenen
On Wed, Mar 28, 2018 at 1:34 PM, Martijn Coenen wrote: > On Wed, Mar 28, 2018 at 1:28 PM, Greg KH 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

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-04-23 Thread Martijn Coenen
On Wed, Mar 28, 2018 at 1:34 PM, Martijn Coenen wrote: > On Wed, Mar 28, 2018 at 1:28 PM, Greg KH 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

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-03-28 Thread Martijn Coenen
On Wed, Mar 28, 2018 at 1:28 PM, Greg KH 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

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-03-28 Thread Martijn Coenen
On Wed, Mar 28, 2018 at 1:28 PM, Greg KH 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

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-03-28 Thread Greg KH
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

Re: [PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-03-28 Thread Greg KH
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

[PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-03-28 Thread Martijn Coenen
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:

[PATCH v2] ANDROID: binder: prevent transactions into own process.

2018-03-28 Thread Martijn Coenen
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: