[hive] branch master updated: HIVE-24097: correct NPE exception in HiveMetastoreAuthorizer (Sam An reviewed by Naveen Gangam)

[ngangam]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5e951b3  HIVE-24097: correct NPE exception in HiveMetastoreAuthorizer 
(Sam An reviewed by Naveen Gangam)
5e951b3 is described below

commit 5e951b33f894002defa1d19372d29075379a15f7
Author: Sam An <sam...@cloudera.com>
AuthorDate: Mon Aug 31 14:05:38 2020 -0700

    HIVE-24097: correct NPE exception in HiveMetastoreAuthorizer (Sam An 
reviewed by Naveen Gangam)
---
 .../authorization/plugin/metastore/HiveMetaStoreAuthorizer.java        | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
 
b/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
index 8dbeeee..36bcb89 100644
--- 
a/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
+++ 
b/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/metastore/HiveMetaStoreAuthorizer.java
@@ -534,8 +534,9 @@ public class HiveMetaStoreAuthorizer extends 
MetaStorePreEventListener implement
   private boolean skipAuthorization(HiveMetaStoreAuthzInfo authzContext) {
     LOG.debug("==> HiveMetaStoreAuthorizer.skipAuthorization()");
 
+    //If HMS does not check the event type, it will leave it as null. We don't 
try to authorize null pointer. 
     if(authzContext == null){
-      return false;
+      return true;
     }
     boolean ret = false;
     UserGroupInformation ugi = null;