Re: [Qemu-devel] [PATCH] Adds null check for DisplayStatus

[Anthony Liguori <aliguori@us.ibm.com>]

Stefano Stabellini wrote:
> Allocate a DisplaySurface in dumb_display_init if none else does it.
> The DisplaySurface will be used for the qemu monitor, serial and
> parallel ports, etc.
>
> Signed-off-by: Andrew May  <acmay@acmay.homeip.net>
> Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
>   

Applied.  Thanks.

Regards,

Anthony Liguori

> diff --git a/vl.c b/vl.c
> index 63d954b..b246e47 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2778,6 +2778,11 @@ DisplayState *get_displaystate(void)
>  static void dumb_display_init(void)
>  {
>      DisplayState *ds = qemu_mallocz(sizeof(DisplayState));
> +    if (ds == NULL) {
> +        fprintf(stderr, "dumb_display_init: DisplayState allocation failed\n");
> +        exit(1);
> +    }
> +    ds->surface = qemu_create_displaysurface(640, 480, 32, 640 * 4);
>      register_displaystate(ds);
>  }
>  
>
>
>
>