You are viewing a plain text version of this content. The canonical link for it is here.
Posted to c-dev@axis.apache.org by "Alex Mantaut (JIRA)" <ji...@apache.org> on 2013/04/16 16:05:16 UTC

[jira] [Commented] (AXIS2C-1632) Patch to fix Stability issues and memory leaks on hash keys

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

Alex Mantaut commented on AXIS2C-1632:
--------------------------------------

need to create a new interface to keep backward compatibility
                
> Patch to fix Stability issues and memory leaks on hash keys
> -----------------------------------------------------------
>
>                 Key: AXIS2C-1632
>                 URL: https://issues.apache.org/jira/browse/AXIS2C-1632
>             Project: Axis2-C
>          Issue Type: Bug
>          Components: core/engine
>            Reporter: Alex Mantaut
>              Labels: patch
>         Attachments: axis2c-1632.patch
>
>
> Hash seems to keep a pointer to the keys of the elements it stores.Right now that memory is managed by the caller of the method... In some cases it causes a memory leak, but as the key memory is external to the hash it is a potential segfault (I think this could be related to other existing segfaults which are difficult to identify)
> I proposed a patch to create a copy of the key for the hash to use (the hash releases that memory automatically when freed)...
> I looked trough all the code to correct the calls to axutil_hash_set() in order to unify the memory management, and runned all the samples to check it worked correctly (I know hash it's used in a lot of places)
> Please review the patch to see if any changes need to be done on it...

--
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

---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@axis.apache.org
For additional commands, e-mail: c-dev-help@axis.apache.org