commit bf6571fbfb7361ec370e0905471ab2c83040a8fb
author juyuchen <juyuchen@google.com> Fri Nov 30 18:50:47 2018 +0800
committer juyuchen <juyuchen@google.com> Tue Dec 11 10:58:22 2018 +0800
tree fce83e6bad602fde931108792c6ad9773d0a5c54
parent e880e0c92622543ee4066312471d90adb4b468b4

audio: fix memory corruption.

Race condition:

There is a race condition between audio primary hal thread and soundtrigger
hal thread when they want to get snd_card number.

Memory leakage:

my_data.kvpairs in platforom_info_init function doesn't be destroyed
when use str_parms_create().

Bug: 119328981
Test: manual audio test

Change-Id: Ib86b3cf986a7213007bf9f073ef714a02b28a111
Signed-off-by: juyuchen <juyuchen@google.com>

hal/msm8916/platform.c

diff --git a/hal/msm8916/platform.c b/hal/msm8916/platform.c
index 2bf552c..2707c89 100644
--- a/hal/msm8916/platform.c
+++ b/hal/msm8916/platform.c
@@ -1132,7 +1132,8 @@
     /* Initialize ACDB and PCM ID's */
     strlcpy(platform_info_path, PLATFORM_INFO_XML_PATH, MAX_MIXER_XML_PATH);
     resolve_config_file(platform_info_path);
-    platform_info_init(platform_info_path, my_data);
+    platform_info_init(platform_info_path, my_data,
+                       true, &platform_set_parameters);
 
     my_data->acdb_handle = dlopen(LIB_ACDB_LOADER, RTLD_NOW);
     if (my_data->acdb_handle == NULL) {