On Thu, Feb 02, 2017 at 04:21:39PM +0100, Michal Privoznik wrote:
On 02/02/2017 02:56 PM, Daniel P. Berrange wrote:
On Thu, Feb 02, 2017 at 02:49:48PM +0100, Michal Privoznik wrote:
Before 9c17d665fdc5f (v1.3.2 - I know, right?) it was possible to have the following interface configuration:
<interface type='ethernet'/> <script path=''/> </interface>
This resulted in -netdev tap,script=,.. Fortunately, qemu helped us to get away with this as it just ignored the empty script path. However, after the commit mentioned above it's libvirtd who is executing the script. Unfortunately without special case-ing empty script path.
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
This was always invalid input and the fact that it happened to work in the past is just luck. Since this has been broken since 1.3.2 there's plenty of libvirt releases that reject this, and so any app realistically has to be fixed to omit the <script> entirely if they want any portability. As such I don't think we need to "fix" this.
Fair enough. Although, 1.3.2 was not that long time ago (~a year). BTW: it's openstack who is still using this.
Seems nova has already been fixed in both master & stable branches https://bugs.launchpad.net/nova/+bug/1649527 Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|