Set GUC "is_superuser" in all processes that set AuthenticatedUserId.
authorNoah Misch <[email protected]>
Mon, 6 Nov 2023 14:14:13 +0000 (06:14 -0800)
committerNoah Misch <[email protected]>
Mon, 6 Nov 2023 14:14:13 +0000 (06:14 -0800)
It was always false in single-user mode, in autovacuum workers, and in
background workers.  This had no specifically-identified security
consequences, but non-core code or future work might make it
security-relevant.  Back-patch to v11 (all supported versions).

Jelte Fennema-Nio.  Reported by Jelte Fennema-Nio.

src/backend/utils/init/miscinit.c

index 182d666852f3a80183ccda0762999065a87af1d4..cfc5afaa6fd7046aaae9bb3f4e2a6345c09b53fc 100644 (file)
@@ -842,6 +842,14 @@ InitializeSessionUserIdStandalone(void)
 
        AuthenticatedUserId = BOOTSTRAP_SUPERUSERID;
        SetSessionUserId(BOOTSTRAP_SUPERUSERID, true);
+
+       /*
+        * XXX This should set SetConfigOption("session_authorization"), too.
+        * Since we don't, C code will get NULL, and current_setting() will get an
+        * empty string.
+        */
+       SetConfigOption("is_superuser", "on",
+                                       PGC_INTERNAL, PGC_S_DYNAMIC_DEFAULT);
 }
 
 /*