commit a5ba5f23a7facbacefbef5a9541a5a35955570e3
author Dhananjay Kumar <dhakumar@codeaurora.org> Wed Nov 30 16:01:29 2016 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Fri Jan 06 02:22:14 2017 -0800
tree 80cd539393e05acf750a5d6c0ad25dbfb418a4f0
parent 767f9bba3e7bb91965f192afa90f55f01b171e25

hal: close mixer at platform deinit

Close mixer opened by platform during deinit to avoid fd leak.
This mixer fd can be accumulated in a prcocess if hal is loaded-unloaded
several times.

CRs-Fixed: 1095695
Change-Id: Ib5a6f0628f178f897cd67d80f32de7e3515aa9f0

hal/msm8916/platform.c

diff --git a/hal/msm8916/platform.c b/hal/msm8916/platform.c
index fd591a7..8902f63 100644
--- a/hal/msm8916/platform.c
+++ b/hal/msm8916/platform.c
@@ -2212,6 +2212,11 @@
         my_data->edid_info = NULL;
     }
 
+    if (my_data->adev->mixer) {
+        mixer_close(my_data->adev->mixer);
+        my_data->adev->mixer = NULL;
+    }
+
     free(platform);
     /* deinit usb */
     audio_extn_usb_deinit();

hal/msm8974/platform.c

diff --git a/hal/msm8974/platform.c b/hal/msm8974/platform.c
index df373b6..c0d3ebe 100644
--- a/hal/msm8974/platform.c
+++ b/hal/msm8974/platform.c
@@ -1935,6 +1935,11 @@
         my_data->edid_info = NULL;
     }
 
+    if (my_data->adev->mixer) {
+        mixer_close(my_data->adev->mixer);
+        my_data->adev->mixer = NULL;
+    }
+
     free(platform);
     /* deinit usb */
     audio_extn_usb_deinit();