diff options
author | Mike Kelly <mike.kelly@arm.com> | 2023-07-31 12:48:53 +0100 |
---|---|---|
committer | mike.kelly <mike.kelly@arm.com> | 2023-08-01 16:02:04 +0000 |
commit | a92b3f3d441139e106f60d1c56ffbe4eaf41140f (patch) | |
tree | 87042cbf3a60987d5f7bd5714326e4f1c67674be | |
parent | 60101b3eeccfafd1c9bada96674e4f0b167c7870 (diff) | |
download | android-nn-driver-a92b3f3d441139e106f60d1c56ffbe4eaf41140f.tar.gz |
IVGCVSW-7900 Added parameter checks to executeFenced
* Added parameter checks to ensure that the native_handles will work
with this function.
Signed-off-by: Mike Kelly <mike.kelly@arm.com>
Change-Id: I0f07de04b24f930d8025461178ba4cdb470eb5b4
-rw-r--r-- | ArmnnPreparedModel_1_3.cpp | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/ArmnnPreparedModel_1_3.cpp b/ArmnnPreparedModel_1_3.cpp index ceeb3c48..1827d900 100644 --- a/ArmnnPreparedModel_1_3.cpp +++ b/ArmnnPreparedModel_1_3.cpp @@ -434,6 +434,21 @@ Return<void> ArmnnPreparedModel_1_3<HalVersion>::executeFenced(const V1_3::Reque auto fenceNativeHandle = fenceWaitFor[index].getNativeHandle(); if (!fenceNativeHandle) { + ALOGE("ArmnnPreparedModel_1_3::executeFenced null native handle."); + cb(V1_3::ErrorStatus::INVALID_ARGUMENT, hidl_handle(nullptr), nullptr); + return Void(); + } + + if (fenceNativeHandle->numFds != 1) + { + ALOGE("ArmnnPreparedModel_1_3::executeFenced invalid fenceHandle numFds."); + cb(V1_3::ErrorStatus::INVALID_ARGUMENT, hidl_handle(nullptr), nullptr); + return Void(); + } + + if (fenceNativeHandle->numInts != 0) + { + ALOGE("ArmnnPreparedModel_1_3::executeFenced invalid fenceHandle numInts."); cb(V1_3::ErrorStatus::INVALID_ARGUMENT, hidl_handle(nullptr), nullptr); return Void(); } |