[PATCH v6 12/13] security: Always forget labels for TPM state directory
In the case of outgoing migration, we avoid restoring the
remembered labels for the TPM state directory because doing so
would risk cutting off storage access for the target node.
Even in that case though, we should still forget (unref) the
remembered labels: if we don't, the source node will keep
thinking that the state directory is in use.
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
src/security/security_selinux.c | 54 +++++++++++++++++++++++++++++++++
1 file changed, 54 insertions(+)
diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c
index 3e213a553b..4f13d305d9 100644
--- a/src/security/security_selinux.c
+++ b/src/security/security_selinux.c
@@ -210,6 +210,51 @@ virSecuritySELinuxRecallLabel(const char *path,
}
+/**
+ * virSecuritySELinuxForgetLabels:
+ * @path: file or directory to work on
+ *
+ * Forgets rememebered SELinux labels for @path, including its
+ * children if it is a directory.
+ *
+ * This is intended to be used in cleanup paths, so failure to forget
+ * a single label is not considered fatal; instead, a best-effort
+ * attempt to continue and forget as many labels as possible will be
+ * made.
+ *
+ * Returns: 0 on success, <0 on failure
+ */
+static int
+virSecuritySELinuxForgetLabels(const char *path)
+{
+ int ret = 0;
+ struct dirent *ent;
+ g_autoptr(DIR) dir = NULL;
+ g_autofree char *con = NULL;
+
+ if (virSecuritySELinuxRecallLabel(path, &con) < 0)
+ VIR_WARN("Failed to forget remembered SELinux labels for %s, ignoring",
path);
+
+ if (!virFileIsDir(path))
+ return 0;
+
+ if (virDirOpen(&dir, path) < 0)
+ return -1;
+
+ while ((ret = virDirRead(dir, &ent, path)) > 0) {
+ g_autofree char *spath = NULL;
+ g_autofree char *scon = NULL;
+
+ spath = g_strdup_printf("%s/%s", path, ent->d_name);
+
+ if (virSecuritySELinuxRecallLabel(spath, &scon) < 0)
+ VIR_WARN("Failed to forget remembered SELinux labels for %s,
ignoring", spath);
+ }
+
+ return ret;
+}
+
+
static int virSecuritySELinuxSetFilecon(virSecurityManager *mgr,
const char *path,
const char *tcon,
@@ -3709,6 +3754,15 @@ virSecuritySELinuxRestoreTPMLabels(virSecurityManager *mgr,
if (restoreTPMStateLabel) {
ret = virSecuritySELinuxRestoreFileLabels(mgr,
def->tpms[i]->data.emulator.storagepath);
+ } else {
+ g_autofree char *oldlabel = NULL;
+
+ /* Even if we're not restoring the original label for the
+ * TPM state directory, we should still forget any
+ * remembered label so that a subsequent attempt at TPM
+ * startup will not fail due to the state directory being
+ * considered as still in use */
+
ignore_value(virSecuritySELinuxForgetLabels(def->tpms[i]->data.emulator.storagepath));
}
if (ret == 0 &&
--
2.46.0