commit a8b82c2db5c12d9fc5662ac902898ae2d8acc1da
author Lingfeng Yang <lfy@google.com> Wed Jul 17 17:26:21 2019 -0700
committer Alistair Delva <adelva@google.com> Wed Dec 11 10:25:06 2019 -0800
tree 531bcf173558dad582c2b6282dd943955abd88dc
parent c36abe48f1e444fc5734405a33feb99064c69633

Fix memory leak in strdup(str_parms_to_str(..))

str_parms_to_str already uses asprintf which allocates;
no need to strdup, or we leak.

bug: 36655968
Change-Id: Iadd4b06f21823be8e4fa96e9b95f8ca2ee4c4c86

guest/hals/audio/audio_hw.c

diff --git a/guest/hals/audio/audio_hw.c b/guest/hals/audio/audio_hw.c
index f0979a9..40396d7 100644
--- a/guest/hals/audio/audio_hw.c
+++ b/guest/hals/audio/audio_hw.c
@@ -377,7 +377,7 @@
     }
 
     if (get) {
-        str = strdup(str_parms_to_str(reply));
+        str = str_parms_to_str(reply);
     }
     else {
         ALOGD("%s Unsupported paramter: %s", __FUNCTION__, keys);
@@ -906,7 +906,7 @@
     }
 
     if (get) {
-        str = strdup(str_parms_to_str(reply));
+        str = str_parms_to_str(reply);
     }
     else {
         ALOGD("%s Unsupported paramter: %s", __FUNCTION__, keys);