[libvirt] [PATCH 5/8] Avoid unsafe use of /proc/$PID/root in LXC USB hotplug

Eric Blake eblake at redhat.com
Tue Feb 18 04:58:48 UTC 2014


On 02/17/2014 09:39 AM, Daniel P. Berrange wrote:
> Rewrite lxcDomainAttachDeviceHostdevSubsysUSBLive function
> to use the virProcessRunInMountNamespace helper. This avoids
> risk of a malicious guest replacing /dev with a absolute
> symlink, tricking the driver into changing the host OS
> filesystem.
> 
> Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
> ---
>  src/lxc/lxc_driver.c | 73 ++++++++++++++++------------------------------------
>  1 file changed, 22 insertions(+), 51 deletions(-)
> 

>  
>      if (!(usb = virUSBDeviceNew(def->source.subsys.u.usb.bus,
> -                                def->source.subsys.u.usb.device, vroot)))
> +                                def->source.subsys.u.usb.device, NULL)))

It looks like all callers of virUSBDeviceNew pass NULL once this series
is over.  Can we clean things up in a followup to kill the dead
parameter?  But this patch is fine as-is.

ACK.


-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140217/1e92644b/attachment-0001.sig>


More information about the libvir-list mailing list