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

ASF GitHub Bot commented on THRIFT-2817:
----------------------------------------

Github user bivas commented on the pull request:

    https://github.com/apache/thrift/pull/273#issuecomment-94269795
  
    We're still experiencing read errors with this fix merged (`Not enough 
frame size 4085 to read 4096 bytes`)
    
    The commit is missing a `return` statement in case of `framesize` smaller 
than the `buffer` and `err == nil ` looks strange
    
    
     


> TSimpleJSONProtocol reads beyond end of message
> -----------------------------------------------
>
>                 Key: THRIFT-2817
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2817
>             Project: Thrift
>          Issue Type: Bug
>          Components: Go - Library
>            Reporter: Chi Vinh Le
>            Assignee: Jens Geyer
>             Fix For: 0.9.3
>
>
> The TSimpleJSONProtocol reads beyond end of message, which prevent processing 
> the message until more data arrives.
> https://github.com/apache/thrift/pull/264
> EDIT: New pull request here:
> https://github.com/apache/thrift/pull/273



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

Reply via email to