motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-26 11:32:23 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

util/compatfd.c: Replaced a malloc call with g_malloc.

Browse source 
Replaced a call to malloc() and its respective call to free()
with g_malloc() and g_free().

g_malloc() is preferred more than g_try_* functions, which
return NULL on error, when the size of the requested
allocation  is small. This is because allocating few
bytes should not be a problem in a healthy system.
Otherwise, the system is already in a critical state.

Subsequently, removed NULL-checking after g_malloc().

Signed-off-by: Mahmoud Mandour <ma.mandourr@gmail.com>
Message-Id: <20210315105814.5188-3-ma.mandourr@gmail.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
This commit is contained in:
Mahmoud Mandour 2021-03-15 12:58:14 +02:00 • committed by Thomas Huth
parent 20868330a9
commit e0c5a18efc
1 changed files with 2 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

8
util/compatfd.c
View file

@ -72,14 +72,10 @@ static int qemu_signalfd_compat(const sigset_t *mask)
QemuThread thread; QemuThread thread;
int fds[2]; int fds[2];
info = malloc(sizeof(*info)); info = g_malloc(sizeof(*info));
if (info == NULL) {
errno = ENOMEM;
return -1;
}
if (pipe(fds) == -1) { if (pipe(fds) == -1) {
free(info); g_free(info);
return -1; return -1;
} }