diff options
| author | Andres Freund <andres@anarazel.de> | 2022-08-22 20:16:50 -0700 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2022-08-22 20:16:50 -0700 |
| commit | cd063344fb801a90a40923a5b8aefe4eb8d80762 (patch) | |
| tree | a9248822b82059ddb2cd7b9f36856d6a1b180f81 /src/include/utils/pgstat_internal.h | |
| parent | ba8321349bc02423aa4d49ebb5d579ec313cf930 (diff) | |
| download | postgresql-cd063344fb801a90a40923a5b8aefe4eb8d80762.tar.gz postgresql-cd063344fb801a90a40923a5b8aefe4eb8d80762.zip | |
pgstat: Acquire lock when reading variable-numbered stats
Somewhere during the development of the patch acquiring a lock during read
access to variable-numbered stats got lost. The missing lock acquisition won't
cause corruption, but can lead to reading torn values when accessing
stats. Add the missing lock acquisitions.
Reported-by: Greg Stark <stark@mit.edu>
Reviewed-by: "Drouvot, Bertrand" <bdrouvot@amazon.com>
Reviewed-by: Andres Freund <andres@anarazel.de>
Author: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Discussion: https://postgr.es/m/CAM-w4HMYkM_DkYhWtUGV+qE_rrBxKOzOF0+5faozxO3vXrc9wA@mail.gmail.com
Backpatch: 15-
Diffstat (limited to 'src/include/utils/pgstat_internal.h')
| -rw-r--r-- | src/include/utils/pgstat_internal.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/include/utils/pgstat_internal.h b/src/include/utils/pgstat_internal.h index 9303d05427f..901d2041d66 100644 --- a/src/include/utils/pgstat_internal.h +++ b/src/include/utils/pgstat_internal.h @@ -581,6 +581,7 @@ extern void pgstat_detach_shmem(void); extern PgStat_EntryRef *pgstat_get_entry_ref(PgStat_Kind kind, Oid dboid, Oid objoid, bool create, bool *found); extern bool pgstat_lock_entry(PgStat_EntryRef *entry_ref, bool nowait); +extern bool pgstat_lock_entry_shared(PgStat_EntryRef *entry_ref, bool nowait); extern void pgstat_unlock_entry(PgStat_EntryRef *entry_ref); extern bool pgstat_drop_entry(PgStat_Kind kind, Oid dboid, Oid objoid); extern void pgstat_drop_all_entries(void); |