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

ASF GitHub Bot commented on PROTON-2199:
----------------------------------------

jiridanek opened a new pull request #244:
URL: https://github.com/apache/qpid-proton/pull/244


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> memory leak in c++ object inspect
> ---------------------------------
>
>                 Key: PROTON-2199
>                 URL: https://issues.apache.org/jira/browse/PROTON-2199
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: cpp-binding
>            Reporter: Jeff Eaton
>            Assignee: Jiri Daněk
>            Priority: Major
>              Labels: easyfix
>             Fix For: proton-c-0.32.0
>
>
> The {{pn_ptr_base::inspect()}} function in {{cpp/src/object.cpp}} calls 
> {{pn_string(NULL)}} on line 36 and never frees the allocated memory with 
> {{pn_free()}}.
> The fix is to create the std::string as a temporary variable, then free the 
> pn_string_t, then return the temporary std::string, like this:
> {{std::string tmp(pn_string_get(s));}}
> {{pn_free(s);}}
> {{return tmp;}}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to