[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests: test-bdrv-graph-mod: fix memory leak
From: |
Li Qiang |
Subject: |
Re: [Qemu-devel] [PATCH] tests: test-bdrv-graph-mod: fix memory leak |
Date: |
Tue, 12 Mar 2019 08:31:10 +0800 |
Philippe Mathieu-Daudé <address@hidden> 于2019年3月10日周日 下午10:34写道:
> On 3/10/19 12:34 PM, Li Qiang wrote:
> > Fixes: 2dbfadf
>
> ^ Please keep tags together (with Signed-off-by, ...)
>
> > Spotted by ASAN when 'make check'.
>
> I'm not native English speaker but I'd say:
>
>
Thanks your advice.
Hope the maintainer will do this minor adjustment.
Thanks,
Li Qiang
> Spotted by ASAN [with] 'make check'.
>
> or
>
> Spotted by ASAN [while running] 'make check'.
>
> Here goes:
>
> "Fixes: 2dbfadf"
>
> > Signed-off-by: Li Qiang <address@hidden>
> > ---
> > tests/test-bdrv-graph-mod.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/test-bdrv-graph-mod.c b/tests/test-bdrv-graph-mod.c
> > index 458dfa6661..8bf0fe735d 100644
> > --- a/tests/test-bdrv-graph-mod.c
> > +++ b/tests/test-bdrv-graph-mod.c
> > @@ -117,6 +117,7 @@ static void test_update_perm_tree(void)
> >
> > bdrv_unref(bs);
> > blk_unref(root);
> > + error_free(local_err);
>
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
>
> > }
> >
> > /*
> >
>