]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4:dsdb/tests: let OwnerGroupDescriptorTests.test_141() set the required ACE explicitly
authorStefan Metzmacher <metze@samba.org>
Mon, 20 Mar 2023 11:04:37 +0000 (12:04 +0100)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 22 Mar 2023 22:10:32 +0000 (22:10 +0000)
All other tests use the same logic and run before, which means the ACE
is already there and is implicitly required.

As we want to cleanup the ACE after each test in the next step,
as the tests should not have side effects for other tests, e.g.
'blackbox.dbcheck'.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=15338

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source4/dsdb/tests/python/sec_descriptor.py

index bc432bdaa7423feee9faded13c074666df2187f2..2c197a846a53cefa6caa6ccac3367aa162e5a120 100755 (executable)
@@ -868,6 +868,9 @@ class OwnerGroupDescriptorTests(DescriptorTests):
         self.check_user_belongs(self.get_users_domain_dn(user_name), [])
         # Open Ldb connection with the tested user
         _ldb = self.get_ldb_connection(user_name, "samba123@")
+        # Change Schema partition descriptor
+        mod = "(A;;CC;;;AU)"
+        self.sd_utils.dacl_add_ace(self.schema_dn, mod)
         # Create a custom security descriptor
         # NB! Problematic owner part won't accept DA only <User Sid> !!!
         user_sid = self.sd_utils.get_object_sid(self.get_users_domain_dn(user_name))