commit 0a3f05b2f22d387180d6ebce6485f18007f8d4f0
author Sharad Sangle <assangle@codeaurora.org> Thu Jul 20 13:05:55 2017 +0530
committer Shiv Maliyappanahalli <smaliyap@codeaurora.org> Tue Aug 15 14:51:32 2017 -0700
tree 9a37db54c5b65ae242da6ff6da39ce273916b125
parent fcd412a67b31d4d4e0169e017f04e4ce2f0c67a8

hal: fix LLVM issues

Fix potential memory leak issue in add_new_sndcard
and monitor_thread_loop

Change-Id: I62b8b1457e27377f3ec9e05d2fbf566ab22358ed

hal/audio_extn/sndmonitor.c

diff --git a/hal/audio_extn/sndmonitor.c b/hal/audio_extn/sndmonitor.c
index dd7f92f..1860cad 100644
--- a/hal/audio_extn/sndmonitor.c
+++ b/hal/audio_extn/sndmonitor.c
@@ -153,9 +153,10 @@
 
     char *state = read_state(fd);
 
-    if (!state)
+    if (!state) {
+        free(s);
         return -1;
-
+    }
     bool online = state && !strcmp(state, "ONLINE");
 
     ALOGV("card %d initial state %s %d", card, state, online);
@@ -547,7 +548,8 @@
             ++i;
         }
     }
-
+    if (pfd)
+        free(pfd);
     return NULL;
 }