You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by rm...@apache.org on 2020/12/09 05:27:09 UTC

[ranger] branch master updated: RANGER-3108:NPE in RangerPolicyRepository.init

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

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


The following commit(s) were added to refs/heads/master by this push:
     new bb4c84f  RANGER-3108:NPE in RangerPolicyRepository.init
bb4c84f is described below

commit bb4c84f97367d79370508635ae4c23abda6cc3a0
Author: Ramesh Mani <rm...@cloudera.com>
AuthorDate: Tue Dec 8 20:55:33 2020 -0800

    RANGER-3108:NPE in RangerPolicyRepository.init
---
 .../ranger/plugin/policyengine/RangerPolicyRepository.java   | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java b/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
index 2655103..38faddd 100644
--- a/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
+++ b/agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java
@@ -1079,12 +1079,14 @@ public class RangerPolicyRepository {
                 LOG.debug("rowFilter policy evaluation order: #" + (++order) + " - policy id=" + policy.getId() + "; name=" + policy.getName() + "; evalOrder=" + policyEvaluator.getEvalOrder());
             }
 
-            LOG.debug("audit policy evaluation order: " + this.auditPolicyEvaluators.size() + " policies");
-            order = 0;
-            for(RangerPolicyEvaluator policyEvaluator : this.auditPolicyEvaluators) {
-                RangerPolicy policy = policyEvaluator.getPolicy();
+            LOG.debug("audit policy evaluation order: " + ((this.auditPolicyEvaluators != null) ? this.auditPolicyEvaluators.size() : 0  + " policies"));
+            if (this.auditPolicyEvaluators != null) {
+                order = 0;
+                for(RangerPolicyEvaluator policyEvaluator : this.auditPolicyEvaluators) {
+                    RangerPolicy policy = policyEvaluator.getPolicy();
 
-                LOG.debug("rowFilter policy evaluation order: #" + (++order) + " - policy id=" + policy.getId() + "; name=" + policy.getName() + "; evalOrder=" + policyEvaluator.getEvalOrder());
+                    LOG.debug("rowFilter policy evaluation order: #" + (++order) + " - policy id=" + policy.getId() + "; name=" + policy.getName() + "; evalOrder=" + policyEvaluator.getEvalOrder());
+                }
             }
         }
     }