Re: [Qemu-devel] [PATCH] virtio-serial-bus: fix ports_map allocation on init

[Anthony Liguori <anthony@codemonkey.ws>]

On 05/19/2010 04:31 AM, Amit Shah wrote:
> From: Alon Levy<alevy@redhat.com>
>
> Fix for too small allocation to ports_map
>
> Signed-off-by: Alon Levy<alevy@redhat.com>
> Signed-off-by: Amit Shah<amit.shah@redhat.com>
>    

Applied.  Thanks.

Regards,

Anthony Liguori

> ---
>   hw/virtio-serial-bus.c |    3 ++-
>   1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c
> index 3ce95e8..7f9d28f 100644
> --- a/hw/virtio-serial-bus.c
> +++ b/hw/virtio-serial-bus.c
> @@ -774,7 +774,8 @@ VirtIODevice *virtio_serial_init(DeviceState *dev, uint32_t max_nr_ports)
>       }
>
>       vser->config.max_nr_ports = max_nr_ports;
> -    vser->ports_map = qemu_mallocz((max_nr_ports + 31) / 32);
> +    vser->ports_map = qemu_mallocz(((max_nr_ports + 31) / 32)
> +        * sizeof(vser->ports_map[0]));
>       /*
>        * Reserve location 0 for a console port for backward compat
>        * (old kernel, new qemu)
>