[PATCH] android: binder: check result of binder_get_thread() in binder_poll()

2017-05-08 Thread Dmitry Torokhov
If binder_get_thread() fails to give us a thread data, we should avoid dereferencing a NULL pointer and return POLLERR instead. Signed-off-by: Dmitry Torokhov --- drivers/android/binder.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/drivers/android/binder.c b/

Re: [PATCH] android: binder: check result of binder_get_thread() in binder_poll()

2017-08-22 Thread Greg Kroah-Hartman
On Mon, May 08, 2017 at 01:43:23PM -0700, Dmitry Torokhov wrote: > If binder_get_thread() fails to give us a thread data, we should avoid > dereferencing a NULL pointer and return POLLERR instead. > > Signed-off-by: Dmitry Torokhov > Reviewed-by: Douglas Anderson > --- > drivers/android/binder.

Re: [PATCH] android: binder: check result of binder_get_thread() in binder_poll()

2017-05-08 Thread John Stultz
On Mon, May 8, 2017 at 1:43 PM, Dmitry Torokhov wrote: > If binder_get_thread() fails to give us a thread data, we should avoid > dereferencing a NULL pointer and return POLLERR instead. > > Signed-off-by: Dmitry Torokhov Pulling Todd Kjos in on this too. -john > --- > drivers/android/binder.c

Re: [PATCH] android: binder: check result of binder_get_thread() in binder_poll()

2017-05-08 Thread Doug Anderson
Dmitry, On Mon, May 8, 2017 at 1:46 PM, John Stultz wrote: > On Mon, May 8, 2017 at 1:43 PM, Dmitry Torokhov > wrote: >> If binder_get_thread() fails to give us a thread data, we should avoid >> dereferencing a NULL pointer and return POLLERR instead. >> >> Signed-off-by: Dmitry Torokhov > > Pu