[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 3/6] loader: extract rom_free() function
From: |
Alistair Francis |
Subject: |
Re: [Qemu-devel] [PATCH v4 3/6] loader: extract rom_free() function |
Date: |
Wed, 8 Aug 2018 14:32:31 -0700 |
On Fri, Aug 3, 2018 at 7:47 AM, Stefan Hajnoczi <address@hidden> wrote:
> The next patch will need to free a rom. There is already code to do
> this in rom_add_file().
>
> Note that rom_add_file() uses:
>
> rom = g_malloc0(sizeof(*rom));
> ...
> if (rom->fw_dir) {
> g_free(rom->fw_dir);
> g_free(rom->fw_file);
> }
>
> The conditional is unnecessary since g_free(NULL) is a no-op.
>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
Reviewed-by: Alistair Francis <address@hidden>
Alistair
> ---
> hw/core/loader.c | 21 ++++++++++++---------
> 1 file changed, 12 insertions(+), 9 deletions(-)
>
> diff --git a/hw/core/loader.c b/hw/core/loader.c
> index bbb6e65bb5..0c72e7c05a 100644
> --- a/hw/core/loader.c
> +++ b/hw/core/loader.c
> @@ -847,6 +847,17 @@ struct Rom {
> static FWCfgState *fw_cfg;
> static QTAILQ_HEAD(, Rom) roms = QTAILQ_HEAD_INITIALIZER(roms);
>
> +/* rom->data must be heap-allocated (do not use with rom_add_elf_program())
> */
> +static void rom_free(Rom *rom)
> +{
> + g_free(rom->data);
> + g_free(rom->path);
> + g_free(rom->name);
> + g_free(rom->fw_dir);
> + g_free(rom->fw_file);
> + g_free(rom);
> +}
> +
> static inline bool rom_order_compare(Rom *rom, Rom *item)
> {
> return ((uintptr_t)(void *)rom->as > (uintptr_t)(void *)item->as) ||
> @@ -995,15 +1006,7 @@ err:
> if (fd != -1)
> close(fd);
>
> - g_free(rom->data);
> - g_free(rom->path);
> - g_free(rom->name);
> - if (fw_dir) {
> - g_free(rom->fw_dir);
> - g_free(rom->fw_file);
> - }
> - g_free(rom);
> -
> + rom_free(rom);
> return -1;
> }
>
> --
> 2.17.1
>
>
- [Qemu-devel] [PATCH v4 0/6] arm: add Cortex M0 CPU model and hex file loader, Stefan Hajnoczi, 2018/08/03
- [Qemu-devel] [PATCH v4 2/6] target/arm: add "cortex-m0" CPU model, Stefan Hajnoczi, 2018/08/03
- [Qemu-devel] [PATCH v4 3/6] loader: extract rom_free() function, Stefan Hajnoczi, 2018/08/03
- [Qemu-devel] [PATCH v4 1/6] hw/arm: make bitbanded IO optional on ARMv7-M, Stefan Hajnoczi, 2018/08/03
- [Qemu-devel] [PATCH v4 4/6] loader: add rom transaction API, Stefan Hajnoczi, 2018/08/03
- [Qemu-devel] [PATCH v4 5/6] loader: Implement .hex file loader, Stefan Hajnoczi, 2018/08/03