diff options
| author | Stefan Weil | 2012-09-04 22:14:19 +0200 |
|---|---|---|
| committer | Stefan Hajnoczi | 2012-09-23 08:35:29 +0200 |
| commit | ad11ad77748bdd8016370db210751683dc038dd6 (patch) | |
| tree | a4b9696072345e03e7a4ad4c750721b8ed1008a9 /linux-user/syscall.c | |
| parent | qemu-timer: simplify qemu_run_timers (diff) | |
| download | qemu-ad11ad77748bdd8016370db210751683dc038dd6.tar.gz qemu-ad11ad77748bdd8016370db210751683dc038dd6.tar.xz qemu-ad11ad77748bdd8016370db210751683dc038dd6.zip | |
linux-user: Remove redundant null check and replace free by g_free
Report from smatch:
linux-user/syscall.c:3632 do_ioctl_dm(220) info:
redundant null check on big_buf calling free()
'big_buf' was allocated by g_malloc0, therefore free was also
replaced by g_free.
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Stefan Hajnoczi <stefanha@gmail.com>
Diffstat (limited to 'linux-user/syscall.c')
| -rw-r--r-- | linux-user/syscall.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 6257a04d0a..471d0605f7 100644 --- a/linux-user/syscall.c +++ b/linux-user/syscall.c @@ -3628,9 +3628,7 @@ static abi_long do_ioctl_dm(const IOCTLEntry *ie, uint8_t *buf_temp, int fd, unlock_user(argptr, arg, target_size); } out: - if (big_buf) { - free(big_buf); - } + g_free(big_buf); return ret; } |