diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2023-10-14 16:13:11 -0400 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2023-10-14 16:13:11 -0400 |
commit | fcdd6689d09c3950efa8a79db872f507069adb92 (patch) | |
tree | 4d91d7e02ab02f2bc5160edd627db48c9d761a6c /src/backend/commands/dbcommands.c | |
parent | 403ac226ddd6071245b7b283861c26960ea7293f (diff) | |
download | postgresql-fcdd6689d09c3950efa8a79db872f507069adb92.tar.gz postgresql-fcdd6689d09c3950efa8a79db872f507069adb92.zip |
Harden xxx_is_visible() functions against concurrent object drops.
For the same reasons given in commit 403ac226d, adjust these
functions to not assume that checking SearchSysCacheExists can
guarantee success of a later fetch.
This follows the same internal API choices made in the earlier commit:
add a function XXXExt(oid, is_missing) and use that to eliminate
the need for a separate existence check. The changes are very
straightforward, though tedious. For the moment I just made the new
functions static in namespace.c, but we could export them if a need
emerges.
Per bug #18014 from Alexander Lakhin. Given the lack of hard evidence
that there's a bug in non-debug builds, I'm content to fix this only
in HEAD.
Discussion: https://postgr.es/m/18014-28c81cb79d44295d@postgresql.org
Diffstat (limited to 'src/backend/commands/dbcommands.c')
0 files changed, 0 insertions, 0 deletions