The branch, master has been updated
       via  31d3227 s4:lib/messaging: make irpc_request private and remove 
unused code
       via  dd71542 s4:libcli/composite: remove unused composite_continue_irpc()
       via  a9078aa s4:lib/messaging/tests: make use of irpc_binding_handle()
       via  a65d1eb s4:lib/messaging: use irpc_binding_handle() for python 
bindings
       via  993d37b s4:wrepl_server: use irpc_binding_handle_by_name() in 
wreplsrv_scavenging_replica_active_records()
       via  f43ec0e s4:wrepl_server: use irpc_binding_handle_by_name() in 
r_do_release_demand()
       via  2e07f82 s4:wrepl_server: use irpc_binding_handle_by_name() in 
r_do_challenge()
       via  8202cf7 s4:auth_winbind: use irpc_binding_handle_by_name()
       via  52d4a97 s4:libcli/wbclient: use irpc_binding_handle_by_name()
       via  5c73c84 s4:rpc_server/netlogon: use irpc_binding_handle_by_name()
       via  e117177 s4:smbd/service_task: use irpc_binding_handle_by_name()
       via  ff0362f s4:dsdb/kcc: use irpc_binding_handle_by_name()
       via  3c41505 s4:libcli/finddcs: use irpc_binding_handle_by_name()
       via  f2422a0 s4:rpc_server/common: use irpc_binding_handle_by_name() in 
dcesrv_irpc_forward_rpc_call()
       via  611357a s4:lib/messaging: add irpc_binding_handle_by_name() helper 
function
       via  e26f0ab s4:lib/messaging: add irpc dcerpc_binding_handle backend
       via  328c708 s4:lib/messaging: move messaging prototypes from irpc.h to 
messaging.h
       via  8740cae s4:lib/messaging/tests: fix some compiler warnings
       via  705f4c2 s4:auth_winbind: remove unused winbind_samba3 backend
       via  0f35d51 s4:auth_winbind: fix segfault in 
winbind_check_password_wbclient()
       via  5b0e0ac s4:auth_winbind: fix compiler warnings
       via  14ffb7d nsswitch/libwbclient: LIBWBCLIENT depends on 
LIBWINBIND-CLIENT
      from  fdffa90 s4-drs: A quick fix for DRSUAPI_EXOP_FSMO_RID_ALLOC 
extended_op handling

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 31d3227074a63ab7f66f74eca019884bb2e236da
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 16:05:38 2010 +0200

    s4:lib/messaging: make irpc_request private and remove unused code
    
    metze

commit dd7154234c267254cea199638062c5be1cd8b253
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 15:59:44 2010 +0200

    s4:libcli/composite: remove unused composite_continue_irpc()
    
    metze

commit a9078aad3c970e325e71f72f9f2c09e7136082cd
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 15:42:56 2010 +0200

    s4:lib/messaging/tests: make use of irpc_binding_handle()
    
    metze

commit a65d1eb05ce2eb0369e7489205c57191cd09c989
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 15:21:16 2010 +0200

    s4:lib/messaging: use irpc_binding_handle() for python bindings
    
    metze

commit 993d37baf589da85f5285f4a0edc45da692cc192
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 13:18:14 2010 +0200

    s4:wrepl_server: use irpc_binding_handle_by_name() in 
wreplsrv_scavenging_replica_active_records()
    
    metze

commit f43ec0e7516a1039172ce3e7f26ad47eb0271310
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 13:05:32 2010 +0200

    s4:wrepl_server: use irpc_binding_handle_by_name() in r_do_release_demand()
    
    metze

commit 2e07f82f7cb905d5892d5dec7966ff17725bbc79
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 12:58:10 2010 +0200

    s4:wrepl_server: use irpc_binding_handle_by_name() in r_do_challenge()
    
    metze

commit 8202cf7966b8125ac175ceb8cef18037c9a8cc31
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 16:59:05 2010 +0200

    s4:auth_winbind: use irpc_binding_handle_by_name()
    
    metze

commit 52d4a97afc4b09f860e84f059c419d33bfec18ff
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 12:33:17 2010 +0200

    s4:libcli/wbclient: use irpc_binding_handle_by_name()
    
    metze

