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

Posted to dev@bookkeeper.apache.org by "Sijie Guo (Updated) (JIRA)" <ji...@apache.org> on 2011/11/04 18:21:51 UTC

[jira] [Updated] (BOOKKEEPER-107) memory leak in HostAddress of hedwig c++ client

     [ https://issues.apache.org/jira/browse/BOOKKEEPER-107?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sijie Guo updated BOOKKEEPER-107:
---------------------------------

    Attachment: BOOKKEEPER-107.patch

attach a patch:
1) use freeaddrinfo instead of free
2) remove socket_addr from HostAddress, since we just need to keep ip and port.
3) in clientimpl, it is not necessary to parse address from string each time creating channel. just parse it once when clientimpl initializes and keep it in variable defaultHost which could be used directly when creating channels.
                
> memory leak in HostAddress of hedwig c++ client
> -----------------------------------------------
>
>                 Key: BOOKKEEPER-107
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-107
>             Project: Bookkeeper
>          Issue Type: Bug
>          Components: hedwig-client
>    Affects Versions: 3.4.0
>            Reporter: Sijie Guo
>             Fix For: 4.0.0
>
>         Attachments: BOOKKEEPER-107.patch
>
>
> should use freeaddrinfo to free struct addrinfo, instead of using free directly.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira