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

x/vulndb: potential Go vuln in github.com/containerd/containerd: GHSA-hmfx-3pcx-653p #1574

Closed
GoVulnBot opened this issue Feb 16, 2023 · 3 comments
Assignees

Comments

@GoVulnBot
Copy link

In GitHub Security Advisory GHSA-hmfx-3pcx-653p, there is a vulnerability in the following Go packages or modules:

Unit Fixed Vulnerable Ranges
github.com/containerd/containerd 1.6.18 >= 1.6.0, < 1.6.18

Cross references:

See doc/triage.md for instructions on how to triage this report.

modules:
  - module: github.com/containerd/containerd
    versions:
      - introduced: 1.6.0
        fixed: 1.6.18
    packages:
      - package: github.com/containerd/containerd
  - module: github.com/containerd/containerd
    versions:
      - introduced: TODO (earliest fixed "1.5.18", vuln range "<= 1.5.17")
    packages:
      - package: github.com/containerd/containerd
description: |-
    ### Impact

    A bug was found in containerd where supplementary groups are not set up properly inside a container.  If an attacker has direct access to a container and manipulates their supplementary group access, they may be able to use supplementary group access to bypass primary group restrictions in some cases, potentially gaining access to sensitive information or gaining the ability to execute code in that container.

    Downstream applications that use the containerd client library may be affected as well.

    ### Patches
    This bug has been fixed in containerd v1.6.18 and v.1.5.18.  Users should update to these versions and recreate containers to resolve this issue.  Users who rely on a downstream application that uses containerd's client library should check that application for a separate advisory and instructions.

    ### Workarounds

    Ensure that the `"USER $USERNAME"` Dockerfile instruction is not used.  Instead, set the container entrypoint to a value similar to `ENTRYPOINT ["su", "-", "user"]` to allow `su` to properly set up supplementary groups.

    ### References

    - https://www.benthamsgaze.org/2022/08/22/vulnerability-in-linux-containers-investigation-and-mitigation/
    - Docker/Moby: CVE-2022-36109, fixed in Docker 20.10.18
    - CRI-O: CVE-2022-2995, fixed in CRI-O 1.25.0
    - Podman: CVE-2022-2989, fixed in Podman 3.0.1 and 4.2.0
    - Buildah: CVE-2022-2990, fixed in Buildah 1.27.1

    Note that CVE IDs apply to a particular implementation, even if an issue is common.

    ### For more information

    If you have any questions or comments about this advisory:

    * Open an issue in [containerd](https://github.com/containerd/containerd/issues/new/choose)
    * Email us at [security@containerd.io](mailto:security@containerd.io)

    To report a security issue in containerd:
    * [Report a new vulnerability](https://github.com/containerd/containerd/security/advisories/new)
    * Email us at [security@containerd.io](mailto:security@containerd.io)
cves:
  - CVE-2023-25173
ghsas:
  - GHSA-hmfx-3pcx-653p
references:
  - advisory: https://github.com/containerd/containerd/security/advisories/GHSA-hmfx-3pcx-653p
  - advisory: https://github.com/advisories/GHSA-hmfx-3pcx-653p

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/469099 mentions this issue: data/reports: add GO-2023-1574.yaml

@tatianab
Copy link
Contributor

Review for appropriate use of alias vs related

@gopherbot
Copy link
Contributor

Change https://go.dev/cl/607456 mentions this issue: data/reports: update 6 reports

gopherbot pushed a commit that referenced this issue Aug 21, 2024
Fix reports which won't pass an upcoming lint check
by merging / collapsing their version ranges.

  - data/reports/GO-2022-0617.yaml
  - data/reports/GO-2023-1573.yaml
  - data/reports/GO-2023-1574.yaml
  - data/reports/GO-2023-1730.yaml
  - data/reports/GO-2023-1946.yaml
  - data/reports/GO-2024-2784.yaml

Updates #617
Updates #1573
Updates #1574
Updates #1730
Updates #1946
Updates #2784

Change-Id: If02308deccab77b00cf10cb3619263e456d1ea64
Reviewed-on: https://go-review.googlesource.com/c/vulndb/+/607456
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Tatiana Bradley <tatianabradley@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants