From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id B6F043858402; Tue, 4 Jan 2022 23:49:46 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org B6F043858402 From: "blubban at gmail dot com" To: gcc-bugs@gcc.gnu.org Subject: [Bug libstdc++/103911] New: std::from_chars shouldn't call isdigit Date: Tue, 04 Jan 2022 23:49:46 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: new X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: libstdc++ X-Bugzilla-Version: unknown X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: blubban at gmail dot com X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: bug_id short_desc product version bug_status bug_severity priority component assigned_to reporter target_milestone Message-ID: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 04 Jan 2022 23:49:46 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D103911 Bug ID: 103911 Summary: std::from_chars shouldn't call isdigit Product: gcc Version: unknown Status: UNCONFIRMED Severity: normal Priority: P3 Component: libstdc++ Assignee: unassigned at gcc dot gnu.org Reporter: blubban at gmail dot com Target Milestone: --- In header , function __from_chars_alnum calls std::isdigit(). In addition to looking weird ("everyone" knows ctype is locale dependent and charconv is not), this can cause trouble in a handful of rare cases. If another thread concurrently calls setlocale, that's a race condition; additionally, if the locale contains digits other than '0'..'9', from_chars= can return wrong answer. (For example, some versions of Windows libc think 0xB2, 0xB3 and 0xB9 are the digits =C2=B2=C2=B3=C2=B9 in the "us" locale.) GCC will, by default, replace isdigit with c>=3D'0' && c<=3D'9'; to reprodu= ce the above, use -fno-builtin or Clang. (Bonus issue: A comment on that function says it applies to bases 11 to 26. Shouldn't that be 11 to 36?)=