commit a6ec0d950f5f1b04382444dc9f2a07930021e6bd
author Pallavi <pallavim@codeaurora.org> Thu Jul 13 17:32:22 2017 +0530
committer Garmond Leung <garmondl@codeaurora.org> Tue Sep 05 11:13:30 2017 -0700
tree fce6f915c667aef82f05f9f5b8c3849939900215
parent 6c4d381b49edc6558f550ad60031dc1766db0fb5

hal: fix null pointer references and data initialization errors

  - add null pointer check and initialize uninitialized variables

CRs-Fixed: 2075786
Change-Id: I12f53d12c2e7c36155b1abd10f5a39fb0318da28

qahw_api/test/trans_loopback_test.c

diff --git a/qahw_api/test/trans_loopback_test.c b/qahw_api/test/trans_loopback_test.c
index a0673de..009e288 100644
--- a/qahw_api/test/trans_loopback_test.c
+++ b/qahw_api/test/trans_loopback_test.c
@@ -237,6 +237,8 @@
 {
     int rc=0;
     tlb_hdmi_config_t hdmi_config = {0};
+    transcode_loopback_config_t *transcode_loopback_config = &g_trnscode_loopback_config;
+    transcode_loopback_config->source_port_config.source_port_state = SOURCE_PORT_INACTIVE;
     switch(source_port_type)
     {
         case SOURCE_PORT_HDMI :
@@ -491,8 +493,9 @@
                     }
                     j++;
                 }
-                fprintf(log_file,"devpath = %s, switch_name = %s \n",
-                                         dev_path, switch_name);
+
+                if ((dev_path != NULL) && (switch_name != NULL)) 
+                    fprintf(log_file,"devpath = %s, switch_name = %s \n",dev_path, switch_name);
 
                 if((DEV_NODE_CHECK(tlb_hdmi_in_audio_dev_path, dev_path) == 0)  || (DEV_NODE_CHECK(tlb_hdmi_in_audio_sample_rate_dev_path, dev_path) == 0)
                 || (DEV_NODE_CHECK(tlb_hdmi_in_audio_state_dev_path, dev_path) == 0)