tizenaudio-sink/source: Fix resource leak of pcm handle

[Version] 5.0.62
[Profile] Common
[Issue Type] Resource leak

Signed-off-by: KimJeongYeon <jeongyeon.kim@samsung.com>
Change-Id: Ieab7b58bacf8e7d8707df20c11db60046d7587b6

diff --git a/packaging/pulseaudio-modules-tizen.spec b/packaging/pulseaudio-modules-tizen.spec
index b88166c..f38fd4d 100644 (file)
--- a/packaging/pulseaudio-modules-tizen.spec
+++ b/packaging/pulseaudio-modules-tizen.spec
@@ -1,6 +1,6 @@
 Name:             pulseaudio-modules-tizen
 Summary:          Pulseaudio modules for Tizen
-Version:          5.0.61
+Version:          5.0.62
 Release:          0
 Group:            Multimedia/Audio
 License:          LGPL-2.1+

diff --git a/src/module-tizenaudio-sink.c b/src/module-tizenaudio-sink.c
index 60105d1..99ae7f6 100644 (file)
--- a/src/module-tizenaudio-sink.c
+++ b/src/module-tizenaudio-sink.c
@@ -589,5 +589,10 @@ void pa__done(pa_module*m) {
     if (u->rtpoll)
         pa_rtpoll_free(u->rtpoll);
 
+    if (u->pcm_handle) {
+        pa_hal_interface_pcm_stop(u->hal_interface, u->pcm_handle);
+        pa_hal_interface_pcm_close(u->hal_interface, u->pcm_handle);
+    }
+
     pa_xfree(u);
 }

diff --git a/src/module-tizenaudio-source.c b/src/module-tizenaudio-source.c
index 79a4664..0578cc0 100644 (file)
--- a/src/module-tizenaudio-source.c
+++ b/src/module-tizenaudio-source.c
@@ -538,5 +538,10 @@ void pa__done(pa_module*m) {
     if (u->rtpoll)
         pa_rtpoll_free(u->rtpoll);
 
+    if (u->pcm_handle) {
+        pa_hal_interface_pcm_stop(u->hal_interface, u->pcm_handle);
+        pa_hal_interface_pcm_close(u->hal_interface, u->pcm_handle);
+    }
+
     pa_xfree(u);
 }