This is an automated email from the ASF dual-hosted git repository.

yiguolei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new b68001aee5 [fix](priv) fix duplicated priv check when check column 
priv (#17446)
b68001aee5 is described below

commit b68001aee54528b02a93cb4a735f2d62b96f3b04
Author: Yulei-Yang <yulei.yang0...@gmail.com>
AuthorDate: Tue Mar 7 08:51:55 2023 +0800

    [fix](priv) fix duplicated priv check when check column priv (#17446)
    
    when executing select stmt, columns privilege check will be invoked 
multiple times(column number in select stmt)
    
    Issue Number: close #xxx
---
 .../java/org/apache/doris/mysql/privilege/AccessControllerManager.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/AccessControllerManager.java
 
b/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/AccessControllerManager.java
index 271ee18f03..2e588ea792 100644
--- 
a/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/AccessControllerManager.java
+++ 
b/fe/fe-core/src/main/java/org/apache/doris/mysql/privilege/AccessControllerManager.java
@@ -153,7 +153,7 @@ public class AccessControllerManager {
             PrivPredicate wanted) throws UserException {
         boolean hasGlobal = sysAccessController.checkGlobalPriv(currentUser, 
wanted);
         CatalogAccessController accessController = 
getAccessControllerOrDefault(ctl);
-        for (TableName tableName : tableToColsMap.keys()) {
+        for (TableName tableName : tableToColsMap.keySet()) {
             accessController.checkColsPriv(hasGlobal, currentUser, ctl, 
tableName.getDb(),
                     tableName.getTbl(), tableToColsMap.get(tableName), wanted);
         }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to