[AAA-151] Previous password continues to work after password change Created: 21/Nov/17 Updated: 28/Nov/17 Resolved: 28/Nov/17 |
|
| Status: | Resolved |
| Project: | aaa |
| Component/s: | General |
| Affects Version/s: | None |
| Fix Version/s: | Carbon-SR3, Nitrogen-SR1 |
| Type: | Bug | Priority: | High |
| Reporter: | Vaibhav Hemant Dixit | Assignee: | Ryan Goulding |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Description |
|
#security-status: confirmed-leaked This issue has been confirmed as a security vulnerability in Vaibhav Hemant Dixit reported the following security vulnerability to the security mailing list: Severity : OPENDAYLIGHT AUTHENTICATION BREACHED Issue: After updating the password, the login is successful with both OLD and NEW passwords Steps to reproduce: Start the controller. Observation: The admin user can authenticate using both OLD and NEW passwords. |
| Comments |
| Comment by Ryan Goulding [ 21/Nov/17 ] |
|
Proposed fix: https://git.opendaylight.org/gerrit/#/q/topic:AAA-151 Affects CLI only; the REST endpoints do Claim cache invalidation already. Reboot works because the ClaimCache is force-flushed upon exit of the Java process. Although the CLI requires system access for (bin/client) or SSH access to the Karaf process, the effects are still bad because an admin expects that the old password should not work after he/she has changed it. He or she invokes the change-user-password Karaf CLI command expecting that the old credentials will no longer be accepted. Not sure how we want to handle this, but I'd imagine it involves CVE and proper documentation. The fixes will be merged to the affected branches (carbon, nitrogen & master) as they pass jenkins-releng. |
| Comment by Luke Hinds [ 22/Nov/17 ] |
|
|
| Comment by Luke Hinds [ 22/Nov/17 ] |
|
rgoulding et al, please verify the impact description:
|
| Comment by Ryan Goulding [ 28/Nov/17 ] |
|
My apologies, I was out for the Holiday weekend. This seems accurate to me. Thanks for your help with this issue. |