Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SSSD does not handle group renames safely #2118

Closed
sssd-bot opened this issue May 2, 2020 · 0 comments
Closed

SSSD does not handle group renames safely #2118

sssd-bot opened this issue May 2, 2020 · 0 comments

Comments

@sssd-bot
Copy link

sssd-bot commented May 2, 2020

Cloned from Pagure issue: https://pagure.io/SSSD/sssd/issue/1076

  • Created at 2011-11-02 14:36:16 by sgallagh
  • Closed as Duplicate
  • Assigned to nobody

SSSD has a hard restriction in the SysDB that only one group can exist with a particular ID.

When a group is renamed on the server, it becomes an order-of-operations issue whether or not we handle it correctly. If we look up the old group name first, we'll see it's removed and delete it from our cache. If we ask for the new group name, we'll get an error attempting to save it to the cache because the IDs will match.

It's well-documented that we don't support multiple entries with the same GID and that doing so will result in unexpected behaviour, so I'd like to recommend the following change:

When we attempt to save a group to the sysdb and the lookup reveals that another group exists already with that GID, instead of failing we should remove the old group. If there actually ARE two groups with the same GID, the effect will be that they'll regularly remove each other, causing a lot more LDAP lookups (and confusing offline access), but this isn't a supported configuration anyway.

In the case that it's actually a rename, we'll handle things much more gracefully.

Comments


Comment from jhrozek at 2011-11-02 15:14:54

A similar (if not the same) issue is being tracked in https://fedorahosted.org/sssd/ticket/1040


Comment from sgallagh at 2011-11-02 15:30:07

Fields changed

resolution: => duplicate
status: new => closed


Comment from dpal at 2012-01-19 02:14:08

Fields changed

rhbz: => 0


Comment from simo at 2012-03-08 15:25:44

Fields changed

milestone: NEEDS_TRIAGE => void


Comment from sgallagh at 2017-02-24 14:50:28

Metadata Update from @sgallagh:

  • Issue set to the milestone: void
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant