diff options
author | 2023-01-20 22:32:08 +0000 | |
---|---|---|
committer | 2023-01-20 22:32:08 +0000 | |
commit | eec41c20fccde72e81a00bf6577633a4e6200b4c (patch) | |
tree | 89caee5cc20e1bfcf20c6666225554f863e028a8 | |
parent | b621667caa69883ac5c2b4bb9fdd0256fb07ec1d (diff) | |
parent | 0beb956917b34471e5709a6ce2ab47d83011b1e6 (diff) |
Merge "Check for null data buffer in AParcel_marshal"
-rw-r--r-- | libs/binder/ndk/parcel.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libs/binder/ndk/parcel.cpp b/libs/binder/ndk/parcel.cpp index 86930229ce..94f72d96f6 100644 --- a/libs/binder/ndk/parcel.cpp +++ b/libs/binder/ndk/parcel.cpp @@ -700,6 +700,9 @@ binder_status_t AParcel_marshal(const AParcel* parcel, uint8_t* buffer, size_t s return STATUS_BAD_VALUE; } const uint8_t* internalBuffer = parcel->get()->data(); + if (internalBuffer == nullptr) { + return STATUS_UNEXPECTED_NULL; + } memcpy(buffer, internalBuffer + start, len); return STATUS_OK; } |