binder: avoid potential data leakage when copying txn
authorTodd Kjos <tkjos@google.com>
Tue, 30 Nov 2021 18:51:50 +0000 (10:51 -0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 3 Dec 2021 13:29:39 +0000 (14:29 +0100)
commit6d98eb95b450a75adb4516a1d33652dc78d2b20c
tree6a2414f8bbba0be2d87aa7faa881c3c2cc8a6fce
parentfe6b1869243f23a485a106c214bcfdc7aa0ed593
binder: avoid potential data leakage when copying txn

Transactions are copied from the sender to the target
first and objects like BINDER_TYPE_PTR and BINDER_TYPE_FDA
are then fixed up. This means there is a short period where
the sender's version of these objects are visible to the
target prior to the fixups.

Instead of copying all of the data first, copy data only
after any needed fixups have been applied.

Fixes: 457b9a6f09f0 ("Staging: android: add binder driver")
Reviewed-by: Martijn Coenen <maco@android.com>
Acked-by: Christian Brauner <christian.brauner@ubuntu.com>
Signed-off-by: Todd Kjos <tkjos@google.com>
Link: https://lore.kernel.org/r/20211130185152.437403-3-tkjos@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/android/binder.c