From: Victor Stinner Date: Fri, 20 Mar 2026 16:26:18 +0000 (+0100) Subject: [3.14] gh-146196: Fix Undefined Behavior in _PyUnicodeWriter_WriteASCIIString() ... X-Git-Tag: v3.14.4~102 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=6e73225c323ee2c1facc013425adf2b786dc8d47;p=thirdparty%2FPython%2Fcpython.git [3.14] gh-146196: Fix Undefined Behavior in _PyUnicodeWriter_WriteASCIIString() (#146201) (#146220) gh-146196: Fix Undefined Behavior in _PyUnicodeWriter_WriteASCIIString() (#146201) Avoid calling memcpy(data + writer->pos, NULL, 0) which has an undefined behavior. (cherry picked from commit cd10a2e65c25682095f6ee4a9b9a181938a50d2e) Co-authored-by: Shamil --- diff --git a/Misc/NEWS.d/next/Core_and_Builtins/2026-03-20-13-55-14.gh-issue-146196.Zg70Kb.rst b/Misc/NEWS.d/next/Core_and_Builtins/2026-03-20-13-55-14.gh-issue-146196.Zg70Kb.rst new file mode 100644 index 000000000000..9e03c1bbb0e1 --- /dev/null +++ b/Misc/NEWS.d/next/Core_and_Builtins/2026-03-20-13-55-14.gh-issue-146196.Zg70Kb.rst @@ -0,0 +1,2 @@ +Fix potential Undefined Behavior in :c:func:`PyUnicodeWriter_WriteASCII` by +adding a zero-length check. Patch by Shamil Abdulaev. diff --git a/Objects/unicodeobject.c b/Objects/unicodeobject.c index 53f219eb185d..4a457c4ac9ff 100644 --- a/Objects/unicodeobject.c +++ b/Objects/unicodeobject.c @@ -14054,6 +14054,10 @@ _PyUnicodeWriter_WriteASCIIString(_PyUnicodeWriter *writer, if (len == -1) len = strlen(ascii); + if (len == 0) { + return 0; + } + assert(ucs1lib_find_max_char((const Py_UCS1*)ascii, (const Py_UCS1*)ascii + len) < 128); if (writer->buffer == NULL && !writer->overallocate) {