commit a669eeafd45db1a27e2b4ad08e4158ddf6771262
author himta ram <hram@codeaurora.org> Mon Feb 04 14:56:06 2019 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Thu Mar 28 05:22:14 2019 -0700
tree f5dc54d787ceff6411952e843a756a92c0b2352b
parent ef396aeca2bbef0263256a6bb7e7b7092636d4ed

FM: Fix KW issue in helium code

check "val" for NULL and return if val is NULL.

CRs-Fixed: 2424902
Change-Id: I2cb7464c06e39fbdf346709a43003caf7fb6a5b3

helium/radio_helium_hal.c

diff --git a/helium/radio_helium_hal.c b/helium/radio_helium_hal.c
index 019db8c..947402f 100644
--- a/helium/radio_helium_hal.c
+++ b/helium/radio_helium_hal.c
@@ -1977,23 +1977,31 @@
     ALOGE("%s: cmd = 0x%x", __func__, cmd);
     switch(cmd) {
     case HCI_FM_HELIUM_FREQ:
-        if (!val)
+        if (!val) {
             ret = -FM_HC_STATUS_NULL_POINTER;
+            goto end;
+        }
         *val = hal->radio->fm_st_rsp.station_rsp.station_freq;
         break;
     case HCI_FM_HELIUM_UPPER_BAND:
-        if (!val)
+        if (!val) {
             ret = -FM_HC_STATUS_NULL_POINTER;
+            goto end;
+        }
         *val = hal->radio->recv_conf.band_high_limit;
         break;
     case HCI_FM_HELIUM_LOWER_BAND:
-        if (!val)
+        if (!val) {
             ret = -FM_HC_STATUS_NULL_POINTER;
+            goto end;
+        }
         *val = hal->radio->recv_conf.band_low_limit;
         break;
     case HCI_FM_HELIUM_AUDIO_MUTE:
-        if (!val)
+        if (!val) {
             ret = -FM_HC_STATUS_NULL_POINTER;
+            goto end;
+        }
         *val = hal->radio->mute_mode.hard_mute;
         break;
     case HCI_FM_HELIUM_SINR_SAMPLES: