diff options
author | Tom Lane <tgl@sss.pgh.pa.us> | 2009-10-02 18:13:26 +0000 |
---|---|---|
committer | Tom Lane <tgl@sss.pgh.pa.us> | 2009-10-02 18:13:26 +0000 |
commit | 9b5ac366f922cee5bbb73180fe400dffb5c13136 (patch) | |
tree | c119c76e508b6601f3a6035f4ecb95a7403ff0ff /src/backend/utils/adt/network.c | |
parent | 3bf3527fe05d84bb3b4bb1f7c68d9afeaf79d733 (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/utils/adt/network.c')
0 files changed, 0 insertions, 0 deletions