diff options
| author | Christopher Tate <ctate@google.com> | 2015-05-27 17:53:02 -0700 |
|---|---|---|
| committer | The Android Automerger <android-build@google.com> | 2015-08-14 12:25:33 -0700 |
| commit | 415cb192a586c9f2261a6653e2c681fce5af743e (patch) | |
| tree | a9f9f223944d8a00fa8c374dffad81a7701451a6 | |
| parent | fecb040045ccfb50c2cb8a0220745136c7e0220f (diff) | |
| download | frameworks_native-android-5.1.1_r22.zip frameworks_native-android-5.1.1_r22.tar.gz frameworks_native-android-5.1.1_r22.tar.bz2 | |
Disregard alleged binder entities beyond parcel boundsandroid-5.1.1_r22android-5.1.1_r17
When appending one parcel's contents to another, ignore binder
objects within the source Parcel that appear to lie beyond the
formal bounds of that Parcel's data buffer.
Bug 17312693
Change-Id: If592a260f3fcd9a56fc160e7feb2c8b44c73f514
(cherry picked from commit 27182be9f20f4f5b48316666429f09b9ecc1f22e)
| -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 99c934c..87ce5d0 100644 --- a/libs/binder/Parcel.cpp +++ b/libs/binder/Parcel.cpp @@ -411,7 +411,7 @@ status_t Parcel::appendFrom(const Parcel *parcel, size_t offset, size_t len) // Count objects in range for (int i = 0; i < (int) size; i++) { size_t off = objects[i]; - if ((off >= offset) && (off < offset + len)) { + if ((off >= offset) && (off + sizeof(flat_binder_object) <= offset + len)) { if (firstIndex == -1) { firstIndex = i; } |