summaryrefslogtreecommitdiffstats
path: root/hw/vfio/platform.c
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2015-11-10 12:11:08 -0700
committerAlex Williamson <alex.williamson@redhat.com>2015-11-10 12:11:08 -0700
commitbdd81addf4033ce26e6cd180b060f63095f3ded9 (patch)
tree3f5637ba3c46fe0a198d1d52c42221f71237922e /hw/vfio/platform.c
parent0282abf078c3353a178ab77a115828ce333181dd (diff)
downloadhqemu-bdd81addf4033ce26e6cd180b060f63095f3ded9.zip
hqemu-bdd81addf4033ce26e6cd180b060f63095f3ded9.tar.gz
vfio: Use g_new() & friends where that makes obvious sense
g_new(T, n) is neater than g_malloc(sizeof(T) * n). It's also safer, for two reasons. One, it catches multiplication overflowing size_t. Two, it returns T * rather than void *, which lets the compiler catch more type errors. This commit only touches allocations with size arguments of the form sizeof(T). Same Coccinelle semantic patch as in commit b45c03f. Signed-off-by: Markus Armbruster <armbru@redhat.com> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Diffstat (limited to 'hw/vfio/platform.c')
-rw-r--r--hw/vfio/platform.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/vfio/platform.c b/hw/vfio/platform.c
index 5c1156c..289b498 100644
--- a/hw/vfio/platform.c
+++ b/hw/vfio/platform.c
@@ -478,7 +478,7 @@ static int vfio_populate_device(VFIODevice *vbasedev)
struct vfio_region_info reg_info = { .argsz = sizeof(reg_info) };
VFIORegion *ptr;
- vdev->regions[i] = g_malloc0(sizeof(VFIORegion));
+ vdev->regions[i] = g_new0(VFIORegion, 1);
ptr = vdev->regions[i];
reg_info.index = i;
ret = ioctl(vbasedev->fd, VFIO_DEVICE_GET_REGION_INFO, &reg_info);
OpenPOWER on IntegriCloud