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: stable@vger.kernel.org # 4.4+ Acked-by: Christian Brauner <christian.brauner@ubuntu.com> Signed-off-by: Todd Kjos <tkjos@google.com> Change-Id: I9b1769a3510fed250bb21859ef8beebabe034c66 Link: https://lore.kernel.org/r/20211112180720.2858135-1-tkjos@google.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent fa55b7d commit c21a80c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

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)