Description
symfony/polyfill-intl-idn provides a userland implementation of idn_to_utf8() and idn_to_ascii() for runtimes that lack the intl extension. Its Idn::process() method decodes labels prefixed with xn-- using Punycode but never enforces the validity criterion added in UTS #46 revision 33 Section 4 step 4.1.2: after a successful Punycode decode, the result must contain at least one non-ASCII code point.
As a consequence, xn-- labels whose Punycode payload is empty (xn--) or decodes to a string made of only ASCII code points (e.g. xn--kc1zs4-) are accepted by the polyfill while PHP's native ext-intl rejects them with IDNA_ERROR_INVALID_ACE_LABEL. Originally unequal domain names are therefore regarded as equal, which can lead to blacklist bypassing, inconsistent URL parsing and server-side request forgery (similar to CVE-2024-12224).
Example with IDNA_USE_STD3_RULES | IDNA_CHECK_BIDI | IDNA_CHECK_CONTEXTJ | IDNA_NONTRANSITIONAL_TO_ASCII:
| Input |
Polyfill output |
Native ext-intl output |
poc.xn--kc1zs4-.com |
poc.kc1zs4.com |
false (errors=1024) |
poc.kc1zs4.xn-- |
poc.kc1zs4. |
false (errors=1024) |
Applications using the polyfill to canonicalise or compare hostnames inherit the inconsistency.
Resolution
Idn::process() now records IDNA_ERROR_INVALID_ACE_LABEL when a Punycode payload decodes to an empty string or to a string containing only ASCII code points, matching the native ext-intl behaviour and UTS #46 revision 33.
The patch for this issue is available here for branch 1.x.
Credits
Symfony would like to thank Nazy Mad for reporting the issue and Nicolas Grekas for providing the fix.
References
Description
symfony/polyfill-intl-idnprovides a userland implementation ofidn_to_utf8()andidn_to_ascii()for runtimes that lack theintlextension. ItsIdn::process()method decodes labels prefixed withxn--using Punycode but never enforces the validity criterion added in UTS #46 revision 33 Section 4 step 4.1.2: after a successful Punycode decode, the result must contain at least one non-ASCII code point.As a consequence,
xn--labels whose Punycode payload is empty (xn--) or decodes to a string made of only ASCII code points (e.g.xn--kc1zs4-) are accepted by the polyfill while PHP's nativeext-intlrejects them withIDNA_ERROR_INVALID_ACE_LABEL. Originally unequal domain names are therefore regarded as equal, which can lead to blacklist bypassing, inconsistent URL parsing and server-side request forgery (similar to CVE-2024-12224).Example with
IDNA_USE_STD3_RULES | IDNA_CHECK_BIDI | IDNA_CHECK_CONTEXTJ | IDNA_NONTRANSITIONAL_TO_ASCII:ext-intloutputpoc.xn--kc1zs4-.compoc.kc1zs4.comfalse(errors=1024)poc.kc1zs4.xn--poc.kc1zs4.false(errors=1024)Applications using the polyfill to canonicalise or compare hostnames inherit the inconsistency.
Resolution
Idn::process()now recordsIDNA_ERROR_INVALID_ACE_LABELwhen a Punycode payload decodes to an empty string or to a string containing only ASCII code points, matching the nativeext-intlbehaviour and UTS #46 revision 33.The patch for this issue is available here for branch 1.x.
Credits
Symfony would like to thank Nazy Mad for reporting the issue and Nicolas Grekas for providing the fix.
References