diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2021-09-15 06:23:00 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-15 06:23:00 -0400 |
| commit | 10873c6f494fbc7a43365000f6f9702c21741077 (patch) | |
| tree | 8eb5e73ac7144c4918d95c31a9b0b68746038b53 /pkg/api/handlers/compat | |
| parent | aff64dda65514064ccbf7fcaf78293e111539eb6 (diff) | |
| parent | 9b04e17893dbf91326c0f68ce7627d7bb07cf515 (diff) | |
| download | podman-10873c6f494fbc7a43365000f6f9702c21741077.tar.gz podman-10873c6f494fbc7a43365000f6f9702c21741077.tar.bz2 podman-10873c6f494fbc7a43365000f6f9702c21741077.zip | |
Merge pull request #11578 from jelly/handle_nil_pointer_deref
api: handle nil pointer dereference in api endpoints
Diffstat (limited to 'pkg/api/handlers/compat')
| -rw-r--r-- | pkg/api/handlers/compat/containers.go | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/pkg/api/handlers/compat/containers.go b/pkg/api/handlers/compat/containers.go index 26e1bf00b..a15fdb553 100644 --- a/pkg/api/handlers/compat/containers.go +++ b/pkg/api/handlers/compat/containers.go @@ -104,8 +104,12 @@ func ListContainers(w http.ResponseWriter, r *http.Request) { } filterMap, err := util.PrepareFilters(r) + if err != nil { + utils.Error(w, "Something went wrong.", http.StatusInternalServerError, errors.Wrapf(err, "failed to decode filter parameters for %s", r.URL.String())) + return + } - if dErr := decoder.Decode(&query, r.URL.Query()); dErr != nil || err != nil { + if err := decoder.Decode(&query, r.URL.Query()); err != nil { utils.Error(w, "Something went wrong.", http.StatusInternalServerError, errors.Wrapf(err, "failed to parse parameters for %s", r.URL.String())) return } |