[libvirt] [PATCH] lxc: report correct error
Laine Stump
laine at laine.org
Wed May 4 04:48:31 UTC 2011
On 05/03/2011 05:04 PM, Eric Blake wrote:
> Clang noticed a dead assignment, which turned out to be the use
> of the wrong variable. rc starts life as -1, and is only ever
> assigned to 0 just before a successful cleanup.
>
> * src/lxc/lxc_driver.c (lxcSetupInterfaces): Don't call
> virReportSystemError(-1).
> ---
> src/lxc/lxc_driver.c | 3 +--
> 1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c
> index e905302..b94941d 100644
> --- a/src/lxc/lxc_driver.c
> +++ b/src/lxc/lxc_driver.c
> @@ -1100,10 +1100,9 @@ static int lxcSetupInterfaces(virConnectPtr conn,
> }
>
> if ((ret = brAddInterface(brctl, bridge, parentVeth)) != 0) {
> - virReportSystemError(rc,
> + virReportSystemError(ret,
> _("Failed to add %s device to %s"),
> parentVeth, bridge);
> - rc = -1;
> goto error_exit;
> }
>
ACK.
More information about the libvir-list
mailing list