[
https://issues.apache.org/jira/browse/THRIFT-135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12632790#action_12632790
]
Bryan Duxbury commented on THRIFT-135:
--------------------------------------
How about we make a wrapper Set implementation that throws
IllegalArgumentException if you try to add null, and use that as the
implementation in generated structs?
> Nulls in set<string> throw an exception in Java
> -----------------------------------------------
>
> Key: THRIFT-135
> URL: https://issues.apache.org/jira/browse/THRIFT-135
> Project: Thrift
> Issue Type: Bug
> Components: Compiler (Java)
> Reporter: David Reiss
>
> From Amit Sudharshan:
> I recently noticed a bug(feature?) in
> com.facebook.thrift.protocol.TBinaryProtocol.writeString where if it is
> passed a null pointer it will throw NPE.
> Now, the autogenerated stub code tries to prevent this, however we recently
> came across a case where we had a Set<String> which contained a "NULL" (legal
> in java). Thrift tests to see if the set is non-null and implicitely whether
> it has any elements, both of these pass in this case, and so the null string
> is passed to the writeString method where we get the NPE.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
