commit 2583c250c288835a0b8e002b4ea04ec545f184d3
author Venkat Thogaru <vthogaru@codeaurora.org> Tue Jun 16 16:02:12 2020 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Tue Jun 16 04:37:08 2020 -0700
tree 8fbe5dd994dd803d52b1e31d5eea145b18f91d21
parent cdb534d9cdd5e3893f4e00c5a13ed745a3c9b5be

display: Fix null pointer dereference

Change-Id: Iff047b79330854c2e2bd8fd07f18c842dec0bc2d

services/config/src/device_impl.cpp

diff --git a/services/config/src/device_impl.cpp b/services/config/src/device_impl.cpp
index 8d036cd..6cbdb18 100644
--- a/services/config/src/device_impl.cpp
+++ b/services/config/src/device_impl.cpp
@@ -575,6 +575,10 @@
   int32_t error = intf_->GetSupportedDSIBitClks(*disp_id, &bit_clks);
 
   bit_clks_data = reinterpret_cast<uint64_t *>(malloc(sizeof(uint64_t) * bit_clks.size()));
+  if (bit_clks_data == NULL) {
+    _hidl_cb(-EINVAL, {}, {});
+    return;
+  }
   for (int i = 0; i < bit_clks.size(); i++) {
     bit_clks_data[i] = bit_clks[i];
   }