On 12/15/2011 04:05 AM, Stefan Hajnoczi wrote:
> Commit f462141f18ffdd75847f6459ef83d90b831d12c0 introduced clean up code
> when usb_qdev_init() fails. Unfortunately it calls .handle_destroy()
> when .init() was never invoked or failed. This can lead to crashes when
> .handle_destroy() tries to clean up things that were never initialized.
>
> This patch is careful to undo only those steps that completed along the
> usb_qdev_init() code path. It's not as pretty as the unified error
> handling in f462141f18ffdd75847f6459ef83d90b831d12c0 but it's necessary.
>
> Signed-off-by: Stefan Hajnoczi<email address hidden>
On 12/15/2011 04:05 AM, Stefan Hajnoczi wrote: 5847f6459ef83d9 0b831d12c0 introduced clean up code 5847f6459ef83d9 0b831d12c0 but it's necessary.
> Commit f462141f18ffdd7
> when usb_qdev_init() fails. Unfortunately it calls .handle_destroy()
> when .init() was never invoked or failed. This can lead to crashes when
> .handle_destroy() tries to clean up things that were never initialized.
>
> This patch is careful to undo only those steps that completed along the
> usb_qdev_init() code path. It's not as pretty as the unified error
> handling in f462141f18ffdd7
>
> Signed-off-by: Stefan Hajnoczi<email address hidden>
Applied. Thanks.
Regards,
Anthony Liguori
> --- init(DeviceStat e *qdev, DeviceInfo *base) &dev->strings) ; port(dev) ; >init(dev) ; port(dev) ; attach( dev); exit(qdev) ; exit(qdev) ; exit(DeviceStat e *qdev)
> hw/usb-bus.c | 12 +++++-------
> 1 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/hw/usb-bus.c b/hw/usb-bus.c
> index 8cafb76..8203390 100644
> --- a/hw/usb-bus.c
> +++ b/hw/usb-bus.c
> @@ -77,23 +77,21 @@ static int usb_qdev_
> QLIST_INIT(
> rc = usb_claim_
> if (rc != 0) {
> - goto err;
> + return rc;
> }
> rc = dev->info-
> if (rc != 0) {
> - goto err;
> + usb_release_
> + return rc;
> }
> if (dev->auto_attach) {
> rc = usb_device_
> if (rc != 0) {
> - goto err;
> + usb_qdev_
> + return rc;
> }
> }
> return 0;
> -
> -err:
> - usb_qdev_
> - return rc;
> }
>
> static int usb_qdev_