OrgACRN
/
sof
mirror of https://github.com/thesofproject/sof.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

logger: Fix uuid_dict memory leak 

config->uids_dict has never been freed before.
`out` label has been added to perform `uuid_dict`
free before function return.

Signed-off-by: Karol Trzcinski <karolx.trzcinski@linux.intel.com>
This commit is contained in:
Karol Trzcinski 2020-09-25 16:19:13 +02:00 committed by Liam Girdwood
parent 5b0b1a3db5
commit e3b0aa7103
1 changed files with 11 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
tools/logger/convert.c
View File

@ -853,19 +853,25 @@ int convert(struct convert_config *config)
config->ldc_fd);
if (!count) {
log_err("failed to read uuid section data.\n");
return -ferror(config->ldc_fd);
ret = -ferror(config->ldc_fd);
goto out;
}
if (config->dump_ldc)
return dump_ldc_info();
if (config->dump_ldc) {
ret = dump_ldc_info();
goto out;
}
if (config->filter_config) {
ret = filter_update_firmware();
if (ret) {
log_err("failed to apply trace filter, %d.\n", ret);
return ret;
goto out;
}
}
return logger_read();
ret = logger_read();
out:
free(config->uids_dict);
return ret;
}