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
