Re: [Qemu-devel] [PATCH] fix '-cpu ?' Segfault

["Andreas Färber" <afaerber@suse.de>]

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<jdenemar@redhat.com>
>> Signed-off-by: Eduardo Habkost<ehabkost@redhat.com>
>> ---
>>   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