On Mon, Jul 29, 2019 at 03:16:06PM -0700, Todd Kjos wrote: > From: Todd Kjos <tkjos@xxxxxxxxxxx> > > commit a370003cc301d4361bae20c9ef615f89bf8d1e8a upstream > > There is a race between the binder driver cleaning > up a completed transaction via binder_free_transaction() > and a user calling binder_ioctl(BC_FREE_BUFFER) to > release a buffer. It doesn't matter which is first but > they need to be protected against running concurrently > which can result in a UAF. > > Signed-off-by: Todd Kjos <tkjos@xxxxxxxxxx> > Cc: stable <stable@xxxxxxxxxxxxxxx> # 4.14 4.19 > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > --- > drivers/android/binder.c | 16 ++++++++++++++-- > 1 file changed, 14 insertions(+), 2 deletions(-) Thanks for the backport, now queued up. greg k-h
