diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-10-02 18:13:32 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-10-02 18:13:32 +0000 |
commit | 595f2744b0498e5b82649242aae35553761f3081 (patch) | |
tree | 3a95213fd6fb73b94be6b958b1a1238081bc0781 /src/backend/executor/nodeHash.c | |
parent | fa990959f453a84e3aab11b24b10e0eea75471bc (diff) | |
download | postgresql-595f2744b0498e5b82649242aae35553761f3081.tar.gz postgresql-595f2744b0498e5b82649242aae35553761f3081.zip |
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/nodeHash.c')
0 files changed, 0 insertions, 0 deletions