st-hal: Fix potential NULL pointer dereference
In function update_hw_config_on_start, pointer
conf_levels could be used without allocating
memory or assigning valid address. This could
cause a potential NULL pointer dereference.
Check conf_levels against NULL before using it.
Change-Id: I2bd3257a7dddaaa2ac2ca869b269c069ff655af3
diff --git a/st_session.c b/st_session.c
index 1a9b630..b72e277 100644
--- a/st_session.c
+++ b/st_session.c
@@ -2603,6 +2603,12 @@
if (v_info->merge_fs_soundmodels) {
/* merge_fs_soundmodels is true only for QC SVA UUID */
+ if (conf_levels == NULL) {
+ ALOGE("%s: Unexpected, conf_levels pointer is NULL",
+ __func__);
+ status = -EINVAL;
+ return status;
+ }
/*
* Note:
* For ADSP case, the generated conf levles size must be equal to
@@ -2706,6 +2712,13 @@
} else {
+ if (conf_levels == NULL) {
+ ALOGE("%s: Unexpected, conf_levels pointer is NULL",
+ __func__);
+ status = -EINVAL;
+ return status;
+ }
+
if (!st_ses->lab_enabled && enable_lab)
st_ses->lab_enabled = true;