mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-22 21:55:25 +00:00
qemu: Allow empty script path to <interface/>
Before 9c17d665fd
(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 commit is contained in:
parent
d23dfaae9f
commit
1d9ab0f04a
@ -145,6 +145,16 @@
|
|||||||
libxl: fix dom0 autoballooning with Xen 4.8
|
libxl: fix dom0 autoballooning with Xen 4.8
|
||||||
</summary>
|
</summary>
|
||||||
</change>
|
</change>
|
||||||
|
<change>
|
||||||
|
<summary>
|
||||||
|
qemu: Allow empty script path to <interface/>
|
||||||
|
</summary>
|
||||||
|
<description>
|
||||||
|
Historically, this was always allowed. Unfortunately, due to some
|
||||||
|
rework done for 1.3.2 release a bug was dragged in which suddenly
|
||||||
|
stop allowing domain with such configuration to start.
|
||||||
|
</description>
|
||||||
|
</change>
|
||||||
</section>
|
</section>
|
||||||
</release>
|
</release>
|
||||||
<release version="v3.0.0" date="2017-01-17">
|
<release version="v3.0.0" date="2017-01-17">
|
||||||
|
@ -2615,6 +2615,10 @@ virNetDevRunEthernetScript(const char *ifname, const char *script)
|
|||||||
virCommandPtr cmd;
|
virCommandPtr cmd;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
|
/* Not a bug! Previously we did accept script="" as a NO-OP. */
|
||||||
|
if (STREQ(script, ""))
|
||||||
|
return 0;
|
||||||
|
|
||||||
cmd = virCommandNew(script);
|
cmd = virCommandNew(script);
|
||||||
virCommandAddArgFormat(cmd, "%s", ifname);
|
virCommandAddArgFormat(cmd, "%s", ifname);
|
||||||
virCommandClearCaps(cmd);
|
virCommandClearCaps(cmd);
|
||||||
|
Loading…
Reference in New Issue
Block a user