[libvirt] [PATCH] Add missing error reporting when loading mac filter config for QEMU

[Eric Blake]

On 06/14/2011 08:57 AM, Daniel P. Berrange wrote:
> If the 'mac_filter' configuration parameter is enabled, and there
> is a failure to enable filtering, no error is reported back to
> the caller. Also fix some bogus whitespace indentation for
> hugetlbfs_mount

I probably would have split this into two patches, but oh well.  The net
result is still small enough to be reviewable.

> 
> * src/qemu/qemu_conf.c: Add missing error reporting
> ---
>  src/qemu/qemu_conf.c |   24 ++++++++++++++----------
>  1 files changed, 14 insertions(+), 10 deletions(-)

> @@ -398,12 +398,16 @@ int qemudLoadDriverConfig(struct qemud_driver *driver,
>              virReportSystemError(errno,
>                                   _("failed to enable mac filter in '%s'"),
>                                   __FILE__);
> +            virConfFree(conf);
> +            return -1;
>          }
>  
>          if ((errno = networkDisableAllFrames(driver))) {
>              virReportSystemError(errno,
>                           _("failed to add rule to drop all frames in '%s'"),
>                                   __FILE__);
> +            virConfFree(conf);
> +            return -1;

Here's the meat of the change.

ACK.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20110614/2133474d/attachment-0001.sig>