Re: Re: bug in 2nd-highest-upvoted mb_check_encoding() comment code
[email protected] (Peter Cowburn)
| Newsgroups | php.doc |
|---|---|
| Message-ID | <CAPg3XxKopEycHxsX+76he540DH9PrUdnqg1Sz1TS9S74eU6Akg@mail.gmail.com> |
On Sat, 14 Aug 2021 at 16:25, Hans Henrik Bergan <[email protected]> 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. > > >