You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Jaesang Kim (JIRA)" <ji...@apache.org> on 2014/04/18 08:38:17 UTC

[jira] [Commented] (THRIFT-2478) Thrift c_glib binary protocol may go to infinite loop for bad input

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

Jaesang Kim commented on THRIFT-2478:
-------------------------------------

thrift_procotol.c is different from your patched code.
maybe missing some commit.

this is trunk code.
{code}
result += thrift_protocol_skip (protocol, ftype, error);
{code}

but your code is
{code}
ret = thrift_protocol_skip (protocol, ftype, error);
{code}


> Thrift c_glib binary protocol may go to infinite loop for bad input
> -------------------------------------------------------------------
>
>                 Key: THRIFT-2478
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2478
>             Project: Thrift
>          Issue Type: Bug
>          Components: C glib - Library
>    Affects Versions: 0.9.1
>            Reporter: Xiaodong Ma
>         Attachments: fix_c_glib_binary_protocol_data_fuzzing_cause_infinite_loop.diff
>
>
> Have the patch ready for review. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)