trino icon indicating copy to clipboard operation
trino copied to clipboard

Fix PTF support in TestingAccessControlManager

Open huberty89 opened this issue 2 years ago • 0 comments

Description

Invoked PTF from a SECURITY DEFINER view using LocalQueryRunner cause to call incorrect overloaded method checkCanGrantExecuteFunctionPrivilege. This cause perform a wrong check from SystemAccessControl, this check invoked:

checkCanGrantExecuteFunctionPrivilege(SystemSecurityContext context, String functionName, TrinoPrincipal grantee, boolean grantOption)

instead of:

checkCanGrantExecuteFunctionPrivilege(SystemSecurityContext context, FunctionKind functionKind, CatalogSchemaRoutineName functionName, TrinoPrincipal grantee, boolean grantOption)

Release notes

(x) This is not user-visible or docs only and no release notes are required. ( ) Release notes are required, please propose a release note for me. ( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

huberty89 avatar Sep 20 '22 18:09 huberty89