[issue46716] regrtest didn't respect the timeout when running test_subprocess on AMD64 Windows11 3.x

[STINNER Victor]

STINNER Victor <vstinner at python.org> added the comment:

> It does seem that only the Windows Popen._wait() cannot handle negative timeout values, so the fix should be as simple as coercing the timeout values to >= 0.

Oh. This function should maybe raise an exception if the timeout is negative, and ther caller must replace negative timeout with zero.

> A judicious use of prints in subprocess.py, reveals that the timeout passed to wait() ends up being negative.  That value, once cast to a DWORD, ultimately causes a very long wait (0xfffffff2, in my testing).

This sounds dangerous and must be fixed. Python must not convert negative values to very large positive values.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue46716>
_______________________________________