Bug report

This call is problematic:

It can return NULL in theory.

These calls also can return NULL:

• cpython/Modules/timemodule.c

  Lines 477 to 478 in f526314

  	PyStructSequence_SET_ITEM(v, 9,
  	PyUnicode_DecodeLocale(p->tm_zone, "surrogateescape"));

• cpython/Modules/timemodule.c

  Lines 481 to 482 in f526314

  	PyStructSequence_SET_ITEM(v, 9,
  	PyUnicode_DecodeLocale(zone, "surrogateescape"));

This error guard in the end will only show the last error:

Not the first one. Also: why XDECREF when v cannot be NULL at this point?

Refs #116714

I will send a PR adding our regular macro for the job.

Linked PRs

• gh-118998: Handle errors correctly in tmtotuple in timemodule #118999
• [3.13] gh-118998: Handle errors correctly in tmtotuple in timemodule (GH-118999) #119018
• [3.12] gh-118998: Handle errors correctly in tmtotuple in timemodule (GH-118999) #119019