[44/88] hw/core/loader: use g_new() family of functions

Message ID	20171006235023.11952-45-f4bug@amsat.org
State	New
Headers	show Return-Path: <qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org> From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= <f4bug@amsat.org> To: =?utf-8?q?Marc-Andr=C3=A9_Lureau?= <marcandre.lureau@gmail.com>, Markus Armbruster <armbru@redhat.com>, Eric Blake <eblake@redhat.com>, Alistair Francis <alistair.francis@xilinx.com> Date: Fri, 6 Oct 2017 20:49:39 -0300 Message-Id: <20171006235023.11952-45-f4bug@amsat.org> In-Reply-To: <20171006235023.11952-1-f4bug@amsat.org> References: <20171006235023.11952-1-f4bug@amsat.org> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [PATCH 44/88] hw/core/loader: use g_new() family of functions Precedence: list Cc: Kevin Wolf <kwolf@redhat.com>, =?utf-8?q?Marc-Andr=C3=A9_Lureau?= <marcandre.lureau@redhat.com>, qemu trival <qemu-trivial@nongnu.org>, =?utf-8?q?Philippe_Mathieu-Dau?= =?utf-8?b?ZMOp?= <f4bug@amsat.org>, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" <qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>
Series	use g_new() family of functions \| expand [00/88] use g_new() family of functions [01/88] cocci: script to use g_new() & friends [02/88] cocci: add more g_new() transformations [03/88] cocci: extract typeof() from g_new() [04/88] cocci: avoid use of g_new0() [05/88] cocci: use g_strfreev() [06/88] ARM: use g_new() family of functions [07/88] Audio: use g_new() family of functions [08/88] BT: use g_new() family of functions [09/88] Bootdevice: use g_new() family of functions [10/88] Character devices: use g_new() family of functions [11/88] Cryptodev Backends: use g_new() family of functions [12/88] backends/hostmem: use g_new() family of functions [13/88] Dirty Bitmaps: use g_new() family of functions [14/88] ui/vnc: use g_new() family of functions [15/88] ui/spice: use g_new() family of functions [16/88] Main loop: use g_new() family of functions [17/88] Memory API: use g_new() family of functions [18/88] memory: use g_new() family of functions [19/88] exec: use g_new() family of functions [20/88] Migration: use g_new() family of functions [21/88] NBD: use g_new() family of functions [22/88] LM32: use g_new() family of functions [23/88] M68K: use g_new() family of functions [24/88] MIPS: use g_new() family of functions [25/88] OpenRISC: use g_new() family of functions [26/88] S390: use g_new() family of functions [27/88] SH4: use g_new() family of functions [28/88] disas: use g_new() family of functions [29/88] SPARC: use g_new() family of functions [30/88] QEMU Guest Agent: use g_new() family of functions [31/88] QMP: use g_new() family of functions [32/88] QObject: use g_new() family of functions [33/88] qom: use g_new() family of functions [34/88] qapi: use g_new() family of functions [35/88] Record/replay: use g_new() family of functions [36/88] SLIRP: use g_new() family of functions [37/88] TCG: use g_new() family of functions [38/88] VFIO: use g_new() family of functions [39/88] hw/i386: use g_new() family of functions [40/88] hw/xen: use g_new() family of functions [41/88] target/i386: use g_new() family of functions [42/88] XTFPGA: use g_new() family of functions [43/88] hw/core: use g_new() family of functions [44/88] hw/core/loader: use g_new() family of functions [45/88] hw/display: use g_new() family of functions [46/88] hw/display/vga: use g_new() family of functions [47/88] hw/i2c: use g_new() family of functions [48/88] hw/input: use g_new() family of functions [49/88] hw/misc: use g_new() family of functions [50/88] ivshmem: use g_new() family of functions [51/88] hw/timer: use g_new() family of functions [52/88] i.MX31: use g_new() family of functions [53/88] iSCSI: use g_new() family of functions [54/88] KVM: use g_new() family of functions [55/88] PReP: use g_new() family of functions [56/88] PowerPC: use g_new() family of functions [57/88] ppc: use g_new() family of functions [58/88] ppc4xx: use g_new() family of functions [59/88] ppce500: use g_new() family of functions [60/88] sPAPR: use g_new() family of functions [61/88] tests: use g_new() family of functions [62/88] user: use g_new() family of functions [63/88] bsd-user/syscall: fix a memory leak on EFAULT, use g_new() [64/88] user: use g_strfreev() [65/88] util: use g_new() family of functions [66/88] util/qemu-thread-win32: use g_new() family of functions [67/88] util/coroutine: use g_new() family of functions [68/88] virtio: use g_new() family of functions [69/88] virtio-9p: use g_new() family of functions [70/88] block: use g_new() family of functions [71/88] block: avoid use of g_new0() [72/88] hw/block/nvme: use g_new() family of functions [73/88] hw/block/xen_disk: avoid use of g_new0() [74/88] balloon: use g_new() family of functions [75/88] decnumber: use g_new() family of functions [76/88] dma: use g_new() family of functions [77/88] dump: use g_new() family of functions [78/88] gdb: use g_new() family of functions [79/88] hmp: use g_new() family of functions [80/88] numa: use g_new() family of functions [81/88] hw/acpi: use g_new() family of functions [82/88] hw/ipmi: use g_new() family of functions [83/88] hw/pci: use g_new() family of functions [84/88] hw/scsi: use g_new() family of functions [85/88] hw/usb: use g_new() family of functions [86/88] net/hub: use g_new() family of functions [87/88] hw/net/rocker: use g_new() family of functions [88/88] scripts/coverity-model: use g_new() family of functions

