Fix breakage of get_ps_display() in the PS_USE_NONE case.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 27 Mar 2022 16:57:46 +0000 (12:57 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 27 Mar 2022 16:57:46 +0000 (12:57 -0400)
commit0fb6954aa5012fc0c41af364fb328f90e648f6b7
treeb4a409b95910d5258e964bd65f7ffc9f41e89b5b
parent411b91360f2711e36782b68cd0c9bc6de44d3384
Fix breakage of get_ps_display() in the PS_USE_NONE case.

Commit 8c6d30f21 caused this function to fail to set *displen
in the PS_USE_NONE code path.  If the variable's previous value
had been negative, that'd lead to a memory clobber at some call
sites.  We'd managed not to notice due to very thin test coverage
of such configurations, but this appears to explain buildfarm member
lorikeet's recent struggles.

Credit to Andrew Dunstan for spotting the problem.  Back-patch
to v13 where the bug was introduced.

Discussion: https://postgr.es/m/136102.1648320427@sss.pgh.pa.us
src/backend/utils/misc/ps_status.c