classification
Title: WinError(): Python int too large to convert to C long
Type: behavior Stage:
Components: ctypes, Windows Versions: Python 3.10, Python 3.9, Python 3.8
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: Kelvin You, amaury.forgeotdarc, belopolsky, eryksun, josh.r, meador.inge, paul.moore, steve.dower, tim.golden, zach.ware
Priority: normal Keywords:

Created on 2016-10-19 07:28 by Kelvin You, last changed 2021-02-26 20:32 by eryksun.

Messages (7)
msg278963 - (view) Author: Kelvin You (Kelvin You) Date: 2016-10-19 07:28
// callproc.c
static PyObject *format_error(PyObject *self, PyObject *args)
{
    PyObject *result;
    wchar_t *lpMsgBuf;
    DWORD code = 0;
    if (!PyArg_ParseTuple(args, "|i:FormatError", &code))  
                                  ^ Here the format string should be "|I:FormatError"
        return NULL;
    if (code == 0)
        code = GetLastError();
    lpMsgBuf = FormatError(code);
    if (lpMsgBuf) {
        result = PyUnicode_FromWideChar(lpMsgBuf, wcslen(lpMsgBuf));
        LocalFree(lpMsgBuf);
    } else {
        result = PyUnicode_FromString("<no description>");
    }
    return result;
}
msg279005 - (view) Author: Josh Rosenberg (josh.r) * (Python triager) Date: 2016-10-20 01:16
You can't use I as a format code safely; it silently ignores/wraps overflow on the conversion, where i raises on overflow. The unsigned converters are basically useless for resilient code in 99% of cases.

I *think* I remember some private utility functions for doing this using O& though, not sure if they're available in callproc.c...
msg279008 - (view) Author: Kelvin You (Kelvin You) Date: 2016-10-20 03:35
I report this issue because the function WlanScan(https://msdn.microsoft.com/zh-cn/library/windows/desktop/ms706783(v=vs.85).aspx) returns a error code 0x80342002 when the WLAN is disabled on Windows 10.  ctypes.WinError() raise an exception of 'Python int too large to convert to C long' when handle this error code.
msg279009 - (view) Author: Kelvin You (Kelvin You) Date: 2016-10-20 03:53
Here is the full list of windows error code:
https://msdn.microsoft.com/en-us/library/cc231196.aspx
You can see a lot of error codes is above 0x80000000.
msg279033 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2016-10-20 14:15
All HRESULT values are, since the topmost bit indicates that it's an error, but all the others should be 16-bit positive integers IIRC. I don't think this function is meant to work with HRESULTs, but I could be wrong - fairly sure it's meant for Win32 error codes, though I'd have to dig into the format function it calls.

What function are you calling that produces an HRESULT from GetLastError?
msg279047 - (view) Author: Eryk Sun (eryksun) * (Python triager) Date: 2016-10-20 16:48
Kelvin is calling WlanScan [1], which returns an error code that apparently can include HRESULT (signed) values cast as DWORD (unsigned) values, including 0x80342002 (ERROR_NDIS_DOT11_POWER_STATE_INVALID). 

ctypes.FormatError calls FormatMessage [2] with the flag FORMAT_MESSAGE_FROM_SYSTEM. About half of the system error codes defined in winerror.h consist of COM HRESULT codes. But, to clarify Kelvin's link [3], this does not include NTSTATUS values from kernel-mode system calls. NTSTATUS values require the ntdll.dll message table.

[1]: https://msdn.microsoft.com/en-us/library/ms706783
[2]: https://msdn.microsoft.com/en-us/library/ms679351
[3]: https://msdn.microsoft.com/en-us/library/cc231196

A workaround in this case is to use the default c_int restype to return the error as a signed integer:

    >>> ctypes.FormatError(0x80342002 - 2**32)
    "The wireless local area network interface is powered down and doesn't
    support the requested operation."

A similar workaround works for setting the exit code to an HRESULT or NTSTATUS error code:

    >>> hex(subprocess.call('python -c "import os; os._exit(0xC0000001)"'))
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
    OverflowError: Python int too large to convert to C long
    '0x1'
    >>> hex(subprocess.call('python -c "import sys; sys.exit(0xC0000001)"'))
    '0xffffffff'

    >>> hex(subprocess.call('python -c "import os; os._exit(0xC0000001 - 2**32)"'))
    '0xc0000001'
    >>> hex(subprocess.call('python -c "import sys; sys.exit(0xC0000001 - 2**32)"'))
    '0xc0000001'
msg387744 - (view) Author: Eryk Sun (eryksun) * (Python triager) Date: 2021-02-26 20:32
format_error() can use format "L" (long long) and then check for a value in the accepted range `value >= LONG_MIN && value <= ULONG_MAX`. If the value is out of range, raise OverflowError. Otherwise assign the value to DWORD `code`.
History
Date User Action Args
2021-02-26 20:32:21eryksunsetmessages: + msg387744
versions: + Python 3.8, Python 3.9, Python 3.10, - Python 2.7, Python 3.5, Python 3.6, Python 3.7
2016-10-20 16:48:50eryksunsetnosy: + eryksun
messages: + msg279047
2016-10-20 14:15:49steve.dowersetmessages: + msg279033
2016-10-20 03:53:49Kelvin Yousetmessages: + msg279009
2016-10-20 03:35:15Kelvin Yousetmessages: + msg279008
2016-10-20 01:16:20josh.rsetnosy: + josh.r
messages: + msg279005
2016-10-19 07:59:48serhiy.storchakasetnosy: + belopolsky, amaury.forgeotdarc, paul.moore, tim.golden, meador.inge, zach.ware, steve.dower

type: crash -> behavior
components: + Windows
versions: + Python 3.7, - Python 3.3, Python 3.4
2016-10-19 07:28:00Kelvin Youcreate