Hi On Tue, Jul 4, 2017 at 4:16 PM Peng Hao <peng.h...@zte.com.cn> wrote:
> vhost exists a call for g_file_get_contents, but not call g_free. > > Signed-off-by: Peng Hao<peng.h...@zte.com.cn> > Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> --- > hw/virtio/vhost-backend.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/hw/virtio/vhost-backend.c b/hw/virtio/vhost-backend.c > index 4e31de1..2c481d6 100644 > --- a/hw/virtio/vhost-backend.c > +++ b/hw/virtio/vhost-backend.c > @@ -52,11 +52,13 @@ static int vhost_kernel_memslots_limit(struct > vhost_dev *dev) > &s, NULL, NULL)) { > uint64_t val = g_ascii_strtoull(s, NULL, 10); > if (!((val == G_MAXUINT64 || !val) && errno)) { > + g_free(s); > return val; > } > error_report("ignoring invalid max_mem_regions value in vhost > module:" > " %s", s); > } > + g_free(s); > return limit; > } > > -- > 1.8.3.1 > > > > -- Marc-André Lureau