commit bd26d385693e4f172c348dc27f9d1e23288c50cf
author wangyunjian <wangyunjian@huawei.com> Sat Feb 15 08:29:59 2020 -0500
committer Jens Geyer <jensg@apache.org> Fri Feb 21 21:25:45 2020 +0100
tree 198cb3fa65ce742106f89f2a5132722646b3ae43
parent d8480c9ac14f133d272372530e33619b2a64ddf7

THRIFT-5094: Fix memory leak in thrift_server_set_property()
Client: c_glib
Patch: wangyunjian

This closes #2008

Signed-off-by: wangyunjian <wangyunjian@huawei.com>

lib/c_glib/src/thrift/c_glib/server/thrift_server.c

diff --git a/lib/c_glib/src/thrift/c_glib/server/thrift_server.c b/lib/c_glib/src/thrift/c_glib/server/thrift_server.c
index ccf9153..2109add 100644
--- a/lib/c_glib/src/thrift/c_glib/server/thrift_server.c
+++ b/lib/c_glib/src/thrift/c_glib/server/thrift_server.c
@@ -76,21 +76,27 @@
   switch (property_id)
   {
     case PROP_THRIFT_SERVER_PROCESSOR:
+      g_clear_object (&server->processor);
       server->processor = g_value_dup_object (value);
       break;
     case PROP_THRIFT_SERVER_SERVER_TRANSPORT:
+      g_clear_object (&server->server_transport);
       server->server_transport = g_value_dup_object (value);
       break;
     case PROP_THRIFT_SERVER_INPUT_TRANSPORT_FACTORY:
+      g_clear_object (&server->input_transport_factory);
       server->input_transport_factory = g_value_dup_object (value);
       break;
     case PROP_THRIFT_SERVER_OUTPUT_TRANSPORT_FACTORY:
+      g_clear_object (&server->output_transport_factory);
       server->output_transport_factory = g_value_dup_object (value);
       break;
     case PROP_THRIFT_SERVER_INPUT_PROTOCOL_FACTORY:
+      g_clear_object (&server->input_protocol_factory);
       server->input_protocol_factory = g_value_dup_object (value);
       break;
     case PROP_THRIFT_SERVER_OUTPUT_PROTOCOL_FACTORY:
+      g_clear_object (&server->output_protocol_factory);
       server->output_protocol_factory = g_value_dup_object (value);
       break;
   }