From 803ff1161c131f8300db84d8551dfa545817abbd Mon Sep 17 00:00:00 2001 From: Ramji Jiyani Date: Mon, 2 Aug 2021 22:04:45 +0000 Subject: [PATCH] UPSTREAM: 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: Todd Kjos Signed-off-by: Ramji Jiyani Link: https://lore.kernel.org/r/20210802220446.1938347-1-ramjiyani@google.com Signed-off-by: Greg Kroah-Hartman Bug: 186045972 (cherry picked from commit 1ae14df56cc3e87d56f3c159803a289021f8ef7d) Signed-off-by: Ramji Jiyani Change-Id: I874bab8f32bafcf6a5fb38669c60d1e968f50f59 --- drivers/android/binder.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/android/binder.c b/drivers/android/binder.c index cd688e896239..7d0a1713a513 100644 --- a/drivers/android/binder.c +++ b/drivers/android/binder.c @@ -2705,8 +2705,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);