Bug 50973 – IndexError when setting sambaRID without sambaSID

Bug 50973 - IndexError when setting sambaRID without sambaSID


Summary:	IndexError when setting sambaRID without sambaSID

Status:	NEW

Product:	UCS
Classification:	Unclassified
Component:	UDM (Generic)
Version:	UCS 4.4
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	UMC maintainers
QA Contact:	UMC maintainers

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2020-03-18 12:53 CET by Daniel Tröder
Modified:	2020-04-02 11:19 CEST (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Bug Report
What type of bug is this?:	3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?:	1: Will affect a very few installed domains
How will those affected feel about the bug?:	2: A Pain – users won’t like this once they notice it
User Pain:	0.034
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Daniel Tröder

2020-03-18 12:53:16 CET

There is a bug in the code of groups/group that triggers IndexError: list index out of range when enabling the "samba" option and setting the sambaRID in one step:

line 638, in _ldap_post_modify:

   self._update_sambaPrimaryGroupSID(self.oldattr.get('sambaSID', [])[0], self.groupSid)

dict.get(attr, [])[0] → will always create an IndexError when attr is not set.