This patch reworks the SELinux patch to be only run on demand by the daemon

Added --selinux-enable switch to daemon to enable SELinux labeling.

The daemon will now generate a new unique random SELinux label when a
container starts, and remove it when the container is removed.   The MCS
labels will be stored in the daemon memory.  The labels of containers will
be stored in the container.json file.

When the daemon restarts on boot or if done by an admin, it will read all containers json files and reserve the MCS labels.

A potential problem would be conflicts if you setup thousands of containers,
current scheme would handle ~500,000 containers.

Docker-DCO-1.1-Signed-off-by: Dan Walsh <dwalsh@redhat.com> (github: rhatdan)
Docker-DCO-1.1-Signed-off-by: Dan Walsh <dwalsh@redhat.com> (github: crosbymichael)
This commit is contained in:
Dan Walsh 2014-04-21 17:09:26 -04:00 committed by Michael Crosby
parent 48d893cc6b
commit d0559e92af
4 changed files with 34 additions and 4 deletions

View file

@ -24,3 +24,7 @@ func GetPidCon(pid int) (string, error) {
func Init() { func Init() {
} }
func ReserveLabel(label string) error {
return nil
}

View file

@ -75,3 +75,7 @@ func GetPidCon(pid int) (string, error) {
func Init() { func Init() {
selinux.SelinuxEnabled() selinux.SelinuxEnabled()
} }
func ReserveLabel(label string) {
selinux.ReserveLabel(label)
}

View file

@ -204,6 +204,13 @@ func NewContext(scon string) SELinuxContext {
return c return c
} }
func ReserveLabel(scon string) {
if len(scon) != 0 {
con := strings.SplitN(scon, ":", 4)
mcsAdd(con[3])
}
}
func SelinuxGetEnforce() int { func SelinuxGetEnforce() int {
var enforce int var enforce int
@ -229,8 +236,12 @@ func SelinuxGetEnforceMode() int {
return Disabled return Disabled
} }
func mcsAdd(mcs string) { func mcsAdd(mcs string) error {
if mcsList[mcs] {
return fmt.Errorf("MCS Label already exists")
}
mcsList[mcs] = true mcsList[mcs] = true
return nil
} }
func mcsDelete(mcs string) { func mcsDelete(mcs string) {
@ -283,15 +294,21 @@ func uniqMcs(catRange uint32) string {
} }
} }
mcs = fmt.Sprintf("s0:c%d,c%d", c1, c2) mcs = fmt.Sprintf("s0:c%d,c%d", c1, c2)
if mcsExists(mcs) { if err := mcsAdd(mcs); err != nil {
continue continue
} }
mcsAdd(mcs)
break break
} }
return mcs return mcs
} }
func FreeLxcContexts(scon string) {
if len(scon) != 0 {
con := strings.SplitN(scon, ":", 4)
mcsDelete(con[3])
}
}
func GetLxcContexts() (processLabel string, fileLabel string) { func GetLxcContexts() (processLabel string, fileLabel string) {
var ( var (
val, key string val, key string
@ -344,7 +361,8 @@ func GetLxcContexts() (processLabel string, fileLabel string) {
} }
exit: exit:
mcs := IntToMcs(os.Getpid(), 1024) // mcs := IntToMcs(os.Getpid(), 1024)
mcs := uniqMcs(1024)
scon := NewContext(processLabel) scon := NewContext(processLabel)
scon["level"] = mcs scon["level"] = mcs
processLabel = scon.Get() processLabel = scon.Get()
@ -373,6 +391,8 @@ func CopyLevel(src, dest string) (string, error) {
} }
scon := NewContext(src) scon := NewContext(src)
tcon := NewContext(dest) tcon := NewContext(dest)
mcsDelete(tcon["level"])
mcsAdd(scon["level"])
tcon["level"] = scon["level"] tcon["level"] = scon["level"]
return tcon.Get(), nil return tcon.Get(), nil
} }

View file

@ -31,9 +31,11 @@ func TestSELinux(t *testing.T) {
plabel, flabel = selinux.GetLxcContexts() plabel, flabel = selinux.GetLxcContexts()
t.Log(plabel) t.Log(plabel)
t.Log(flabel) t.Log(flabel)
selinux.FreeLxcContexts(plabel)
plabel, flabel = selinux.GetLxcContexts() plabel, flabel = selinux.GetLxcContexts()
t.Log(plabel) t.Log(plabel)
t.Log(flabel) t.Log(flabel)
selinux.FreeLxcContexts(plabel)
t.Log("getenforce ", selinux.SelinuxGetEnforce()) t.Log("getenforce ", selinux.SelinuxGetEnforce())
t.Log("getenforcemode ", selinux.SelinuxGetEnforceMode()) t.Log("getenforcemode ", selinux.SelinuxGetEnforceMode())
pid := os.Getpid() pid := os.Getpid()