summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Pawan Wagh <waghpawan@google.com> 2023-01-20 22:32:08 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2023-01-20 22:32:08 +0000
commiteec41c20fccde72e81a00bf6577633a4e6200b4c (patch)
tree89caee5cc20e1bfcf20c6666225554f863e028a8
parentb621667caa69883ac5c2b4bb9fdd0256fb07ec1d (diff)
parent0beb956917b34471e5709a6ce2ab47d83011b1e6 (diff)
Merge "Check for null data buffer in AParcel_marshal"
-rw-r--r--libs/binder/ndk/parcel.cpp3
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;
}