On Tue, 4 Jul 2023 at 18:19, Anthony PERARD <anthony.per...@citrix.com> wrote: > > From: Anthony PERARD <anthony.per...@citrix.com> > > Commit 189829399070 ("xen-block: Use specific blockdev driver") > introduced a new error path, without taking care of allocated > resources. > > So only allocate the qdicts after the error check, and free both > `filename` and `driver` when we are about to return and thus taking > care of both success and error path. > > Coverity only spotted the leak of qdicts (*_layer variables). > > Reported-by: Peter Maydell <peter.mayd...@linaro.org> > Fixes: Coverity CID 1508722, 1398649 > Fixes: 189829399070 ("xen-block: Use specific blockdev driver") > Signed-off-by: Anthony PERARD <anthony.per...@citrix.com>
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM