|
 |
711f45 |
From 703ab8c4dfb7f8fd1540c3849ad469d39695a26f Mon Sep 17 00:00:00 2001
|
|
|
711f45 |
From: Florence Blanc-Renaud <flo@redhat.com>
|
|
|
711f45 |
Date: Jan 25 2023 16:57:02 +0000
|
|
|
711f45 |
Subject: trust-add: handle missing msSFU30MaxGidNumber
|
|
|
711f45 |
|
|
|
711f45 |
|
|
|
711f45 |
When ipa trust-add is executed with --range-type ad-trust-posix,
|
|
|
711f45 |
the server tries to find the max uidnumber and max gidnumber
|
|
|
711f45 |
from AD domain controller.
|
|
|
711f45 |
The values are extracted from the entry
|
|
|
711f45 |
CN=<domain>,CN=ypservers,CN=ypServ30,CN=RpcServices,CN=System,<AD suffix>
|
|
|
711f45 |
in the msSFU30MaxUidNumber and msSFU30MaxGidNumber attributes.
|
|
|
711f45 |
|
|
|
711f45 |
msSFU30MaxUidNumber is required but not msSFU30MaxGidNumber.
|
|
|
711f45 |
In case msSFU30MaxGidNumber is missing, the code is currently assigning
|
|
|
711f45 |
a "None" value and later on evaluates the max between this value and
|
|
|
711f45 |
msSFU30MaxUidNumber. The max function cannot compare None and a list
|
|
|
711f45 |
of string and triggers an exception.
|
|
|
711f45 |
|
|
|
711f45 |
To avoid the exception, assign [b'0'] to max gid if msSFU30MaxGidNumber
|
|
|
711f45 |
is missing. This way, the comparison succeeds and max returns the
|
|
|
711f45 |
value from msSFU30MaxUidNumber.
|
|
|
711f45 |
|
|
|
711f45 |
Fixes: https://pagure.io/freeipa/issue/9310
|
|
|
711f45 |
Signed-off-by: Florence Blanc-Renaud <flo@redhat.com>
|
|
|
711f45 |
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
|
|
|
711f45 |
|
|
|
711f45 |
---
|
|
|
711f45 |
|
|
|
711f45 |
diff --git a/ipaserver/plugins/trust.py b/ipaserver/plugins/trust.py
|
|
|
711f45 |
index c074f6d..79264b8 100644
|
|
|
711f45 |
--- a/ipaserver/plugins/trust.py
|
|
|
711f45 |
+++ b/ipaserver/plugins/trust.py
|
|
|
711f45 |
@@ -379,7 +379,10 @@ def add_range(myapi, trustinstance, range_name, dom_sid, *keys, **options):
|
|
|
711f45 |
range_type = u'ipa-ad-trust-posix'
|
|
|
711f45 |
|
|
|
711f45 |
max_uid = info.get('msSFU30MaxUidNumber')
|
|
|
711f45 |
- max_gid = info.get('msSFU30MaxGidNumber', None)
|
|
|
711f45 |
+ # if max_gid is missing, assume 0 and the max will
|
|
|
711f45 |
+ # be obtained from max_uid. We just checked that
|
|
|
711f45 |
+ # msSFU30MaxUidNumber is defined
|
|
|
711f45 |
+ max_gid = info.get('msSFU30MaxGidNumber', [b'0'])
|
|
|
711f45 |
max_id = int(max(max_uid, max_gid)[0])
|
|
|
711f45 |
|
|
|
711f45 |
base_id = int(info.get('msSFU30OrderNumber')[0])
|
|
|
711f45 |
|