util/compatfd.c: Replaced a malloc call with g_malloc.
authorMahmoud Mandour <ma.mandourr@gmail.com>
Mon, 15 Mar 2021 10:58:14 +0000 (12:58 +0200)
committerThomas Huth <thuth@redhat.com>
Fri, 14 May 2021 10:28:01 +0000 (12:28 +0200)
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>
util/compatfd.c

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