Re: [PATCH i-g-t v2 4/7] runner/settings: Fix code_coverage_script leak

[Peter Senna Tschudin <peter.senna@linux.intel.com>]

On 22.01.2025 12:37, Gustavo Sousa wrote:
> Quoting Lucas De Marchi (2025-01-21 19:57:30-03:00)
>> Make sure to free it when clearing settings.
>>
>> Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
> 
> Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Tested-by: Peter Senna Tschudin <peter.senna@linux.intel.com>
Reviewed-by: Peter Senna Tschudin <peter.senna@linux.intel.com>
> 
>> ---
>> runner/settings.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/runner/settings.c b/runner/settings.c
>> index 96377f1de..2787869ee 100644
>> --- a/runner/settings.c
>> +++ b/runner/settings.c
>> @@ -640,6 +640,7 @@ void clear_settings(struct settings *settings)
>>         free(settings->name);
>>         free(settings->test_root);
>>         free(settings->results_path);
>> +        free(settings->code_coverage_script);
>>
>>         free_regexes(&settings->include_regexes);
>>         free_regexes(&settings->exclude_regexes);
>> -- 
>> 2.48.0
>>