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

test_dsa: Avoid leaking LWLock tranches.

Browse Source 
Since this is a test module, leaking a couple of LWLock tranches is
fine, but we want to discourage that pattern in third-party code.
This commit teaches the module to create only one tranche and to
store its ID in shared memory for use by other backends.

Reported-by: Alexander Lakhin <exclusion@gmail.com>
Reviewed-by: Sami Imseih <samimseih@gmail.com>
Discussion: https://postgr.es/m/dd36d384-55df-4fc2-825c-5bc56c950fa9%40gmail.com
This commit is contained in:
Nathan Bossart
2025-11-12 14:57:48 -06:00
parent 1165a933aa
commit c5c74282f2
1 changed files with 19 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
src/test/modules/test_dsa/test_dsa.c
View File

@@ -13,25 +13,35 @@
#include "postgres.h"
#include "fmgr.h"
#include "storage/dsm_registry.h"
#include "storage/lwlock.h"
#include "utils/dsa.h"
#include "utils/resowner.h"
PG_MODULE_MAGIC;
static void
init_tranche(void *ptr)
{
int *tranche_id = (int *) ptr;
*tranche_id = LWLockNewTrancheId("test_dsa");
}
/* Test basic DSA functionality */
PG_FUNCTION_INFO_V1(test_dsa_basic);
Datum
test_dsa_basic(PG_FUNCTION_ARGS)
{
int tranche_id;
int *tranche_id;
bool found;
dsa_area *a;
dsa_pointer p[100];
/* XXX: this tranche is leaked */
tranche_id = LWLockNewTrancheId("test_dsa");
tranche_id = GetNamedDSMSegment("test_dsa", sizeof(int),
init_tranche, &found);
a = dsa_create(tranche_id);
a = dsa_create(*tranche_id);
for (int i = 0; i < 100; i++)
{
p[i] = dsa_allocate(a, 1000);
@@ -62,17 +72,18 @@ PG_FUNCTION_INFO_V1(test_dsa_resowners);
Datum
test_dsa_resowners(PG_FUNCTION_ARGS)
{
int tranche_id;
int *tranche_id;
bool found;
dsa_area *a;
dsa_pointer p[10000];
ResourceOwner oldowner;
ResourceOwner childowner;
/* XXX: this tranche is leaked */
tranche_id = LWLockNewTrancheId("test_dsa");
tranche_id = GetNamedDSMSegment("test_dsa", sizeof(int),
init_tranche, &found);
/* Create DSA in parent resource owner */
a = dsa_create(tranche_id);
a = dsa_create(*tranche_id);
/*
* Switch to child resource owner, and do a bunch of allocations in the