[THRIFT-3378] c_glib service does not handle negative Thrift byte values correctly

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: C glib - Compiler
• Patch Info: Patch Available
• Assignee: Nobuaki Sukegawa
• Reporter: Nobuaki Sukegawa
• Created: 10/Oct/15 01:47
• Updated: 13/Oct/15 22:53
• Resolved: 10/Oct/15 22:52

Description

There's a problematic type cast in generated c_glib service code that treat the sign bit of gint8 as normal bit (i.e. 128).
The problem is illustrated as follows.

int i = 0;
users_handler_func((*gint8)&i);  // Suppose this function tries to set i to -100
store_handler_result(i);         // This stores 156, not -100