Message ID

20171006235023.11952-45-f4bug@amsat.org

State

New

Headers

From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= <f4bug@amsat.org>
To: =?utf-8?q?Marc-Andr=C3=A9_Lureau?= <marcandre.lureau@gmail.com>,
	Markus Armbruster <armbru@redhat.com>, Eric Blake <eblake@redhat.com>, 
	Alistair Francis <alistair.francis@xilinx.com>
Date: Fri,  6 Oct 2017 20:49:39 -0300
Message-Id: <20171006235023.11952-45-f4bug@amsat.org>
In-Reply-To: <20171006235023.11952-1-f4bug@amsat.org>
References: <20171006235023.11952-1-f4bug@amsat.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Subject: [Qemu-devel] [PATCH 44/88] hw/core/loader: use g_new() family of
	functions
Precedence: list
Cc: Kevin Wolf <kwolf@redhat.com>, =?utf-8?q?Marc-Andr=C3=A9_Lureau?=
	<marcandre.lureau@redhat.com>, 	qemu trival <qemu-trivial@nongnu.org>,
	=?utf-8?q?Philippe_Mathieu-Dau?= =?utf-8?b?ZMOp?= <f4bug@amsat.org>,
	qemu-devel@nongnu.org
Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org
Sender: "Qemu-devel"
	<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>

Series

use g_new() family of functions | expand

Comments

Alistair Francis Oct. 10, 2017, 12:03 a.m. UTC | #1

On Fri, Oct 6, 2017 at 4:49 PM, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

Reviewed-by: Alistair Francis <alistair.francis@xilinx.com>

Thanks,
Alistair

> ---
>  hw/core/loader.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/hw/core/loader.c b/hw/core/loader.c
> index 4593061445..def4766dbb 100644
> --- a/hw/core/loader.c
> +++ b/hw/core/loader.c
> @@ -899,7 +899,7 @@ int rom_add_file(const char *file, const char *fw_dir,
>          return -1;
>      }
>
> -    rom = g_malloc0(sizeof(*rom));
> +    rom = g_new0(Rom, 1);
>      rom->name = g_strdup(file);
>      rom->path = qemu_find_file(QEMU_FILE_TYPE_BIOS, rom->name);
>      rom->as = as;
> @@ -996,7 +996,7 @@ MemoryRegion *rom_add_blob(const char *name, const void *blob, size_t len,
>      Rom *rom;
>      MemoryRegion *mr = NULL;
>
> -    rom           = g_malloc0(sizeof(*rom));
> +    rom           = g_new0(Rom, 1);
>      rom->name     = g_strdup(name);
>      rom->as       = as;
>      rom->addr     = addr;
> @@ -1039,7 +1039,7 @@ int rom_add_elf_program(const char *name, void *data, size_t datasize,
>  {
>      Rom *rom;
>
> -    rom           = g_malloc0(sizeof(*rom));
> +    rom = g_new0(Rom, 1);
>      rom->name     = g_strdup(name);
>      rom->addr     = addr;
>      rom->datasize = datasize;
> --
> 2.14.2
>
>

diff --git a/hw/core/loader.c b/hw/core/loader.c
index 4593061445..def4766dbb 100644
--- a/hw/core/loader.c
+++ b/hw/core/loader.c
@@ -899,7 +899,7 @@  int rom_add_file(const char *file, const char *fw_dir,
         return -1;
     }
 
-    rom = g_malloc0(sizeof(*rom));
+    rom = g_new0(Rom, 1);
     rom->name = g_strdup(file);
     rom->path = qemu_find_file(QEMU_FILE_TYPE_BIOS, rom->name);
     rom->as = as;
@@ -996,7 +996,7 @@  MemoryRegion *rom_add_blob(const char *name, const void *blob, size_t len,
     Rom *rom;
     MemoryRegion *mr = NULL;
 
-    rom           = g_malloc0(sizeof(*rom));
+    rom           = g_new0(Rom, 1);
     rom->name     = g_strdup(name);
     rom->as       = as;
     rom->addr     = addr;
@@ -1039,7 +1039,7 @@  int rom_add_elf_program(const char *name, void *data, size_t datasize,
 {
     Rom *rom;
 
-    rom           = g_malloc0(sizeof(*rom));
+    rom = g_new0(Rom, 1);
     rom->name     = g_strdup(name);
     rom->addr     = addr;
     rom->datasize = datasize;

[44/88] hw/core/loader: use g_new() family of functions

Commit Message

Comments

Patch