[libvirt] [PATCH 5/8] Fix thinko in PCI hostdev detach

Daniel P. Berrange berrange at redhat.com
Mon Aug 17 19:05:25 UTC 2009


On Mon, Aug 17, 2009 at 03:10:18PM +0100, Mark McLoughlin wrote:
> * src/qemu_driver.c: Add missing break statement in
>   qemudDomainDetachHostDevice()
> ---
>  src/qemu_driver.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/src/qemu_driver.c b/src/qemu_driver.c
> index 187497f..048e1c0 100644
> --- a/src/qemu_driver.c
> +++ b/src/qemu_driver.c
> @@ -5894,6 +5894,7 @@ static int qemudDomainDetachHostDevice(virConnectPtr conn,
>      switch (hostdev->source.subsys.type) {
>      case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI:
>          ret = qemudDomainDetachHostPciDevice(conn, vm, dev);
> +        break;
>      default:
>          qemudReportError(conn, dom, NULL, VIR_ERR_NO_SUPPORT,
>                           _("hostdev subsys type '%s' not supported"),

ACK


Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|




More information about the libvir-list mailing list