summaryrefslogtreecommitdiffstats
path: root/ui/console.c
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2015-11-03 17:12:03 +0100
committerMichael Tokarev <mjt@tls.msk.ru>2015-11-06 15:42:38 +0300
commitfedf0d35aafc4f1f1e5f6dbc80cb23ae1ae49f0b (patch)
tree0585de21d60bf249c5b37b9a6f742e87b0926377 /ui/console.c
parent374ec0669a1aa3affac7850a16c6cad18221c439 (diff)
downloadhqemu-fedf0d35aafc4f1f1e5f6dbc80cb23ae1ae49f0b.zip
hqemu-fedf0d35aafc4f1f1e5f6dbc80cb23ae1ae49f0b.tar.gz
ui: 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 Tokarev <mjt@tls.msk.ru>
Diffstat (limited to 'ui/console.c')
-rw-r--r--ui/console.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/ui/console.c b/ui/console.c
index f26544e..745c354 100644
--- a/ui/console.c
+++ b/ui/console.c
@@ -450,7 +450,7 @@ static void text_console_resize(QemuConsole *s)
if (s->width < w1)
w1 = s->width;
- cells = g_malloc(s->width * s->total_height * sizeof(TextCell));
+ cells = g_new(TextCell, s->width * s->total_height);
for(y = 0; y < s->total_height; y++) {
c = &cells[y * s->width];
if (w1 > 0) {
OpenPOWER on IntegriCloud