NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit7445869

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 parentd572003 commit7445869Copy full SHA for 7445869

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
`@@ -5684,7 +5684,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5684`	`5684`
`5685`	`5685`	`tuple=SearchSysCache1(FOREIGNSERVEROID,ObjectIdGetDatum(objoid));`
`5686`	`5686`	`if (!HeapTupleIsValid(tuple))`
`5687`		`-elog(ERROR,"cache lookup failed for foreigndata wrapper %u",`
	`5687`	`+elog(ERROR,"cache lookup failed for foreignserver %u",`
`5688`	`5688`	`objoid);`
`5689`	`5689`
`5690`	`5690`	`aclDatum=SysCacheGetAttr(FOREIGNSERVEROID,tuple,`
`@@ -5765,7 +5765,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5765`	`5765`
`5766`	`5766`	`tuple=SearchSysCache1(NAMESPACEOID,ObjectIdGetDatum(objoid));`
`5767`	`5767`	`if (!HeapTupleIsValid(tuple))`
`5768`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`5768`	`+elog(ERROR,"cache lookup failed forschema %u",objoid);`
`5769`	`5769`
`5770`	`5770`	`aclDatum=SysCacheGetAttr(NAMESPACEOID,tuple,`
`5771`	`5771`	`Anum_pg_namespace_nspacl,&isNull);`
`@@ -5807,7 +5807,7 @@ recordExtObjInitPriv(Oid objoid, Oid classoid)`
`5807`	`5807`
`5808`	`5808`	`tuple=SearchSysCache1(TYPEOID,ObjectIdGetDatum(objoid));`
`5809`	`5809`	`if (!HeapTupleIsValid(tuple))`
`5810`		`-elog(ERROR,"cache lookup failed forfunction %u",objoid);`
	`5810`	`+elog(ERROR,"cache lookup failed fortype %u",objoid);`
`5811`	`5811`
`5812`	`5812`	`aclDatum=SysCacheGetAttr(TYPEOID,tuple,Anum_pg_type_typacl,`
`5813`	`5813`	`&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!

Commit7445869

File tree

1 file changed

1 file changed

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

0 commit comments