s3:winbind: Add a generic cache for NDR based parent-child requests

[sfrench/samba-autobuild/.git] / source3 / winbindd / winbindd_locator.c

diff --git a/source3/winbindd/winbindd_locator.c b/source3/winbindd/winbindd_locator.c
index b2a8bd7e3037ff2541a5f2ac62092328d7db05e1..5488394976dfeb6f95ef9b120a61c729023edab2 100644 (file)
--- a/source3/winbindd/winbindd_locator.c
+++ b/source3/winbindd/winbindd_locator.c
@@ -33,7 +33,7 @@ static struct winbindd_child static_locator_child;
 
 void init_locator_child(void)
 {
-       setup_child(&static_locator_child,
+       setup_child(NULL, &static_locator_child,
                    locator_dispatch_table,
                    "log.winbindd", "locator");
 }
@@ -45,11 +45,15 @@ struct winbindd_child *locator_child(void)
 
 void winbindd_dsgetdcname(struct winbindd_cli_state *state)
 {
-       state->request.domain_name
-               [sizeof(state->request.domain_name)-1] = '\0';
+       state->request->data.dsgetdcname.domain_name
+               [sizeof(state->request->data.dsgetdcname.domain_name)-1] = '\0';
+       state->request->data.dsgetdcname.site_name
+               [sizeof(state->request->data.dsgetdcname.site_name)-1] = '\0';
+       state->request->data.dsgetdcname.domain_guid
+               [sizeof(state->request->data.dsgetdcname.domain_guid)-1] = '\0';
 
        DEBUG(3, ("[%5lu]: dsgetdcname for %s\n", (unsigned long)state->pid,
-                 state->request.domain_name));
+                 state->request->data.dsgetdcname.domain_name));
 
        sendto_child(state, locator_child());
 }
@@ -94,53 +98,76 @@ static uint32_t get_dsgetdc_flags(uint32_t wbc_flags)
        return ds_flags;
 }
 
-
 static enum winbindd_result dual_dsgetdcname(struct winbindd_domain *domain,
                                             struct winbindd_cli_state *state)
 {
        NTSTATUS result;
        struct netr_DsRGetDCNameInfo *info = NULL;
-       const char *dc = NULL;
        uint32_t ds_flags = 0;
+       struct GUID guid, *guid_ptr = NULL;
+       const char *guid_str = NULL;
 
-       state->request.domain_name
-               [sizeof(state->request.domain_name)-1] = '\0';
+       state->request->data.dsgetdcname.domain_name
+               [sizeof(state->request->data.dsgetdcname.domain_name)-1] = '\0';
+       state->request->data.dsgetdcname.site_name
+               [sizeof(state->request->data.dsgetdcname.site_name)-1] = '\0';
+       state->request->data.dsgetdcname.domain_guid
+               [sizeof(state->request->data.dsgetdcname.domain_guid)-1] = '\0';
 
        DEBUG(3, ("[%5lu]: dsgetdcname for %s\n", (unsigned long)state->pid,
-                 state->request.domain_name));
-
-       ds_flags = get_dsgetdc_flags(state->request.flags);
+                 state->request->data.dsgetdcname.domain_name));
 
-       result = dsgetdcname(state->mem_ctx, winbind_messaging_context(),
-                            state->request.domain_name,
-                            NULL, NULL, ds_flags, &info);
+       ds_flags = get_dsgetdc_flags(state->request->flags);
 
-       if (!NT_STATUS_IS_OK(result)) {
-               return WINBINDD_ERROR;
+       result = GUID_from_string(state->request->data.dsgetdcname.domain_guid,
+                                 &guid);
+       if (NT_STATUS_IS_OK(result) && !GUID_all_zero(&guid)) {
+               guid_ptr = &guid;
        }
 
-       if (info->dc_address) {
-               dc = strip_hostname(info->dc_address);
-       }
+       result = dsgetdcname(state->mem_ctx,
+                            winbind_messaging_context(),
+                            state->request->data.dsgetdcname.domain_name,
+                            guid_ptr,
+                            state->request->data.dsgetdcname.site_name,
+                            ds_flags,
+                            &info);
 
-       if ((!dc || !is_ipaddress_v4(dc)) && info->dc_unc) {
-               dc = strip_hostname(info->dc_unc);
+       if (!NT_STATUS_IS_OK(result)) {
+               return WINBINDD_ERROR;
        }
 
-       if (!dc || !*dc) {
+       guid_str = GUID_string(state->mem_ctx, &info->domain_guid);
+       if (!guid_str) {
                return WINBINDD_ERROR;
        }
 
-       fstrcpy(state->response.data.dc_name, dc);
+       fstrcpy(state->response->data.dsgetdcname.dc_unc, info->dc_unc);
+       fstrcpy(state->response->data.dsgetdcname.dc_address, info->dc_address);
+       state->response->data.dsgetdcname.dc_address_type = info->dc_address_type;
+       fstrcpy(state->response->data.dsgetdcname.domain_guid, guid_str);
+       fstrcpy(state->response->data.dsgetdcname.domain_name, info->domain_name);
+       fstrcpy(state->response->data.dsgetdcname.forest_name, info->forest_name);
+       state->response->data.dsgetdcname.dc_flags = info->dc_flags;
+       fstrcpy(state->response->data.dsgetdcname.dc_site_name, info->dc_site_name);
+       fstrcpy(state->response->data.dsgetdcname.client_site_name, info->client_site_name);
 
        return WINBINDD_OK;
 }
 
 static const struct winbindd_child_dispatch_table locator_dispatch_table[] = {
        {
+               .name           = "PING",
+               .struct_cmd     = WINBINDD_PING,
+               .struct_fn      = winbindd_dual_ping,
+       },{
                .name           = "DSGETDCNAME",
                .struct_cmd     = WINBINDD_DSGETDCNAME,
                .struct_fn      = dual_dsgetdcname,
+       },{
+               .name           = "NDRCMD",
+               .struct_cmd     = WINBINDD_DUAL_NDRCMD,
+               .struct_fn      = winbindd_dual_ndrcmd,
        },{
                .name           = NULL,
        }