From 2e61d90c44df342d2493ef407ada126ee8c8cb61 Mon Sep 17 00:00:00 2001 From: Carlos Llamas Date: Wed, 26 Apr 2023 02:16:25 +0000 Subject: [PATCH] ANDROID: fix use of plain integer as NULL pointer This patch fixes the following sparse issues: drivers/android/binder.c:1373:70: sparse: sparse: Using plain integer as NULL pointer drivers/android/binder.c:2508:41: sparse: sparse: Using plain integer as NULL pointer Fixes: e107ea9e4dd1 ("ANDROID: vendor_hooks: Add hooks for binder proc transaction") Reported-by: kernel test robot Link: https://lore.kernel.org/oe-kbuild-all/202304150607.IuUhkfxB-lkp@intel.com/ Cc: zhengding chen Change-Id: I272b3239e2d81bfdd28d1fe412f2c3d0731ec50f Signed-off-by: Carlos Llamas --- drivers/android/binder.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/android/binder.c b/drivers/android/binder.c index 16e0ba315abf..66152cc321a9 100644 --- a/drivers/android/binder.c +++ b/drivers/android/binder.c @@ -1399,7 +1399,8 @@ err_no_ref: */ static void binder_free_ref(struct binder_ref *ref) { - trace_android_vh_binder_del_ref(ref->proc ? ref->proc->tsk : 0, ref->data.desc); + trace_android_vh_binder_del_ref(ref->proc ? ref->proc->tsk : NULL, + ref->data.desc); if (ref->node) binder_free_node(ref->node); kfree(ref->death); @@ -2871,7 +2872,8 @@ static int binder_proc_transaction(struct binder_transaction *t, thread = binder_select_thread_ilocked(proc); trace_android_vh_binder_proc_transaction(current, proc->tsk, - thread ? thread->task : 0, node->debug_id, t->code, pending_async); + thread ? thread->task : NULL, node->debug_id, t->code, + pending_async); if (thread) { binder_transaction_priority(thread->task, t, node_prio,