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/TCompactProtocol.cs

diff --git a/lib/netstd/Thrift/Protocol/TCompactProtocol.cs b/lib/netstd/Thrift/Protocol/TCompactProtocol.cs
index 1fd7e50..5ca4841 100644
--- a/lib/netstd/Thrift/Protocol/TCompactProtocol.cs
+++ b/lib/netstd/Thrift/Protocol/TCompactProtocol.cs
@@ -816,8 +816,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.Double: return 8;  // uses fixedLongToBytes() which always writes 8 bytes
                 case TType.Byte: return sizeof(byte);
@@ -825,7 +825,7 @@
                 case TType.I32: return sizeof(byte);  // zigzag
                 case TType.I64: return sizeof(byte);  // zigzag
                 case TType.String: return sizeof(byte);  // 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(byte);  // element count
                 case TType.Set: return sizeof(byte);  // element count
                 case TType.List: return sizeof(byte);  // element count