[PATCH 08/13] securityselinuxhelper: Use g_new0() instead of malloc()+memset() combo

Michal Privoznik mprivozn at redhat.com
Thu Aug 3 10:36:15 UTC 2023


Inside of securityselinuxhelper we still use malloc() +
memset(.., 0, ...) combo. Convert it to g_new0().

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 tests/securityselinuxhelper.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/tests/securityselinuxhelper.c b/tests/securityselinuxhelper.c
index c32c90c17e..797f090a7e 100644
--- a/tests/securityselinuxhelper.c
+++ b/tests/securityselinuxhelper.c
@@ -173,9 +173,8 @@ int getfilecon_raw(const char *path, char **con)
     }
     if (len < 0)
         return -1;
-    if (!(constr = malloc(len+1)))
-        return -1;
-    memset(constr, 0, len);
+
+    constr = g_new0(char, len + 1);
     if (getxattr(path, "user.libvirt.selinux", constr, len) < 0) {
         free(constr);
         return -1;
-- 
2.41.0



More information about the libvir-list mailing list