virSecuritySELinuxRestoreFileLabels: Refactor variable freeing

Declare 'filename' inside the loop that is using it and use automatic
freeing.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c
index f042da4a139501962abb098a98863ace93a694e6..854e46954e5a911c93feb3ab3d4010ee92a8fad4 100644 (file)
--- a/src/security/security_selinux.c
+++ b/src/security/security_selinux.c
@@ -3698,7 +3698,6 @@ virSecuritySELinuxRestoreFileLabels(virSecurityManager *mgr,
 {
     int ret = 0;
     struct dirent *ent;
-    char *filename = NULL;
     g_autoptr(DIR) dir = NULL;
 
     if ((ret = virSecuritySELinuxRestoreFileLabel(mgr, path, true)))
@@ -3711,9 +3710,8 @@ virSecuritySELinuxRestoreFileLabels(virSecurityManager *mgr,
         return -1;
 
     while ((ret = virDirRead(dir, &ent, path)) > 0) {
-        filename = g_strdup_printf("%s/%s", path, ent->d_name);
+        g_autofree char *filename = g_strdup_printf("%s/%s", path, ent->d_name);
         ret = virSecuritySELinuxRestoreFileLabel(mgr, filename, true);
-        VIR_FREE(filename);
         if (ret < 0)
             break;
     }