commit 5c73c84f291573e835e9286c8b3a0831bc3c7dad
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 12:09:36 2010 +0200

    s4:rpc_server/netlogon: use irpc_binding_handle_by_name()
    
    metze

commit e117177307fe42e0e6ef92ee66df1154ffe3b9ec
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 11:39:15 2010 +0200

    s4:smbd/service_task: use irpc_binding_handle_by_name()
    
    metze

commit ff0362fc3567ec2dab902c685f7715a6070a63e3
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 11:34:17 2010 +0200

    s4:dsdb/kcc: use irpc_binding_handle_by_name()
    
    metze

commit 3c4150522cc85fd5774ca3346ff11f9d481bca7a
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 11:13:01 2010 +0200

    s4:libcli/finddcs: use irpc_binding_handle_by_name()
    
    metze

commit f2422a0faab4443649633d6b378a4357eff194f2
Author: Stefan Metzmacher <[email protected]>
Date:   Mon Aug 30 13:45:48 2010 +0200

    s4:rpc_server/common: use irpc_binding_handle_by_name() in 
dcesrv_irpc_forward_rpc_call()
    
    metze

commit 611357aee31ad58632a2b029a91d950a2cffb341
Author: Stefan Metzmacher <[email protected]>
Date:   Mon Aug 30 13:44:41 2010 +0200

    s4:lib/messaging: add irpc_binding_handle_by_name() helper function
    
    metze

commit e26f0abe9183ac46fd8b749f7696f404414b780e
Author: Stefan Metzmacher <[email protected]>
Date:   Mon Aug 30 13:44:01 2010 +0200

    s4:lib/messaging: add irpc dcerpc_binding_handle backend
    
    metze

commit 328c7084f2356dfad5c482fac4cd783b8027ab6b
Author: Stefan Metzmacher <[email protected]>
Date:   Tue Aug 31 13:25:59 2010 +0200

    s4:lib/messaging: move messaging prototypes from irpc.h to messaging.h
    
    metze

commit 8740caece0a236d7db4c8efa757766532b06691b
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 15:42:28 2010 +0200

    s4:lib/messaging/tests: fix some compiler warnings
    
    metze

commit 705f4c205685b0a167402ff54e37abd75b92a72d
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 10:39:18 2010 +0200

    s4:auth_winbind: remove unused winbind_samba3 backend
    
    This uses the winbind protocol directly, which needs to be avoided!
    
    metze

commit 0f35d51ab65b974d3362e0b6f7ebff7b41fb5835
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 16:15:27 2010 +0200

    s4:auth_winbind: fix segfault in winbind_check_password_wbclient()
    
    We should only look at err if WBC_ERR_AUTH_ERROR is returned.
    
    metze

commit 5b0e0acc819dedba3ef631e66be6c632fdf23e77
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 10:57:01 2010 +0200

    s4:auth_winbind: fix compiler warnings
    
    metze

commit 14ffb7d699df450a4fc5fc362cec7f804cae922b
Author: Stefan Metzmacher <[email protected]>
Date:   Fri Sep 3 13:27:01 2010 +0200

    nsswitch/libwbclient: LIBWBCLIENT depends on LIBWINBIND-CLIENT
    
    metze

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

Summary of changes:
 nsswitch/libwbclient/wscript_build            |    1 +
 source4/auth/ntlm/auth_winbind.c              |  156 ++--------
 source4/auth/ntlm/wscript_build               |    2 +-
 source4/dsdb/kcc/kcc_periodic.c               |   45 +++-
 source4/dsdb/wscript_build                    |    2 +-
 source4/lib/messaging/irpc.h                  |   73 +----
 source4/lib/messaging/messaging.c             |  447 +++++++++++++++++--------
 source4/lib/messaging/messaging.h             |   25 ++
 source4/lib/messaging/pymessaging.c           |   70 ++++-
 source4/lib/messaging/tests/irpc.c            |   75 +++--
 source4/lib/messaging/wscript_build           |    2 +-
 source4/libcli/composite/composite.c          |   11 -
 source4/libcli/composite/composite.h          |    5 -
 source4/libcli/finddcs.c                      |   40 ++-
 source4/libcli/wbclient/wbclient.c            |   86 ++---
 source4/libcli/wbclient/wbclient.h            |    5 +-
 source4/libcli/wbclient/wscript_build         |    2 +-
 source4/libcli/wscript_build                  |    2 +-
 source4/librpc/wscript_build                  |    5 +
 source4/rpc_server/common/forward.c           |   41 ++-
 source4/rpc_server/netlogon/dcerpc_netlogon.c |   17 +-
 source4/rpc_server/wscript_build              |    2 +-
 source4/smbd/service_task.c                   |   11 +-
 source4/smbd/wscript_build                    |    2 +-
 source4/wrepl_server/wrepl_apply_records.c    |  121 ++++----
 source4/wrepl_server/wrepl_scavenging.c       |   41 ++-
 source4/wrepl_server/wscript_build            |    2 +-
 27 files changed, 715 insertions(+), 576 deletions(-)


