Revert "More correct fix (hopefully :-) for any memory leaks."
authorKarolin Seeger <kseeger@samba.org>
Tue, 10 Jun 2008 07:13:02 +0000 (09:13 +0200)
committerKarolin Seeger <kseeger@samba.org>
Tue, 10 Jun 2008 07:13:02 +0000 (09:13 +0200)
This reverts commit 9bb18b330164584905c383cf3eff2d077b250e86.

source/libsmb/namequery.c

index 24d7ee1a9c5b98056de496d905756f3f107ecebe..c987890e69ff4ee51ac819c3abdde62d2f33c87a 100644 (file)
@@ -1835,9 +1835,6 @@ static NTSTATUS get_dc_list(const char *domain,
        NTSTATUS status;
        TALLOC_CTX *ctx = talloc_init("get_dc_list");
 
-       *ip_list = NULL;
-       *count = 0;
-
        if (!ctx) {
                return NT_STATUS_NO_MEMORY;
        }
@@ -1917,7 +1914,7 @@ static NTSTATUS get_dc_list(const char *domain,
 
        p = pserver;
        while (next_token_talloc(ctx, &p, &name, LIST_SEP)) {
-               if (!done_auto_lookup && strequal(name, "*")) {
+               if (strequal(name, "*")) {
                        status = internal_resolve_name(domain, 0x1C, sitename,
                                                       &auto_ip_list,
                                                       &auto_count,
@@ -2058,12 +2055,6 @@ static NTSTATUS get_dc_list(const char *domain,
 
   out:
 
-       if (!NT_STATUS_IS_OK(status)) {
-               SAFE_FREE(return_iplist);
-               *ip_list = NULL;
-               *count = 0;
-       }
-
        SAFE_FREE(auto_ip_list);
        TALLOC_FREE(ctx);
        return status;
@@ -2083,9 +2074,6 @@ NTSTATUS get_sorted_dc_list( const char *domain,
        NTSTATUS status;
        enum dc_lookup_type lookup_type = DC_NORMAL_LOOKUP;
 
-       *ip_list = NULL;
-       *count = 0;
-
        DEBUG(8,("get_sorted_dc_list: attempting lookup "
                "for name %s (sitename %s) using [%s]\n",
                domain,
@@ -2099,8 +2087,6 @@ NTSTATUS get_sorted_dc_list( const char *domain,
        status = get_dc_list(domain, sitename, ip_list,
                        count, lookup_type, &ordered);
        if (!NT_STATUS_IS_OK(status)) {
-               SAFE_FREE(*ip_list);
-               *count = 0;
                return status;
        }
 
@@ -2131,8 +2117,6 @@ NTSTATUS get_kdc_list( const char *realm,
                        count, DC_KDC_ONLY, &ordered);
 
        if (!NT_STATUS_IS_OK(status)) {
-               SAFE_FREE(*ip_list);
-               *count = 0;
                return status;
        }