[PATCH 2/9] Free channels in RedsState destructor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



RedsState is only freed at application exit, so this leak didn't matter
much in practice.
---
 server/reds.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/server/reds.c b/server/reds.c
index c03ed18..c2ac4d9 100644
--- a/server/reds.c
+++ b/server/reds.c
@@ -3681,8 +3681,12 @@ SPICE_GNUC_VISIBLE void spice_server_destroy(SpiceServer *reds)
     free(reds->config);
     if (reds->main_channel) {
         main_channel_close(reds->main_channel);
+        g_object_unref(reds->main_channel);
     }
 
+    if (reds->inputs_channel)
+        g_object_unref(reds->inputs_channel);
+
 #ifdef RED_STATISTICS
     if (reds->stat_shm_name) {
         shm_unlink(reds->stat_shm_name);
-- 
2.7.4

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]