On 08/05/2010 01:24 PM, Chris Lalancette wrote: > On 08/05/10 - 02:12:52PM, Doug Goldstein wrote: >> Fix the error checking to use the return value from brAddTap() instead >> of checking the current errno value which might have been changed by >> clean up calls inside of brAddTap(). >> >> Signed-off-by: Doug Goldstein <cardoe@xxxxxxxxxx> >> --- >> src/qemu/qemu_conf.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c >> index e92021a..7c0e354 100644 >> --- a/src/qemu/qemu_conf.c >> +++ b/src/qemu/qemu_conf.c >> @@ -1689,7 +1689,7 @@ qemudNetworkIfaceConnect(virConnectPtr conn, >> tapmac, >> vnet_hdr, >> &tapfd))) { >> - if (errno == ENOTSUP) { >> + if (err == ENOTSUP) { >> /* In this particular case, give a better diagnostic. */ >> qemuReportError(VIR_ERR_INTERNAL_ERROR, >> _("Failed to add tap interface to bridge. " > > Makes sense to me. > > ACK I've pushed patch 2 and 3, but agree with Chris's NACK on patch 1. -- Eric Blake eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list