diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2021-06-21 11:17:49 +0200 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2021-06-21 11:17:49 +0200 |
| commit | 97b7134186490b36e01efc9d2feaf7038c666457 (patch) | |
| tree | daa5107a35e6f28752185ef4326f6fa0ad5bb0ae | |
| parent | 69a58bfe4ab05567a8fab8bdce7f3095ed06b99c (diff) | |
| download | postgresql-97b7134186490b36e01efc9d2feaf7038c666457.tar.gz postgresql-97b7134186490b36e01efc9d2feaf7038c666457.zip | |
amcheck: Fix code comments
Code comments were claiming that verify_heapam() was checking
privileges on the relation it was operating on, but it didn't actually
do that. Perhaps earlier versions of the patch did that, but now the
access is regulated by privileges on the function. Remove the wrong
comments.
| -rw-r--r-- | contrib/amcheck/verify_heapam.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/contrib/amcheck/verify_heapam.c b/contrib/amcheck/verify_heapam.c index d8b3fd3d4f9..a3caee7cdd3 100644 --- a/contrib/amcheck/verify_heapam.c +++ b/contrib/amcheck/verify_heapam.c @@ -298,7 +298,7 @@ verify_heapam(PG_FUNCTION_ARGS) rsinfo->setDesc = ctx.tupdesc; MemoryContextSwitchTo(old_context); - /* Open relation, check relkind and access method, and check privileges */ + /* Open relation, check relkind and access method */ ctx.rel = relation_open(relid, AccessShareLock); sanity_check_relation(ctx.rel); @@ -524,8 +524,7 @@ verify_heapam(PG_FUNCTION_ARGS) } /* - * Check that a relation's relkind and access method are both supported, - * and that the caller has select privilege on the relation. + * Check that a relation's relkind and access method are both supported. */ static void sanity_check_relation(Relation rel) |