Bug 751314

Summary:	SSSD does not sanitize broken memberUID correctly
Product:	Red Hat Enterprise Linux 6	Reporter:	Jakub Hrozek <jhrozek>
Component:	sssd	Assignee:	Stephen Gallagher <sgallagh>
Status:	CLOSED DEFERRED	QA Contact:	IDM QE LIST <seceng-idm-qe-list>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	6.2	CC:	dpal, grajaiya, jgalipea, prc
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:	When a group contains certain incorrect multi-valued memberUID values, SSSD fails to sanitize the values properly. The memberUID value should only contain one username. As a result, SSSD creates incorrect users, using the broken memberUID values as their usernames. This, for example, causes problems during cache indexing.	Story Points:	---
Clone Of:		Environment:
Last Closed:	2011-11-14 18:31:06 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jakub Hrozek 2011-11-04 10:03:26 UTC

Description of problem:
When a group contains certain incorrect multivalued memberUID values, SSSD fails to sanitize the value properly. The memberUID value should contain one username, so SSSD creates fake users using the broken memberUID as their username. This causes problems during cache indexing, for example.

Version-Release number of selected component (if applicable):
sssd-1.5.1-66

How reproducible:
always

Steps to Reproduce:
1. Create a group similar to this one:

dn: badmemberuid,ou=Groups,dc=example,dc=com
memberUid: userone,usertwo,userthree
gidNumber: 5418
cn: badmemberuid
objectClass: posixGroup
objectClass: top

2. getent group badmemberuid
3. ldbsearch -H /var/lib/sss/db/cache_example.com.ldb

Actual results:
The commas are not treated as special characters.

Expected results:
The commas are escaped with a '\' to sanitize them.

The DN should be sanitized and upgrades (which index the cache) should perform fine.

Comment 1 Jakub Hrozek 2011-11-04 10:05:45 UTC

Upstream ticket:
https://fedorahosted.org/sssd/ticket/1079

Comment 3 Jakub Hrozek 2011-11-04 10:20:32 UTC

Too late for 6.2. We agreed to document this issue in release notes.