Re: [Acl-devel] [PATCH] Fix minor issues found in getfacl.c

[Andreas Grünbacher]

Jaska and Kamil,

2015-09-08 14:26 GMT+02:00 Kamil Dudka <address@hidden>:
> On Tuesday 08 September 2015 14:37:56 Jaska Uimonen wrote:
>> Hi,
>>
>> I was assigned to fix some static analysis issues
>> in my company. There we're couple identified in acl.
>>
>> These are all in getfacl.c and are quite minor things,
>> but here they are anyway... they are mainly concerning
>> not freeing things returned from acl_get_identifier.

Thanks.

> Looks good to me.  I would personally prefer explicit type conversion when
> freeing id_p instead of changing its declaration

Or making the variable non-const in the first place. Also I would
prefer not to expand the life span of those temporary variables too
much. How about the attached fix?

Andreas

0001-getfacl-Fix-minor-resource-leak.patch
Description: Text Data