diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2020-06-09 21:16:12 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-06-09 21:16:12 -0400 |
| commit | 2fa7f03a358c2be4521030f24086e12fb990746a (patch) | |
| tree | 7f41fdafae0074d5c17e9cf91b1a04ad06284083 /pkg/domain | |
| parent | fbe09d78e91c9ac5cadc8b00a67c7d7f89d64868 (diff) | |
| parent | 817dbdfdf8ee10c524a15f7247a3742cb926f96b (diff) | |
| download | podman-2fa7f03a358c2be4521030f24086e12fb990746a.tar.gz podman-2fa7f03a358c2be4521030f24086e12fb990746a.tar.bz2 podman-2fa7f03a358c2be4521030f24086e12fb990746a.zip | |
Merge pull request #6542 from mheon/fix_pod_kill_signalerr
Ensure signal validation happens first in pod kill
Diffstat (limited to 'pkg/domain')
| -rw-r--r-- | pkg/domain/infra/tunnel/pods.go | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/pkg/domain/infra/tunnel/pods.go b/pkg/domain/infra/tunnel/pods.go index b93c48aab..1eb1fffd7 100644 --- a/pkg/domain/infra/tunnel/pods.go +++ b/pkg/domain/infra/tunnel/pods.go @@ -7,6 +7,7 @@ import ( "github.com/containers/libpod/pkg/bindings/pods" "github.com/containers/libpod/pkg/domain/entities" "github.com/containers/libpod/pkg/specgen" + "github.com/containers/libpod/pkg/util" "github.com/pkg/errors" ) @@ -19,6 +20,12 @@ func (ic *ContainerEngine) PodKill(ctx context.Context, namesOrIds []string, opt var ( reports []*entities.PodKillReport ) + + _, err := util.ParseSignal(options.Signal) + if err != nil { + return nil, err + } + foundPods, err := getPodsByContext(ic.ClientCxt, options.All, namesOrIds) if err != nil { return nil, err |