Re: Re: bug in 2nd-highest-upvoted mb_check_encoding() comment code
| From: | Peter Cowburn | Date: | Sat, 14 Aug 2021 16:35:08 +0000 |
| Subject: | Re: Re: bug in 2nd-highest-upvoted mb_check_encoding() comment code | ||
| References: | 1 | Groups: | php.doc |
| Request: | Send a blank email to phpdoc+get-969387984@lists.php.net to get a copy of this message | ||
On Sat, 14 Aug 2021 at 16:25, Hans Henrik Bergan <divinity76@gmail.com>
wrote:
> @Peter Cowburn
> PS it would appear that you (accidentally?) deleted every single comment,
> instead of just comment 95289
>
Thanks. It was not accidental.
>
>
>> > the comment
>> >
>> > https://www.php.net/manual/en/function.mb-check-encoding.php#95289
>> > is
>> the
>> > 2nd-highest upvoted comment, and makes some dubious claims, but the code
>> > provided is bugged and should not be used,
>> > as a specific example it incorrectly validates "\xC0\x81" , which is
>> > illegal in UTF8 (quote RFC3629: The octet values C0, C1, F5 to FF never
>> > appear - ref
>> > https://datatracker.ietf.org/doc/html/rfc3629 )
>> >
>> > so.. can we at least get an editor's note that the code is bugged,
>> > incorrectly validates "\xC0\x81", and should not be used?
>> Instead of doing this, I have deleted the referenced note.
>
>
>