diff options
| author | Martijn Coenen <maco@google.com> | 2018-04-04 11:46:56 +0200 |
|---|---|---|
| committer | Tim Schumacher <timschumi@gmx.de> | 2018-07-16 20:24:36 +0200 |
| commit | 057269956b5b0e45b415eba96c7ed87b46d624ad (patch) | |
| tree | e7e1533e66b4b1226eb437fc97d102c2c82fd35a | |
| parent | b2cc7deac7747b0b1f5156f9f4b22413eb410d5f (diff) | |
| download | frameworks_native-057269956b5b0e45b415eba96c7ed87b46d624ad.tar.gz frameworks_native-057269956b5b0e45b415eba96c7ed87b46d624ad.tar.bz2 frameworks_native-057269956b5b0e45b415eba96c7ed87b46d624ad.zip | |
Don't pad before calling writeInPlace().
writeInplace() itself already pads securely, by masking off
the padded bytes. If the padding is done before calling
writeInplace(), no mask is applied, and heap data can leak.
Bug: 77237570
Test: builds
Change-Id: Ide27a0002d4ed4196530430760245b971f6a3f44
Merged-In: Ide27a0002d4ed4196530430760245b971f6a3f44
(cherry picked from commit f8542381b72a7bb2452a5278a00ca8c34edbf8a0)
(cherry picked from commit 732132b765cd7b667f16cf32f0fe4c852d7d44dd)
Change-Id: Id65e4573e18ab68b804f1cf63a6977a71da01e5d
| -rw-r--r-- | libs/binder/Parcel.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/binder/Parcel.cpp b/libs/binder/Parcel.cpp index 4690a8233..f474d9907 100644 --- a/libs/binder/Parcel.cpp +++ b/libs/binder/Parcel.cpp @@ -1055,7 +1055,7 @@ status_t Parcel::write(const FlattenableHelperInterface& val) if (err) return err; // payload - void* const buf = this->writeInplace(pad_size(len)); + void* const buf = this->writeInplace(len); if (buf == NULL) return BAD_VALUE; |