-
Anthony PERARD authored
Commit 18982939 ("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.maydell@linaro.org>
Fixes: Coverity CID 1508722, 1398649
Fixes: 18982939 ("xen-block: Use specific blockdev driver")
Signed-off-by:
Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by:
Paul Durrant <paul@xen.org>
Reviewed-by:
Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20230704171819.42564-1-anthony.perard@citrix.com>
Signed-off-by:
Anthony PERARD <anthony.perard@citrix.com>Anthony PERARD authoredCommit 18982939 ("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.maydell@linaro.org>
Fixes: Coverity CID 1508722, 1398649
Fixes: 18982939 ("xen-block: Use specific blockdev driver")
Signed-off-by:
Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by:
Paul Durrant <paul@xen.org>
Reviewed-by:
Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20230704171819.42564-1-anthony.perard@citrix.com>
Signed-off-by:
Anthony PERARD <anthony.perard@citrix.com>
Loading