Fix: allow NULL for 'PyBytes_FromStringAndSize'.

Author: fangerer

graalpython/com.oracle.graal.python.test/src/tests/cpyext/test_bytes.py

@@ -76,6 +76,20 @@ def compile_module(self, name):
         arguments=("char* str", "Py_ssize_t sz"),
     )
 
+    # PyBytes_FromStringAndSize
+    test_PyBytes_FromStringAndSizeNULL = CPyExtFunction(
+        lambda args: len(b"\x00"*args[0]),
+        lambda: ( (128, ), ),
+        code = """PyObject* PyBytes_FromStringAndSizeNULL(Py_ssize_t n) {
+            // we are return the length because the content is random (uninitialized)
+            return PyBytes_Size(PyBytes_FromStringAndSize(NULL, n));
+        }
+        """,
+        resultspec="n",
+        argspec='n',
+        arguments=["Py_ssize_t n"],
+    )
+
     # PyBytes_FromString
     test_PyBytes_FromString = CPyExtFunction(
         lambda arg: bytes(arg[0], "utf-8"),

graalpython/lib-graalpython/python_cext.py

@@ -216,7 +216,7 @@ def PyTruffle_Object_LEN(obj):
 ##################### BYTES
 
 def PyBytes_FromStringAndSize(string, size, encoding):
-    if string is not None:
+    if string is not None and string is not Py_NoValue():
         return bytes(string, encoding)
     assert size >= 0;
     return PyTruffle_Bytes_EmptyWithCapacity(size, native_null)