First time in a while - Coverity complained this morning [...]
diff --git a/src/lxc/lxc_fuse.c b/src/lxc/lxc_fuse.c index e73b4d0690..c4223f4e06 100644 --- a/src/lxc/lxc_fuse.c +++ b/src/lxc/lxc_fuse.c @@ -326,10 +326,10 @@ int lxcSetupFuse(virLXCFusePtr *f, virDomainDefPtr def) *f = fuse;
^^ Event use_after_free: Using freed pointer "fuse". Also see events: [alias][freed_arg]
return ret; cleanup1: - VIR_FREE(fuse->mountpoint); + g_free(fuse->mountpoint); virMutexDestroy(&fuse->lock); cleanup2: - VIR_FREE(fuse); + g_free(fuse);
^^ Event freed_arg: "g_free" frees "fuse". A fuse = NULL; here will make coverity happy, but not sure if that's standard any more... The VIR_FREE would have done thta for us IIRC. John [...]