diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2005-11-05 03:04:53 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2005-11-05 03:04:53 +0000 |
commit | 48052de722a502408275fa3b7aee7de430265fc0 (patch) | |
tree | 88e3bb1280caae1ee6f4e83724a8f3aa65d25353 /src/backend/access/gist/gistget.c | |
parent | 95af2633c3e3d64a3cc508409a74ef29de25c852 (diff) | |
download | postgresql-48052de722a502408275fa3b7aee7de430265fc0.tar.gz |
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.
Diffstat (limited to 'src/backend/access/gist/gistget.c')
0 files changed, 0 insertions, 0 deletions