[ 
https://issues.apache.org/jira/browse/THRIFT-3276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14952116#comment-14952116
 ] 

Hudson commented on THRIFT-3276:
--------------------------------

SUCCESS: Integrated in Thrift #1683 (See 
[https://builds.apache.org/job/Thrift/1683/])
THRIFT-3276 Binary data does not decode correctly using the (roger: rev 
a175437f66fa1a0b36233e7dd40b061d471276ff)
* lib/javame/src/org/apache/thrift/protocol/TJSONProtocol.java
* lib/cpp/src/thrift/protocol/TJSONProtocol.cpp
* test/known_failures_Linux.json
* lib/java/src/org/apache/thrift/protocol/TJSONProtocol.java
* lib/cpp/src/thrift/protocol/TJSONProtocol.h
* lib/csharp/src/Protocol/TJSONProtocol.cs


> Binary data does not decode correctly using the TJSONProtocol when the base64 
> encoded data is padded.
> -----------------------------------------------------------------------------------------------------
>
>                 Key: THRIFT-3276
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3276
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library, Java - Library
>    Affects Versions: 0.9.1
>            Reporter: Steve Niemitz
>            Assignee: Nobuaki Sukegawa
>
> If the base64 encoded JSON string has padding bytes at the end, the resulting 
> binary data returned in the thrift object ends up with extra padding bytes 
> (0xFF) at the end.
> It seems like this is caused by the TJSONProtocol [using the padded 
> length|https://github.com/apache/thrift/blob/0.9.1/lib/java/src/org/apache/thrift/protocol/TJSONProtocol.java#L760]
>  when it tries to decode the last chunk, which results in the '=' being 
> converted into binary data.  The DECODE_TABLE defaults to -1 (0xFF), which is 
> how it gets into the final output.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to