Re: PATCH: Implement CDROM media change for QEMU/KVM driver

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

 



Richard W.M. Jones wrote:
> Richard W.M. Jones wrote:
> >+    strcat(cmd, newdisk->src);
> 
> Also, is quoting/escaping required?  In a naive libvirt-based app, it's 
> plausible that the string is provided by the user and could contain \n 
> to send arbitrary commands to the qemu console.

Agreed.  We can use something like qemudEscapeShellArg for that.
This (untested) patch adds qemudEscapeArg for non-shell arguments.

-jim

--
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]