[libvirt] [PATCH] network/bridge_driver.c: avoid potential NULL-dereference

[Matthias Bolte]

2010/1/5 Jim Meyering <jim at meyering.net>:
> obvious typo:
>
> >From 6f0810192dc6d9b223f2f973812fd787f398576c Mon Sep 17 00:00:00 2001
> From: Jim Meyering <meyering at redhat.com>
> Date: Tue, 5 Jan 2010 15:48:42 +0100
> Subject: [PATCH] network/bridge_driver.c: avoid potential NULL-dereference
>
> * src/network/bridge_driver.c (networkBuildDnsmasqArgv): Correct
> test for NULL *argv.
> ---
>  src/network/bridge_driver.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
> index abee78c..69809b3 100644
> --- a/src/network/bridge_driver.c
> +++ b/src/network/bridge_driver.c
> @@ -1,7 +1,7 @@
>  /*
>  * driver.c: core driver methods for managing qemu guests
>  *
> - * Copyright (C) 2006-2009 Red Hat, Inc.
> + * Copyright (C) 2006-2010 Red Hat, Inc.
>  * Copyright (C) 2006 Daniel P. Berrange
>  *
>  * This library is free software; you can redistribute it and/or
> @@ -513,7 +513,7 @@ networkBuildDnsmasqArgv(virConnectPtr conn,
>     return 0;
>
>  no_memory:
> -    if (argv) {
> +    if (*argv) {
>         for (i = 0; (*argv)[i]; i++)
>             VIR_FREE((*argv)[i]);
>         VIR_FREE(*argv);
> --
> 1.6.6.384.g14e6a
>

ACK.

Matthias