Changeset truncated at 500 lines:

diff --git a/nsswitch/libwbclient/wscript_build 
b/nsswitch/libwbclient/wscript_build
index ee730e4..c6e48c5 100644
--- a/nsswitch/libwbclient/wscript_build
+++ b/nsswitch/libwbclient/wscript_build
@@ -2,6 +2,7 @@
 
 bld.SAMBA_SUBSYSTEM('LIBWBCLIENT',
        source='wbc_guid.c wbc_idmap.c wbclient.c wbc_pam.c wbc_pwd.c wbc_sid.c 
wbc_util.c',
+       deps='LIBWINBIND-CLIENT'
        )
 
 
diff --git a/source4/auth/ntlm/auth_winbind.c b/source4/auth/ntlm/auth_winbind.c
index 51a1f4d..0dc52b9 100644
--- a/source4/auth/ntlm/auth_winbind.c
+++ b/source4/auth/ntlm/auth_winbind.c
@@ -25,43 +25,23 @@
 #include "auth/auth.h"
 #include "auth/ntlm/auth_proto.h"
 #include "auth/auth_sam_reply.h"
-#include "nsswitch/winbind_client.h"
-#include "librpc/gen_ndr/ndr_netlogon.h"
-#include "librpc/gen_ndr/ndr_winbind.h"
+#include "librpc/gen_ndr/ndr_winbind_c.h"
 #include "lib/messaging/irpc.h"
 #include "param/param.h"
 #include "nsswitch/libwbclient/wbclient.h"
 #include "libcli/security/dom_sid.h"
 
-static NTSTATUS get_info3_from_ndr(TALLOC_CTX *mem_ctx, struct 
winbindd_response *response, struct netr_SamInfo3 *info3)
-{
-       size_t len = response->length - sizeof(struct winbindd_response);
-       if (len > 4) {
-               enum ndr_err_code ndr_err;
-               DATA_BLOB blob;
-               blob.length = len - 4;
-               blob.data = (uint8_t *)(((char *)response->extra_data.data) + 
4);
-
-               ndr_err = ndr_pull_struct_blob(&blob, mem_ctx, 
-                              info3,
-                             (ndr_pull_flags_fn_t)ndr_pull_netr_SamInfo3);
-               if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
-                       return ndr_map_error2ntstatus(ndr_err);
-               }
-
-               return NT_STATUS_OK;
-       } else {
-               DEBUG(2, ("get_info3_from_ndr: No info3 struct found!\n"));
-               return NT_STATUS_UNSUCCESSFUL;
-       }
-}
-
 static NTSTATUS get_info3_from_wbcAuthUserInfo(TALLOC_CTX *mem_ctx,
                                               struct wbcAuthUserInfo *info,
                                               struct netr_SamInfo3 *info3)
 {
        int i, j;
        struct samr_RidWithAttribute *rids = NULL;
+       struct dom_sid *user_sid;
+       struct dom_sid *group_sid;
+
+       user_sid = (struct dom_sid *)(void *)&info->sids[0].sid;
+       group_sid = (struct dom_sid *)(void *)&info->sids[1].sid;
 
        info3->base.last_logon = info->logon_time;
        info3->base.last_logoff = info->logoff_time;
@@ -101,10 +81,10 @@ static NTSTATUS get_info3_from_wbcAuthUserInfo(TALLOC_CTX 
*mem_ctx,
                return NT_STATUS_INVALID_PARAMETER;
        }
 
-       dom_sid_split_rid(mem_ctx, (struct dom_sid2 *) &info->sids[0].sid,
+       dom_sid_split_rid(mem_ctx, user_sid,
                          &info3->base.domain_sid,
                          &info3->base.rid);
-       dom_sid_split_rid(mem_ctx, (struct dom_sid2 *) &info->sids[1].sid, NULL,
+       dom_sid_split_rid(mem_ctx, group_sid, NULL,
                          &info3->base.primary_gid);
 
        /* We already handled the first two, now take care of the rest */
@@ -115,9 +95,11 @@ static NTSTATUS get_info3_from_wbcAuthUserInfo(TALLOC_CTX 
*mem_ctx,
        NT_STATUS_HAVE_NO_MEMORY(rids);
 
        for (i = 2, j = 0; i < info->num_sids; ++i, ++j) {
+               struct dom_sid *tmp_sid;
+               tmp_sid = (struct dom_sid *)(void *)&info->sids[1].sid;
+
                rids[j].attributes = info->sids[i].attributes;
-               dom_sid_split_rid(mem_ctx,
-                                 (struct dom_sid2 *) &info->sids[i].sid,
+               dom_sid_split_rid(mem_ctx, tmp_sid,
                                  NULL, &rids[j].rid);
        }
        info3->base.groups.rids = rids;
@@ -138,89 +120,6 @@ static NTSTATUS winbind_want_check(struct 
auth_method_context *ctx,
        return NT_STATUS_OK;
 }
 
-/*
- Authenticate a user with a challenge/response
- using the samba3 winbind protocol
-*/
-static NTSTATUS winbind_check_password_samba3(struct auth_method_context *ctx,
-                                             TALLOC_CTX *mem_ctx,
-                                             const struct 
auth_usersupplied_info *user_info, 
-                                             struct auth_serversupplied_info 
**server_info)
-{
-       struct winbindd_request request;
-       struct winbindd_response response;
-        NSS_STATUS result;
-       NTSTATUS nt_status;
-       struct netr_SamInfo3 info3;             
-
-       /* Send off request */
-       const struct auth_usersupplied_info *user_info_temp;    
-       nt_status = encrypt_user_info(mem_ctx, ctx->auth_ctx, 
-                                     AUTH_PASSWORD_RESPONSE, 
-                                     user_info, &user_info_temp);
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               return nt_status;
-       }
-       user_info = user_info_temp;
-
-       ZERO_STRUCT(request);
-       ZERO_STRUCT(response);
-       request.flags = WBFLAG_PAM_INFO3_NDR;
-
-       request.data.auth_crap.logon_parameters = user_info->logon_parameters;
-
-       safe_strcpy(request.data.auth_crap.user,
-                      user_info->client.account_name, sizeof(fstring));
-       safe_strcpy(request.data.auth_crap.domain,
-                      user_info->client.domain_name, sizeof(fstring));
-       safe_strcpy(request.data.auth_crap.workstation,
-                      user_info->workstation_name, sizeof(fstring));
-
-       memcpy(request.data.auth_crap.chal, ctx->auth_ctx->challenge.data.data, 
sizeof(request.data.auth_crap.chal));
-
-       request.data.auth_crap.lm_resp_len = 
MIN(user_info->password.response.lanman.length,
-                                                
sizeof(request.data.auth_crap.lm_resp));
-       request.data.auth_crap.nt_resp_len = 
MIN(user_info->password.response.nt.length, 
-                                                
sizeof(request.data.auth_crap.nt_resp));
-
-       memcpy(request.data.auth_crap.lm_resp, 
user_info->password.response.lanman.data,
-              request.data.auth_crap.lm_resp_len);
-       memcpy(request.data.auth_crap.nt_resp, 
user_info->password.response.nt.data,
-              request.data.auth_crap.nt_resp_len);
-
-       result = winbindd_request_response(WINBINDD_PAM_AUTH_CRAP, &request, 
&response);
-
-       nt_status = NT_STATUS(response.data.auth.nt_status);
-       NT_STATUS_NOT_OK_RETURN(nt_status);
-
-       if (result == NSS_STATUS_SUCCESS && response.extra_data.data) {
-               union netr_Validation validation;
-
-               nt_status = get_info3_from_ndr(mem_ctx, &response, &info3);
-               SAFE_FREE(response.extra_data.data);
-               NT_STATUS_NOT_OK_RETURN(nt_status); 
-
-               validation.sam3 = &info3;
-               nt_status = make_server_info_netlogon_validation(mem_ctx, 
-                                                                
user_info->client.account_name, 
-                                                                3, &validation,
-                                                                server_info);
-               return nt_status;
-       } else if (result == NSS_STATUS_SUCCESS && !response.extra_data.data) {
-               DEBUG(0, ("Winbindd authenticated the user [%s]\\[%s], "
-                         "but did not include the required info3 reply!\n", 
-                         user_info->client.domain_name, 
user_info->client.account_name));
-               return NT_STATUS_INSUFFICIENT_LOGON_INFO;
-       } else if (NT_STATUS_IS_OK(nt_status)) {
-               DEBUG(1, ("Winbindd authentication for [%s]\\[%s] failed, "
-                         "but no error code is available!\n", 
-                         user_info->client.domain_name, 
user_info->client.account_name));
-               return NT_STATUS_NO_LOGON_SERVERS;
-       }
-
-        return nt_status;
-}
-
 struct winbind_check_password_state {
        struct winbind_SamLogon req;
 };
@@ -235,7 +134,7 @@ static NTSTATUS winbind_check_password(struct 
auth_method_context *ctx,
                                       struct auth_serversupplied_info 
**server_info)
 {
        NTSTATUS status;
-       struct server_id *winbind_servers;
+       struct dcerpc_binding_handle *irpc_handle;
        struct winbind_check_password_state *s;
        const struct auth_usersupplied_info *user_info_new;
        struct netr_IdentityInfo *identity_info;
@@ -248,8 +147,10 @@ static NTSTATUS winbind_check_password(struct 
auth_method_context *ctx,
        s = talloc(mem_ctx, struct winbind_check_password_state);
        NT_STATUS_HAVE_NO_MEMORY(s);
 
-       winbind_servers = irpc_servers_byname(ctx->auth_ctx->msg_ctx, s, 
"winbind_server");
-       if ((winbind_servers == NULL) || (winbind_servers[0].id == 0)) {
+       irpc_handle = irpc_binding_handle_by_name(s, ctx->auth_ctx->msg_ctx,
+                                                 "winbind_server",
+                                                 &ndr_table_winbind);
+       if (irpc_handle == NULL) {
                DEBUG(0, ("Winbind authentication for [%s]\\[%s] failed, " 
                          "no winbind_server running!\n",
                          user_info->client.domain_name, 
user_info->client.account_name));
@@ -310,9 +211,7 @@ static NTSTATUS winbind_check_password(struct 
auth_method_context *ctx,
 
        s->req.in.validation_level      = 3;
 
-       status = IRPC_CALL(ctx->auth_ctx->msg_ctx, winbind_servers[0],
-                          winbind, WINBIND_SAMLOGON,
-                          &s->req, s);
+       status = dcerpc_winbind_SamLogon_r(irpc_handle, s, &s->req);
        NT_STATUS_NOT_OK_RETURN(status);
 
        status = make_server_info_netlogon_validation(mem_ctx,
@@ -385,13 +284,17 @@ static NTSTATUS winbind_check_password_wbclient(struct 
auth_method_context *ctx,
                user_info->password.response.nt.data;
 
        wbc_status = wbcAuthenticateUserEx(&params, &info, &err);
-       if (!WBC_ERROR_IS_OK(wbc_status)) {
+       if (wbc_status == WBC_ERR_AUTH_ERROR) {
                DEBUG(1, ("error was %s (0x%08x)\nerror message was '%s'\n",
                      err->nt_string, err->nt_status, err->display_string));
 
                nt_status = NT_STATUS(err->nt_status);
                wbcFreeMemory(err);
                NT_STATUS_NOT_OK_RETURN(nt_status);
+       } else if (!WBC_ERROR_IS_OK(wbc_status)) {
+               DEBUG(1, ("wbcAuthenticateUserEx: failed with %u - %s\n",
+                       wbc_status, wbcErrorString(wbc_status)));
+               return NT_STATUS_LOGON_FAILURE;
        }
        nt_status = get_info3_from_wbcAuthUserInfo(mem_ctx, info, &info3);
        wbcFreeMemory(info);
@@ -405,13 +308,6 @@ static NTSTATUS winbind_check_password_wbclient(struct 
auth_method_context *ctx,
 
 }
 
-static const struct auth_operations winbind_samba3_ops = {
-       .name           = "winbind_samba3",
-       .get_challenge  = auth_get_challenge_not_implemented,
-       .want_check     = winbind_want_check,
-       .check_password = winbind_check_password_samba3
-};
-
 static const struct auth_operations winbind_ops = {
        .name           = "winbind",
        .get_challenge  = auth_get_challenge_not_implemented,
@@ -430,12 +326,6 @@ _PUBLIC_ NTSTATUS auth_winbind_init(void)
 {
        NTSTATUS ret;
 
-       ret = auth_register(&winbind_samba3_ops);
-       if (!NT_STATUS_IS_OK(ret)) {
-               DEBUG(0,("Failed to register 'winbind_samba3' auth 
backend!\n"));
-               return ret;
-       }
-
        ret = auth_register(&winbind_ops);
        if (!NT_STATUS_IS_OK(ret)) {
                DEBUG(0,("Failed to register 'winbind' auth backend!\n"));
diff --git a/source4/auth/ntlm/wscript_build b/source4/auth/ntlm/wscript_build
index 673b5fc..c3ce364 100644
--- a/source4/auth/ntlm/wscript_build
+++ b/source4/auth/ntlm/wscript_build
@@ -27,7 +27,7 @@ bld.SAMBA_MODULE('auth_winbind',
        source='auth_winbind.c',
        subsystem='auth',
        init_function='auth_winbind_init',
-       deps='NDR_WINBIND MESSAGING LIBWINBIND-CLIENT LIBWBCLIENT'
+       deps='RPC_NDR_WINBIND MESSAGING LIBWBCLIENT'
        )
 
 
diff --git a/source4/dsdb/kcc/kcc_periodic.c b/source4/dsdb/kcc/kcc_periodic.c
index 0955c99..28ff404 100644
--- a/source4/dsdb/kcc/kcc_periodic.c
+++ b/source4/dsdb/kcc/kcc_periodic.c
@@ -33,7 +33,7 @@
 #include "librpc/gen_ndr/ndr_misc.h"
 #include "librpc/gen_ndr/ndr_drsuapi.h"
 #include "librpc/gen_ndr/ndr_drsblobs.h"
-#include "librpc/gen_ndr/ndr_irpc.h"
+#include "librpc/gen_ndr/ndr_irpc_c.h"
 #include "param/param.h"
 
 /*
@@ -97,6 +97,11 @@ static bool check_MasterNC(struct kccsrv_partition *p, 
struct repsFromToBlob *r,
        return false;
 }
 
+struct kccsrv_notify_drepl_server_state {
+       struct dreplsrv_refresh r;
+};
+
+static void kccsrv_notify_drepl_server_done(struct tevent_req *subreq);
 
 /**
  * Force dreplsrv to update its state as topology is changed
@@ -104,16 +109,44 @@ static bool check_MasterNC(struct kccsrv_partition *p, 
struct repsFromToBlob *r,
 static void kccsrv_notify_drepl_server(struct kccsrv_service *s,
                                       TALLOC_CTX *mem_ctx)
 {
-       struct dreplsrv_refresh r;
-       struct server_id *drepl_srv;
+       struct kccsrv_notify_drepl_server_state *state;
+       struct dcerpc_binding_handle *irpc_handle;
+       struct tevent_req *subreq;
 
-       drepl_srv = irpc_servers_byname(s->task->msg_ctx, mem_ctx, "dreplsrv");
-       if (!drepl_srv) {
+       state = talloc_zero(mem_ctx, struct kccsrv_notify_drepl_server_state);
+       if (state == NULL) {
+               return;
+       }
+
+       irpc_handle = irpc_binding_handle_by_name(state, s->task->msg_ctx,
+                                                 "dreplsrv", &ndr_table_irpc);
+       if (irpc_handle == NULL) {
                /* dreplsrv is not running yet */
+               TALLOC_FREE(state);
+               return;
+       }
+
+       subreq = dcerpc_dreplsrv_refresh_r_send(state, s->task->event_ctx,
+                                               irpc_handle, &state->r);
+       if (subreq == NULL) {
+               TALLOC_FREE(state);
                return;
        }
+       tevent_req_set_callback(subreq, kccsrv_notify_drepl_server_done, state);
+}
+
+static void kccsrv_notify_drepl_server_done(struct tevent_req *subreq)
+{
+       struct kccsrv_notify_drepl_server_state *state =
+               tevent_req_callback_data(subreq,
+               struct kccsrv_notify_drepl_server_state);
+       NTSTATUS status;
+
+       status = dcerpc_dreplsrv_refresh_r_recv(subreq, state);
+       TALLOC_FREE(subreq);
 
-       IRPC_CALL(s->task->msg_ctx, drepl_srv[0], irpc, DREPLSRV_REFRESH, &r, 
mem_ctx);
+       /* we don't care about errors */
+       TALLOC_FREE(state);
 }
 
 /*
diff --git a/source4/dsdb/wscript_build b/source4/dsdb/wscript_build
index 3b3dae9..d3961f5 100644
--- a/source4/dsdb/wscript_build
+++ b/source4/dsdb/wscript_build
@@ -38,7 +38,7 @@ bld.SAMBA_MODULE('KCC_SRV',
        autoproto='kcc/kcc_service_proto.h',
        subsystem='service',
        init_function='server_service_kcc_init',
-       deps='SAMDB process_model RPC_NDR_DRSUAPI'
+       deps='SAMDB process_model RPC_NDR_IRPC RPC_NDR_DRSUAPI'
        )
 
 
diff --git a/source4/lib/messaging/irpc.h b/source4/lib/messaging/irpc.h
index 6607eb4..4b49795 100644
--- a/source4/lib/messaging/irpc.h
+++ b/source4/lib/messaging/irpc.h
@@ -54,78 +54,25 @@ typedef NTSTATUS (*irpc_function_t)(struct irpc_message *, 
void *r);
                           NDR_ ## funcname, \
                          (irpc_function_t)function, private_data)
 
-/* make a irpc call */
-#define IRPC_CALL(msg_ctx, server_id, pipename, funcname, ptr, ctx) \
-   irpc_call(msg_ctx, server_id, &ndr_table_ ## pipename, NDR_ ## funcname, 
ptr, ctx)
-
-#define IRPC_CALL_SEND(msg_ctx, server_id, pipename, funcname, ptr, ctx) \
-   irpc_call_send(msg_ctx, server_id, &ndr_table_ ## pipename, NDR_ ## 
funcname, ptr, ctx)
-
-
-/*
-  a pending irpc call
-*/
-struct irpc_request {
-       struct messaging_context *msg_ctx;
-       const struct ndr_interface_table *table;
-       int callnum;
-       int callid;
-       void *r;
-       NTSTATUS status;
-       bool done;
-       bool reject_free;
-       TALLOC_CTX *mem_ctx;
-       struct {
-               void (*fn)(struct irpc_request *);
-               void *private_data;
-       } async;
-};
-
-struct loadparm_context;
-
-typedef void (*msg_callback_t)(struct messaging_context *msg, void 
*private_data,
-                              uint32_t msg_type, 
-                              struct server_id server_id, DATA_BLOB *data);
-
-NTSTATUS messaging_send(struct messaging_context *msg, struct server_id 
server, 
-                       uint32_t msg_type, const DATA_BLOB *data);
-NTSTATUS messaging_register(struct messaging_context *msg, void *private_data,
-                           uint32_t msg_type, 
-                           msg_callback_t fn);
-NTSTATUS messaging_register_tmp(struct messaging_context *msg, void 
*private_data,
-                               msg_callback_t fn, uint32_t *msg_type);
-struct messaging_context *messaging_init(TALLOC_CTX *mem_ctx, 
-                                        const char *dir,
-                                        struct server_id server_id, 
-                                        struct tevent_context *ev);
-struct messaging_context *messaging_client_init(TALLOC_CTX *mem_ctx, 
-                                        const char *dir,
-                                        struct tevent_context *ev);
-NTSTATUS messaging_send_ptr(struct messaging_context *msg, struct server_id 
server, 
-                           uint32_t msg_type, void *ptr);
-void messaging_deregister(struct messaging_context *msg, uint32_t msg_type, 
void *private_data);
-
-
-
+struct ndr_interface_table;
 
 NTSTATUS irpc_register(struct messaging_context *msg_ctx, 
                       const struct ndr_interface_table *table, 
                       int call, irpc_function_t fn, void *private_data);
-struct irpc_request *irpc_call_send(struct messaging_context *msg_ctx, 
-                                   struct server_id server_id, 
-                                   const struct ndr_interface_table *table, 
-                                   int callnum, void *r, TALLOC_CTX *ctx);
-NTSTATUS irpc_call_recv(struct irpc_request *irpc);
-NTSTATUS irpc_call(struct messaging_context *msg_ctx, 
-                  struct server_id server_id, 
-                  const struct ndr_interface_table *table, 
-                  int callnum, void *r, TALLOC_CTX *ctx);
+
+struct dcerpc_binding_handle *irpc_binding_handle(TALLOC_CTX *mem_ctx,
+                                       struct messaging_context *msg_ctx,
+                                       struct server_id server_id,
+                                       const struct ndr_interface_table 
*table);
+struct dcerpc_binding_handle *irpc_binding_handle_by_name(TALLOC_CTX *mem_ctx,
+                                       struct messaging_context *msg_ctx,
+                                       const char *dest_task,
+                                       const struct ndr_interface_table 
*table);
 
 NTSTATUS irpc_add_name(struct messaging_context *msg_ctx, const char *name);
 struct server_id *irpc_servers_byname(struct messaging_context *msg_ctx, 
TALLOC_CTX *mem_ctx, const char *name);
 void irpc_remove_name(struct messaging_context *msg_ctx, const char *name);
 NTSTATUS irpc_send_reply(struct irpc_message *m, NTSTATUS status);
-struct server_id messaging_get_server_id(struct messaging_context *msg_ctx);
 
 #endif
 
diff --git a/source4/lib/messaging/messaging.c 
b/source4/lib/messaging/messaging.c
index 290210e..e50e50f 100644
--- a/source4/lib/messaging/messaging.c
+++ b/source4/lib/messaging/messaging.c
@@ -33,10 +33,23 @@
 #include "../tdb/include/tdb.h"
 #include "../lib/util/util_tdb.h"
 #include "cluster/cluster.h"
+#include "../lib/util/tevent_ntstatus.h"
 
 /* change the message version with any incompatible changes in the protocol */
 #define MESSAGING_VERSION 1
 
+/*
+  a pending irpc call
+*/
+struct irpc_request {
+       struct messaging_context *msg_ctx;
+       int callid;
+       struct {
+               void (*handler)(struct irpc_request *irpc, struct irpc_message 
*m);
+               void *private_data;
+       } incoming;
+};
+
 struct messaging_context {
        struct server_id server_id;
        struct socket_context *sock;
@@ -678,24 +691,11 @@ NTSTATUS irpc_register(struct messaging_context *msg_ctx,
 static void irpc_handler_reply(struct messaging_context *msg_ctx, struct 
irpc_message *m)
 {
        struct irpc_request *irpc;
-       enum ndr_err_code ndr_err;
 
        irpc = (struct irpc_request *)idr_find(msg_ctx->idr, m->header.callid);
        if (irpc == NULL) return;
 
-       /* parse the reply data */
-       ndr_err = irpc->table->calls[irpc->callnum].ndr_pull(m->ndr, NDR_OUT, 
irpc->r);
-       if (NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {
-               irpc->status = m->header.status;
-               talloc_steal(irpc->mem_ctx, m);
-       } else {
-               irpc->status = ndr_map_error2ntstatus(ndr_err);


-- 
Samba Shared Repository

Reply via email to