|
From: | Anthony Liguori |
Subject: | Re: [Qemu-trivial] [Qemu-devel] [PATCH] hw/tc58128.c: Remove unnecessary check for g_malloc failure |
Date: | Fri, 11 Nov 2011 13:45:12 -0600 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.21) Gecko/20110831 Lightning/1.0b2 Thunderbird/3.1.13 |
On 11/09/2011 01:34 PM, Peter Maydell wrote:
Remove a check for g_malloc failing: this never happens. Also use g_malloc rather than g_malloc0 as we immediately memset the entire region and so zero-initialising it is pointless. Spotted by Coverity (see bug 887883). Signed-off-by: Peter Maydell<address@hidden>
Applied. Thanks. Regards, Anthony Liguori
--- hw/tc58128.c | 6 +----- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/hw/tc58128.c b/hw/tc58128.c index ee3ecad..4ce80b1 100644 --- a/hw/tc58128.c +++ b/hw/tc58128.c @@ -30,12 +30,8 @@ static void init_dev(tc58128_dev * dev, const char *filename) int ret, blocks; dev->state = WAIT; - dev->flash_contents = g_malloc0(FLASH_SIZE); + dev->flash_contents = g_malloc(FLASH_SIZE); memset(dev->flash_contents, 0xff, FLASH_SIZE); - if (!dev->flash_contents) { - fprintf(stderr, "could not alloc memory for flash\n"); - exit(1); - } if (filename) { /* Load flash image skipping the first block */ ret = load_image(filename, dev->flash_contents + 528 * 32);
[Prev in Thread] | Current Thread | [Next in Thread] |