[JAMES-1213] Don't compare references but values

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0-beta3
• Fix Version/s: 3.0-beta3
• Component/s: UsersStore & UsersRepository
• Environment: All
• Assignee: Felix Knecht
• Reporter: Felix Knecht
• Created: 25/Mar/11 05:46
• Updated: 18/Jul/11 19:43
• Resolved: 25/Mar/11 05:47

Description

JCRUser compares references instead of values for a String property:

public boolean verifyPassword(String pass) {
final boolean result;
if (pass == null)

{ result = false; }

else

{ result = hashedSaltedPassword == hashPassword(userName, pass); }

return result;
}