Repository: ranger Updated Branches: refs/heads/master d6d5c2c31 -> 2135a0037
RANGER-1345: Added a null check in order to avoid possible Null pointer dereference errors Signed-off-by: Velmurugan Periasamy <v...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/ranger/repo Commit: http://git-wip-us.apache.org/repos/asf/ranger/commit/2135a003 Tree: http://git-wip-us.apache.org/repos/asf/ranger/tree/2135a003 Diff: http://git-wip-us.apache.org/repos/asf/ranger/diff/2135a003 Branch: refs/heads/master Commit: 2135a00370e47d1b46b60d4c39dee82cb1c7fad7 Parents: d6d5c2c Author: Sailaja Polavarapu <spolavar...@hortonworks.com> Authored: Mon Feb 6 16:21:46 2017 -0800 Committer: Velmurugan Periasamy <v...@apache.org> Committed: Tue Feb 7 20:54:58 2017 -0500 ---------------------------------------------------------------------- .../ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ranger/blob/2135a003/ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java ---------------------------------------------------------------------- diff --git a/ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java b/ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java index 4c95907..f93083c 100644 --- a/ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java +++ b/ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java @@ -414,7 +414,7 @@ private static final Logger LOG = Logger.getLogger(LdapPolicyMgrUserGroupBuilder } List<String> oldUsers = new ArrayList<String>(); - if (groupUserInfo.getXuserInfo() != null) { + if (groupUserInfo != null && groupUserInfo.getXuserInfo() != null) { for (XUserInfo xUserInfo : groupUserInfo.getXuserInfo()) { oldUsers.add(xUserInfo.getName()); }
