NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7ad458e

committed

Fix come incorrect elog() messages in aclchk.c

Three error strings used with cache lookup failures were referring toincorrect object types for ACL checks:- Schemas- Types- Foreign ServersThere errors should never be triggered, but if they do incorrectinformation would be reported.Author: Justin PryzbyDiscussion:https://postgr.es/m/20221222153041.GN1153@telsasoft.comBackpatch-through: 11

1 parent97431d6 commit7ad458eCopy full SHA for 7ad458e

File tree

1 file changed

-3

lines changed

src/backend/catalog
- aclchk.c

1 file changed

-3

lines changed

`‎src/backend/catalog/aclchk.c‎`

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -5752,7 +5752,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5752`	`5752`
`5753`	`5753`	`tuple=SearchSysCache1(FOREIGNSERVEROID,ObjectIdGetDatum(objoid));`
`5754`	`5754`	`if (!HeapTupleIsValid(tuple))`
`5755`		`-elog(ERROR,"cache lookup failed for foreigndata wrapper %u",`
	`5755`	`+elog(ERROR,"cache lookup failed for foreignserver %u",`
`5756`	`5756`	`objoid);`
`5757`	`5757`
`5758`	`5758`	`aclDatum=SysCacheGetAttr(FOREIGNSERVEROID,tuple,`
`@@ -5838,7 +5838,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5838`	`5838`
`5839`	`5839`	`tuple=SearchSysCache1(NAMESPACEOID,ObjectIdGetDatum(objoid));`
`5840`	`5840`	`if (!HeapTupleIsValid(tuple))`
`5841`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`5841`	`+elog(ERROR,"cache lookup failed forschema %u",objoid);`
`5842`	`5842`
`5843`	`5843`	`aclDatum=SysCacheGetAttr(NAMESPACEOID,tuple,`
`5844`	`5844`	`Anum_pg_namespace_nspacl,&isNull);`
`@@ -5880,7 +5880,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5880`	`5880`
`5881`	`5881`	`tuple=SearchSysCache1(TYPEOID,ObjectIdGetDatum(objoid));`
`5882`	`5882`	`if (!HeapTupleIsValid(tuple))`
`5883`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`5883`	`+elog(ERROR,"cache lookup failed fortype %u",objoid);`
`5884`	`5884`
`5885`	`5885`	`aclDatum=SysCacheGetAttr(TYPEOID,tuple,Anum_pg_type_typacl,`
`5886`	`5886`	`&isNull);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit7ad458e

File tree

1 file changed

1 file changed

`‎src/backend/catalog/aclchk.c‎`

0 commit comments