On Fri, Mar 19, 2021 at 03:25:27PM +0200, Mahmoud Mandour wrote: > @@ -588,7 +587,7 @@ out: > } > > pthread_mutex_destroy(&req->ch.lock); > - free(fbuf.mem); > + g_free(fbuf.mem); > free(req); ^--- was FVRequest allocation changed in a previous patch? Maybe an earlier patch forgot to use g_free() here. Aside from this: Reviewed-by: Stefan Hajnoczi