From: David Reiss <dreiss@apache.org>
Date: Wed, 11 Jun 2008 00:59:34 +0000 (+0000)
Subject: Fix thrift_buffered_transport leak in erl bindings
X-Git-Tag: 0.2.0~731
X-Git-Url: https://source.supwisdom.com/gerrit/gitweb?a=commitdiff_plain;h=a5a53dbfcc22df3e50aa419c5e317d609c90dc9e;p=common%2Fthrift.git

Fix thrift_buffered_transport leak in erl bindings

Summary:
  - Was leaking processes when the client cleanly disconnected since the
    thrift_processor process was exiting with 'normal' Reason. Changed it
    to 'protocol_closed'

Test plan:
  - Telnet to thrift service port and close (^]close) before the read timeout
    elapses. Verify that erlang:system_info(process_count) does not increase
    after doing this multiple times.


git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@666430 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/lib/alterl/src/thrift_processor.erl b/lib/alterl/src/thrift_processor.erl
index 2d2fe372..8f323523 100644
--- a/lib/alterl/src/thrift_processor.erl
+++ b/lib/alterl/src/thrift_processor.erl
@@ -33,7 +33,7 @@ loop(State = #state{in_protocol = IProto,
             loop(State);
         {error, closed} ->
             % error_logger:info_msg("Client disconnected~n"),
-            ok
+            exit(protocol_closed)
     end.
 
 handle_function(State = #state{in_protocol = IProto,