diff mbox series

[73/88] hw/block/xen_disk: avoid use of g_new0()

Message ID	20171006235023.11952-74-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>, Jeff Cody <jcody@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Max Reitz <mreitz@redhat.com>, Jan Beulich <jbeulich@suse.com> Date: Fri, 6 Oct 2017 20:50:08 -0300 Message-Id: <20171006235023.11952-74-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 73/88] hw/block/xen_disk: avoid use of g_new0() Precedence: list Cc: qemu-block@nongnu.org, qemu trival <qemu-trivial@nongnu.org>, qemu-devel@nongnu.org, =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= <f4bug@amsat.org>, Paul Durrant <paul.durrant@citrix.com>, Anthony Perard <anthony.perard@citrix.com>, =?utf-8?q?Marc-Andr=C3=A9_Lureau?= <marcandre.lureau@redhat.com> 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

Commit Message

Philippe Mathieu-Daudé Oct. 6, 2017, 11:50 p.m. UTC

From: Jan Beulich <jbeulich@suse.com>

Prefer g_new() / g_new0() to be farther backwards compatible with older
glib versions. As there's no point in zeroing the allocation here (the
loop right afterwards fully initializes the memory), use the former.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Acked-by: Anthony Perard <anthony.perard@citrix.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
[PMD: rebase & subject tweak]
---
 hw/block/xen_disk.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff mbox series

Patch

diff --git a/hw/block/xen_disk.c b/hw/block/xen_disk.c
index 536e2ee735..6d2fd4d284 100644
--- a/hw/block/xen_disk.c
+++ b/hw/block/xen_disk.c
@@ -1232,7 +1232,7 @@  static int blk_connect(struct XenDevice *xendev)
         return -1;
     }
 
-    domids = g_new0(uint32_t, blkdev->nr_ring_ref);
+    domids = g_new(uint32_t, blkdev->nr_ring_ref);
     for (i = 0; i < blkdev->nr_ring_ref; i++) {
         domids[i] = blkdev->xendev.dom;
     }