ANDROID: binder: init desired_prio.sched_policy before use it
authorGanesh Mahendran <opensource.ganesh@gmail.com>
Wed, 27 Sep 2017 07:12:25 +0000 (15:12 +0800)
committerAmit Pundir <amit.pundir@linaro.org>
Mon, 18 Dec 2017 15:41:22 +0000 (21:11 +0530)
In function binder_transaction_priority(), we access
desired_prio before initialzing it.

This patch fix this.

Change-Id: I9d14d50f9a128010476a65b52631630899a44633
Signed-off-by: Ganesh Mahendran <opensource.ganesh@gmail.com>
drivers/android/binder.c

index ba83f0da15d120443453aaf557e85e5c56df6ec0..fbf044ff73a30075953885f0f28976bff9cbcbe7 100644 (file)
@@ -1188,7 +1188,7 @@ static void binder_transaction_priority(struct task_struct *task,
                                        struct binder_priority node_prio,
                                        bool inherit_rt)
 {
-       struct binder_priority desired_prio;
+       struct binder_priority desired_prio = t->priority;
 
        if (t->set_priority_called)
                return;
@@ -1200,9 +1200,6 @@ static void binder_transaction_priority(struct task_struct *task,
        if (!inherit_rt && is_rt_policy(desired_prio.sched_policy)) {
                desired_prio.prio = NICE_TO_PRIO(0);
                desired_prio.sched_policy = SCHED_NORMAL;
-       } else {
-               desired_prio.prio = t->priority.prio;
-               desired_prio.sched_policy = t->priority.sched_policy;
        }
 
        if (node_prio.prio < t->priority.prio ||