diff options
| author | Markus Armbruster <armbru@redhat.com> | 2015-09-14 13:54:03 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2015-09-16 17:33:33 +0200 |
| commit | 2d528d45ecf5ee3c1a566a9f3d664464925ef830 (patch) | |
| tree | a5a41630f4c90434e690f8cb5c5b92b67296520f /backends/testdev.c | |
| parent | 2cd8af2d44268ffd4d224d6c297e8c644c01fa8d (diff) | |
| download | qemu-2d528d45ecf5ee3c1a566a9f3d664464925ef830.tar.gz qemu-2d528d45ecf5ee3c1a566a9f3d664464925ef830.tar.bz2 qemu-2d528d45ecf5ee3c1a566a9f3d664464925ef830.zip | |
qemu-char: 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>
Message-Id: <1442231643-23630-1-git-send-email-armbru@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'backends/testdev.c')
| -rw-r--r-- | backends/testdev.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/backends/testdev.c b/backends/testdev.c index eba396aeb9..1429152d05 100644 --- a/backends/testdev.c +++ b/backends/testdev.c @@ -113,8 +113,8 @@ CharDriverState *chr_testdev_init(void) TestdevCharState *testdev; CharDriverState *chr; - testdev = g_malloc0(sizeof(TestdevCharState)); - testdev->chr = chr = g_malloc0(sizeof(CharDriverState)); + testdev = g_new0(TestdevCharState, 1); + testdev->chr = chr = g_new0(CharDriverState, 1); chr->opaque = testdev; chr->chr_write = testdev_write; |