Skip to content

Commit

Permalink
bpo-28269: Replace strcasecmp with system function _stricmp. (pythonG…
Browse files Browse the repository at this point in the history
  • Loading branch information
gongminmin authored and serhiy-storchaka committed Jul 13, 2019
1 parent b5bbb8a commit 05f2d84
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 19 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Replace use of :c:func:`strcasecmp` for the system function :c:func:`_stricmp`. Patch by Minmin Gong.
20 changes: 1 addition & 19 deletions Python/dynload_win.c
Original file line number Diff line number Diff line change
Expand Up @@ -38,24 +38,6 @@ const char *_PyImport_DynLoadFiletab[] = {
NULL
};

/* Case insensitive string compare, to avoid any dependencies on particular
C RTL implementations */

static int strcasecmp (const char *string1, const char *string2)
{
int first, second;

do {
first = tolower(*string1);
second = tolower(*string2);
string1++;
string2++;
} while (first && first == second);

return (first - second);
}


/* Function to return the name of the "python" DLL that the supplied module
directly imports. Looks through the list of imported modules and
returns the first entry that starts with "python" (case sensitive) and
Expand Down Expand Up @@ -297,7 +279,7 @@ dl_funcptr _PyImport_FindSharedFuncptrWindows(const char *prefix,
import_python = GetPythonImport(hDLL);

if (import_python &&
strcasecmp(buffer,import_python)) {
_stricmp(buffer,import_python)) {
PyErr_Format(PyExc_ImportError,
"Module use of %.150s conflicts "
"with this version of Python.",
Expand Down

0 comments on commit 05f2d84

Please sign in to comment.