[PATCH v4 3/4] lxc: remove domain definition from lxc network struct

Michal Privoznik mprivozn at redhat.com
Mon Feb 3 10:06:18 UTC 2020


On 2/3/20 2:28 AM, Julio Faracco wrote:
> Domain definition is useless now inside network structure. This pointer
> was required because new network definition was being added each time
> that a new network type appeared. So, this should be processed into
> old function `lxcNetworkParseDataType()`. Now, as it was moved to an
> array, it can be handle together each interface pointer.
> 
> Signed-off-by: Julio Faracco <jcfaracco at gmail.com>
> ---
>   src/lxc/lxc_native.c | 17 +++++++++--------
>   1 file changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/src/lxc/lxc_native.c b/src/lxc/lxc_native.c
> index 8bbe205659..99539a0205 100644
> --- a/src/lxc/lxc_native.c
> +++ b/src/lxc/lxc_native.c
> @@ -414,7 +414,6 @@ lxcCreateHostdevDef(int mode, int type, const char *data)
>   typedef struct _lxcNetworkParseData lxcNetworkParseData;
>   typedef lxcNetworkParseData *lxcNetworkParseDataPtr;
>   struct _lxcNetworkParseData {
> -    virDomainDefPtr def;
>       char *type;
>       char *link;
>       char *mac;
> @@ -470,13 +469,16 @@ lxcAddNetworkRouteDefinition(const char *address,
>   }
>   
>   static int
> -lxcAddNetworkDefinition(lxcNetworkParseData *data)
> +lxcAddNetworkDefinition(virDomainDefPtr def, lxcNetworkParseData *data)
>   {
>       virDomainNetDefPtr net = NULL;
>       virDomainHostdevDefPtr hostdev = NULL;
>       bool isPhys, isVlan = false;
>       size_t i;
>   
> +    if (!data)
> +        return -1;

Is this check really necessary? I mean, can @data be NULL? Because from 
my understanding of the code it can not. But even if it can, we need to 
report an error here because the caller doesn't.

You don't need to resend, I can fix it before pushing. The rest looks good.

Michal




More information about the libvir-list mailing list