[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/4] vl: Free accel_list in configure_accelerators
From: |
Laurent Vivier |
Subject: |
Re: [PATCH 2/4] vl: Free accel_list in configure_accelerators |
Date: |
Thu, 9 Jan 2020 09:24:08 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 |
On 09/01/2020 09:18, Thomas Huth wrote:
> On 09/01/2020 03.17, Richard Henderson wrote:
>> We allocate the list with g_strsplit, so free it too.
>> This freeing was lost during one of the rearrangements.
>>
>> Fixes: 6f6e1698a68c
>> Signed-off-by: Richard Henderson <address@hidden>
>> ---
>> vl.c | 4 +++-
>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>
>> diff --git a/vl.c b/vl.c
>> index be79b03c1a..c9329fe699 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -2748,7 +2748,6 @@ static int do_configure_accelerator(void *opaque,
>> QemuOpts *opts, Error **errp)
>> static void configure_accelerators(const char *progname)
>> {
>> const char *accel;
>> - char **accel_list, **tmp;
>> bool init_failed = false;
>>
>> qemu_opts_foreach(qemu_find_opts("icount"),
>> @@ -2756,6 +2755,8 @@ static void configure_accelerators(const char
>> *progname)
>>
>> accel = qemu_opt_get(qemu_get_machine_opts(), "accel");
>> if (QTAILQ_EMPTY(&qemu_accel_opts.head)) {
>> + char **accel_list, **tmp;
>> +
>> if (accel == NULL) {
>> /* Select the default accelerator */
>> if (!accel_find("tcg") && !accel_find("kvm")) {
>> @@ -2787,6 +2788,7 @@ static void configure_accelerators(const char
>> *progname)
>> error_report("invalid accelerator %s", *tmp);
>> }
>> }
>> + g_strfreev(accel_list);
>> } else {
>> if (accel != NULL) {
>> error_report("The -accel and \"-machine accel=\" options are
>> incompatible");
>>
>
> FYI, a fix for this is already part of Laurent's "Trivial branch
> patches" PULL request from yesterday.
https://patchew.org/QEMU/address@hidden/address@hidden/
Thanks,
Laurent