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(a)gentoo.org>
> ---
> 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(a)redhat.com +1-801-349-2682
Libvirt virtualization library
http://libvirt.org