diff options
author | Noah Misch <noah@leadboat.com> | 2023-05-08 06:14:07 -0700 |
---|---|---|
committer | Noah Misch <noah@leadboat.com> | 2023-05-08 06:14:11 -0700 |
commit | dbd5795e7539ec9e15c0d4ed2d05b1b18d2a3b09 (patch) | |
tree | a3e3c6e7fd55be595f0234b2546c468e6bb4242c /src/backend/executor/nodeRecursiveunion.c | |
parent | 8229bfe91def5878b498996ab24b62950edd9e40 (diff) |
Replace last PushOverrideSearchPath() call with set_config_option().
The two methods don't cooperate, so set_config_option("search_path",
...) has been ineffective under non-empty overrideStack. This defect
enabled an attacker having database-level CREATE privilege to execute
arbitrary code as the bootstrap superuser. While that particular attack
requires v13+ for the trusted extension attribute, other attacks are
feasible in all supported versions.
Standardize on the combination of NewGUCNestLevel() and
set_config_option("search_path", ...). It is newer than
PushOverrideSearchPath(), more-prevalent, and has no known
disadvantages. The "override" mechanism remains for now, for
compatibility with out-of-tree code. Users should update such code,
which likely suffers from the same sort of vulnerability closed here.
Back-patch to v11 (all supported versions).
Alexander Lakhin. Reported by Alexander Lakhin.
Security: CVE-2023-2454
Diffstat (limited to 'src/backend/executor/nodeRecursiveunion.c')
0 files changed, 0 insertions, 0 deletions