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