qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] softmmu: fix crash with invalid -M memory-backend=


From: Paolo Bonzini
Subject: Re: [PATCH] softmmu: fix crash with invalid -M memory-backend=
Date: Tue, 17 Mar 2020 16:40:56 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0

On 17/03/20 16:13, Igor Mammedov wrote:
> On Mon,  9 Mar 2020 15:51:55 +0100
> Marc-André Lureau <address@hidden> wrote:
> 
>> Fixes: fe64d06afc1c5d895f220c268cfe4d5f1e65d44e ("vl.c: ensure that
>> ram_size matches size of machine.memory-backend")
>> Signed-off-by: Marc-André Lureau <address@hidden>
> 
> Paolo,
> 
> Can you pick up this patch for your pull req? 

I've done it already, but I can send more.

Paolo

>> ---
>>  softmmu/vl.c | 5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/softmmu/vl.c b/softmmu/vl.c
>> index 5549f4b619..38e9c404f2 100644
>> --- a/softmmu/vl.c
>> +++ b/softmmu/vl.c
>> @@ -4287,6 +4287,11 @@ void qemu_init(int argc, char **argv, char **envp)
>>  
>>          backend = object_resolve_path_type(current_machine->ram_memdev_id,
>>                                             TYPE_MEMORY_BACKEND, NULL);
>> +        if (!backend) {
>> +            error_report("Memory backend '%s' not found",
>> +                         current_machine->ram_memdev_id);
>> +            exit(EXIT_FAILURE);
>> +        }
>>          backend_size = object_property_get_uint(backend, "size",  
>> &error_abort);
>>          if (have_custom_ram_size && backend_size != ram_size) {
>>                  error_report("Size specified by -m option must match size 
>> of "
> 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]