diff options
author | Markus Armbruster | 2015-09-14 13:50:44 +0200 |
---|---|---|
committer | Michael Roth | 2015-10-20 01:28:06 +0200 |
commit | f3a06403b82c7f036564e4caf18b52ce6885fcfb (patch) | |
tree | b44a6798691f6b40575983d264b8ea153dfd04e2 /qga/channel-win32.c | |
parent | build: qemu-ga: add 'qemu-ga' build target for w32 (diff) | |
download | qemu-f3a06403b82c7f036564e4caf18b52ce6885fcfb.tar.gz qemu-f3a06403b82c7f036564e4caf18b52ce6885fcfb.tar.xz qemu-f3a06403b82c7f036564e4caf18b52ce6885fcfb.zip |
qga: 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>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
Diffstat (limited to 'qga/channel-win32.c')
-rw-r--r-- | qga/channel-win32.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qga/channel-win32.c b/qga/channel-win32.c index 04fa5e4d1d..215b15a7f5 100644 --- a/qga/channel-win32.c +++ b/qga/channel-win32.c @@ -322,7 +322,7 @@ static gboolean ga_channel_open(GAChannel *c, GAChannelMethod method, GAChannel *ga_channel_new(GAChannelMethod method, const gchar *path, GAChannelCallback cb, gpointer opaque) { - GAChannel *c = g_malloc0(sizeof(GAChannel)); + GAChannel *c = g_new0(GAChannel, 1); SECURITY_ATTRIBUTES sec_attrs; if (!ga_channel_open(c, method, path)) { |