diff options
| author | Christian Heimes <cheimes@redhat.com> | 2020-08-11 13:29:17 +0200 |
|---|---|---|
| committer | Matthew Heon <matthew.heon@pm.me> | 2020-08-20 12:16:52 -0400 |
| commit | 7fb53bc240cea153fb054bcd307d3b1a8945a435 (patch) | |
| tree | 0ca9c31af6e4718f620ab7f8143dfecf3ad99a79 | |
| parent | 76ae0c907591b60374febdd8bd41f90cbc5444b6 (diff) | |
| download | podman-7fb53bc240cea153fb054bcd307d3b1a8945a435.tar.gz podman-7fb53bc240cea153fb054bcd307d3b1a8945a435.tar.bz2 podman-7fb53bc240cea153fb054bcd307d3b1a8945a435.zip | |
Use set for systemd commands
Signed-off-by: Christian Heimes <cheimes@redhat.com>
| -rw-r--r-- | pkg/specgen/generate/container_create.go | 7 | ||||
| -rw-r--r-- | pkg/varlinkapi/create.go | 7 |
2 files changed, 12 insertions, 2 deletions
diff --git a/pkg/specgen/generate/container_create.go b/pkg/specgen/generate/container_create.go index 630e59854..b31bc91e0 100644 --- a/pkg/specgen/generate/container_create.go +++ b/pkg/specgen/generate/container_create.go @@ -153,7 +153,12 @@ func createContainerOptions(ctx context.Context, rt *libpod.Runtime, s *specgen. } if len(command) > 0 { - if command[0] == "/usr/sbin/init" || command[0] == "/sbin/init" || command[0] == "/usr/local/sbin/init" || (filepath.Base(command[0]) == "systemd") { + useSystemdCommands := map[string]bool{ + "/sbin/init": true, + "/usr/sbin/init": true, + "/usr/local/sbin/init": true, + } + if useSystemdCommands[command[0]] || (filepath.Base(command[0]) == "systemd") { useSystemd = true } } diff --git a/pkg/varlinkapi/create.go b/pkg/varlinkapi/create.go index 249505486..7661173b8 100644 --- a/pkg/varlinkapi/create.go +++ b/pkg/varlinkapi/create.go @@ -704,7 +704,12 @@ func ParseCreateOpts(ctx context.Context, c *GenericCLIResults, runtime *libpod. if err != nil { return nil, errors.Wrapf(err, "cannot parse bool %s", c.String("systemd")) } - if x && (command[0] == "/usr/sbin/init" || command[0] == "/sbin/init" || command[0] == "/usr/local/sbin/init" || (filepath.Base(command[0]) == "systemd")) { + useSystemdCommands := map[string]bool{ + "/sbin/init": true, + "/usr/sbin/init": true, + "/usr/local/sbin/init": true, + } + if x && (useSystemdCommands[command[0]] || (filepath.Base(command[0]) == "systemd")) { systemd = true } } |