NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit4dbe72d

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 parent4fceb45 commit4dbe72dCopy full SHA for 4dbe72d

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
`@@ -5652,7 +5652,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5652`	`5652`
`5653`	`5653`	`tuple=SearchSysCache1(FOREIGNSERVEROID,ObjectIdGetDatum(objoid));`
`5654`	`5654`	`if (!HeapTupleIsValid(tuple))`
`5655`		`-elog(ERROR,"cache lookup failed for foreigndata wrapper %u",`
	`5655`	`+elog(ERROR,"cache lookup failed for foreignserver %u",`
`5656`	`5656`	`objoid);`
`5657`	`5657`
`5658`	`5658`	`aclDatum=SysCacheGetAttr(FOREIGNSERVEROID,tuple,`
`@@ -5738,7 +5738,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5738`	`5738`
`5739`	`5739`	`tuple=SearchSysCache1(NAMESPACEOID,ObjectIdGetDatum(objoid));`
`5740`	`5740`	`if (!HeapTupleIsValid(tuple))`
`5741`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`5741`	`+elog(ERROR,"cache lookup failed forschema %u",objoid);`
`5742`	`5742`
`5743`	`5743`	`aclDatum=SysCacheGetAttr(NAMESPACEOID,tuple,`
`5744`	`5744`	`Anum_pg_namespace_nspacl,&isNull);`
`@@ -5780,7 +5780,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5780`	`5780`
`5781`	`5781`	`tuple=SearchSysCache1(TYPEOID,ObjectIdGetDatum(objoid));`
`5782`	`5782`	`if (!HeapTupleIsValid(tuple))`
`5783`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`5783`	`+elog(ERROR,"cache lookup failed fortype %u",objoid);`
`5784`	`5784`
`5785`	`5785`	`aclDatum=SysCacheGetAttr(TYPEOID,tuple,Anum_pg_type_typacl,`
`5786`	`5786`	`&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!

Commit4dbe72d

File tree

1 file changed

1 file changed

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

0 commit comments