more buffer validation

Bug: 294464949
Test: hidl_test
Test: libHidlBase_parcel_fuzzer with test case
Change-Id: I1f27c6d1d8a3fb189d510443c17f44aaff0488f2

1 files changed, 5 insertions, 0 deletions

diff --git a/Parcel.cpp b/Parcel.cpp
index a20d98c..77e2161 100644
--- a/Parcel.cpp
+++ b/Parcel.cpp
@@ -1453,6 +1453,11 @@ status_t Parcel::readNullableNativeHandleNoDup(const native_handle_t **handle,
         // writable memory, and the handle returned from here will actually be
         // used (rather than be ignored).
         if (embedded) {
+            if(!validateBufferParent(parent_buffer_handle, parent_offset)) {
+                ALOGE("Buffer in parent %zu offset %zu invalid.", parent_buffer_handle, parent_offset);
+                return BAD_VALUE;
+            }
+
             binder_buffer_object *parentBuffer =
                 reinterpret_cast<binder_buffer_object*>(mData + mObjects[parent_buffer_handle]);