diff options
| author | wolfbeast <mcwerewolf@gmail.com> | 2017-11-21 11:13:05 +0100 |
|---|---|---|
| committer | wolfbeast <mcwerewolf@gmail.com> | 2018-02-08 19:29:05 +0100 |
| commit | a58d1701b272b528a031f6d50d4632ada140c8a2 (patch) | |
| tree | 0f4d80e5e5b144558f5e2700241e90a5227004d7 /netwerk | |
| parent | a4a316fbc0c979d29e5ada3ab709c15e934b8639 (diff) | |
| download | uxp-a58d1701b272b528a031f6d50d4632ada140c8a2.tar.gz | |
Force punycode display for IDNs with a <dotless-i/j, combining mark above> sequence.
Diffstat (limited to 'netwerk')
| -rw-r--r-- | netwerk/dns/nsIDNService.cpp | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/netwerk/dns/nsIDNService.cpp b/netwerk/dns/nsIDNService.cpp index d4f31027e6..73a189b67e 100644 --- a/netwerk/dns/nsIDNService.cpp +++ b/netwerk/dns/nsIDNService.cpp @@ -797,6 +797,7 @@ bool nsIDNService::isLabelSafe(const nsAString &label) Script lastScript = Script::INVALID; uint32_t previousChar = 0; + uint32_t baseChar = 0; // last non-diacritic seen (base char for marks) uint32_t savedNumberingSystem = 0; // Simplified/Traditional Chinese check temporarily disabled -- bug 857481 #if 0 @@ -846,11 +847,19 @@ bool nsIDNService::isLabelSafe(const nsAString &label) } } - // Check for consecutive non-spacing marks - if (previousChar != 0 && - previousChar == ch && - GetGeneralCategory(ch) == HB_UNICODE_GENERAL_CATEGORY_NON_SPACING_MARK) { - return false; + if (GetGeneralCategory(ch) == HB_UNICODE_GENERAL_CATEGORY_NON_SPACING_MARK) { + // Check for consecutive non-spacing marks + if (previousChar != 0 && previousChar == ch) { + return false; + } + // Check for diacritics on dotless-i or dotless-j, which would be + // indistinguishable from normal accented letter. + if ((baseChar == 0x0237 || baseChar == 0x0131) && + ((ch >= 0x0300 && ch <= 0x0314) || ch == 0x031a)) { + return false; + } + } else { + baseChar = ch; } // Simplified/Traditional Chinese check temporarily disabled -- bug 857481 |