commit adabc385d09e0aa5086e4909b359d84578032faf
author Jack Pham <quic_jackp@quicinc.com> Mon Oct 17 00:17:50 2022 -0700
committer Jack Pham <quic_jackp@quicinc.com> Mon Oct 17 00:17:50 2022 -0700
tree f5ca86a6d4f72982ee9a571efafb03641e61ebf4
parent 2b977acc6d8d72d62cbd760c16624c0fe3ba80e9

UsbGadgetHal: Add null check in getCurrentUsbFunctions()

Avoid dereferencing if the callback parameter is null.

Change-Id: I9da79fd92f44abf0175e2ca311f0247b49bace1d

hal/UsbGadget.cpp

diff --git a/hal/UsbGadget.cpp b/hal/UsbGadget.cpp
index 65a8c09..10173e0 100644
--- a/hal/UsbGadget.cpp
+++ b/hal/UsbGadget.cpp
@@ -107,6 +107,8 @@
 
 Return<void> UsbGadget::getCurrentUsbFunctions(
     const sp<V1_0::IUsbGadgetCallback> &callback) {
+  if (!callback) return Void();
+
   Return<void> ret = callback->getCurrentUsbFunctionsCb(
       mCurrentUsbFunctions, mCurrentUsbFunctionsApplied
                                 ? Status::FUNCTIONS_APPLIED