Skip to content

XSS via missing Binding syntax validation

Moderate
crewjam published GHSA-267v-3v32-g6q5 Oct 14, 2023

Package

gomod github.com/crewjam/saml (Go)

Affected versions

< 0.4.14

Patched versions

0.4.14

Description

Impact

The package does not validate the ACS Location URI according to the SAML binding being parsed.

If abused, this flaw allows attackers to register malicious Service Providers at the IdP and inject Javascript in the ACS endpoint definition, achieving Cross-Site-Scripting (XSS) in the IdP context during the redirection at the end of a SAML SSO Flow.

Consequently, an attacker may perform any authenticated action as the victim once the victim’s browser loaded the SAML IdP initiated SSO link for the malicious service provider.

Note: The severity is considered “High” because the SP registration is commonly an unrestricted operation in IdPs, hence not requiring particular permissions or publicly accessible to ease the IdP interoperability.

Patches

This issue is fixed in 0.4.14

Workarounds

Users of the package can perform external validation of URLs provided in SAML metadata, or restrict the ability for end-users to upload arbitrary metadata.

References

This issue was reported by Francesco Lacerenza from Doyensec

Severity

Moderate

CVE ID

CVE-2023-45683

Weaknesses

No CWEs

Credits