gh-151126: Add missing PyErr_NoMemory in _winapi module

[sobolevn]

This place misses it:

cpython/Modules/_winapi.c

Lines 1196 to 1198 in 9fdbade

	ret = PyMem_Malloc(*size);
	if (ret == NULL)
	goto cleanup;

It is used as:

attribute_list->handle_list = gethandlelist(value, "handle_list", &handle_list_size);
if (attribute_list->handle_list == NULL && PyErr_Occurred()) {
    ret = -1;
    goto cleanup;
}

It would be rather strange to swallow MemoryError here.
Before that we allow KeyError to pass silently and return early if this code path should not be taken:

value = PyMapping_GetItemString(mapping, name);
if (!value) {
    PyErr_Clear();
    return NULL;
}

---

All other error paths in the same function set exceptions, except this if:

cpython/Modules/_winapi.c

Lines 1278 to 1282 in 9fdbade

	attribute_list->attribute_list = PyMem_Malloc(attribute_list_size);
	if (attribute_list->attribute_list == NULL) {
	ret = -1;
	goto cleanup;
	}

Later this NULL is returned from _winapi.CreateProcess without an exception set.

• Issue: Many places have missing PyErr_NoMemory() call after failing memory allocation #151126

[miss-islington-app]

Thanks @sobolevn for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15.
🐍🍒⛏🤖

[bedevere-app]

GH-151180 is a backport of this pull request to the 3.15 branch.

[bedevere-app]

GH-151181 is a backport of this pull request to the 3.14 branch.

[bedevere-app]

GH-151182 is a backport of this pull request to the 3.13 branch.

[serhiy-storchaka]

I am not sure this caused crash. At least not in the first case. But adding PyErr_NoMemory() was correct.