[libvirt] [PATCH] minor usb masstorage hotadd cleanup

Hi, this just brings qemudDomainAttachUsbMassstorageDevice in line with the rest of the code: * don't allow to add the same target device name more than once * escape paths for qemu's monitor command Cheers, -- Guido

On Fri, Oct 17, 2008 at 12:44:13PM +0200, Guido Günther wrote:
Hi, this just brings qemudDomainAttachUsbMassstorageDevice in line with the rest of the code:
* don't allow to add the same target device name more than once * escape paths for qemu's monitor command
Good idea ! Applied and commited, thanks ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

On Tue, Oct 21, 2008 at 03:09:48PM +0200, Daniel Veillard wrote:
On Fri, Oct 17, 2008 at 12:44:13PM +0200, Guido Günther wrote:
Hi, this just brings qemudDomainAttachUsbMassstorageDevice in line with the rest of the code:
* don't allow to add the same target device name more than once * escape paths for qemu's monitor command
Good idea ! Applied and commited, Would it make sense to push the check for duplicate device names further up in the long term? -- Guido
participants (2)
-
Daniel Veillard
-
Guido Günther