Re: [libvirt] [PATCH 1/1] libvirt patch to write a mcs translation file to /run/setrans directory

From: Dan Walsh <dwalsh(a)redhat.com&gt; mcstransd is a translation tool that can translate MCS Labels into human understandable code. I have patched it to watch for translation files in the /run/setrans directory. This allows us to run commands like ps -eZ and see system_u:system_r:svirt_t:Fedora18 rather then system_u:system_r:svirt_t:s0:c1,c2. When used with containers it would make an easy way to list all processes within a container using ps -eZ | grep Fedora18 --- src/security/security_selinux.c | 56 ++++++++++++++++++++++++++++++++++++++++- 1 file changed, 55 insertions(+), 1 deletion(-) diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c index 61ff1de..bf32755 100644 --- a/src/security/security_selinux.c +++ b/src/security/security_selinux.c @@ -82,6 +82,55 @@ virSecuritySELinuxRestoreSecurityTPMFileLabelInt(virSecurityManagerPtr mgr, virDomainTPMDefPtr tpm); +static int +virSecuritySELinuxAddMCSFile(const char *name, + const char *label) +{ + int rc = -1;

+ char *tmp = NULL; + context_t con = NULL; + + if (virAsprintf(&tmp, "%s/%s", SELINUX_TRANS_DIR, name) == -1) {

+ virReportOOMError(); + return -1; + } + + if (! (con = context_new(label))) {

+ virReportSystemError(errno, "%s", + _("unable to allocate security context")); + goto done; + } + rc = virFileWriteStr(tmp, context_range_get(con), 0); + if (rc) {

+ virReportSystemError(errno, + _("unable to create MCS file %s"), tmp);

+ }

+ +done:

+ VIR_FREE(tmp); + context_free(con); + return rc;

+} + +static int +virSecuritySELinuxRemoveMCSFile(const char *name) +{ + char *tmp=NULL; + int rc = 0;

+ if (virAsprintf(&tmp, "%s/%s", SELINUX_TRANS_DIR, name) == -1) {

+ }

+ virReportSystemError(errno, + _("Unable to remove MCS file %s"), tmp); + rc = -1;

+ }

+ return rc;

+} + /* * Returns 0 on success, 1 if already reserved, or -1 on fatal error */

@@ -1977,6 +2026,11 @@ virSecuritySELinuxSetSecurityProcessLabel(virSecurityManagerPtr mgr ATTRIBUTE_UN return -1; } + if (virSecuritySELinuxAddMCSFile(def->name, secdef->label)) {

+ if (security_getenforce() == 1) + return -1;

+ }

+ if (setexeccon_raw(secdef->label) == -1) { virReportSystemError(errno, _("unable to set security context '%s'"),