Re: [PATCH] qemu: Allow empty script path to <interface/>

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

 



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@xxxxxxxxxx>
> > 
> > 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.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1412834

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/ :|

--
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]
  Powered by Linux