axis-c-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Senaka Fernando (JIRA)" <>
Subject [jira] Commented: (AXIS2C-873) Function axiom_document_build_all[Alastair FETTES]
Date Fri, 04 Jan 2008 05:04:34 GMT


Senaka Fernando commented on AXIS2C-873:

Hi Alastair & Lahiru,

+1 for pointing this out. We must document this 2 avoid possible leak. Parameters don't require
a mention however, only the return value matters.


> Function axiom_document_build_all[Alastair FETTES]
> --------------------------------------------------
>                 Key: AXIS2C-873
>                 URL:
>             Project: Axis2-C
>          Issue Type: Bug
>    Affects Versions: Current (Nightly)
>         Environment: Ubuntu 7.04
>            Reporter: Lahiru Gunathilake
>             Fix For: Current (Nightly)
> <snippet>
> axiom_node_t* axiom_document_build_all(
>       struct axiom_document*     document,
>       const axutil_env_t*        env)
> </snippet>
> Problem: The caller of this function is not given responsibility for the return value
of type axiom_node_t*, since (as of v1.1.0) the return value is free-ed by the function axiom_document_free.
If the caller calls the function axiom_document_free_self, the return value is NOT free-ed.
So a memory leak is possible here.
> Solution: Document memory ownership for parameters and return value.  Investigate the
use of free functions and pointers in regards to possible memory leak.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message