[PATCH] lib: fwts_log: free log_file on error exit
Colin Ian King
colin.king at canonical.com
Wed May 8 06:43:55 UTC 2013
On 08/05/13 03:10, Alex Hung wrote:
> On 05/07/2013 03:03 PM, Colin King wrote:
>> From: Colin Ian King <colin.king at canonical.com>
>>
>> overity CID#997342: Resource leak
>>
>> log_file needs to be free'd on log_file->fp == NULL error
>> condition.
>>
>> Signed-off-by: Colin Ian King <colin.king at canonical.com>
>> ---
>> src/lib/src/fwts_log.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/src/lib/src/fwts_log.c b/src/lib/src/fwts_log.c
>> index 4667a76..41214da 100644
>> --- a/src/lib/src/fwts_log.c
>> +++ b/src/lib/src/fwts_log.c
>> @@ -674,6 +674,7 @@ fwts_log *fwts_log_open(
>>
>> if (log_file->fp == NULL) {
>> fwts_log_close(newlog);
>> + free(log_file);
>> return NULL;
>> }
>> }
>>
>
> The patch looks good. Is it also needed to free(log_file) in the below
> if-statement just a little above?
>
> if ((newname = fwts_log_filename(filename, mask)) == NULL) {
> fwts_log_close(newlog);
> return NULL;
> }
>
>
Well spotted. I will submit version 2 of the patch including this. Thanks.
More information about the fwts-devel
mailing list