[libvirt] [PATCH] vbox: network: make sure driver not be NULL in virRegisterNetworkDriver
Eric Blake
eblake at redhat.com
Fri Oct 24 14:47:47 UTC 2014
On 10/24/2014 03:31 AM, Shanzhi Yu wrote:
> libvirtd will report below error if does not make sure driver not be NULL
> in virRegisterNetworkDriver
>
> $ libvirtd
> 2014-10-24 09:24:36.443+0000: 28876: info : libvirt version: 1.2.10
> 2014-10-24 09:24:36.443+0000: 28876: error : virRegisterNetworkDriver:549 : driver in virRegisterNetworkDriver must not be NULL
> 2014-10-24 09:24:36.443+0000: 28876: error : virDriverLoadModule:99 : Failed module registration vboxNetworkRegister
>
> Signed-off-by: Shanzhi Yu <shyu at redhat.com>
> ---
> src/vbox/vbox_driver.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/vbox/vbox_driver.c b/src/vbox/vbox_driver.c
> index 743a488..ff69069 100644
> --- a/src/vbox/vbox_driver.c
> +++ b/src/vbox/vbox_driver.c
> @@ -152,7 +152,7 @@ int vboxNetworkRegister(void)
> if (VBoxCGlueInit(&uVersion) == 0)
> networkDriver = vboxGetNetworkDriver(uVersion);
>
> - if (virRegisterNetworkDriver(networkDriver) < 0)
> + if ((networkDriver != NULL) && (virRegisterNetworkDriver(networkDriver) < 0))
Over-parenthesized. Sufficient to write:
if (networkDriver && virRegisterNetworkDriver(networkDriver) < 0)
Or did you botch the logic, and really mean:
if (!networkDriver || virRegisterNetworkDriver(networkDriver) < 0)
Furthermore, Dan's recent patch series will probably overhaul all of
this anyways, so it may be easier to just wait for his patches to land.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 539 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20141024/b5b3ed02/attachment-0001.sig>
More information about the libvir-list
mailing list