[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Andreas Grünbacher
Subject: Re: [Acl-devel] [PATCH] Fix minor issues found in getfacl.c
Date: Tue, 8 Sep 2015 14:30:15 +0200

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.


> 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?


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

reply via email to

[Prev in Thread] Current Thread [Next in Thread]