NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite3897a3

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 parent1a3daa5 commite3897a3Copy full SHA for e3897a3

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
`@@ -6100,7 +6100,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`6100`	`6100`
`6101`	`6101`	`tuple=SearchSysCache1(FOREIGNSERVEROID,ObjectIdGetDatum(objoid));`
`6102`	`6102`	`if (!HeapTupleIsValid(tuple))`
`6103`		`-elog(ERROR,"cache lookup failed for foreigndata wrapper %u",`
	`6103`	`+elog(ERROR,"cache lookup failed for foreignserver %u",`
`6104`	`6104`	`objoid);`
`6105`	`6105`
`6106`	`6106`	`aclDatum=SysCacheGetAttr(FOREIGNSERVEROID,tuple,`
`@@ -6186,7 +6186,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`6186`	`6186`
`6187`	`6187`	`tuple=SearchSysCache1(NAMESPACEOID,ObjectIdGetDatum(objoid));`
`6188`	`6188`	`if (!HeapTupleIsValid(tuple))`
`6189`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`6189`	`+elog(ERROR,"cache lookup failed forschema %u",objoid);`
`6190`	`6190`
`6191`	`6191`	`aclDatum=SysCacheGetAttr(NAMESPACEOID,tuple,`
`6192`	`6192`	`Anum_pg_namespace_nspacl,&isNull);`
`@@ -6228,7 +6228,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`6228`	`6228`
`6229`	`6229`	`tuple=SearchSysCache1(TYPEOID,ObjectIdGetDatum(objoid));`
`6230`	`6230`	`if (!HeapTupleIsValid(tuple))`
`6231`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`6231`	`+elog(ERROR,"cache lookup failed fortype %u",objoid);`
`6232`	`6232`
`6233`	`6233`	`aclDatum=SysCacheGetAttr(TYPEOID,tuple,Anum_pg_type_typacl,`
`6234`	`6234`	`&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!

Commite3897a3

File tree

1 file changed

1 file changed

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

0 commit comments