Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

vulnerability with ldap authentication

#555
  • Issue Type: Bug
  • Rundeck version: 1.4.2

Issue: vulnerability using ldap authentication

When an user is connect one time with the good login/pass,
anyone can connect to the web gui using the same login with any pass
oO and have all the user power...


Reproduce: (Enter steps to reproduce bug here.)
1- Connect the web GUI one user on the Computer1 with good credential
2- try to connect the same user with any pass on the Computer2
3- It will work with any password oO

Reported by Desaules · April 13th, 2012 @ 08:13 AM

State: resolved
Milestone: Rundeck 1.4.3
Assigned to: Greg Schueler Greg Schueler

Activity

  1. Greg Schueler
    Greg Schueler
    • State changed from new to open
    • Tag set to authentication, bug, ldap
    • Milestone set to Rundeck 1.4.3
    • Milestone order changed from 124 to 0

    analysis: if "forceBindingLogin" is set to true for the JettyCachingLdapLoginModule login module, then after a user logs in successfully there is a failure to verify the user credentials for the duration of the "cacheDurationMillis" for subsequent log ins.

    security:

    This means that while the user information is cached, the credentials are not checked for subsequent logins for that user.

    mitigation:

    Either set "forceBindingLogin" to false, or disable the caching behavior to prevent this issue.

    To disable the cache: In the jaas login module config file for the JettyCachingLdapLoginModule, set the "cacheDurationMillis" to 0.

    fix:

    the cache needs to use the user credentials to verify login attempts, not just the username.

    April 13th, 2012 @ 01:59 PM

  2. Greg Schueler
    Greg Schueler
    • State changed from open to needs_verification

    (from [0b91204303ffb47adee715961467b4fe85eef9a9]) Use digest of user+password as cache key

    [#555 vulnerability with ldap authentication state:needs_verification]

    require username+password to be valid before
    allowing use of cached user info.
    https://github.com/dtolabs/rundeck/commit/0b91204303ffb47adee715961...

    April 13th, 2012 @ 05:55 PM

  3. Greg Schueler
    Greg Schueler

    (from [35b34140896f761749d06dfd32d12863d77152e9]) Use digest of user+password as cache key

    [#555 vulnerability with ldap authentication state:needs_verification]

    require username+password to be valid before
    allowing use of cached user info.
    https://github.com/dtolabs/rundeck/commit/35b34140896f761749d06dfd3...

    April 13th, 2012 @ 05:55 PM

  4. Greg Schueler
    Greg Schueler
    • State changed from needs_verification to resolved
    • Milestone order changed from 35 to 0

    June 21st, 2012 @ 11:08 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.