chore: raises OSError if getting locale info failed

Author: mokurin000

Lib/locale.py

@@ -557,6 +557,9 @@ def getdefaultlocale(envvars=('LC_ALL', 'LC_CTYPE', 'LANG', 'LANGUAGE')):
         1766.  code and encoding can be None in case the values cannot
         be determined.
 
+        Raises OSError on Windows if failed to get locale info (unlikely
+        to happen on modern Windows).
+
     """
 
     return _getdefaultlocale(envvars)
@@ -572,12 +575,7 @@ def _getdefaultlocale(envvars=('LC_ALL', 'LC_CTYPE', 'LANG', 'LANGUAGE')):
     else:
         # add other platform-specific processing here, if
         # necessary...
-        # for Windows, for any reason if the code was
-        # not avaliable, fall back to POSIX behaviour
-        if sys.platform == "win32" and code is None:
-            pass
-        else:
-            return code, encoding
+        return code, encoding
 
     # fall back on POSIX behaviour
     import os

Modules/_localemodule.c

@@ -29,6 +29,7 @@ This software comes with no warranty. Use at your own risk.
 #    define WIN32_LEAN_AND_MEAN
 #  endif
 #  include <windows.h>
+#  include "pyerrors.h"             // PyErr_SetFromWindowsErr()
 #endif
 
 PyDoc_STRVAR(locale__doc__, "Support for POSIX locales.");
@@ -556,8 +557,8 @@ _locale__getdefaultlocale_impl(PyObject *module)
     }
 
     /* cannot determine the language code (very unlikely) */
-    Py_INCREF(Py_None);
-    return Py_BuildValue("Os", Py_None, encoding);
+    /* if any of GetLocaleInfoA above failed */
+    return PyErr_SetFromWindowsErr(0);
 }
 #endif