Re: [PATCH] perf script: Fix possible memory leaks

From: Namhyung Kim
Date: Mon Aug 11 2014 - 03:45:47 EST


Hi Adrian,

On Fri, 01 Aug 2014 14:32:19 +0300, Adrian Hunter wrote:
> On 1/08/2014 11:24 a.m., Namhyung Kim wrote:
>> Some paths in perf script don't call perf_session__delete() after
>> creating a new session.
>>

[SNIP]
>> - perf_session__delete(session);
>> cleanup_scripting();
>> +out_delete:
>> + perf_session__delete(session);
>
> Some of the db-export scripting I do relies on the session
> being deleted before the script is stopped. I would prefer
> the original order i.e. perf_session__delete() then
> cleanup_scripting()

Okay, I'll fix it and send v2.

Thanks,
Namhyung
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/