commit f79031e87531464b84d911acf9aee57cac8b83d8
author David Reiss <dreiss@apache.org> Fri Jul 06 21:43:48 2007 +0000
committer David Reiss <dreiss@apache.org> Fri Jul 06 21:43:48 2007 +0000
tree f24a3e9989a757cf75e15eef59d43bbc24628628
parent 58dfb4fbe82c346814c2e4f33b6045468bfd68c6

Fixed possible Thrift serialization bug

Summary: TBinaryProtocol.cpp was making assumptions about evaluation order
that are not guaranteed by the standard.  Now it does not (at least in these
functions).

Trac Bug: #

Blame Rev:

Reviewed By: mcslee

Test Plan: still compiles.

Revert Plan: svn revert

Notes:

EImportant:


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

lib/cpp/src/protocol/TBinaryProtocol.cpp

diff --git a/lib/cpp/src/protocol/TBinaryProtocol.cpp b/lib/cpp/src/protocol/TBinaryProtocol.cpp
index 2645601..621e5dc 100644
--- a/lib/cpp/src/protocol/TBinaryProtocol.cpp
+++ b/lib/cpp/src/protocol/TBinaryProtocol.cpp
@@ -57,15 +57,17 @@
                                             const int32_t seqid) {
   if (strict_write_) {
     int32_t version = (VERSION_1) | ((int32_t)messageType);
-    return
-      writeI32(version) +
-      writeString(name) +
-      writeI32(seqid);
+    uint32_t wsize = 0;
+    wsize += writeI32(version);
+    wsize += writeString(name);
+    wsize += writeI32(seqid);
+    return wsize;
   } else {
-    return 
-      writeString(name) + 
-      writeByte((int8_t)messageType) +
-      writeI32(seqid);
+    uint32_t wsize = 0;
+    wsize += writeString(name);
+    wsize += writeByte((int8_t)messageType);
+    wsize += writeI32(seqid);
+    return wsize;
   }
 }
 
@@ -84,9 +86,10 @@
 uint32_t TBinaryProtocol::writeFieldBegin(const string& name,
                                           const TType fieldType,
                                           const int16_t fieldId) {
-  return
-    writeByte((int8_t)fieldType) +
-    writeI16(fieldId);
+  uint32_t wsize = 0;
+  wsize += writeByte((int8_t)fieldType);
+  wsize += writeI16(fieldId);
+  return wsize;
 }
 
 uint32_t TBinaryProtocol::writeFieldEnd() {
@@ -101,10 +104,11 @@
 uint32_t TBinaryProtocol::writeMapBegin(const TType keyType,
                                         const TType valType,
                                         const uint32_t size) {
-  return
-    writeByte((int8_t)keyType) +
-    writeByte((int8_t)valType) +
-    writeI32((int32_t)size);
+  uint32_t wsize = 0;
+  wsize += writeByte((int8_t)keyType);
+  wsize += writeByte((int8_t)valType);
+  wsize += writeI32((int32_t)size);
+  return wsize;
 }
 
 uint32_t TBinaryProtocol::writeMapEnd() {
@@ -113,9 +117,10 @@
 
 uint32_t TBinaryProtocol::writeListBegin(const TType elemType,
                                          const uint32_t size) {
-  return
-    writeByte((int8_t) elemType) +
-    writeI32((int32_t)size);
+  uint32_t wsize = 0;
+  wsize += writeByte((int8_t) elemType);
+  wsize += writeI32((int32_t)size);
+  return wsize;
 }
 
 uint32_t TBinaryProtocol::writeListEnd() {
@@ -124,9 +129,11 @@
 
 uint32_t TBinaryProtocol::writeSetBegin(const TType elemType,
                                         const uint32_t size) {
+  uint32_t wsize = 0;
   return
-    writeByte((int8_t)elemType) +
-    writeI32((int32_t)size);
+  wsize += writeByte((int8_t)elemType);
+  wsize += writeI32((int32_t)size);
+  return wsize;
 }
 
 uint32_t TBinaryProtocol::writeSetEnd() {