[PATCH 0/2] hw/arm/xlnx-zynqmp.c: fix some error-handling code

Peter Maydell posted 2 patches 4 years ago
Test docker-mingw@fedora passed
Test docker-quick@centos7 passed
Test checkpatch passed
Test FreeBSD passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200324134947.15384-1-peter.maydell@linaro.org
Maintainers: Alistair Francis <alistair@alistair23.me>, Peter Maydell <peter.maydell@linaro.org>, "Edgar E. Iglesias" <edgar.iglesias@gmail.com>
hw/arm/xlnx-zynqmp.c | 27 ++++++++++++++++++++++++++-
1 file changed, 26 insertions(+), 1 deletion(-)
[PATCH 0/2] hw/arm/xlnx-zynqmp.c: fix some error-handling code
Posted by Peter Maydell 4 years ago
This series fixes some minor bugs in error-handling paths
in the xlnx-zynqmp device. The first is a memory leak spotted
by Coverity; the second is some missing error-propagation code
that I noticed while writing the first patch.

thanks
-- PMM

Peter Maydell (2):
  hw/arm/xlnx-zynqmp.c: Avoid memory leak in error-return path
  hw/arm/xlnx-zynqmp.c: Add missing error-propagation code

 hw/arm/xlnx-zynqmp.c | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

-- 
2.20.1


Re: [PATCH 0/2] hw/arm/xlnx-zynqmp.c: fix some error-handling code
Posted by Edgar E. Iglesias 4 years ago
On Tue, Mar 24, 2020 at 2:49 PM Peter Maydell <peter.maydell@linaro.org>
wrote:

> This series fixes some minor bugs in error-handling paths
> in the xlnx-zynqmp device. The first is a memory leak spotted
> by Coverity; the second is some missing error-propagation code
> that I noticed while writing the first patch.
>

Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>


>
> thanks
> -- PMM
>
> Peter Maydell (2):
>   hw/arm/xlnx-zynqmp.c: Avoid memory leak in error-return path
>   hw/arm/xlnx-zynqmp.c: Add missing error-propagation code
>
>  hw/arm/xlnx-zynqmp.c | 27 ++++++++++++++++++++++++++-
>  1 file changed, 26 insertions(+), 1 deletion(-)
>
> --
> 2.20.1
>
>