s3-libads: Also handle the DS_WEB_SERVICE_REQUIRED flag in check_cldap_reply_required...

Signed-off-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>

diff --git a/selftest/knownfail.d/getdcname b/selftest/knownfail.d/getdcname
index 55bb6c822240f7d8082e948bf97594dbc1b03246..30d5fc7eda2e521b30a88e9c9f11c6a48166091b 100644 (file)
--- a/selftest/knownfail.d/getdcname
+++ b/selftest/knownfail.d/getdcname
@@ -1,4 +1,3 @@
 ^samba.tests.getdcname.samba.tests.getdcname.GetDCNameEx.test_get_dc_direct_need_2012r2_but_not_found\(fl2008r2dc\)
 ^samba.tests.getdcname.samba.tests.getdcname.GetDCNameEx.test_get_dc_direct_need_2012r2_but_not_found_fallback
 ^samba.tests.getdcname.samba.tests.getdcname.GetDCNameEx.test_get_dc_direct_need_web_but_not_found
-^samba.tests.getdcname.samba.tests.getdcname.GetDCNameEx.test_get_dc_winbind_need_web_but_not_found

diff --git a/source3/libads/cldap.c b/source3/libads/cldap.c
index 623d98dcdf41661f2710076ec1c3d02d780afc0a..56c2537ffa985d736dbe8129078932b642bc585a 100644 (file)
--- a/source3/libads/cldap.c
+++ b/source3/libads/cldap.c
@@ -79,6 +79,9 @@ bool check_cldap_reply_required_flags(uint32_t ret_flags,
        if (req_flags & DS_TIMESERV_REQUIRED)
                RETURN_ON_FALSE(ret_flags & NBT_SERVER_TIMESERV);
 
+       if (req_flags & DS_WEB_SERVICE_REQUIRED)
+               RETURN_ON_FALSE(ret_flags & NBT_SERVER_ADS_WEB_SERVICE);
+
        if (req_flags & DS_WRITABLE_REQUIRED)
                RETURN_ON_FALSE(ret_flags & NBT_SERVER_WRITABLE);