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

Marshall McMullen commented on ZOOKEEPER-1636:
----------------------------------------------

[~thawan] This patch looks really good to me. In our environment where I 
implemented and tested the original multi op code we have an obscenely large 
jute buffer size so I never would have hit this bug. Really good find. Patch 
looks good to me and the unit test is an awesome addition.

+1 for me.
                
> c-client crash when zoo_amulti failed 
> --------------------------------------
>
>                 Key: ZOOKEEPER-1636
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1636
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.4.3
>            Reporter: Thawan Kooburat
>            Assignee: Thawan Kooburat
>            Priority: Critical
>         Attachments: ZOOKEEPER-1636.patch, ZOOKEEPER-1636.patch
>
>
> deserialize_response for multi operation don't handle the case where the 
> server fail to send back response. (Eg. when multi packet is too large) 
> c-client will try to process completion of all sub-request as if the 
> operation is successful and will eventually cause SIGSEGV

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to