[libvirt] [PATCH v3 04/21] LXC from native: migrate fstab and lxc.mount.entry

John Ferlan jferlan at redhat.com
Thu Feb 13 11:34:52 UTC 2014


New Coverity complaint regarding this change ...

<...snip...>

>  virDomainDefPtr
>  lxcParseConfigString(const char *config)
>  {
> @@ -115,6 +343,7 @@ lxcParseConfigString(const char *config)
>                         _("failed to generate uuid"));
>          goto error;
>      }
> +
>      vmdef->id = -1;
>      vmdef->mem.max_balloon = 64 * 1024;
>  
> @@ -127,6 +356,8 @@ lxcParseConfigString(const char *config)
>       * minimum required to make XML parsing pass */
>      vmdef->maxvcpus = 1;
>  
> +    vmdef->nfss = 0;
> +
>      if (VIR_STRDUP(vmdef->os.type, "exe") < 0)
>          goto error;
>  
> @@ -142,6 +373,18 @@ lxcParseConfigString(const char *config)
>      if (lxcSetRootfs(vmdef, properties) < 0)
>          goto error;
>  
> +    /* Look for fstab: we shouldn't have it */
> +    if (virConfGetValue(properties, "lxc.mount")) {
> +        virReportError(VIR_ERR_ARGUMENT_UNSUPPORTED, "%s",
> +                       _("lxc.mount found, use lxc.mount.entry lines instead"));
> +        goto error;
> +    }
> +
> +    /* Loop over lxc.mount.entry to add filesystem devices for them */
> +    value = virConfGetValue(properties, "lxc.mount.entry");

(1) Event returned_pointer: 	Pointer "value" returned by
"virConfGetValue(properties, "lxc.mount.entry")" is never used.


'value' is set, but never used.


John
> +    if (virConfWalk(properties, lxcFstabWalkCallback, vmdef) < 0)
> +        goto error;
> +
>      goto cleanup;
>  




More information about the libvir-list mailing list