Re: Review Request 63534: RANGER-1870:Modify the logic for buildUnixUserList and parseMembers method in UnixUserGroupBuilder class

[Qiang Zhang]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63534/
-----------------------------------------------------------

(Updated 十一月 3, 2017, 2:23 a.m.)


Review request for ranger, Ankita Sinha, Don Bosco Durai, Colm O hEigeartaigh, 
Gautam Borad, Madhan Neethiraj, pengjianhua, Ramesh Mani, Selvamohan Neethiraj, 
sam  rome, and Velmurugan Periasamy.


Bugs: RANGER-1870
    https://issues.apache.org/jira/browse/RANGER-1870


Repository: ranger


Description
-------

Modify the logic for buildUnixUserList and parseMembers method in 
UnixUserGroupBuilder class 
1.In the buildUnixUserList method:
int len = tokens.length;
if (len = 3) groupId = tokens [3]; case ArrayIndexOutOfBoundsException.
2.Another similar error in the parseMembers method:
String[] tokens = line.split(":");
if (tokens.length = 2) String groupId = tokens [2]; case 
ArrayIndexOutOfBoundsException.


Diffs
-----

  
ugsync/src/main/java/org/apache/ranger/unixusersync/process/UnixUserGroupBuilder.java
 045fe3f 


Diff: https://reviews.apache.org/r/63534/diff/1/


Testing (updated)
-------

tested it.


Thanks,

Qiang Zhang