Pull pending binder patches into chromeos-4.12 |
||
Issue descriptionQueued in -next: d53bebdf4d77 binder: fix memory corruption in binder_transaction binder 52b81611f209 binder: fix an ret value override 1c363eaece27 android: binder: fix type mismatch warning from git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git char-misc-linus
,
Sep 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/kernel/+/6cd5652ecf5f5be464ff7e4989902ec971503b0c commit 6cd5652ecf5f5be464ff7e4989902ec971503b0c Author: Xu YiPing <xuyiping@hisilicon.com> Date: Wed Sep 20 23:37:54 2017 FROMGIT: 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 BUG= chromium:767131 TEST=Build and run Change-Id: If28c80d1da73a0eabf49b896e31bb298c1aee068 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> Signed-off-by: Guenter Roeck <groeck@chromium.org> (cherry picked from commit 52b81611f209 git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git char-misc-linus) Reviewed-on: https://chromium-review.googlesource.com/675772 Reviewed-by: Dylan Reid <dgreid@chromium.org> [modify] https://crrev.com/6cd5652ecf5f5be464ff7e4989902ec971503b0c/drivers/android/binder.c
,
Sep 20 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/kernel/+/0821466c83f7ea3e78ea345ae3e2f4e344ec2346 commit 0821466c83f7ea3e78ea345ae3e2f4e344ec2346 Author: Xu YiPing <xuyiping@hisilicon.com> Date: Wed Sep 20 23:37:56 2017 FROMGIT: binder: fix memory corruption in binder_transaction binder commit 7a4408c6bd3e ("binder: make sure accesses to proc/thread are safe") made a change to enqueue tcomplete to thread->todo before enqueuing the transaction. However, in err_dead_proc_or_thread case, the tcomplete is directly freed, without dequeued. It may cause the thread->todo list to be corrupted. So, dequeue it before freeing. BUG= chromium:767131 TEST=Build and run Change-Id: I0914f1469ea2e0eaea4f554e26a88ffe55a4cfaf Fixes: 7a4408c6bd3e ("binder: make sure accesses to proc/thread are safe") Signed-off-by: Xu YiPing <xuyiping@hisilicon.com> Signed-off-by: Todd Kjos <tkjos@google.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Guenter Roeck <groeck@chromium.org> (cherry picked from commit d53bebdf4d77 git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git char-misc-linus) Reviewed-on: https://chromium-review.googlesource.com/675773 Reviewed-by: Dylan Reid <dgreid@chromium.org> [modify] https://crrev.com/0821466c83f7ea3e78ea345ae3e2f4e344ec2346/drivers/android/binder.c
,
Sep 21 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by bugdroid1@chromium.org
, Sep 20 2017