[jira] [Commented] (RANGER-3232) A Potential NPE in Issue RANGER-980

["Pradeep Agrawal (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/RANGER-3232?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17450373#comment-17450373 ] 

Pradeep Agrawal commented on RANGER-3232:
-----------------------------------------

+1 for the patch. Please raise the review request in the review board https://reviews.apache.org/

> A Potential NPE in Issue RANGER-980
> -----------------------------------
>
>                 Key: RANGER-3232
>                 URL: https://issues.apache.org/jira/browse/RANGER-3232
>             Project: Ranger
>          Issue Type: Improvement
>          Components: usersync
>    Affects Versions: 2.2.0
>            Reporter: Shiyou xin
>            Priority: Major
>             Fix For: 3.0.0, 2.2.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> --- a/ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
> +++ b/ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
> @@ -661,7 +661,7 @@ private void computeUserDelta(Map<String, Map<String, String>> sourceUsers) {
> } else {
> XUserInfo oldUser = userCache.get(userName);
> Map<String, String> oldUserAttrs = oldUser.getOtherAttrsMap();
> String oldUserDN = oldUserAttrs.get(UgsyncCommonConstants.FULL_NAME);
> if (StringUtils.equalsIgnoreCase(userDN, oldUserDN)
> && StringUtils.equalsIgnoreCase(oldUserAttrs.get(UgsyncCommonConstants.SYNC_SOURCE), newUserAttrs.get(UgsyncC
> && StringUtils.equalsIgnoreCase(oldUserAttrs.get(UgsyncCommonConstants.LDAP_URL), newUserAttrs.get(UgsyncComm



--
This message was sent by Atlassian Jira
(v8.20.1#820001)