Skip to content

Commit c21a80c

Browse files
toddkjosgregkh
authored andcommitted
binder: fix test regression due to sender_euid change
This is a partial revert of commit 29bc22a ("binder: use euid from cred instead of using task"). Setting sender_euid using proc->cred caused some Android system test regressions that need further investigation. It is a partial reversion because subsequent patches rely on proc->cred. Fixes: 29bc22a ("binder: use euid from cred instead of using task") Cc: [email protected] # 4.4+ Acked-by: Christian Brauner <[email protected]> Signed-off-by: Todd Kjos <[email protected]> Change-Id: I9b1769a3510fed250bb21859ef8beebabe034c66 Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent fa55b7d commit c21a80c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/android/binder.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2710,7 +2710,7 @@ static void binder_transaction(struct binder_proc *proc,
27102710
t->from = thread;
27112711
else
27122712
t->from = NULL;
2713-
t->sender_euid = proc->cred->euid;
2713+
t->sender_euid = task_euid(proc->tsk);
27142714
t->to_proc = target_proc;
27152715
t->to_thread = target_thread;
27162716
t->code = tr->code;

0 commit comments

Comments
 (0)