]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4:selftest: only run ad_member with AUTH_LEVEL_CONNECT_LSA=1
authorStefan Metzmacher <metze@samba.org>
Mon, 23 Sep 2024 13:13:59 +0000 (15:13 +0200)
committerAndreas Schneider <asn@cryptomilk.org>
Thu, 10 Oct 2024 14:01:04 +0000 (14:01 +0000)
We only want to test against
'allow dcerpc auth level connect:lsarpc = yes' once
in order to have the related code tests.
We use the ad_memeber for that special test and
use the default on the tested ADDC.

This reveals some knownfails, which will be fixed in
the next commit...

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

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
selftest/knownfail.d/test_lsa_multi_auth_connect [new file with mode: 0644]
selftest/target/Samba4.pm
source4/selftest/tests.py

diff --git a/selftest/knownfail.d/test_lsa_multi_auth_connect b/selftest/knownfail.d/test_lsa_multi_auth_connect
new file mode 100644 (file)
index 0000000..0cd69dc
--- /dev/null
@@ -0,0 +1 @@
+^samba.tests.dcerpc.raw_protocol.samba.tests.dcerpc.raw_protocol.TestDCERPC_BIND.test_lsa_multi_auth_connect...chgdcpass
index 4b289fb27aeddf4094d5764c61fa410853bf9585..e917f65fc36ae24447ea62bfbecaae5d2ae40056 100755 (executable)
@@ -2208,7 +2208,6 @@ sub provision_chgdcpass($$)
 
        my $extra_smb_conf = "
        check password script = $self->{srcdir}/selftest/checkpassword_arg1.sh ${unacceptable_password}
-       allow dcerpc auth level connect:lsarpc = yes
        dcesrv:max auth states = 8
         drs:broken_samba_4.5_get_anc_emulation = true
         drs:get_tgt_support = false
index c4f980196464c8be317db3a64ad38f50b968cd54..69d670fe7b466648b46abbc4160b3dc1131fb925 100755 (executable)
@@ -1368,14 +1368,19 @@ planoldpythontestsuite(
 planoldpythontestsuite("ad_dc_ntvfs", "samba.tests.dcerpc.dnsserver", extra_args=['-U"$USERNAME%$PASSWORD"'])
 planoldpythontestsuite("ad_dc", "samba.tests.dcerpc.dnsserver", extra_args=['-U"$USERNAME%$PASSWORD"'])
 
-for env in ["chgdcpass", "ad_member"]:
-    planoldpythontestsuite(env, "samba.tests.dcerpc.raw_protocol",
-                           environ={"MAX_NUM_AUTH": "8",
-                                    "ALLOW_BIND_AUTH_PAD": "1",
-                                    "AUTH_LEVEL_CONNECT_LSA": "1",
-                                    "LEGACY_BIND_NACK_NO_REASON": "1",
-                                    "USERNAME": "$DC_USERNAME",
-                                    "PASSWORD": "$DC_PASSWORD"})
+planoldpythontestsuite("chgdcpass", "samba.tests.dcerpc.raw_protocol",
+                       environ={"MAX_NUM_AUTH": "8",
+                                "ALLOW_BIND_AUTH_PAD": "1",
+                                "LEGACY_BIND_NACK_NO_REASON": "1",
+                                "USERNAME": "$DC_USERNAME",
+                                "PASSWORD": "$DC_PASSWORD"})
+planoldpythontestsuite("ad_member", "samba.tests.dcerpc.raw_protocol",
+                       environ={"MAX_NUM_AUTH": "8",
+                                "ALLOW_BIND_AUTH_PAD": "1",
+                                "AUTH_LEVEL_CONNECT_LSA": "1",
+                                "LEGACY_BIND_NACK_NO_REASON": "1",
+                                "USERNAME": "$DC_USERNAME",
+                                "PASSWORD": "$DC_PASSWORD"})
 
 if have_heimdal_support:
     planoldpythontestsuite("ad_dc_smb1:local", "samba.tests.auth_log", extra_args=['-U"$USERNAME%$PASSWORD"'],