Re: [libvirt] [PATCH] Fix USB device re-labelling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Mark McLoughlin wrote:
> A simple misplaced break out of a switch results in:
> 
>   libvir: error : Failed to open file '/sys/bus/pci/devices/0000:00:54c./vendor': No such file or directory
>   libvir: error : Failed to open file '/sys/bus/pci/devices/0000:00:54c./device': No such file or directory
>   libvir: error : this function is not supported by the hypervisor: Failed to read product/vendor ID for 0000:00:54c.
> 
> when trying to passthrough a USB host device to qemu.
> 
> * src/security_selinux.c: fix a switch/break thinko
> ---
>  src/security/security_selinux.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c
> index bdd9eb2..d08d502 100644
> --- a/src/security/security_selinux.c
> +++ b/src/security/security_selinux.c
> @@ -492,12 +492,11 @@ SELinuxSetSecurityHostdevLabel(virConnectPtr conn,
>  
>              ret = usbDeviceFileIterate(conn, usb, SELinuxSetSecurityUSBLabel, vm);
>              usbFreeDevice(conn, usb);
> -
> -            break;
>          } else {
>              /* XXX deal with product/vendor better */
>              ret = 0;
>          }
> +        break;

D'oh.  ACK.

-- 
Chris Lalancette

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]