From e8d9caa436db2931613d7cb4a4a8b3933d5f6ded Mon Sep 17 00:00:00 2001 From: Thomas Munro Date: Fri, 13 Jul 2018 16:12:03 +1200 Subject: [PATCH] Accept invalidation messages in InitializeSessionUserId(). If the authentication method modified the system catalogs through a separate database connection (say, to create a new role on the fly), make sure syscache sees the changes before we try to find the user. Author: Thomas Munro Reviewed-by: Tom Lane, Andres Freund Discussion: https://postgr.es/m/CAEepm%3D3_h0_cgmz5PMyab4xk_OFrg6G5VCN%3DnF4chFXM9iFOqA%40mail.gmail.com --- src/backend/utils/init/miscinit.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/backend/utils/init/miscinit.c b/src/backend/utils/init/miscinit.c index 4bb28938c2..865119d272 100644 --- a/src/backend/utils/init/miscinit.c +++ b/src/backend/utils/init/miscinit.c @@ -48,6 +48,7 @@ #include "storage/procarray.h" #include "utils/builtins.h" #include "utils/guc.h" +#include "utils/inval.h" #include "utils/memutils.h" #include "utils/pidfile.h" #include "utils/syscache.h" @@ -594,6 +595,13 @@ InitializeSessionUserId(const char *rolename, Oid roleid) /* call only once */ AssertState(!OidIsValid(AuthenticatedUserId)); + /* + * Make sure syscache entries are flushed for recent catalog changes. + * This allows us to find roles that were created on-the-fly during + * authentication. + */ + AcceptInvalidationMessages(); + if (rolename != NULL) { roleTup = SearchSysCache1(AUTHNAME, PointerGetDatum(rolename));