[
https://issues.apache.org/jira/browse/AXIS2C-985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12568821#action_12568821
]
senakafdo edited comment on AXIS2C-985 at 2/13/08 11:05 PM:
------------------------------------------------------------------
Hi Dinesh,
There seems to be an issue in the provided patch. tok->start = start is wrong.
What if someone frees the parameters passed after calling this method. Won't we
lead into a dangling pointer situation. tok->start = strdup(start) is the
proper way i guess. And, that is what was happening with some of the replaced
code.
Regards,
Senaka
was (Author: senakafdo):
Hi Dinesh,
There seems to be an issue in the provided patch. tok->start = start is wrong.
What if someone frees the parameters passed after calling this method. Won't we
lead into a dangling pointer situation. tok->start = strdup(start) is the
proper way i guess. And, that is what was happening with some of the replaced
code. I think
Regards,
Senaka
> There's a leak due to wrong token create
> -----------------------------------------
>
> Key: AXIS2C-985
> URL: https://issues.apache.org/jira/browse/AXIS2C-985
> Project: Axis2-C
> Issue Type: Bug
> Affects Versions: Current (Nightly)
> Environment: Ubuntu 7.10
> Reporter: Lahiru Gunathilake
> Assignee: Dinesh Premalal
> Fix For: 1.3.0
>
> Attachments: diff_leak_fix.txt
>
>
> There's a wrong token creation in guththila_init method and it cause a
> 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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
