diff options
| author | Markus Armbruster <armbru@redhat.com> | 2015-09-14 13:50:44 +0200 |
|---|---|---|
| committer | Michael Roth <mdroth@linux.vnet.ibm.com> | 2015-10-19 18:28:06 -0500 |
| commit | f3a06403b82c7f036564e4caf18b52ce6885fcfb (patch) | |
| tree | b44a6798691f6b40575983d264b8ea153dfd04e2 /qga/channel-win32.c | |
| parent | fafcaf1d7434bd3a44a5cd6a98594b3ec12d83de (diff) | |
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)) { |