[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