commit f58a7f6147d87158de8b7857907f54bb87b333eb
author Dong Jinguang <dongjinguang@huawei.com> Thu Dec 07 21:09:43 2017 +0800
committer Dong Jinguang <dongjinguang@huawei.com> Fri Dec 08 09:25:17 2017 +0800
tree 1f0a66bd566b25eaeb6da18082f6e7cc4e9f05d5
parent dd5e6e374ce2b5d46edc2fa0059f874595a883ff

tinyalsa: check NULL for config

pcm_open() was not checking config and crashed memory leak

Test: built and ran on device without errors
Change-Id: Icdd6271b0c5d6107b8bfc043f26a63b2829712b3

pcm.c

diff --git a/pcm.c b/pcm.c
index 120d182..4ae321b 100644
--- a/pcm.c
+++ b/pcm.c
@@ -882,8 +882,11 @@
     char fn[256];
     int rc;
 
+    if (!config) {
+        return &bad_pcm; /* TODO: could support default config here */
+    }
     pcm = calloc(1, sizeof(struct pcm));
-    if (!pcm || !config)
+    if (!pcm)
         return &bad_pcm; /* TODO: could support default config here */
 
     pcm->config = *config;