[
https://issues.apache.org/jira/browse/THRIFT-5383?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17310997#comment-17310997
]
Jens Geyer commented on THRIFT-5383:
------------------------------------
> this line appears to be unnecessary.
I think "misplaced" would be the better word here. The check should be placed
before the read actually happens. Otherwise it makes no sense, it protects
nothing.
> TJSONProtocol Java readString throws on unnecessary bounds check
> -----------------------------------------------------------------
>
> Key: THRIFT-5383
> URL: https://issues.apache.org/jira/browse/THRIFT-5383
> Project: Thrift
> Issue Type: Bug
> Components: Java - Library
> Affects Versions: 0.14.0, 0.14.1
> Reporter: Aaron St. George
> Priority: Major
>
> [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();
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
