Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(deps): update module github.com/docker/docker to v25 [security] (release-2.9.x) - abandoned #12288

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 21, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
github.com/docker/docker v23.0.8+incompatible -> v25.0.6+incompatible age adoption passing confidence

Moby's external DNS requests from 'internal' networks could lead to data exfiltration

CVE-2024-29018 / GHSA-mq39-4gv4-mvpx / GO-2024-2659

More information

Details

Moby is an open source container framework originally developed by Docker Inc. as Docker. It is a key component of Docker Engine, Docker Desktop, and other distributions of container tooling or runtimes. As a batteries-included container runtime, Moby comes with a built-in networking implementation that enables communication between containers, and between containers and external resources.

Moby's networking implementation allows for creating and using many networks, each with their own subnet and gateway. This feature is frequently referred to as custom networks, as each network can have a different driver, set of parameters, and thus behaviors. When creating a network, the --internal flag is used to designate a network as internal. The internal attribute in a docker-compose.yml file may also be used to mark a network internal, and other API clients may specify the internal parameter as well.

When containers with networking are created, they are assigned unique network interfaces and IP addresses (typically from a non-routable RFC 1918 subnet). The root network namespace (hereafter referred to as the 'host') serves as a router for non-internal networks, with a gateway IP that provides SNAT/DNAT to/from container IPs.

Containers on an internal network may communicate between each other, but are precluded from communicating with any networks the host has access to (LAN or WAN) as no default route is configured, and firewall rules are set up to drop all outgoing traffic. Communication with the gateway IP address (and thus appropriately configured host services) is possible, and the host may communicate with any container IP directly.

In addition to configuring the Linux kernel's various networking features to enable container networking, dockerd directly provides some services to container networks. Principal among these is serving as a resolver, enabling service discovery (looking up other containers on the network by name), and resolution of names from an upstream resolver.

