Reject nonminmal representations of UTF8 at the beginning of the 3-byte BMP range
ClosedPublic
Actions

Authored by epriestley on Feb 23 2014, 8:35 PM.

Tags

None

Referenced Files

	F15464327: D8313.id19764.diff
	Wed, Apr 2, 8:30 AM

	F15456427: D8313.id19764.diff
	Sun, Mar 30, 9:37 AM

	F15454790: D8313.id19779.diff
	Sat, Mar 29, 8:26 PM

	F15449164: D8313.id19774.diff
	Fri, Mar 28, 8:40 AM

	F15449005: D8313.id.diff
	Fri, Mar 28, 8:07 AM

	F15446747: D8313.diff
	Thu, Mar 27, 8:01 PM

	F15430557: D8313.id.diff
	Mon, Mar 24, 8:10 AM

	F15428638: D8313.id19774.diff
	Sun, Mar 23, 9:26 PM

Subscribers

Details

Reviewers

arice
btrahan

Maniphest Tasks

T1191: Comments are truncated at first non-base-plane character

Commits

rPHU563e600eba85: Reject nonminmal representations of UTF8 at the beginning of the 3-byte BMP…

Summary

Ref T1191. These byte ranges (\xE0\x80\x80 through \xE0\x9F\xBF) are alternate representations of characters with a different preferred minimal representation. MySQL and mbstring both reject them, and we should too.

Test Plan

Ran unit tests.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

arice accepted this revision.Feb 23 2014, 11:15 PM

arice added inline comments.

src/utils/utf8.php
126	*like

epriestley added inline comments.Feb 23 2014, 11:17 PM

src/utils/utf8.php
126	o why thank you

"lik" -> "like"

Closed by commit rPHU563e600eba85 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

utils/

__tests__/

PhutilUTF8TestCase.php

2 lines

utf8.php

21 lines

Diff 19779

View Options

src/utils/tests/PhutilUTF8TestCase.php

View Options

Reject nonminmal representations of UTF8 at the beginning of the 3-byte BMP rangeClosedPublicActions