From dfc383cf3975ccb1053ef7431d1dfdcff1dfefea Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Wed, 11 Apr 2018 23:39:48 +0300 Subject: [PATCH] Make local copy of client hostnames in backend status array. The other strings, application_name and query string, were snapshotted to local memory in pgstat_read_current_status(), but we forgot to do that for client hostnames. As a result, the client hostname would appear to change in the local copy, if the client disconnected. Backpatch to all supported versions. Author: Edmund Horner Reviewed-by: Michael Paquier Discussion: https://www.postgresql.org/message-id/CAMyN-kA7aOJzBmrYFdXcc7Z0NmW%2B5jBaf_m%3D_-77uRNyKC9r%3DA%40mail.gmail.com --- src/backend/postmaster/pgstat.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c index 437c91faa23..c562c0c0434 100644 --- a/src/backend/postmaster/pgstat.c +++ b/src/backend/postmaster/pgstat.c @@ -2786,6 +2786,7 @@ pgstat_read_current_status(void) PgBackendStatus *localtable; PgBackendStatus *localentry; char *localappname, + *localclienthostname, *localactivity; int i; @@ -2801,6 +2802,9 @@ pgstat_read_current_status(void) localappname = (char *) MemoryContextAlloc(pgStatLocalContext, NAMEDATALEN * MaxBackends); + localclienthostname = (char *) + MemoryContextAlloc(pgStatLocalContext, + NAMEDATALEN * MaxBackends); localactivity = (char *) MemoryContextAlloc(pgStatLocalContext, pgstat_track_activity_query_size * MaxBackends); @@ -2832,6 +2836,8 @@ pgstat_read_current_status(void) */ strcpy(localappname, (char *) beentry->st_appname); localentry->st_appname = localappname; + strcpy(localclienthostname, (char *) beentry->st_clienthostname); + localentry->st_clienthostname = localclienthostname; strcpy(localactivity, (char *) beentry->st_activity); localentry->st_activity = localactivity; } @@ -2850,6 +2856,7 @@ pgstat_read_current_status(void) { localentry++; localappname += NAMEDATALEN; + localclienthostname += NAMEDATALEN; localactivity += pgstat_track_activity_query_size; localNumBackends++; } -- 2.39.5