pgsql: Fix bogus time printout in walreceiver's debug log messages. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Fix bogus time printout in walreceiver's debug log messages.
Date
Msg-id E1WW6I5-0002mk-Ue@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix bogus time printout in walreceiver's debug log messages.

The displayed sendtime and receipttime were always exactly equal, because
somebody forgot that timestamptz_to_str returns a static buffer (thereby
simplifying life for most callers, at the cost of complicating it for those
who need two results concurrently).  Apply the same pstrdup solution used
by the other call sites with this issue.  Back-patch to 9.2 where the
faulty code was introduced.  Per bug #9849 from Haruka Takatsuka, though
this is not exactly his patch.

Possibly we should change timestamptz_to_str's API, but I wouldn't want
to do so in the back branches.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/8120c7452a51a773ad7a249b55557439f39d41ef

Modified Files
--------------
src/backend/replication/walreceiver.c |   14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: Fix some compiler warnings that clang emits with -pedantic.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix bogus time printout in walreceiver's debug log messages.