You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@zookeeper.apache.org by "Raul Gutierrez Segales (JIRA)" <ji...@apache.org> on 2014/11/04 18:37:34 UTC

[jira] [Commented] (ZOOKEEPER-2073) Memory leak on zookeeper_close

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

Raul Gutierrez Segales commented on ZOOKEEPER-2073:
---------------------------------------------------

Thanks for the patch [~dgosselin]. I see there's 4 places where bptr needs to be free inside zookeeper_process(), maybe we should try to unify that...

> Memory leak on zookeeper_close
> ------------------------------
>
>                 Key: ZOOKEEPER-2073
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2073
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.6.0
>            Reporter: Dave Gosselin
>            Priority: Critical
>             Fix For: 3.6.0
>
>         Attachments: ZOOKEEPER-2073.patch
>
>
> When running valgrind against a zookeeper client using the C API, I noticed an occasional memory leak on zookeeper_close.  I traced the issue to a regression added by fix ZOOKEEPER-804.  The attached patch fixes the regression and the associated memory leak.



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