From: Michal Privoznik <mprivozn@redhat.com>
Date: Wed, 7 Oct 2015 09:16:08 +0000 (+0200)
Subject: security_dac: Fix TODO marks
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=d37d8f78c07bcb72c40a568970a84da3ed92b221;p=libvirt.git

security_dac: Fix TODO marks

Correctly mark the places where we need to remember and recall
file ownership. We don't want to mislead any potential developer.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
---

diff --git a/src/security/security_dac.c b/src/security/security_dac.c
index 864d75b2fd..0dfe570ba3 100644
--- a/src/security/security_dac.c
+++ b/src/security/security_dac.c
@@ -312,6 +312,7 @@ virSecurityDACSetOwnershipInternal(virSecurityDACDataPtr priv,
 static int
 virSecurityDACSetOwnership(const char *path, uid_t uid, gid_t gid)
 {
+    /* XXX record previous ownership */
     return virSecurityDACSetOwnershipInternal(NULL, NULL, path, uid, gid);
 }
 
@@ -324,7 +325,7 @@ virSecurityDACRestoreSecurityFileLabelInternal(virSecurityDACDataPtr priv,
     VIR_INFO("Restoring DAC user and group on '%s'",
              NULLSTR(src ? src->path : path));
 
-    /* XXX record previous ownership */
+    /* XXX recall previous ownership */
     return virSecurityDACSetOwnershipInternal(priv, src, path, 0, 0);
 }