commit 845a87a8158e29ca4b3f176ab8b335a9302db300
author Hasnain Lakhani <m.hasnain.lakhani@gmail.com> Tue May 27 22:31:42 2025 -0700
committer Jens Geyer <Jens-G@users.noreply.github.com> Fri Jun 06 21:37:35 2025 +0200
tree 35685858fb2f7529706805fa56e1c41892c2dabf
parent 2f8347c0a59e9c4ee3276c1618a6130bf3c5d15b

[THRIFT-5871] Improve MAX_MESSAGE_SIZE_CHECK and friends

lib/netstd/Thrift/Protocol/TBinaryProtocol.cs

diff --git a/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs b/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
index 44fa9f7..fc5be9a 100644
--- a/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
+++ b/lib/netstd/Thrift/Protocol/TBinaryProtocol.cs
@@ -452,8 +452,8 @@
         {
             switch (type)
             {
-                case TType.Stop: return 0;
-                case TType.Void: return 0;
+                case TType.Stop: return 1;  // T_STOP needs to count itself
+                case TType.Void: return 1;  // T_VOID needs to count itself
                 case TType.Bool: return sizeof(byte);
                 case TType.Byte: return sizeof(byte);
                 case TType.Double: return sizeof(double);
@@ -461,7 +461,7 @@
                 case TType.I32: return sizeof(int);
                 case TType.I64: return sizeof(long);
                 case TType.String: return sizeof(int);  // string length
-                case TType.Struct: return 0;  // empty struct
+                case TType.Struct: return 1;  // empty struct needs at least 1 byte for the T_STOP
                 case TType.Map: return sizeof(int);  // element count
                 case TType.Set: return sizeof(int);  // element count
                 case TType.List: return sizeof(int);  // element count