summaryrefslogtreecommitdiff
path: root/src/backend/access/rmgrdesc/standbydesc.c
diff options
context:
space:
mode:
authorTom Lane2025-07-25 20:37:29 +0000
committerTom Lane2025-07-25 20:37:29 +0000
commit0f9d4d7c12dcebe951061763ca23ee3b6477e7ca (patch)
tree3696d0d57af5bf432cb2d9a75d2567dd000b278e /src/backend/access/rmgrdesc/standbydesc.c
parent73873805fb3627cb23937c750fa83ffd8f16fc6c (diff)
Silence leakage complaint about postgres_fdw's InitPgFdwOptions.HEADmaster
Valgrind complains that the PQconninfoOption array returned by libpq is leaked. We apparently believed that we could suppress that warning by storing that array's address in a static variable. However, modern C compilers are bright enough to optimize the static variable away. We could escalate that arms race by making the variable global. But on the whole it seems better to revise the code so that it can free libpq's result properly. The only thing that costs us is copying the parameter-name keywords; which seems like a pretty negligible cost in a function that runs at most once per process. Author: Tom Lane <tgl@sss.pgh.pa.us> Reviewed-by: Matheus Alcantara <matheusssilv97@gmail.com> Discussion: https://postgr.es/m/2976982.1748049023@sss.pgh.pa.us
Diffstat (limited to 'src/backend/access/rmgrdesc/standbydesc.c')
0 files changed, 0 insertions, 0 deletions