From 53ec1abec228e5121d5de4acaab08ed26ad70b9c Mon Sep 17 00:00:00 2001 From: Shiming Zhang Date: Wed, 21 Jul 2021 10:41:03 +0800 Subject: [PATCH] remotes/docker/pusher.go: Fix missing Close() Signed-off-by: Shiming Zhang --- remotes/docker/pusher.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/remotes/docker/pusher.go b/remotes/docker/pusher.go index eae5a251d..97ed66a6a 100644 --- a/remotes/docker/pusher.go +++ b/remotes/docker/pusher.go @@ -143,13 +143,16 @@ func (p dockerPusher) push(ctx context.Context, desc ocispec.Descriptor, ref str // TODO: Set updated time? }, }) + resp.Body.Close() return nil, errors.Wrapf(errdefs.ErrAlreadyExists, "content %v on remote", desc.Digest) } } else if resp.StatusCode != http.StatusNotFound { err := remoteserrors.NewUnexpectedStatusErr(resp) log.G(ctx).WithField("resp", resp).WithField("body", string(err.(remoteserrors.ErrUnexpectedStatus).Body)).Debug("unexpected response") + resp.Body.Close() return nil, err } + resp.Body.Close() } if isManifest {