From: David Reiss <dreiss@apache.org>
Date: Fri, 9 May 2008 07:14:10 +0000 (+0000)
Subject: C++/TFramedTransport: Don't write an empty frame on flush.
X-Git-Tag: 0.2.0~791
X-Git-Url: https://source.supwisdom.com/gerrit/gitweb?a=commitdiff_plain;h=80097936bd1258f074a226396f9546d98d060964;p=common%2Fthrift.git

C++/TFramedTransport: Don't write an empty frame on flush.


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

diff --git a/lib/cpp/src/transport/TBufferTransports.cpp b/lib/cpp/src/transport/TBufferTransports.cpp
index 0deef1bf..b26fd3d3 100644
--- a/lib/cpp/src/transport/TBufferTransports.cpp
+++ b/lib/cpp/src/transport/TBufferTransports.cpp
@@ -230,8 +230,10 @@ void TFramedTransport::flush()  {
   sz_nbo = (int32_t)htonl((uint32_t)(sz_hbo));
   memcpy(wBuf_.get(), (uint8_t*)&sz_nbo, sizeof(sz_nbo));
 
-  // Write size and frame body.
-  transport_->write(wBuf_.get(), sizeof(sz_nbo)+sz_hbo);
+  if (sz_hbo > 0) {
+    // Write size and frame body.
+    transport_->write(wBuf_.get(), sizeof(sz_nbo)+sz_hbo);
+  }
 
   // Reset our pointers.
   wBase_ = wBuf_.get();