It's just because in patch 1/4 of this series we use
find_machine_info(machine->cpu_model), which could be a NULL pointer.
Indeed this patch can be avoided reworking a bit the calling function code.
Regards,
alvise
On Mon, Jan 5, 2015 at 4:36 PM, Peter Maydell <peter.mayd...@linaro.org> wrote:
> On 21 November 2014 at 18:07, Alvise Rigo <a.r...@virtualopensystems.com>
> wrote:
>> Signed-off-by: Alvise Rigo <a.r...@virtualopensystems.com>
>> ---
>> hw/arm/virt.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/hw/arm/virt.c b/hw/arm/virt.c
>> index e8d527d..4e7b869 100644
>> --- a/hw/arm/virt.c
>> +++ b/hw/arm/virt.c
>> @@ -151,6 +151,10 @@ static VirtBoardInfo *find_machine_info(const char *cpu)
>> {
>> int i;
>>
>> + if (!cpu) {
>> + return NULL;
>> + }
>> +
>> for (i = 0; i < ARRAY_SIZE(machines); i++) {
>> if (strcmp(cpu, machines[i].cpu_model) == 0) {
>> return &machines[i];
>
> What's the motivation for this change? We can never call this
> function with a NULL pointer at the moment...
>
> thanks
> -- PMM
