diff options
| author | Xu YiPing <xuyiping@hisilicon.com> | 2017-09-05 10:25:38 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-09-18 16:06:00 +0200 |
| commit | 52b81611f209da5f49019260522633e994e241b5 (patch) | |
| tree | 6e184c6958790f5949792b082488b6cfd567b754 /drivers/android | |
| parent | 1c363eaece2752c5f8b1b874cb4ae435de06aa66 (diff) | |
| download | linux-52b81611f209da5f49019260522633e994e241b5.tar.bz2 | |
binder: fix an ret value override
commit 372e3147df70 ("binder: guarantee txn complete / errors delivered
in-order") incorrectly defined a local ret value. This ret value will
be invalid when out of the if block
Fixes: 372e3147df70 ("binder: refactor binder ref inc/dec for thread safety")
Signed-off-by: Xu YiPing <xuyiping@hislicon.com>
Signed-off-by: Todd Kjos <tkjos@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/android')
| -rw-r--r-- | drivers/android/binder.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/android/binder.c b/drivers/android/binder.c index 72af95c9ea22..b257bb0e2cfe 100644 --- a/drivers/android/binder.c +++ b/drivers/android/binder.c @@ -2326,7 +2326,6 @@ static int binder_translate_handle(struct flat_binder_object *fp, (u64)node->ptr); binder_node_unlock(node); } else { - int ret; struct binder_ref_data dest_rdata; binder_node_unlock(node); |