OSDN Git Service

Repair an error introduced by log_line_prefix patch: it is not acceptable
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 5 Nov 2005 03:04:53 +0000 (03:04 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 5 Nov 2005 03:04:53 +0000 (03:04 +0000)
commit48052de722a502408275fa3b7aee7de430265fc0
tree88e3bb1280caae1ee6f4e83724a8f3aa65d25353
parent95af2633c3e3d64a3cc508409a74ef29de25c852
Repair an error introduced by log_line_prefix patch: it is not acceptable
to assume that the string pointer passed to set_ps_display is good forever.
There's no need to anyway since ps_status.c itself saves the string, and
we already had an API (get_ps_display) to return it.
I believe this explains Jim Nasby's report of intermittent crashes in
elog.c when %i format code is in use in log_line_prefix.
While at it, repair a previously unnoticed problem: on some platforms such as
Darwin, the string returned by get_ps_display was blank-padded to the maximum
length, meaning that lock.c's attempt to append " waiting" to it never worked.
src/backend/postmaster/postmaster.c
src/backend/storage/lmgr/lock.c
src/backend/utils/error/elog.c
src/backend/utils/misc/ps_status.c
src/include/libpq/libpq-be.h
src/include/utils/ps_status.h