database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-11 05:41:32 +03:00
Code

Make local copy of client hostnames in backend status array.

Browse Source 
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
This commit is contained in:
Heikki Linnakangas 2018-04-11 23:39:48 +03:00
parent 494f3cb5bb
commit 74dc05e01e
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/backend/postmaster/pgstat.c
View File

@ -3058,6 +3058,7 @@ pgstat_read_current_status(void)
LocalPgBackendStatus *localtable; LocalPgBackendStatus *localtable;
LocalPgBackendStatus *localentry; LocalPgBackendStatus *localentry;
char *localappname, char *localappname,
*localclienthostname,
*localactivity; *localactivity;
#ifdef USE_SSL #ifdef USE_SSL
PgBackendSSLStatus *localsslstatus; PgBackendSSLStatus *localsslstatus;
@ -3076,6 +3077,9 @@ pgstat_read_current_status(void)
localappname = (char *) localappname = (char *)
MemoryContextAlloc(pgStatLocalContext, MemoryContextAlloc(pgStatLocalContext,
NAMEDATALEN * MaxBackends); NAMEDATALEN * MaxBackends);
localclienthostname = (char *)
MemoryContextAlloc(pgStatLocalContext,
NAMEDATALEN * MaxBackends);
localactivity = (char *) localactivity = (char *)
MemoryContextAlloc(pgStatLocalContext, MemoryContextAlloc(pgStatLocalContext,
pgstat_track_activity_query_size * MaxBackends); pgstat_track_activity_query_size * MaxBackends);
@ -3116,6 +3120,8 @@ pgstat_read_current_status(void)
*/ */
strcpy(localappname, (char *) beentry->st_appname); strcpy(localappname, (char *) beentry->st_appname);
localentry->backendStatus.st_appname = localappname; localentry->backendStatus.st_appname = localappname;
strcpy(localclienthostname, (char *) beentry->st_clienthostname);
localentry->backendStatus.st_clienthostname = localclienthostname;
strcpy(localactivity, (char *) beentry->st_activity); strcpy(localactivity, (char *) beentry->st_activity);
localentry->backendStatus.st_activity = localactivity; localentry->backendStatus.st_activity = localactivity;
localentry->backendStatus.st_ssl = beentry->st_ssl; localentry->backendStatus.st_ssl = beentry->st_ssl;
@ -3147,6 +3153,7 @@ pgstat_read_current_status(void)
localentry++; localentry++;
localappname += NAMEDATALEN; localappname += NAMEDATALEN;
localclienthostname += NAMEDATALEN;
localactivity += pgstat_track_activity_query_size; localactivity += pgstat_track_activity_query_size;
#ifdef USE_SSL #ifdef USE_SSL
localsslstatus++; localsslstatus++;