acl_from_text leaking memory
volker at vwsoft.com
volker at vwsoft.com
Mon Nov 16 22:01:30 UTC 2009
On 11/16/09 22:21, Jim Wilcoxson wrote:
> The man page is correct and should not be changed.
>
> In the example program I submitted, it does call acl_free; this is not
> where the leak occurs. The leak occurs because of a temporary string
> that acl_from_text allocates to parse the text.
>
> Jim
>
> On 11/16/09, Gary Jennejohn <gary.jennejohn at freenet.de> wrote:
>> On Mon, 16 Nov 2009 21:12:47 +0100
>> volker at vwsoft.com wrote:
>>
>>> you may want to have a look at the manpage acl_from_text(3):
>>>
>>> "...This function may cause memory to be allocated. The caller should
>>> free any releasable memory, when the new ACL is no longer required, by
>>> calling acl_free(3) with the (void *)acl_t as an argument."
>>>
>>> Please use an acl_free(void *obj_p) call afterwards to avoid leaking
>>> memory.
>>>
>> The suggested fix was appplied to HEAD today. Apparently, the man page
>> should
>> now be updated.
>>
>> ---
>> Gary Jennejohn
>>
>
Yes, I see and c199317 fixed that leak correctly. Jim is right - the
manpage still should not be changed as the caller is still responsible
for free'ing allocated memory.
Volker
More information about the freebsd-hackers
mailing list