[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] fix '-cpu ?' Segfault
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH] fix '-cpu ?' Segfault |
Date: |
Thu, 22 Mar 2012 14:46:48 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:10.0.2) Gecko/20120215 Thunderbird/10.0.2 |
Am 22.03.2012 14:28, schrieb Stefan Berger:
> On 03/21/2012 08:33 AM, Eduardo Habkost wrote:
>> Fix stupid copy&paste mistake at commit
>> ecf40beae7dcbb057d4f115207f9d8276832a774: I moved code around but kept
>> "optarg" on the cpu_list() call.
>>
>> Reported-by: Jiri Denemark<address@hidden>
>> Signed-off-by: Eduardo Habkost<address@hidden>
>> ---
>> vl.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/vl.c b/vl.c
>> index 112b0e0..0fccf50 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -3196,7 +3196,7 @@ int main(int argc, char **argv, char **envp)
>> cpudef_init();
>>
>> if (cpu_model&& *cpu_model == '?') {
>> - list_cpus(stdout,&fprintf, optarg);
>> + list_cpus(stdout,&fprintf, cpu_model);
>> exit(0);
>> }
>>
> Does -cpu ? actually work then? I only see the list of cpus when using
> -cpu ?_ for example.
You're right, it should be cpu_model + 1 from what I recall, i.e. just
the optional argument, not the whole model string.
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg