The branch, v4-0-stable has been updated
       via  3be3266 VERSION: Disable git snapshots for the 4.0.24 release.
       via  65a088d WHATSNEW: Add release notes for Samba 4.0.24.
       via  3d221ef CVE-2014-8143:dsdb-samldb: Check for extended access rights 
before we allow changes to userAccountControl
       via  01a4bd7 CVE-2014-8143:dsdb: Allow use of 
dsdb_autotransaction_request outside util.c
       via  5acd6c0 CVE-2014-8143:pydsdb: Pull in UF_USE_AES_KEYS flag
       via  392523a CVE-2014-8143:auth: Force talloc type of session_info 
pointer to match
       via  003ae6a VERSION: Bump version up to 4.0.24...
      from  21764ef VERSION: Disable git snapshots for the 4.0.23 release.

https://git.samba.org/?p=samba.git;a=shortlog;h=v4-0-stable


- Log -----------------------------------------------------------------
commit 3be3266359da9ee6cd3aac970a81f019e8c389eb
Author: Karolin Seeger <[email protected]>
Date:   Mon Jan 12 21:05:40 2015 +0100

    VERSION: Disable git snapshots for the 4.0.24 release.
    
    Signed-off-by: Karolin Seeger <[email protected]>

commit 65a088d2d418c4285de26a5636f0e7a38ae28ec9
Author: Karolin Seeger <[email protected]>
Date:   Mon Jan 12 21:01:33 2015 +0100

    WHATSNEW: Add release notes for Samba 4.0.24.
    
    Signed-off-by: Karolin Seeger <[email protected]>

commit 3d221efd635601e8f1ba08e018a248472a36d5df
Author: Andrew Bartlett <[email protected]>
Date:   Thu Dec 4 17:23:29 2014 +1300

    CVE-2014-8143:dsdb-samldb: Check for extended access rights before we allow 
changes to userAccountControl
    
    This requires an additional control to be used in the
    LSA server to add domain trust account objects.
    
    Bug: https://bugzilla.samba.org/show_bug.cgi?id=10993
    
    Signed-off-by: Andrew Bartlett <[email protected]>
    Reviewed-by: Stefan Metzmacher <[email protected]>

commit 01a4bd717e1ae61f207bd0ae3109ee20846ff426
Author: Andrew Bartlett <[email protected]>
Date:   Mon Dec 8 14:20:21 2014 +1300

    CVE-2014-8143:dsdb: Allow use of dsdb_autotransaction_request outside util.c
    
    Bug: https://bugzilla.samba.org/show_bug.cgi?id=10993
    
    Change-Id: If6bc90305a1e9a5a92562a01ba7e44330de91cc1
    Pair-programmed-with: Garming Sam <[email protected]>
    Signed-off-by: Andrew Bartlett <[email protected]>
    Signed-off-by: Garming Sam <[email protected]>
    Reviewed-by: Stefan Metzmacher <[email protected]>

commit 5acd6c08f1b9c79a820d3f6b907563bcd1101e15
Author: Andrew Bartlett <[email protected]>
Date:   Mon Dec 8 12:19:19 2014 +1300

    CVE-2014-8143:pydsdb: Pull in UF_USE_AES_KEYS flag
    
    Bug: https://bugzilla.samba.org/show_bug.cgi?id=10993
    
    Change-Id: I36ad5ebc5d8a4811c41b59af90a3add4ae5fd857
    Signed-off-by: Andrew Bartlett <[email protected]>
    Reviewed-by: Garming Sam <[email protected]>
    Reviewed-by: Stefan Metzmacher <[email protected]>

commit 392523a227671eb1570d1815c8280fe001d8a435
Author: Andrew Bartlett <[email protected]>
Date:   Tue Nov 11 15:23:02 2014 +1300

    CVE-2014-8143:auth: Force talloc type of session_info pointer to match
    
    This helps us keep things safe in LDB where we put this in a opaque pointer.
    
    Bug: https://bugzilla.samba.org/show_bug.cgi?id=10993
    
    Andrew Bartlett
    
    Change-Id: I46fe53ba655ca0810c276b72fbca524884cdf22d
    Signed-off-by: Andrew Bartlett <[email protected]>
    Reviewed-by: Garming Sam <[email protected]>
    Reviewed-by: Stefan Metzmacher <[email protected]>

commit 003ae6a1a56732e9e2ddd39a19f5f50426b39854
Author: Karolin Seeger <[email protected]>
Date:   Sun Dec 7 21:47:37 2014 +0100

    VERSION: Bump version up to 4.0.24...
    
    ...and re-enable git snapshots.
    
    Signed-off-by: Karolin Seeger <[email protected]>
    (cherry picked from commit b1adeee4aed02f3066f9a96f20daeb3dce43b68c)

-----------------------------------------------------------------------

Summary of changes:
 VERSION                                 |   2 +-
 WHATSNEW.txt                            |  53 ++++++++-
 librpc/idl/security.idl                 |  13 ++-
 source4/auth/session.c                  |   5 +
 source4/dsdb/common/util.c              |   4 +-
 source4/dsdb/pydsdb.c                   |   1 +
 source4/dsdb/samdb/ldb_modules/samldb.c | 192 +++++++++++++++++++++++++++++++-
 source4/dsdb/samdb/samdb.h              |   6 +
 source4/rpc_server/lsa/dcesrv_lsa.c     |  15 ++-
 source4/setup/schema_samba4.ldif        |   1 +
 10 files changed, 281 insertions(+), 11 deletions(-)


Changeset truncated at 500 lines:

diff --git a/VERSION b/VERSION
index e0e7f36..81f48a9 100644
--- a/VERSION
+++ b/VERSION
@@ -25,7 +25,7 @@
 ########################################################
 SAMBA_VERSION_MAJOR=4
 SAMBA_VERSION_MINOR=0
-SAMBA_VERSION_RELEASE=23
+SAMBA_VERSION_RELEASE=24
 
 ########################################################
 # If a official release has a serious bug              #
diff --git a/WHATSNEW.txt b/WHATSNEW.txt
index c74580b..777997f 100644
--- a/WHATSNEW.txt
+++ b/WHATSNEW.txt
@@ -1,4 +1,53 @@
                    ==============================
+                   Release Notes for Samba 4.0.24
+                          January 15, 2015
+                   ==============================
+
+
+This is a security release in order to address CVE-2014-8143 (Elevation
+of privilege to Active Directory Domain Controller).
+
+o  CVE-2014-8143:
+   Samba's AD DC allows the administrator to delegate
+   creation of user or computer accounts to specific users or groups.
+
+   However, all released versions of Samba's AD DC did not implement the
+   additional required check on the UF_SERVER_TRUST_ACCOUNT bit in the
+   userAccountControl attributes.
+
+
+Changes since 4.0.23:
+---------------------
+
+o   Andrew Bartlett <[email protected]>
+    * BUG 10993: CVE-2014-8143: dsdb-samldb: Check for extended access
+      rights before we allow changes to userAccountControl.
+
+
+######################################################################
+Reporting bugs & Development Discussion
+#######################################
+
+Please discuss this release on the samba-technical mailing list or by
+joining the #samba-technical IRC channel on irc.freenode.net.
+
+If you do report problems then please try to send high quality
+feedback. If you don't provide vital information to help us track down
+the problem then you will probably be ignored.  All bug reports should
+be filed under the Samba 4.0 product in the project's Bugzilla
+database (https://bugzilla.samba.org/).
+
+
+======================================================================
+== Our Code, Our Bugs, Our Responsibility.
+== The Samba Team
+======================================================================
+
+
+Release notes for older releases follow:
+----------------------------------------
+
+                   ==============================
                    Release Notes for Samba 4.0.23
                          December 08, 2014
                    ==============================
@@ -89,8 +138,8 @@ database (https://bugzilla.samba.org/).
 ======================================================================
 
 
-Release notes for older releases follow:
-----------------------------------------
+----------------------------------------------------------------------
+
 
                    ==============================
                    Release Notes for Samba 4.0.22
diff --git a/librpc/idl/security.idl b/librpc/idl/security.idl
index d886b51..2600d49 100644
--- a/librpc/idl/security.idl
+++ b/librpc/idl/security.idl
@@ -653,14 +653,21 @@ interface security
        const string GUID_DRS_CHANGE_RID_MASTER       = 
"d58d5f36-0a98-11d1-adbb-00c04fd8d5cd";
        const string GUID_DRS_CHANGE_SCHEMA_MASTER    = 
"e12b56b6-0a95-11d1-adbb-00c04fd8d5cd";
        const string GUID_DRS_GET_CHANGES             = 
"1131f6aa-9c07-11d1-f79f-00c04fc2dcd2";
+       const string GUID_DRS_REPL_SYNCRONIZE         = 
"1131f6ab-9c07-11d1-f79f-00c04fc2dcd2";
+       const string GUID_DRS_MANAGE_TOPOLOGY         = 
"1131f6ac-9c07-11d1-f79f-00c04fc2dcd2";
        const string GUID_DRS_GET_ALL_CHANGES         = 
"1131f6ad-9c07-11d1-f79f-00c04fc2dcd2";
+       const string GUID_DRS_RO_REPL_SECRET_SYNC     = 
"1131f6ae-9c07-11d1-f79f-00c04fc2dcd2";
        const string GUID_DRS_GET_FILTERED_ATTRIBUTES = 
"89e95b76-444d-4c62-991a-0facbeda640c";
-       const string GUID_DRS_MANAGE_TOPOLOGY         = 
"1131f6ac-9c07-11d1-f79f-00c04fc2dcd2";
        const string GUID_DRS_MONITOR_TOPOLOGY        = 
"f98340fb-7c5b-4cdb-a00b-2ebdfa115a96";
-       const string GUID_DRS_REPL_SYNCRONIZE         = 
"1131f6ab-9c07-11d1-f79f-00c04fc2dcd2";
-       const string GUID_DRS_RO_REPL_SECRET_SYNC     = 
"1131f6ae-9c07-11d1-f79f-00c04fc2dcd2";
        const string GUID_DRS_USER_CHANGE_PASSWORD    = 
"ab721a53-1e2f-11d0-9819-00aa0040529b";
        const string GUID_DRS_FORCE_CHANGE_PASSWORD   = 
"00299570-246d-11d0-a768-00aa006e0529";
+        const string GUID_DRS_UPDATE_PASSWORD_NOT_REQUIRED_BIT
+                                                     = 
"280f369c-67c7-438e-ae98-1d46f3c6f541";
+        const string GUID_DRS_UNEXPIRE_PASSWORD       = 
"ccc2dc7d-a6ad-4a7a-8846-c04e3cc53501";
+        const string GUID_DRS_ENABLE_PER_USER_REVERSIBLY_ENCRYPTED_PASSWORD
+                                                     = 
"05c74c5e-4deb-43b4-bd9f-86664c2a7fd5";
+        const string GUID_DRS_DS_INSTALL_REPLICA      = 
"9923a32a-3607-11d2-b9be-0000f87a36b2";
+
 
        /***************************************************************/
        /* validated writes guids */
diff --git a/source4/auth/session.c b/source4/auth/session.c
index bb0b5bc..8c85302 100644
--- a/source4/auth/session.c
+++ b/source4/auth/session.c
@@ -204,6 +204,11 @@ struct auth_session_info 
*auth_session_info_from_transport(TALLOC_CTX *mem_ctx,
 {
        struct auth_session_info *session_info;
        session_info = talloc_steal(mem_ctx, 
session_info_transport->session_info);
+       /*
+        * This is to allow us to check the type of this pointer using
+        * talloc_get_type()
+        */
+       talloc_set_name(session_info, "struct auth_session_info");
 #ifdef HAVE_GSS_IMPORT_CRED
        if (session_info_transport->exported_gssapi_credentials.length) {
                struct cli_credentials *creds;
diff --git a/source4/dsdb/common/util.c b/source4/dsdb/common/util.c
index 8e40776..1f685dc 100644
--- a/source4/dsdb/common/util.c
+++ b/source4/dsdb/common/util.c
@@ -1012,8 +1012,8 @@ int samdb_msg_set_uint(struct ldb_context *sam_ldb, 
TALLOC_CTX *mem_ctx,
 /*
  * Handle ldb_request in transaction
  */
-static int dsdb_autotransaction_request(struct ldb_context *sam_ldb,
-                                       struct ldb_request *req)
+int dsdb_autotransaction_request(struct ldb_context *sam_ldb,
+                                struct ldb_request *req)
 {
        int ret;
 
diff --git a/source4/dsdb/pydsdb.c b/source4/dsdb/pydsdb.c
index 99e239e..9f8db69 100644
--- a/source4/dsdb/pydsdb.c
+++ b/source4/dsdb/pydsdb.c
@@ -1142,6 +1142,7 @@ void initdsdb(void)
        ADD_DSDB_FLAG(UF_TRUSTED_TO_AUTHENTICATE_FOR_DELEGATION);
        ADD_DSDB_FLAG(UF_NO_AUTH_DATA_REQUIRED);
        ADD_DSDB_FLAG(UF_PARTIAL_SECRETS_ACCOUNT);
+       ADD_DSDB_FLAG(UF_USE_AES_KEYS);
 
        /* groupType flags */
        ADD_DSDB_FLAG(GTYPE_SECURITY_BUILTIN_LOCAL_GROUP);
diff --git a/source4/dsdb/samdb/ldb_modules/samldb.c 
b/source4/dsdb/samdb/ldb_modules/samldb.c
index da9c966..20f5932 100644
--- a/source4/dsdb/samdb/ldb_modules/samldb.c
+++ b/source4/dsdb/samdb/ldb_modules/samldb.c
@@ -33,6 +33,7 @@
 #include "includes.h"
 #include "libcli/ldap/ldap_ndr.h"
 #include "ldb_module.h"
+#include "auth/auth.h"
 #include "dsdb/samdb/samdb.h"
 #include "dsdb/samdb/ldb_modules/util.h"
 #include "dsdb/samdb/ldb_modules/ridalloc.h"
@@ -943,6 +944,10 @@ static int samldb_schema_info_update(struct samldb_ctx *ac)
 }
 
 static int samldb_prim_group_tester(struct samldb_ctx *ac, uint32_t rid);
+static int samldb_check_user_account_control_acl(struct samldb_ctx *ac,
+                                                struct dom_sid *sid,
+                                                uint32_t user_account_control,
+                                                uint32_t 
user_account_control_old);
 
 /*
  * "Objectclass" trigger (MS-SAMR 3.1.1.8.1)
@@ -1037,7 +1042,6 @@ static int samldb_objectclass_trigger(struct samldb_ctx 
*ac)
                el = ldb_msg_find_element(ac->msg, "userAccountControl");
                if (el != NULL) {
                        uint32_t user_account_control, account_type;
-
                        /* Step 1.3: "userAccountControl" -> "sAMAccountType" 
mapping */
                        user_account_control = 
ldb_msg_find_attr_as_uint(ac->msg,
                                                                         
"userAccountControl",
@@ -1134,6 +1138,12 @@ static int samldb_objectclass_trigger(struct samldb_ctx 
*ac)
                                        return ret;
                                }
                        }
+
+                       ret = samldb_check_user_account_control_acl(ac, NULL,
+                                                                   
user_account_control, 0);
+                       if (ret != LDB_SUCCESS) {
+                               return ret;
+                       }
                }
                break;
        }
@@ -1421,6 +1431,172 @@ static int samldb_prim_group_trigger(struct samldb_ctx 
*ac)
        return ret;
 }
 
+/**
+ * Validate that the restriction in point 5 of MS-SAMR 3.1.1.8.10 
userAccountControl is honoured
+ *
+ */
+static int samldb_check_user_account_control_acl(struct samldb_ctx *ac,
+                                                struct dom_sid *sid,
+                                                uint32_t user_account_control,
+                                                uint32_t 
user_account_control_old)
+{
+       int i, ret = 0;
+       bool need_acl_check = false;
+       struct ldb_result *res;
+       const char * const sd_attrs[] = {"ntSecurityDescriptor", NULL};
+       struct security_token *user_token;
+       struct security_descriptor *domain_sd;
+       struct ldb_dn *domain_dn = 
ldb_get_default_basedn(ldb_module_get_ctx(ac->module));
+       const struct uac_to_guid {
+               uint32_t uac;
+               const char *oid;
+               const char *guid;
+               enum sec_privilege privilege;
+               bool delete_is_privileged;
+               const char *error_string;
+       } map[] = {
+               {
+                       .uac = UF_PASSWD_NOTREQD,
+                       .guid = GUID_DRS_UPDATE_PASSWORD_NOT_REQUIRED_BIT,
+                       .error_string = "Adding the UF_PASSWD_NOTREQD bit in 
userAccountControl requires the Update-Password-Not-Required-Bit right that was 
not given on the Domain object"
+               },
+               {
+                       .uac = UF_DONT_EXPIRE_PASSWD,
+                       .guid = GUID_DRS_UNEXPIRE_PASSWORD,
+                       .error_string = "Adding the UF_DONT_EXPIRE_PASSWD bit 
in userAccountControl requires the Unexpire-Password right that was not given 
on the Domain object"
+               },
+               {
+                       .uac = UF_ENCRYPTED_TEXT_PASSWORD_ALLOWED,
+                       .guid = 
GUID_DRS_ENABLE_PER_USER_REVERSIBLY_ENCRYPTED_PASSWORD,
+                       .error_string = "Adding the 
UF_ENCRYPTED_TEXT_PASSWORD_ALLOWED bit in userAccountControl requires the 
Enable-Per-User-Reversibly-Encrypted-Password right that was not given on the 
Domain object"
+               },
+               {
+                       .uac = UF_SERVER_TRUST_ACCOUNT,
+                       .guid = GUID_DRS_DS_INSTALL_REPLICA,
+                       .error_string = "Adding the UF_SERVER_TRUST_ACCOUNT bit 
in userAccountControl requires the DS-Install-Replica right that was not given 
on the Domain object"
+               },
+               {
+                       .uac = UF_PARTIAL_SECRETS_ACCOUNT,
+                       .guid = GUID_DRS_DS_INSTALL_REPLICA,
+                       .error_string = "Adding the UF_PARTIAL_SECRETS_ACCOUNT 
bit in userAccountControl requires the DS-Install-Replica right that was not 
given on the Domain object"
+               },
+               {
+                       .uac = UF_INTERDOMAIN_TRUST_ACCOUNT,
+                       .oid = DSDB_CONTROL_PERMIT_INTERDOMAIN_TRUST_UAC_OID,
+                       .error_string = "Updating the 
UF_INTERDOMAIN_TRUST_ACCOUNT bit in userAccountControl is not permitted over 
LDAP.  This bit is restricted to the LSA CreateTrustedDomain interface",
+                       .delete_is_privileged = true
+               },
+               {
+                       .uac = UF_TRUSTED_FOR_DELEGATION,
+                       .privilege = SEC_PRIV_ENABLE_DELEGATION,
+                       .delete_is_privileged = true,
+                       .error_string = "Updating the UF_TRUSTED_FOR_DELEGATION 
bit in userAccountControl is not permitted without the 
SeEnableDelegationPrivilege"
+               },
+               {
+                       .uac = UF_TRUSTED_TO_AUTHENTICATE_FOR_DELEGATION,
+                       .privilege = SEC_PRIV_ENABLE_DELEGATION,
+                       .delete_is_privileged = true,
+                       .error_string = "Updating the 
UF_TRUSTED_TO_AUTHENTICATE_FOR_DELEGATION bit in userAccountControl is not 
permitted without the SeEnableDelegationPrivilege"
+               }
+
+       };
+
+       if (dsdb_module_am_system(ac->module)) {
+               return LDB_SUCCESS;
+       }
+
+       for (i = 0; i < ARRAY_SIZE(map); i++) {
+               if (user_account_control & map[i].uac) {
+                       need_acl_check = true;
+                       break;
+               }
+       }
+       if (need_acl_check == false) {
+               return LDB_SUCCESS;
+       }
+
+       user_token = acl_user_token(ac->module);
+       if (user_token == NULL) {
+               return LDB_ERR_INSUFFICIENT_ACCESS_RIGHTS;
+       }
+
+       ret = dsdb_module_search_dn(ac->module, ac, &res,
+                                   domain_dn,
+                                   sd_attrs,
+                                   DSDB_FLAG_NEXT_MODULE | 
DSDB_SEARCH_SHOW_DELETED,
+                                   ac->req);
+       if (ret != LDB_SUCCESS) {
+               return ret;
+       }
+       if (res->count != 1) {
+               return ldb_module_operr(ac->module);
+       }
+
+       ret = dsdb_get_sd_from_ldb_message(ldb_module_get_ctx(ac->module),
+                                          ac, res->msgs[0], &domain_sd);
+
+       if (ret != LDB_SUCCESS) {
+               return ret;
+       }
+
+       for (i = 0; i < ARRAY_SIZE(map); i++) {
+               uint32_t this_uac_new = user_account_control & map[i].uac;
+               uint32_t this_uac_old = user_account_control_old & map[i].uac;
+               if (this_uac_new != this_uac_old) {
+                       if (this_uac_old != 0) {
+                               if (map[i].delete_is_privileged == false) {
+                                       continue;
+                               }
+                       }
+                       if (map[i].oid) {
+                               struct ldb_control *control = 
ldb_request_get_control(ac->req, map[i].oid);
+                               if (control == NULL) {
+                                       ret = 
LDB_ERR_INSUFFICIENT_ACCESS_RIGHTS;
+                               }
+                       } else if (map[i].privilege != SEC_PRIV_INVALID) {
+                               bool have_priv = 
security_token_has_privilege(user_token,
+                                                                             
map[i].privilege);
+                               if (have_priv == false) {
+                                       ret = 
LDB_ERR_INSUFFICIENT_ACCESS_RIGHTS;
+                               }
+                       } else {
+                               ret = acl_check_extended_right(ac, domain_sd,
+                                                              user_token,
+                                                              map[i].guid,
+                                                              
SEC_ADS_CONTROL_ACCESS,
+                                                              sid);
+                       }
+                       if (ret != LDB_SUCCESS) {
+                               break;
+                       }
+               }
+       }
+       if (ret == LDB_ERR_INSUFFICIENT_ACCESS_RIGHTS) {
+               switch (ac->req->operation) {
+               case LDB_ADD:
+                       ldb_asprintf_errstring(ldb_module_get_ctx(ac->module),
+                                              "Failed to add %s: %s",
+                                              
ldb_dn_get_linearized(ac->msg->dn),
+                                              map[i].error_string);
+                       break;
+               case LDB_MODIFY:
+                       ldb_asprintf_errstring(ldb_module_get_ctx(ac->module),
+                                              "Failed to modify %s: %s",
+                                              
ldb_dn_get_linearized(ac->msg->dn),
+                                              map[i].error_string);
+                       break;
+               default:
+                       return ldb_module_operr(ac->module);
+               }
+               if (map[i].guid) {
+                       dsdb_acl_debug(domain_sd, acl_user_token(ac->module),
+                                      domain_dn,
+                                      true,
+                                      10);
+               }
+       }
+       return ret;
+}
 
 /**
  * This function is called on LDB modify operations. It performs some 
additions/
@@ -1432,9 +1608,10 @@ static int samldb_user_account_control_change(struct 
samldb_ctx *ac)
        uint32_t user_account_control, old_user_account_control, account_type;
        struct ldb_message_element *el;
        struct ldb_message *tmp_msg;
+       struct dom_sid *sid;
        int ret;
        struct ldb_result *res;
-       const char * const attrs[] = { "userAccountControl", "objectClass", 
NULL };
+       const char * const attrs[] = { "userAccountControl", "objectClass", 
"objectSid", NULL };
        unsigned int i;
        bool is_computer = false;
 
@@ -1563,6 +1740,17 @@ static int samldb_user_account_control_change(struct 
samldb_ctx *ac)
                el->flags = LDB_FLAG_MOD_REPLACE;
        }
 
+       sid = samdb_result_dom_sid(res, res->msgs[0], "objectSid");
+       if (sid == NULL) {
+               return ldb_module_operr(ac->module);
+       }
+
+       ret = samldb_check_user_account_control_acl(ac, sid, 
user_account_control,
+                                                   old_user_account_control);
+       if (ret != LDB_SUCCESS) {
+               return ret;
+       }
+
        return LDB_SUCCESS;
 }
 
diff --git a/source4/dsdb/samdb/samdb.h b/source4/dsdb/samdb/samdb.h
index 7605c65..c848d87 100644
--- a/source4/dsdb/samdb/samdb.h
+++ b/source4/dsdb/samdb/samdb.h
@@ -135,6 +135,12 @@ struct dsdb_control_password_change {
  */
 #define DSDB_CONTROL_SEC_DESC_PROPAGATION_OID "1.3.6.1.4.1.7165.4.3.21"
 
+/*
+ * passed when creating a interdomain trust account through LSA
+ * to relax constraints in the samldb ldb module.
+ */
+#define DSDB_CONTROL_PERMIT_INTERDOMAIN_TRUST_UAC_OID "1.3.6.1.4.1.7165.4.3.23"
+
 #define DSDB_EXTENDED_REPLICATED_OBJECTS_OID "1.3.6.1.4.1.7165.4.4.1"
 struct dsdb_extended_replicated_object {
        struct ldb_message *msg;
diff --git a/source4/rpc_server/lsa/dcesrv_lsa.c 
b/source4/rpc_server/lsa/dcesrv_lsa.c
index bdd0777..648e31d 100644
--- a/source4/rpc_server/lsa/dcesrv_lsa.c
+++ b/source4/rpc_server/lsa/dcesrv_lsa.c
@@ -791,6 +791,7 @@ static NTSTATUS add_trust_user(TALLOC_CTX *mem_ctx,
                               struct trustAuthInOutBlob *in,
                               struct ldb_dn **user_dn)
 {
+       struct ldb_request *req;
        struct ldb_message *msg;
        struct ldb_dn *dn;
        uint32_t i;
@@ -851,7 +852,19 @@ static NTSTATUS add_trust_user(TALLOC_CTX *mem_ctx,
        }
 
        /* create the trusted_domain user account */
-       ret = ldb_add(sam_ldb, msg);
+       ret = ldb_build_add_req(&req, sam_ldb, mem_ctx, msg, NULL, NULL,
+                               ldb_op_default_callback, NULL);
+       if (ret != LDB_SUCCESS) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       ret = ldb_request_add_control(req, 
DSDB_CONTROL_PERMIT_INTERDOMAIN_TRUST_UAC_OID,
+                                     false, NULL);
+       if (ret != LDB_SUCCESS) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       ret = dsdb_autotransaction_request(sam_ldb, req);
        if (ret != LDB_SUCCESS) {
                DEBUG(0,("Failed to create user record %s: %s\n",
                         ldb_dn_get_linearized(msg->dn),
diff --git a/source4/setup/schema_samba4.ldif b/source4/setup/schema_samba4.ldif
index 94aedb0..22f0bc1 100644
--- a/source4/setup/schema_samba4.ldif
+++ b/source4/setup/schema_samba4.ldif
@@ -197,6 +197,7 @@
 #Allocated: DSDB_CONTROL_DBCHECK_MODIFY_RO_REPLICA 1.3.6.1.4.1.7165.4.3.19.1
 #Allocated: DSDB_CONTROL_PASSWORD_BYPASS_LAST_SET_OID 1.3.6.1.4.1.7165.4.3.20
 #Allocated: DSDB_CONTROL_SEC_DESC_PROPAGATION_OID 1.3.6.1.4.1.7165.4.3.21
+#Allocated: DSDB_CONTROL_PERMIT_INTERDOMAIN_TRUST_UAC_OID 
1.3.6.1.4.1.7165.4.3.23
 
 # Extended 1.3.6.1.4.1.7165.4.4.x
 #Allocated: DSDB_EXTENDED_REPLICATED_OBJECTS_OID 1.3.6.1.4.1.7165.4.4.1


-- 
Samba Shared Repository

Reply via email to