[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 02/10] hcd-xhci: check & correct param before
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v7 02/10] hcd-xhci: check & correct param before using it |
Date: |
Thu, 17 Nov 2016 14:48:03 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Cao jin <address@hidden> writes:
> usb_xhci_realize() corrects invalid values of property "intrs"
> automatically, but the uncorrected value is passed to msi_init(),
> which chokes on invalid values. Delay that until after the
> correction.
>
> Resources allocated by usb_xhci_init() are leaked when msi_init()
> fails. Fix by calling it after msi_init().
>
> CC: Gerd Hoffmann <address@hidden>
> CC: Markus Armbruster <address@hidden>
> CC: Marcel Apfelbaum <address@hidden>
> CC: Michael S. Tsirkin <address@hidden>
> Signed-off-by: Cao jin <address@hidden>
> ---
> In previous rounds, usb_xhci_init() is moved too far from its original place,
> which results the segfault(XHCIState->numports is initialized in this func),
> now move it adjacent to msi_init code hunk.
Thanks for explaining what you changed.
Reviewed-by: Markus Armbruster <address@hidden>
- [Qemu-devel] [PATCH v7 00/10] Convert msix_init() to error, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 01/10] msix: Follow CODING_STYLE, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 02/10] hcd-xhci: check & correct param before using it, Cao jin, 2016/11/14
- Re: [Qemu-devel] [PATCH v7 02/10] hcd-xhci: check & correct param before using it,
Markus Armbruster <=
- [Qemu-devel] [PATCH v7 06/10] megasas: remove unnecessary megasas_use_msix(), Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 04/10] megasas: change behaviour of msix switch, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 10/10] msi_init: convert assert to return -errno, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 07/10] megasas: undo the overwrites of msi user configuration, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 09/10] vmxnet3: remove unnecessary internal msix flag, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 05/10] hcd-xhci: change behaviour of msix switch, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 03/10] pci: Convert msix_init() to Error and fix callers to check it, Cao jin, 2016/11/14
- [Qemu-devel] [PATCH v7 08/10] vmxnet3: fix reference leak issue, Cao jin, 2016/11/14
- Re: [Qemu-devel] [PATCH v7 00/10] Convert msix_init() to error, Michael S. Tsirkin, 2016/11/14