[thrift] branch master updated: THRIFT-5122: Fix memory leak on accept error in thrift_simple_server_serve() Client: c_glib Patch: wangyunjian

[je...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jensg pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/thrift.git


The following commit(s) were added to refs/heads/master by this push:
     new b1628cf  THRIFT-5122: Fix memory leak on accept error in thrift_simple_server_serve() Client: c_glib Patch: wangyunjian
b1628cf is described below

commit b1628cf8bcbe6c87b117004beb3de6316417739c
Author: wangyunjian <wa...@huawei.com>
AuthorDate: Sun Mar 1 16:27:16 2020 +0800

    THRIFT-5122: Fix memory leak on accept error in thrift_simple_server_serve()
    Client: c_glib
    Patch: wangyunjian
    
    This closes #2039
    
    Signed-off-by: wangyunjian <wa...@huawei.com>
---
 lib/c_glib/src/thrift/c_glib/server/thrift_simple_server.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/lib/c_glib/src/thrift/c_glib/server/thrift_simple_server.c b/lib/c_glib/src/thrift/c_glib/server/thrift_simple_server.c
index 53e2508..0c89459 100644
--- a/lib/c_glib/src/thrift/c_glib/server/thrift_simple_server.c
+++ b/lib/c_glib/src/thrift/c_glib/server/thrift_simple_server.c
@@ -83,7 +83,10 @@ thrift_simple_server_serve (ThriftServer *server, GError **error)
         g_object_unref (input_protocol);
         g_object_unref (output_protocol);
       }
-
+      if ((*error) != NULL) {
+        g_message ("thrift_simple_server_serve : %s", (*error)->message);
+        g_clear_error (error);
+      }
       if (t != NULL)
       {
         g_object_unref (t);