GH-94808: Cover `PyOS_mystrnicmp` and `PyOS_mystricmp` (gh-102469)

This commit is contained in:
Artem Mukhin 2023-03-22 12:35:27 +01:00 committed by GitHub
parent e0c63b7267
commit 0a60deaeaf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 69 additions and 1 deletions

View File

@ -169,7 +169,7 @@
@MODULE__XXTESTFUZZ_TRUE@_xxtestfuzz _xxtestfuzz/_xxtestfuzz.c _xxtestfuzz/fuzzer.c @MODULE__XXTESTFUZZ_TRUE@_xxtestfuzz _xxtestfuzz/_xxtestfuzz.c _xxtestfuzz/fuzzer.c
@MODULE__TESTBUFFER_TRUE@_testbuffer _testbuffer.c @MODULE__TESTBUFFER_TRUE@_testbuffer _testbuffer.c
@MODULE__TESTINTERNALCAPI_TRUE@_testinternalcapi _testinternalcapi.c @MODULE__TESTINTERNALCAPI_TRUE@_testinternalcapi _testinternalcapi.c
@MODULE__TESTCAPI_TRUE@_testcapi _testcapimodule.c _testcapi/vectorcall.c _testcapi/vectorcall_limited.c _testcapi/heaptype.c _testcapi/unicode.c _testcapi/getargs.c _testcapi/pytime.c _testcapi/datetime.c _testcapi/docstring.c _testcapi/mem.c _testcapi/watchers.c _testcapi/long.c _testcapi/float.c _testcapi/structmember.c _testcapi/exceptions.c _testcapi/code.c @MODULE__TESTCAPI_TRUE@_testcapi _testcapimodule.c _testcapi/vectorcall.c _testcapi/vectorcall_limited.c _testcapi/heaptype.c _testcapi/unicode.c _testcapi/getargs.c _testcapi/pytime.c _testcapi/datetime.c _testcapi/docstring.c _testcapi/mem.c _testcapi/watchers.c _testcapi/long.c _testcapi/float.c _testcapi/structmember.c _testcapi/exceptions.c _testcapi/code.c _testcapi/pyos.c
@MODULE__TESTCLINIC_TRUE@_testclinic _testclinic.c @MODULE__TESTCLINIC_TRUE@_testclinic _testclinic.c
# Some testing modules MUST be built as shared libraries. # Some testing modules MUST be built as shared libraries.

View File

@ -38,6 +38,7 @@ int _PyTestCapi_Init_Float(PyObject *module);
int _PyTestCapi_Init_Structmember(PyObject *module); int _PyTestCapi_Init_Structmember(PyObject *module);
int _PyTestCapi_Init_Exceptions(PyObject *module); int _PyTestCapi_Init_Exceptions(PyObject *module);
int _PyTestCapi_Init_Code(PyObject *module); int _PyTestCapi_Init_Code(PyObject *module);
int _PyTestCapi_Init_PyOS(PyObject *module);
#ifdef LIMITED_API_AVAILABLE #ifdef LIMITED_API_AVAILABLE
int _PyTestCapi_Init_VectorcallLimited(PyObject *module); int _PyTestCapi_Init_VectorcallLimited(PyObject *module);

60
Modules/_testcapi/pyos.c Normal file
View File

