Fix buggy logic in isTempNamespaceInUse()
authorMichael Paquier <michael@paquier.xyz>
Wed, 15 Jan 2020 04:58:41 +0000 (13:58 +0900)
committerMichael Paquier <michael@paquier.xyz>
Wed, 15 Jan 2020 04:58:41 +0000 (13:58 +0900)
commit0fca3d0a4ec297bff5a5cb01dfe345e0f63d7d63
tree962c344943e01846c6e1739c7e1e4a82555c28a1
parent29af9c542f6cdc9a6456b175fdcdfe7a28b77d6c
Fix buggy logic in isTempNamespaceInUse()

The logic introduced in this routine as of 246a6c8 would report an
incorrect result when a session calls it to check if the temporary
namespace owned by the session is in use or not.  It is possible to
optimize more the routine in this case to avoid a PGPROC lookup, but
let's keep the logic simple.  As this routine is used only by autovacuum
for now, there were no live bugs, still let's be correct for any future
code involving it.

Author: Michael Paquier
Reviewed-by: Julien Rouhaud
Discussion: https://postgr.es/m/20200113093703.GA41902@paquier.xyz
Backpatch-through: 11
src/backend/catalog/namespace.c