Re: [PATCH 5/7] virsh: fix error messages in iface-bridge

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

 



On 2012年11月28日 22:42, Osier Yang wrote:
On 2012年11月28日 21:34, Ján Tomko wrote:
The error messages did not correspond to the attributes they printed.
---
tools/virsh-interface.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/virsh-interface.c b/tools/virsh-interface.c
index 9ceb122..3be6d60 100644
--- a/tools/virsh-interface.c
+++ b/tools/virsh-interface.c
@@ -813,13 +813,13 @@ cmdInterfaceBridge(vshControl *ctl, const vshCmd
*cmd)
*/
if (!xmlSetProp(if_node, BAD_CAST "type", BAD_CAST if_type)) {
vshError(ctl, _("Failed to set new slave interface type to '%s' in xml
document"),
- if_name);
+ if_type);
goto cleanup;
}

if (!xmlSetProp(if_node, BAD_CAST "name", BAD_CAST if_name)) {
vshError(ctl, _("Failed to set new slave interface name to '%s' in xml
document"),
- br_name);
+ if_name);

With the indentions fixed.

goto cleanup;
}


ACK

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

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