samba-cvs.cvs
[Top] [All Lists]

[SCM] Samba Shared Repository - branch master updated - bf5c80c8eb3051ff

Subject: [SCM] Samba Shared Repository - branch master updated - bf5c80c8eb3051ff2d04629b2b4a74b0c86d726d
From: Günther Deschner
Date: Thu, 9 Oct 2008 05:37:47 -0500 CDT
The branch, master has been updated
       via  bf5c80c8eb3051ff2d04629b2b4a74b0c86d726d (commit)
       via  4e04c5ae1bb7ade41c0b28bbf9ec7cdb4930d9ae (commit)
       via  76a8da6f711ce13de6c4e3e1af6714b0475b71f5 (commit)
      from  1d83fbffae23325961fd80873c93c06cae5f7a4e (commit)

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


- Log -----------------------------------------------------------------
commit bf5c80c8eb3051ff2d04629b2b4a74b0c86d726d
Author: Günther Deschner <gd@xxxxxxxxx>
Date:   Sat Sep 27 03:11:59 2008 +0200

    krb5-locator: fix dsgetdcname caller.
    
    Guenther

commit 4e04c5ae1bb7ade41c0b28bbf9ec7cdb4930d9ae
Author: Günther Deschner <gd@xxxxxxxxx>
Date:   Sat Sep 27 03:11:33 2008 +0200

    wbinfo: fix dsgetdcname caller.
    
    Guenther

commit 76a8da6f711ce13de6c4e3e1af6714b0475b71f5
Author: Günther Deschner <gd@xxxxxxxxx>
Date:   Sat Sep 27 03:09:07 2008 +0200

    s3-winbindd: use new, richer structures in WINBINDD_DSGETDCNAME 
implementation.
    
    Guenther

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

Summary of changes:
 source3/nsswitch/wbinfo.c               |   14 ++++++--
 source3/nsswitch/winbind_krb5_locator.c |   27 +++++++++++---
 source3/winbindd/winbindd_locator.c     |   61 ++++++++++++++++++++-----------
 3 files changed, 73 insertions(+), 29 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/nsswitch/wbinfo.c b/source3/nsswitch/wbinfo.c
index 60524d1..fc49be4 100644
--- a/source3/nsswitch/wbinfo.c
+++ b/source3/nsswitch/wbinfo.c
@@ -538,8 +538,8 @@ static bool wbinfo_dsgetdcname(const char *domain_name, 
uint32_t flags)
        ZERO_STRUCT(request);
        ZERO_STRUCT(response);
 
-       fstrcpy(request.domain_name, domain_name);
-       request.flags = flags;
+       fstrcpy(request.data.dsgetdcname.domain_name, domain_name);
+       request.data.dsgetdcname.flags = flags;
 
        request.flags |= DS_DIRECTORY_SERVICE_REQUIRED;
 
@@ -553,7 +553,15 @@ static bool wbinfo_dsgetdcname(const char *domain_name, 
uint32_t flags)
 
        /* Display response */
 
-       d_printf("%s\n", response.data.dc_name);
+       d_printf("%s\n", response.data.dsgetdcname.dc_unc);
+       d_printf("%s\n", response.data.dsgetdcname.dc_address);
+       d_printf("%d\n", response.data.dsgetdcname.dc_address_type);
+       d_printf("%s\n", response.data.dsgetdcname.domain_guid);
+       d_printf("%s\n", response.data.dsgetdcname.domain_name);
+       d_printf("%s\n", response.data.dsgetdcname.forest_name);
+       d_printf("0x%08x\n", response.data.dsgetdcname.dc_flags);
+       d_printf("%s\n", response.data.dsgetdcname.dc_site_name);
+       d_printf("%s\n", response.data.dsgetdcname.client_site_name);
 
        return true;
 }
diff --git a/source3/nsswitch/winbind_krb5_locator.c 
b/source3/nsswitch/winbind_krb5_locator.c
index 990c2ca..7eecd13 100644
--- a/source3/nsswitch/winbind_krb5_locator.c
+++ b/source3/nsswitch/winbind_krb5_locator.c
@@ -1,7 +1,7 @@
 /*
    Unix SMB/CIFS implementation.
    kerberos locator plugin
-   Copyright (C) Guenther Deschner 2007
+   Copyright (C) Guenther Deschner 2007-2008
 
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -247,18 +247,19 @@ static bool ask_winbind(const char *realm, char **dcname)
        NSS_STATUS status;
        struct winbindd_request request;
        struct winbindd_response response;
+       const char *dc = NULL;
 
        ZERO_STRUCT(request);
        ZERO_STRUCT(response);
 
-       request.flags = 0x40020600;
+       request.data.dsgetdcname.flags = 0x40020600;
                        /* DS_KDC_REQUIRED |
                        DS_IS_DNS_NAME |
                        DS_RETURN_DNS_NAME |
                        DS_IP_REQUIRED */
 
-       strncpy(request.domain_name, realm,
-               sizeof(request.domain_name)-1);
+       strncpy(request.data.dsgetdcname.domain_name, realm,
+               sizeof(request.data.dsgetdcname.domain_name)-1);
 
        status = winbindd_request_response(WINBINDD_DSGETDCNAME,
                                           &request, &response);
@@ -270,7 +271,23 @@ static bool ask_winbind(const char *realm, char **dcname)
                return false;
        }
 
-       *dcname = strdup(response.data.dc_name);
+       if (response.data.dsgetdcname.dc_address[0] != '\0') {
+               dc = response.data.dsgetdcname.dc_address;
+               if (dc[0] == '\\') dc++;
+               if (dc[0] == '\\') dc++;
+       }
+
+       if (!dc && response.data.dsgetdcname.dc_unc[0] != '\0') {
+               dc = response.data.dsgetdcname.dc_unc;
+               if (dc[0] == '\\') dc++;
+               if (dc[0] == '\\') dc++;
+       }
+
+       if (!dc) {
+               return false;
+       }
+
+       *dcname = strdup(dc);
        if (!*dcname) {
                return false;
        }
diff --git a/source3/winbindd/winbindd_locator.c 
b/source3/winbindd/winbindd_locator.c
index b2a8bd7..b60d235 100644
--- a/source3/winbindd/winbindd_locator.c
+++ b/source3/winbindd/winbindd_locator.c
@@ -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,44 +98,59 @@ 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));
+                 state->request.data.dsgetdcname.domain_name));
 
        ds_flags = get_dsgetdc_flags(state->request.flags);
 
-       result = dsgetdcname(state->mem_ctx, winbind_messaging_context(),
-                            state->request.domain_name,
-                            NULL, NULL, ds_flags, &info);
-
-       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;
 }


-- 
Samba Shared Repository

<Prev in Thread] Current Thread [Next in Thread>
  • [SCM] Samba Shared Repository - branch master updated - bf5c80c8eb3051ff2d04629b2b4a74b0c86d726d, Günther Deschner <=