commit 6746b7ab2c7953ad4b8b7fa0a2654c8b77fed374
author Preetam Singh Ranawat <apranawat@codeaurora.org> Mon Jul 18 16:40:24 2016 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Tue Jul 26 23:48:22 2016 -0700
tree d9873b1622392cb9784b83a3030b0f7b2189a35d
parent 303ff1ab7f61403ce3e43627862a29d2688685e2

hal : Fix input validation errors

 fix miscellaneous errors and add null check
 to avoid following errors
 673141, 673142, 673153, 673154, 673155
 1944609, 1944610

Change-Id: Ica531084375af8404472566a0a664fd0b7f41dd7

hal/msm8916/platform.c

diff --git a/hal/msm8916/platform.c b/hal/msm8916/platform.c
index 82f1934..64475a3 100644
--- a/hal/msm8916/platform.c
+++ b/hal/msm8916/platform.c
@@ -1456,10 +1456,12 @@
     int result;
     char value[PROPERTY_VALUE_MAX];
     cvd_version = calloc(1, MAX_CVD_VERSION_STRING_SIZE);
-    if (!cvd_version)
+    if (!cvd_version) {
         ALOGE("Failed to allocate cvd version");
-    else
+        return -1;
+    } else {
         get_cvd_version(cvd_version, my_data->adev);
+    }
 
     property_get("audio.ds1.metainfo.key",value,"0");
     key = atoi(value);
@@ -2339,7 +2341,7 @@
     struct listnode *node;
     int mode = NATIVE_AUDIO_MODE_INVALID;
 
-    if (!value)
+    if (!value || !parms)
         return ret;
 
     ret = str_parms_get_str(parms, AUDIO_PARAMETER_KEY_NATIVE_AUDIO_MODE,