[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/3] hw/nvram: Fix Memory Leak in Xilinx eFuse QOM
From: |
Francisco Iglesias |
Subject: |
Re: [PATCH 1/3] hw/nvram: Fix Memory Leak in Xilinx eFuse QOM |
Date: |
Mon, 18 Oct 2021 13:21:31 +0200 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On [2021 Oct 15] Fri 13:35:30, Tong Ho wrote:
> Signed-off-by: Tong Ho <tong.ho@xilinx.com>
Reviewed-by: Francisco Iglesias <frasse.iglesias@gmail.com>
> ---
> hw/nvram/xlnx-efuse.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/hw/nvram/xlnx-efuse.c b/hw/nvram/xlnx-efuse.c
> index ee1caab54c..a0fd77b586 100644
> --- a/hw/nvram/xlnx-efuse.c
> +++ b/hw/nvram/xlnx-efuse.c
> @@ -144,10 +144,11 @@ static bool efuse_ro_bits_find(XlnxEFuse *s, uint32_t k)
> bool xlnx_efuse_set_bit(XlnxEFuse *s, unsigned int bit)
> {
> if (efuse_ro_bits_find(s, bit)) {
> + g_autofree char *path = object_get_canonical_path(OBJECT(s));
> +
> qemu_log_mask(LOG_GUEST_ERROR, "%s: WARN: "
> "Ignored setting of readonly efuse bit<%u,%u>!\n",
> - object_get_canonical_path(OBJECT(s)),
> - (bit / 32), (bit % 32));
> + path, (bit / 32), (bit % 32));
> return false;
> }
>
> @@ -202,9 +203,11 @@ static void efuse_realize(DeviceState *dev, Error **errp)
> efuse_ro_bits_sort(s);
>
> if ((s->efuse_size % 32) != 0) {
> + g_autofree char *path = object_get_canonical_path(OBJECT(s));
> +
> error_setg(errp,
> "%s.efuse-size: %u: property value not multiple of 32.",
> - object_get_canonical_path(OBJECT(dev)), s->efuse_size);
> + path, s->efuse_size);
> return;
> }
>
> --
> 2.25.1
>
>
- [PATCH 0/3] hw/nvram: Fix Memory Leaks in Xilinx eFUSE, Tong Ho, 2021/10/15
- [PATCH 2/3] hw/nvram: Fix Memory Leak in Xilinx Versal eFuse device, Tong Ho, 2021/10/15
- [PATCH 1/3] hw/nvram: Fix Memory Leak in Xilinx eFuse QOM, Tong Ho, 2021/10/15
- Re: [PATCH 1/3] hw/nvram: Fix Memory Leak in Xilinx eFuse QOM,
Francisco Iglesias <=
- [PATCH 3/3] hw/nvram: Fix Memory Leak in Xilinx ZynqMP eFuse device, Tong Ho, 2021/10/15
- Re: [PATCH 0/3] hw/nvram: Fix Memory Leaks in Xilinx eFUSE, Philippe Mathieu-Daudé, 2021/10/18
- Re: [PATCH 0/3] hw/nvram: Fix Memory Leaks in Xilinx eFUSE, Laurent Vivier, 2021/10/19