From 48ad0f4e8f36918406b95e498f6750e5d5cf749a Mon Sep 17 00:00:00 2001
From: Daniel J Walsh <dwalsh@redhat.com>
Date: Wed, 1 Jul 2020 09:19:18 -0400
Subject: [PATCH] Don't disable selinux labels if user specifies a security opt
Currenty if the user specifies --pid=host or --ipc=host or --privileged
then we disable SELinux labeling. If the user however specifies
--security-opt label:... Then we assume they want to leave SELinux enabled
and know what they are doing.
This PR will leave SELinux enabled if a user specifies a --security-opt label
option.
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
---
cmd/podman/shared/create.go | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/cmd/podman/shared/create.go b/cmd/podman/shared/create.go
index 7bb2bc896d..dda36826ec 100644
--- a/cmd/podman/shared/create.go
+++ b/cmd/podman/shared/create.go
@@ -198,9 +198,7 @@ func CreateContainer(ctx context.Context, c *GenericCLIResults, runtime *libpod.
func parseSecurityOpt(config *cc.CreateConfig, securityOpts []string, runtime *libpod.Runtime) error {
var labelOpts []string
- if config.PidMode.IsHost() {
- labelOpts = append(labelOpts, label.DisableSecOpt()...)
- } else if config.PidMode.IsContainer() {
+ if config.PidMode.IsContainer() {
ctr, err := runtime.LookupContainer(config.PidMode.Container())
if err != nil {
return errors.Wrapf(err, "container %q not found", config.PidMode.Container())
@@ -212,9 +210,7 @@ func parseSecurityOpt(config *cc.CreateConfig, securityOpts []string, runtime *l
labelOpts = append(labelOpts, secopts...)
}
- if config.IpcMode.IsHost() {
- labelOpts = append(labelOpts, label.DisableSecOpt()...)
- } else if config.IpcMode.IsContainer() {
+ if config.IpcMode.IsContainer() {
ctr, err := runtime.LookupContainer(config.IpcMode.Container())
if err != nil {
return errors.Wrapf(err, "container %q not found", config.IpcMode.Container())
@@ -255,7 +251,14 @@ func parseSecurityOpt(config *cc.CreateConfig, securityOpts []string, runtime *l
return err
}
}
- config.LabelOpts = labelOpts
+ if len(labelOpts) > 0 {
+ config.LabelOpts = labelOpts
+ } else {
+ if config.Privileged || config.IpcMode.IsHost() || config.PidMode.IsHost() {
+ config.LabelOpts = label.DisableSecOpt()
+ }
+ }
+
return nil
}
@@ -795,9 +798,6 @@ func ParseCreateOpts(ctx context.Context, c *GenericCLIResults, runtime *libpod.
if err := parseSecurityOpt(config, c.StringArray("security-opt"), runtime); err != nil {
return nil, err
}
- if config.Privileged && len(config.LabelOpts) == 0 {
- config.LabelOpts = label.DisableSecOpt()
- }
config.SecurityOpts = c.StringArray("security-opt")
warnings, err := verifyContainerResources(config, false)
if err != nil {