* [Qemu-devel] [6890] host_device_remove: remove incorrect check for device name ( Eduardo Habkost)
@ 2009-03-28 15:51 Anthony Liguori
0 siblings, 0 replies; only message in thread
From: Anthony Liguori @ 2009-03-28 15:51 UTC (permalink / raw)
To: qemu-devel
Revision: 6890
http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=6890
Author: aliguori
Date: 2009-03-28 15:51:46 +0000 (Sat, 28 Mar 2009)
Log Message:
-----------
host_device_remove: remove incorrect check for device name (Eduardo Habkost)
There is no need to check for valid prefixes on the the device name
when removing it. If the device name is found on the vlan client list,
it can be removed, regardless of the prefix used on its name.
To reproduce the bug, just run this on the monitor:
(qemu) host_net_add user name=foobar
(qemu) host_net_remove 0 foobar
invalid host network device foobar
(qemu)
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Modified Paths:
--------------
branches/stable_0_10/net.c
Modified: branches/stable_0_10/net.c
===================================================================
--- branches/stable_0_10/net.c 2009-03-28 15:46:15 UTC (rev 6889)
+++ branches/stable_0_10/net.c 2009-03-28 15:51:46 UTC (rev 6890)
@@ -1818,11 +1818,6 @@
VLANState *vlan;
VLANClientState *vc;
- if (!net_host_check_device(device)) {
- term_printf("invalid host network device %s\n", device);
- return;
- }
-
vlan = qemu_find_vlan(vlan_id);
if (!vlan) {
term_printf("can't find vlan %d\n", vlan_id);
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2009-03-28 15:51 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-28 15:51 [Qemu-devel] [6890] host_device_remove: remove incorrect check for device name ( Eduardo Habkost) Anthony Liguori
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).