Commit 1ae14df5 authored by Ramji Jiyani's avatar Ramji Jiyani Committed by Greg Kroah-Hartman
Browse files

binder: Add invalid handle info in user error log 



In the case of a failed transaction, only the thread and process id are
logged. Add the handle info for the reference to the target node in user
error log to aid debugging.

Acked-by: default avatarTodd Kjos <tkjos@google.com>
Signed-off-by: default avatarRamji Jiyani <ramjiyani@google.com>
Link: https://lore.kernel.org/r/20210802220446.1938347-1-ramjiyani@google.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 0fc7ca62

drivers/android/binder.c

+2 −2
Original line number Diff line number Diff line
@@ -2547,8 +2547,8 @@ static void binder_transaction(struct binder_proc *proc, 
						ref->node, &target_proc,

						&return_error);

			} else {

				binder_user_error("%d:%d got transaction to invalid handle\n",

						  proc->pid, thread->pid);

				binder_user_error("%d:%d got transaction to invalid handle, %u\n",

						  proc->pid, thread->pid, tr->target.handle);

				return_error = BR_FAILED_REPLY;

			}

			binder_proc_unlock(proc);