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/api/handlers | |
| 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/api/handlers')
| -rw-r--r-- | pkg/api/handlers/libpod/pods.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/pkg/api/handlers/libpod/pods.go b/pkg/api/handlers/libpod/pods.go index c3f8d5d66..d998167d0 100644 --- a/pkg/api/handlers/libpod/pods.go +++ b/pkg/api/handlers/libpod/pods.go @@ -17,6 +17,7 @@ import ( "github.com/containers/libpod/pkg/util" "github.com/gorilla/schema" "github.com/pkg/errors" + "github.com/sirupsen/logrus" ) func PodCreate(w http.ResponseWriter, r *http.Request) { @@ -375,6 +376,7 @@ func PodKill(w http.ResponseWriter, r *http.Request) { sig, err := util.ParseSignal(signal) if err != nil { utils.InternalServerError(w, errors.Wrapf(err, "unable to parse signal value")) + return } name := utils.GetName(r) pod, err := runtime.LookupPod(name) @@ -382,6 +384,7 @@ func PodKill(w http.ResponseWriter, r *http.Request) { utils.PodNotFound(w, name, err) return } + logrus.Debugf("Killing pod %s with signal %d", pod.ID(), sig) podStates, err := pod.Status() if err != nil { utils.Error(w, "Something went wrong.", http.StatusInternalServerError, err) |