[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] virtio-serial: Fix segfault on guest boot
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2] virtio-serial: Fix segfault on guest boot |
Date: |
Tue, 21 Jun 2011 18:13:45 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Luiz Capitulino <address@hidden> writes:
> If I start qemu with:
>
> # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \
> -device virtio-serial \
> -chardev socket,host=localhost,port=1234,server,nowait,id=foo \
> -device virtserialport,chardev=foo,name=org.qemu.guest_agent
>
> I get a segfault when booting a Fedora 14 guest. The backtrace says:
>
> Program terminated with signal 11, Segmentation fault.
> #0 0x0000000000420850 in handle_control_message (vser=0x3732bd0,
> buf=0x2c173e0, len=8) at
> /home/lcapitulino/src/qmp-unstable/hw/virtio-serial-bus.c:335
> 335 info = DO_UPCAST(VirtIOSerialPortInfo, qdev, port->dev.info);
>
> What's happening is VIRTIO_CONSOLE_DEVICE_READY is a message for the
> whole device, not for an individual port. So port is NULL. This bug was
> introduced by commit a15bb0d6a981de749452a5180fc8084d625671da.
>
> This commit fixes that by making the port returned by find_port_by_id()
> be used only by the VIRTIO_CONSOLE_PORT_READY and
> VIRTIO_CONSOLE_PORT_OPEN messages.
Reviewed-by: Markus Armbruster <address@hidden>