You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "James E. King, III (JIRA)" <ji...@apache.org> on 2016/07/21 04:41:20 UTC

[jira] [Commented] (THRIFT-3885) PHP: Error when readI64 in TCompactProtocol

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

James E. King, III commented on THRIFT-3885:
--------------------------------------------

Does the bug end up causing us to disable some "make cross" tests for PHP?  If so, if we fix the defect we should attempt to re-enable as many php tests as we can to see if they start working?

> PHP: Error when readI64 in TCompactProtocol
> -------------------------------------------
>
>                 Key: THRIFT-3885
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3885
>             Project: Thrift
>          Issue Type: Bug
>          Components: PHP - Library
>    Affects Versions: 0.7, 0.8, 0.9, 0.9.1, 0.9.2, 0.9.3
>         Environment: all
>            Reporter: Chen Zhiming
>              Labels: patch
>
> In line 618 of readI64 of TCompactProtocol, the number of shift bit is error. It must be
> $hi |= (($byte & 0x7f) >> (7 - ($shift - 25)));



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