You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by jk...@apache.org on 2016/11/14 15:32:03 UTC

thrift git commit: THRIFT-3961 terminate the connection to the client if an exception occurs while processing a message Client: C++ Patch: Claudius Heine

Repository: thrift
Updated Branches:
  refs/heads/master c9877fb3b -> 7656793d0


THRIFT-3961 terminate the connection to the client if an exception occurs while processing a message
Client: C++
Patch: Claudius Heine <ch...@denx.de>

This closes #1125


Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/7656793d
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/7656793d
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/7656793d

Branch: refs/heads/master
Commit: 7656793d0de87bded5e4d8c91173b066f3f5daa8
Parents: c9877fb
Author: James E. King, III <jk...@apache.org>
Authored: Mon Nov 14 10:30:05 2016 -0500
Committer: James E. King, III <jk...@apache.org>
Committed: Mon Nov 14 10:30:05 2016 -0500

----------------------------------------------------------------------
 lib/cpp/src/thrift/server/TConnectedClient.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/thrift/blob/7656793d/lib/cpp/src/thrift/server/TConnectedClient.cpp
----------------------------------------------------------------------
diff --git a/lib/cpp/src/thrift/server/TConnectedClient.cpp b/lib/cpp/src/thrift/server/TConnectedClient.cpp
index 9583284..889c885 100644
--- a/lib/cpp/src/thrift/server/TConnectedClient.cpp
+++ b/lib/cpp/src/thrift/server/TConnectedClient.cpp
@@ -84,7 +84,8 @@ void TConnectedClient::run() {
     } catch (const TException& tex) {
       string errStr = string("TConnectedClient processing exception: ") + tex.what();
       GlobalOutput(errStr.c_str());
-      // Continue processing
+      // Disconnect from client, because we could not process the message.
+      done = true;
     }
   }