commit | 2135a00370e47d1b46b60d4c39dee82cb1c7fad7 | [log] [tgz] |
---|---|---|
author | Sailaja Polavarapu <spolavarapu@hortonworks.com> | Mon Feb 06 16:21:46 2017 -0800 |
committer | Velmurugan Periasamy <vel@apache.org> | Tue Feb 07 20:54:58 2017 -0500 |
tree | 3b57c108cdf744b0256f300a2adda4db99e6e0dd | |
parent | d6d5c2c3127f1d47a99637a95d34edb7559ee160 [diff] |
RANGER-1345: Added a null check in order to avoid possible Null pointer dereference errors Signed-off-by: Velmurugan Periasamy <vel@apache.org>
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 @@ } 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()); }