When a DNS request for a name that does not correspond to a container is received, the request is forwarded to the configured upstream resolver (by default, the host's configured resolver). This request is made from the container network namespace: the level of access and routing of traffic is the same as if the request was made by the container itself.

As a consequence of this design, containers solely attached to internal network(s) will be unable to resolve names using the upstream resolver, as the container itself is unable to communicate with that nameserver. Only the names of containers also attached to the internal network are able to be resolved.

Many systems will run a local forwarding DNS resolver, typically present on a loopback address (127.0.0.0/8), such as systemd-resolved or dnsmasq. Common loopback address examples include 127.0.0.1 or 127.0.0.53. As the host and any containers have separate loopback devices, a consequence of the design described above is that containers are unable to resolve names from the host's configured resolver, as they cannot reach these addresses on the host loopback device.

To bridge this gap, and to allow containers to properly resolve names even when a local forwarding resolver is used on a loopback address, dockerd will detect this scenario and instead forward DNS requests from the host/root network namespace. The loopback resolver will then forward the requests to its configured upstream resolvers, as expected.

Impact

Because dockerd will forward DNS requests to the host loopback device, bypassing the container network namespace's normal routing semantics entirely, internal networks can unexpectedly forward DNS requests to an external nameserver.

By registering a domain for which they control the authoritative nameservers, an attacker could arrange for a compromised container to exfiltrate data by encoding it in DNS queries that will eventually be answered by their nameservers. For example, if the domain evil.example was registered, the authoritative nameserver(s) for that domain could (eventually and indirectly) receive a request for this-is-a-secret.evil.example.

Docker Desktop is not affected, as Docker Desktop always runs an internal resolver on a RFC 1918 address.

Patches

Moby releases 26.0.0-rc3, 25.0.5 (released) and 23.0.11 (to be released) are patched to prevent forwarding DNS requests from internal networks.

Workarounds
  • Run containers intended to be solely attached to internal networks with a custom upstream address (--dns argument to docker run, or API equivalent), which will force all upstream DNS queries to be resolved from the container network namespace.
Background
  • yair zak originally reported this issue to the Docker security team.
  • PR https://togithub.com/moby/moby/pull/46609 was opened in public to fix this issue, as it was not originally considered to have a security implication.
  • The official documentation claims that "the --internal flag that will completely isolate containers on a network from any communications external to that network," which necessitated this advisory and CVE.

Severity

  • CVSS Score: 5.9 / 10 (Medium)
  • Vector String: CVSS:3.1/AV:N/AC:H/PR:N/UI:N/S:U/C:H/I:N/A:N

References

This data is provided by OSV and the GitHub Advisory Database (CC-BY 4.0).


Classic builder cache poisoning

CVE-2024-24557 / GHSA-xw73-rw38-6vjc / GO-2024-2512

More information

Details

The classic builder cache system is prone to cache poisoning if the image is built FROM scratch.
Also, changes to some instructions (most important being HEALTHCHECK and ONBUILD) would not cause a cache miss.

An attacker with the knowledge of the Dockerfile someone is using could poison their cache by making them pull a specially crafted image that would be considered as a valid cache candidate for some build steps.

For example, an attacker could create an image that is considered as a valid cache candidate for:

FROM scratch
MAINTAINER Pawel

when in fact the malicious image used as a cache would be an image built from a different Dockerfile.

In the second case, the attacker could for example substitute a different HEALTCHECK command.

Impact

23.0+ users are only affected if they explicitly opted out of Buildkit (DOCKER_BUILDKIT=0 environment variable) or are using the /build API endpoint (which uses the classic builder by default).

All users on versions older than 23.0 could be impacted. An example could be a CI with a shared cache, or just a regular Docker user pulling a malicious image due to misspelling/typosquatting.

Image build API endpoint (/build) and ImageBuild function from github.com/docker/docker/client is also affected as it the uses classic builder by default.

Patches

Patches are included in Moby releases:

  • v25.0.2
  • v24.0.9
  • v23.0.10
Workarounds
  • Use --no-cache or use Buildkit if possible (DOCKER_BUILDKIT=1, it's default on 23.0+ assuming that the buildx plugin is installed).
  • Use Version = types.BuilderBuildKit or NoCache = true in ImageBuildOptions for ImageBuild call.

Severity

  • CVSS Score: 6.9 / 10 (Medium)
  • Vector String: CVSS:3.1/AV:L/AC:H/PR:N/UI:R/S:C/C:L/I:H/A:L

References

This data is provided by OSV and the GitHub Advisory Database (CC-BY 4.0).


Classic builder cache poisoning in github.com/docker/docker

CVE-2024-24557 / GHSA-xw73-rw38-6vjc / GO-2024-2512

More information

Details

Classic builder cache poisoning in github.com/docker/docker

Severity

Unknown

References

This data is provided by OSV and the Go Vulnerability Database (CC-BY 4.0).


Moby authz zero length regression in github.com/moby/moby

CVE-2024-41110 / GHSA-v23v-6jw2-98fq / GO-2024-3005

More information

Details

Moby authz zero length regression in github.com/moby/moby

Severity

Unknown

References

This data is provided by OSV and the Go Vulnerability Database (CC-BY 4.0).


Release Notes

docker/docker (github.com/docker/docker)

v25.0.6+incompatible

Compare Source

v25.0.5+incompatible

Compare Source

v25.0.4+incompatible

Compare Source

v25.0.3+incompatible

Compare Source

v25.0.2+incompatible

Compare Source

v25.0.1+incompatible

Compare Source

v25.0.0+incompatible

Compare Source

v24.0.9+incompatible

Compare Source

v24.0.8+incompatible

Compare Source

v24.0.7+incompatible

Compare Source

v24.0.6+incompatible

Compare Source

v24.0.5+incompatible

Compare Source

v24.0.4+incompatible

Compare Source

v24.0.3+incompatible

Compare Source

v24.0.2+incompatible

Compare Source

v24.0.1+incompatible

Compare Source

v24.0.0+incompatible

Compare Source

v23.0.13+incompatible

Compare Source

v23.0.12+incompatible

Compare Source

v23.0.11+incompatible

Compare Source

v23.0.10+incompatible

Compare Source

v23.0.9+incompatible

Compare Source


Configuration

📅 Schedule: Branch creation - "" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot requested a review from a team as a code owner March 21, 2024 10:22
@renovate renovate bot added area/security dependencies Pull requests that update a dependency file labels Mar 21, 2024
@renovate renovate bot changed the title fix(deps): update module github.com/docker/docker to v24 [security] (release-2.9.x) fix(deps): update module github.com/docker/docker to v24 [security] (release-2.9.x) - autoclosed Apr 18, 2024
@renovate renovate bot closed this Apr 18, 2024
@renovate renovate bot deleted the deps-update/release-2.9.x-go-github.com/docker/docker-vulnerability branch April 18, 2024 19:01
@renovate renovate bot changed the title fix(deps): update module github.com/docker/docker to v24 [security] (release-2.9.x) - autoclosed fix(deps): update module github.com/docker/docker to v24 [security] (release-2.9.x) Apr 21, 2024
@renovate renovate bot reopened this Apr 21, 2024
@renovate renovate bot restored the deps-update/release-2.9.x-go-github.com/docker/docker-vulnerability branch April 21, 2024 09:55
@renovate renovate bot force-pushed the deps-update/release-2.9.x-go-github.com/docker/docker-vulnerability branch from e9ea3ae to 9332efb Compare April 21, 2024 10:04
@periklis periklis force-pushed the deps-update/release-2.9.x-go-github.com/docker/docker-vulnerability branch from 9332efb to 10f762a Compare April 30, 2024 09:15
@renovate renovate bot force-pushed the deps-update/release-2.9.x-go-github.com/docker/docker-vulnerability branch from 10f762a to 1183c7f Compare July 30, 2024 07:38
@renovate renovate bot changed the title fix(deps): update module github.com/docker/docker to v24 [security] (release-2.9.x) fix(deps): update module github.com/docker/docker to v25 [security] (release-2.9.x) Jul 30, 2024
Copy link
Contributor Author

renovate bot commented Jul 30, 2024

ℹ Artifact update notice

File name: go.mod

In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

  • 29 additional dependencies were updated
  • The go directive was updated for compatibility reasons

Details:

Package Change
go 1.20 -> 1.22.5
cloud.google.com/go/pubsub v1.30.0 -> v1.33.0
cloud.google.com/go/storage v1.29.0 -> v1.30.1
github.com/golang/protobuf v1.5.3 -> v1.5.4
github.com/google/go-cmp v0.5.9 -> v0.6.0
github.com/google/uuid v1.3.0 -> v1.4.0
go.etcd.io/bbolt v1.3.6 -> v1.3.10
golang.org/x/sync v0.3.0 -> v0.5.0
google.golang.org/api v0.126.0 -> v0.149.0
google.golang.org/grpc v1.56.3 -> v1.59.0
golang.org/x/oauth2 v0.10.0 -> v0.13.0
cloud.google.com/go v0.110.2 -> v0.110.10
cloud.google.com/go/compute v1.20.1 -> v1.23.3
cloud.google.com/go/iam v0.13.0 -> v1.1.5
cloud.google.com/go/longrunning v0.4.1 -> v0.5.4
github.com/Microsoft/go-winio v0.6.0 -> v0.6.2
github.com/envoyproxy/go-control-plane v0.11.1-0.20230524094728-9239064ad72f -> v0.11.1
github.com/envoyproxy/protoc-gen-validate v0.10.1 -> v1.0.2
github.com/go-logr/logr v1.2.4 -> v1.3.0
github.com/google/s2a-go v0.1.4 -> v0.1.7
github.com/googleapis/enterprise-certificate-proxy v0.2.3 -> v0.3.2
github.com/googleapis/gax-go/v2 v2.11.0 -> v2.12.0
github.com/opencontainers/image-spec v1.0.2 -> v1.1.0
go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.44.0 -> v0.45.0
go.opentelemetry.io/otel v1.18.0 -> v1.21.0
go.opentelemetry.io/otel/metric v1.18.0 -> v1.21.0
go.opentelemetry.io/otel/trace v1.18.0 -> v1.21.0
google.golang.org/genproto v0.0.0-20230530153820-e85fd2cbaebc -> v0.0.0-20231211222908-989df2bf70f3
google.golang.org/genproto/googleapis/api v0.0.0-20230530153820-e85fd2cbaebc -> v0.0.0-20231120223509-83a465c0220f
google.golang.org/genproto/googleapis/rpc v0.0.0-20230530153820-e85fd2cbaebc -> v0.0.0-20231212172506-995d672761c0

@renovate renovate bot changed the title fix(deps): update module github.com/docker/docker to v25 [security] (release-2.9.x) fix(deps): update module github.com/docker/docker to v25 [security] (release-2.9.x) - abandoned Aug 6, 2024
Copy link
Contributor Author

renovate bot commented Aug 6, 2024

Autoclosing Skipped

This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/security dependencies Pull requests that update a dependency file size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant