Miggo Logo

CVE-2018-5387: Wizkunde SAMLBase SAML Bypass

7.5

CVSS Score
3.1

Basic Information

EPSS Score
0.45552%
Published
5/13/2022
Updated
10/5/2023
KEV Status
No
Technology
TechnologyPHP

Technical Details

CVSS Vector
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:H/A:N
Package NameEcosystemVulnerable VersionsFirst Patched Version
gogentooss/samlbasecomposer< 1.2.71.2.7

Vulnerability Intelligence
Miggo AIMiggo AI

Miggo AIRoot Cause Analysis

The vulnerability stems from improper handling of XML comments during text extraction and canonicalization. SAML implementations must consistently process all text nodes when extracting values like NameID. The patch commit (482cdf8c) likely modified these text extraction functions to properly concatenate all text content while ignoring comments, as indicated by the CWE-347 classification and Duo's analysis of XML API pitfalls. The functions handling XML node text extraction (SAML2_Utils::getNodeText) and assertion processing (SAML2_Assertion::processValidatedElement) would be the primary locations where improper text handling would enable signature validation bypass.

Vulnerable functions

Only Mi**o us*rs **n s** t*is s**tion

WAF Protection Rules

WAF Rule

Wizkun** S*ML**s* m*y in*orr**tly utiliz* t** r*sults o* XML *OM tr*v*rs*l *n* **noni**liz*tion *PIs in su** * w*y t**t *n *tt**k*r m*y ** **l* to m*nipul*t* t** S*ML **t* wit*out inv*li**tin* t** *rypto*r*p*i* si*n*tur*, *llowin* t** *tt**k to pot*n

Reasoning

T** vuln*r**ility st*ms *rom improp*r **n*lin* o* XML *omm*nts *urin* t*xt *xtr**tion *n* **noni**liz*tion. S*ML impl*m*nt*tions must *onsist*ntly pro**ss *ll t*xt no**s w**n *xtr**tin* v*lu*s lik* N*m*I*. T** p*t** *ommit (********) lik*ly mo*i*i**