On Wed, Oct 21, 2015 at 19:04:11 +0530, Prasanna Kumar Kalever wrote: ... > --- > block/gluster.c | 420 +++++++++++++++++++++++++++++++++++++++++++++------ > qapi/block-core.json | 62 +++++++- > 2 files changed, 433 insertions(+), 49 deletions(-) > > diff --git a/block/gluster.c b/block/gluster.c > index ededda2..62b6656 100644 > --- a/block/gluster.c > +++ b/block/gluster.c ... > + > > static void qemu_gluster_gconf_free(GlusterConf *gconf) > { > if (gconf) { > - g_free(gconf->host); > g_free(gconf->volume); > g_free(gconf->path); > - g_free(gconf->transport); > + if (gconf->gsconf) { > + g_free(gconf->gsconf[0].host); > + g_free(gconf->gsconf[0].transport); > + g_free(gconf->gsconf); Looks like this leaks second and any further server config struct. > + } > g_free(gconf); > } > } Peter