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/russellhaering/gosaml2: GHSA-5684-g483-2249 #383

Closed
GoVulnBot opened this issue Mar 24, 2022 · 3 comments
Assignees
Labels
excluded: DEPENDENT_VULNERABILITY This vulnerability is downstream of another existing vulnerability report.

Comments

@GoVulnBot
Copy link

In GitHub Security Advisory GHSA-5684-g483-2249, there is a vulnerability in the following Go packages or modules:

Unit Fixed Vulnerable Ranges
github.com/russellhaering/gosaml2 0.5.0 < 0.5.0

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

package: github.com/russellhaering/gosaml2
versions:
  - introduced: v0.0.0
    fixed: v0.5.0
description: |-
    ### Impact
    Given a valid SAML Response, an attacker can potentially modify the document, bypassing signature validation in order to pass off the altered document as a signed one.

    This enables a variety of attacks, including users accessing accounts other than the one to which they authenticated in the identity provider, or full authentication bypass if an external attacker can obtain an expired, signed SAML Response.

    ### Patches
    A patch is available, users of gosaml2 should upgrade to v0.5.0 or higher.

    ### References
    See the [underlying advisory on goxmldsig](https://github.com/russellhaering/goxmldsig/security/advisories/GHSA-q547-gmf8-8jr7) for more details.
published: 2021-05-24T16:59:47Z
last_modified: 2021-10-05T17:07:11Z
ghsas:
  - GHSA-5684-g483-2249

@neild
Copy link
Contributor

neild commented Jul 8, 2022

This is #409, applied to a dependent package.

@neild neild closed this as completed Jul 8, 2022
@tatianab tatianab added excluded: DEPENDENT_VULNERABILITY This vulnerability is downstream of another existing vulnerability report. and removed duplicate labels Jul 28, 2023
@tatianab tatianab self-assigned this Jul 28, 2023
@tatianab
Copy link
Contributor

This needs an excluded report

@tatianab tatianab reopened this Jul 28, 2023
@gopherbot
Copy link
Contributor

Change https://go.dev/cl/513918 mentions this issue: data/excluded: batch add 26 excluded reports

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
excluded: DEPENDENT_VULNERABILITY This vulnerability is downstream of another existing vulnerability report.
Projects
None yet
Development

No branches or pull requests

6 participants