[THRIFT-5383] TJSONProtocol Java readString throws on bounds check

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.14.0, 0.14.1
• Fix Version/s: 0.15.0, 0.14.2
• Component/s: Java - Library
• Assignee: Aaron St. George
• Reporter: Aaron St. George
• Created: 29/Mar/21 15:27
• Updated: 11/Sep/21 12:30
• Resolved: 30/Mar/21 19:36

Description

https://github.com/apache/thrift/blob/7e537505f014d8ede86f30a7500136c2f3e99505/lib/java/src/org/apache/thrift/protocol/TJSONProtocol.java#L976 this line appears to be unnecessary. The following code works with thrift 0.13.0 and fails with an exception on 0.14.x 

TMemoryBuffer buf = new TMemoryBuffer(11); TMemoryBuffer buf = new TMemoryBuffer(11);
TJSONProtocol prot = new TJSONProtocol(buf);
prot.writeString("yeehaw");
prot.readString();

Issue Links

• is duplicated by: THRIFT-5424 Cut release 0.14.2
• links to: GitHub Pull Request #2366