commit 1b4376f20f6930d74c93b9ca05ec8561b87cca5b
author qctecmdr <qctecmdr@localhost> Wed Oct 11 18:28:22 2023 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Wed Oct 11 18:28:21 2023 -0700
tree 8e2cd416df60e3b2312c07204b084c6283f3335b
parent 7f139799900afc1195e473859433e23fca3aa112
parent cd808a8709b6d0d76ffa70224fd1c52e72dceabf

Merge "ipc: HwBinders: add check to avoid OOB in ipc_pal_stream_set_device"

ipc/HwBinders/pal_ipc_server/src/pal_server_wrapper.cpp

diff --git a/ipc/HwBinders/pal_ipc_server/src/pal_server_wrapper.cpp b/ipc/HwBinders/pal_ipc_server/src/pal_server_wrapper.cpp
index 3381842..99174e7 100644
--- a/ipc/HwBinders/pal_ipc_server/src/pal_server_wrapper.cpp
+++ b/ipc/HwBinders/pal_ipc_server/src/pal_server_wrapper.cpp
@@ -881,6 +881,11 @@
     struct pal_device *devices = NULL;
     int cnt = 0;
     int32_t ret = -ENOMEM;
+
+    if (noOfDevices > devs_hidl.size()) {
+        ALOGE("Invalid noOfDevices");
+        return -EINVAL;
+    }
     if (devs_hidl.size()) {
         PalDevice *dev_hidl = NULL;
         devices = (struct pal_device *)calloc (1,