@ -0,0 +1,60 @@
#include "parts.h"
static PyObject *
test_PyOS_mystrnicmp(PyObject *self, PyObject *Py_UNUSED(ignored))
{
assert(PyOS_mystrnicmp("", "", 0) == 0);
assert(PyOS_mystrnicmp("", "", 1) == 0);
assert(PyOS_mystrnicmp("insert", "ins", 3) == 0);
assert(PyOS_mystrnicmp("ins", "insert", 3) == 0);
assert(PyOS_mystrnicmp("insect", "insert", 3) == 0);
assert(PyOS_mystrnicmp("insert", "insert", 6) == 0);
assert(PyOS_mystrnicmp("Insert", "insert", 6) == 0);
assert(PyOS_mystrnicmp("INSERT", "insert", 6) == 0);
assert(PyOS_mystrnicmp("insert", "insert", 10) == 0);
assert(PyOS_mystrnicmp("invert", "insert", 6) == ('v' - 's'));
assert(PyOS_mystrnicmp("insert", "invert", 6) == ('s' - 'v'));
assert(PyOS_mystrnicmp("insert", "ins\0rt", 6) == 'e');
// GH-21845
assert(PyOS_mystrnicmp("insert\0a", "insert\0b", 8) == 0);
Py_RETURN_NONE;
}
static PyObject *
test_PyOS_mystricmp(PyObject *self, PyObject *Py_UNUSED(ignored))
{
assert(PyOS_mystricmp("", "") == 0);
assert(PyOS_mystricmp("insert", "insert") == 0);
assert(PyOS_mystricmp("Insert", "insert") == 0);
assert(PyOS_mystricmp("INSERT", "insert") == 0);
assert(PyOS_mystricmp("insert", "ins") == 'e');
assert(PyOS_mystricmp("ins", "insert") == -'e');
// GH-21845
assert(PyOS_mystricmp("insert", "ins\0rt") == 'e');
assert(PyOS_mystricmp("invert", "insert") == ('v' - 's'));
Py_RETURN_NONE;
}
static PyMethodDef test_methods[] = {
{"test_PyOS_mystrnicmp", test_PyOS_mystrnicmp, METH_NOARGS, NULL},
{"test_PyOS_mystricmp", test_PyOS_mystricmp, METH_NOARGS, NULL},
{NULL},
};
int
_PyTestCapi_Init_PyOS(PyObject *mod)
{
if (PyModule_AddFunctions(mod, test_methods) < 0) {
return -1;
}
return 0;
}

View File

@ -4154,6 +4154,9 @@ PyInit__testcapi(void)
if (_PyTestCapi_Init_Code(m) < 0) { if (_PyTestCapi_Init_Code(m) < 0) {
return NULL; return NULL;
} }
if (_PyTestCapi_Init_PyOS(m) < 0) {
return NULL;
}
#ifndef LIMITED_API_AVAILABLE #ifndef LIMITED_API_AVAILABLE
PyModule_AddObjectRef(m, "LIMITED_API_AVAILABLE", Py_False); PyModule_AddObjectRef(m, "LIMITED_API_AVAILABLE", Py_False);

View File

@ -109,6 +109,7 @@
<ClCompile Include="..\Modules\_testcapi\structmember.c" /> <ClCompile Include="..\Modules\_testcapi\structmember.c" />
<ClCompile Include="..\Modules\_testcapi\exceptions.c" /> <ClCompile Include="..\Modules\_testcapi\exceptions.c" />
<ClCompile Include="..\Modules\_testcapi\code.c" /> <ClCompile Include="..\Modules\_testcapi\code.c" />
<ClCompile Include="..\Modules\_testcapi\pyos.c" />
</ItemGroup> </ItemGroup>
<ItemGroup> <ItemGroup>
<ResourceCompile Include="..\PC\python_nt.rc" /> <ResourceCompile Include="..\PC\python_nt.rc" />

View File

@ -57,6 +57,9 @@
<ClCompile Include="..\Modules\_testcapi\code.c"> <ClCompile Include="..\Modules\_testcapi\code.c">
<Filter>Source Files</Filter> <Filter>Source Files</Filter>
</ClCompile> </ClCompile>
<ClCompile Include="..\Modules\_testcapi\pyos.c">
<Filter>Source Files</Filter>
</ClCompile>
</ItemGroup> </ItemGroup>
<ItemGroup> <ItemGroup>
<ResourceCompile Include="..\PC\python_nt.rc"> <ResourceCompile Include="..\PC\python_nt.rc">