[THRIFT-3827] php CompactProtocol readI64 function has bug, when value has 32bit ~64bit, Example：value=1461563457000

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.9.3
• Fix Version/s: 0.10.0
• Component/s: PHP - Library

Environment

PHP 5.6.15 64bits

• Assignee: lnn1123
• Reporter: lnn1123
• Created: 16/May/16 05:25
• Updated: 28/Sep/16 13:28
• Resolved: 20/Aug/16 09:44

Description

when Varint(int64_val) more than five bytes parseing error,
parse the five byte should be $hi |= (($byte & 0x7f) >> ($shift - 24));
becouse the five byte high bits should be high four bits

Issue Links

• is duplicated by: THRIFT-3885 PHP: Error when readI64 in TCompactProtocol
• links to: GitHub Pull Request #1008