diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-10-02 18:13:19 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-10-02 18:13:19 +0000 |
commit | 81f730521197a56937a10039e92e52c31b9d8847 (patch) | |
tree | 845bf32926246e55c651643b67bc778007555c64 /src/backend/executor/functions.c | |
parent | 9cfc3d21661d21906cc3d235c53e9c1f5ceb8d77 (diff) |
Fix erroneous handling of shared dependencies (ie dependencies on roles)
in CREATE OR REPLACE FUNCTION. The original code would update pg_shdepend
as if a new function was being created, even if it wasn't, with two bad
consequences: pg_shdepend might record the wrong owner for the function,
and any dependencies for roles mentioned in the function's ACL would be lost.
The fix is very easy: just don't touch pg_shdepend at all when doing a
function replacement.
Also update the CREATE FUNCTION reference page, which never explained
exactly what changes and doesn't change in a function replacement.
In passing, fix the CREATE VIEW reference page similarly; there's no
code bug there, but the docs didn't say what happens.
Diffstat (limited to 'src/backend/executor/functions.c')
0 files changed, 0 insertions, 0 deletions