xiaomi-sm8450/android_kernel_xiaomi_sm8450
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

UPSTREAM: binder: Add invalid handle info in user error log

Browse Source 
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 <tkjos@google.com>
Signed-off-by: Ramji Jiyani <ramjiyani@google.com>
Link: https://lore.kernel.org/r/20210802220446.1938347-1-ramjiyani@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Bug: 186045972
(cherry picked from commit 1ae14df56cc3e87d56f3c159803a289021f8ef7d)
Signed-off-by: Ramji Jiyani <ramjiyani@google.com>
Change-Id: I874bab8f32bafcf6a5fb38669c60d1e968f50f59
This commit is contained in:
Ramji Jiyani
2021-08-02 22:04:45 +00:00
parent a40cd23755
commit 803ff1161c
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/android/binder.c
View File

@@ -2705,8 +2705,8 @@ static void binder_transaction(struct binder_proc *proc,
ref->node, &target_proc, ref->node, &target_proc,
&return_error); &return_error);
} else { } else {
binder_user_error("%d:%d got transaction to invalid handle\n", binder_user_error("%d:%d got transaction to invalid handle, %u\n",
proc->pid, thread->pid); proc->pid, thread->pid, tr->target.handle);
return_error = BR_FAILED_REPLY; return_error = BR_FAILED_REPLY;
} }
binder_proc_unlock(proc); binder_proc_unlock(proc);