r22844: Introduce const DATA_BLOB data_blob_null = { NULL, 0, NULL }; and
[abartlet/samba.git/.git] / source3 / auth / auth.c
index a2486acbd1122625e993e687c6c79c5b9f37399a..eb239d3d7df891f53dd262bddc472f84b2bcc488 100644 (file)
@@ -23,6 +23,8 @@
 #undef DBGC_CLASS
 #define DBGC_CLASS DBGC_AUTH
 
+static_decl_auth;
+
 static struct auth_init_function_entry *backends = NULL;
 
 static struct auth_init_function_entry *auth_find_backend_entry(const char *name);
@@ -49,7 +51,7 @@ NTSTATUS smb_register_auth(int version, const char *name, auth_init_function ini
                return NT_STATUS_OBJECT_NAME_COLLISION;
        }
        
-       entry = smb_xmalloc(sizeof(struct auth_init_function_entry));
+       entry = SMB_XMALLOC_P(struct auth_init_function_entry);
        entry->name = smb_xstrdup(name);
        entry->init = init;
 
@@ -77,7 +79,7 @@ static struct auth_init_function_entry *auth_find_backend_entry(const char *name
 
 static const uint8 *get_ntlm_challenge(struct auth_context *auth_context) 
 {
-       DATA_BLOB challenge = data_blob(NULL, 0);
+       DATA_BLOB challenge = data_blob_null;
        const char *challenge_set_by = NULL;
        auth_methods *auth_method;
        TALLOC_CTX *mem_ctx;
@@ -88,6 +90,8 @@ static const uint8 *get_ntlm_challenge(struct auth_context *auth_context)
                return auth_context->challenge.data;
        }
 
+       auth_context->challenge_may_be_modified = False;
+
        for (auth_method = auth_context->auth_method_list; auth_method; auth_method = auth_method->next) {
                if (auth_method->get_chal == NULL) {
                        DEBUG(5, ("auth_get_challenge: module %s did not want to specify a challenge\n", auth_method->name));
@@ -122,11 +126,12 @@ static const uint8 *get_ntlm_challenge(struct auth_context *auth_context)
        if (!challenge_set_by) {
                uchar chal[8];
                
-               generate_random_buffer(chal, sizeof(chal), False);
+               generate_random_buffer(chal, sizeof(chal));
                auth_context->challenge = data_blob_talloc(auth_context->mem_ctx, 
                                                           chal, sizeof(chal));
                
                challenge_set_by = "random";
+               auth_context->challenge_may_be_modified = True;
        } 
        
        DEBUG(5, ("auth_context challenge created by %s\n", challenge_set_by));
@@ -193,7 +198,7 @@ static BOOL check_domain_match(const char *user, const char *domain)
  *                  function auth_get_challenge().  
  *
  * @param server_info If successful, contains information about the authentication, 
- *                    including a SAM_ACCOUNT struct describing the user.
+ *                    including a struct samu struct describing the user.
  *
  * @return An NTSTATUS with NT_STATUS_OK or an appropriate error.
  *
@@ -213,10 +218,10 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
                return NT_STATUS_LOGON_FAILURE;
 
        DEBUG(3, ("check_ntlm_password:  Checking password for unmapped user [%s]\\[%s]@[%s] with the new password interface\n", 
-                 user_info->client_domain.str, user_info->smb_name.str, user_info->wksta_name.str));
+                 user_info->client_domain, user_info->smb_name, user_info->wksta_name));
 
        DEBUG(3, ("check_ntlm_password:  mapped user is: [%s]\\[%s]@[%s]\n", 
-                 user_info->domain.str, user_info->internal_username.str, user_info->wksta_name.str));
+                 user_info->domain, user_info->internal_username, user_info->wksta_name));
 
        if (auth_context->challenge.length != 8) {
                DEBUG(0, ("check_ntlm_password:  Invalid challenge stored for this auth context - cannot continue\n"));
@@ -232,7 +237,7 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
 
 #ifdef DEBUG_PASSWORD
        DEBUG(100, ("user_info has passwords of length %d and %d\n", 
-                   user_info->lm_resp.length, user_info->nt_resp.length));
+                   (int)user_info->lm_resp.length, (int)user_info->nt_resp.length));
        DEBUG(100, ("lm:\n"));
        dump_data(100, user_info->lm_resp.data, user_info->lm_resp.length);
        DEBUG(100, ("nt:\n"));
@@ -240,14 +245,14 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
 #endif
 
        /* This needs to be sorted:  If it doesn't match, what should we do? */
-       if (!check_domain_match(user_info->smb_name.str, user_info->domain.str))
+       if (!check_domain_match(user_info->smb_name, user_info->domain))
                return NT_STATUS_LOGON_FAILURE;
 
        for (auth_method = auth_context->auth_method_list;auth_method; auth_method = auth_method->next) {
                NTSTATUS result;
                
                mem_ctx = talloc_init("%s authentication for user %s\\%s", auth_method->name, 
-                                           user_info->domain.str, user_info->smb_name.str);
+                                           user_info->domain, user_info->smb_name);
 
                result = auth_method->auth(auth_context, auth_method->private_data, mem_ctx, user_info, server_info);
 
@@ -262,10 +267,10 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
 
                if (NT_STATUS_IS_OK(nt_status)) {
                        DEBUG(3, ("check_ntlm_password: %s authentication for user [%s] succeeded\n", 
-                                 auth_method->name, user_info->smb_name.str));
+                                 auth_method->name, user_info->smb_name));
                } else {
                        DEBUG(5, ("check_ntlm_password: %s authentication for user [%s] FAILED with error %s\n", 
-                                 auth_method->name, user_info->smb_name.str, nt_errstr(nt_status)));
+                                 auth_method->name, user_info->smb_name, nt_errstr(nt_status)));
                }
 
                talloc_destroy(mem_ctx);
@@ -276,6 +281,8 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
                }
        }
 
+       /* successful authentication */
+       
        if (NT_STATUS_IS_OK(nt_status)) {
                unix_username = (*server_info)->unix_name;
                if (!(*server_info)->guest) {
@@ -297,18 +304,21 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
                        DEBUG((*server_info)->guest ? 5 : 2, 
                              ("check_ntlm_password:  %sauthentication for user [%s] -> [%s] -> [%s] succeeded\n", 
                               (*server_info)->guest ? "guest " : "", 
-                              user_info->smb_name.str
-                              user_info->internal_username.str
+                              user_info->smb_name, 
+                              user_info->internal_username, 
                               unix_username));
                }
+               
+               return nt_status;
        }
-
-       if (!NT_STATUS_IS_OK(nt_status)) {
-               DEBUG(2, ("check_ntlm_password:  Authentication for user [%s] -> [%s] FAILED with error %s\n", 
-                         user_info->smb_name.str, user_info->internal_username.str, 
-                         nt_errstr(nt_status)));
-               ZERO_STRUCTP(server_info);
-       }
+       
+       /* failed authentication; check for guest lapping */
+       
+       DEBUG(2, ("check_ntlm_password:  Authentication for user [%s] -> [%s] FAILED with error %s\n", 
+                 user_info->smb_name, user_info->internal_username, 
+                 nt_errstr(nt_status)));
+       ZERO_STRUCTP(server_info); 
+       
        return nt_status;
 }
 
@@ -318,9 +328,17 @@ static NTSTATUS check_ntlm_password(const struct auth_context *auth_context,
 
 static void free_auth_context(struct auth_context **auth_context)
 {
-       if (*auth_context != NULL)
+       auth_methods *auth_method;
+
+       if (*auth_context) {
+               /* Free private data of context's authentication methods */
+               for (auth_method = (*auth_context)->auth_method_list; auth_method; auth_method = auth_method->next) {
+                       TALLOC_FREE(auth_method->private_data);
+               }
+
                talloc_destroy((*auth_context)->mem_ctx);
-       *auth_context = NULL;
+               *auth_context = NULL;
+       }
 }
 
 /***************************************************************************
@@ -333,7 +351,7 @@ static NTSTATUS make_auth_context(struct auth_context **auth_context)
 
        mem_ctx = talloc_init("authentication context");
        
-       *auth_context = talloc(mem_ctx, sizeof(**auth_context));
+       *auth_context = TALLOC_P(mem_ctx, struct auth_context);
        if (!*auth_context) {
                DEBUG(0,("make_auth_context: talloc failed!\n"));
                talloc_destroy(mem_ctx);
@@ -373,10 +391,10 @@ BOOL load_auth_module(struct auth_context *auth_context,
        if (p) {
                *p = 0;
                module_params = p+1;
-               trim_string(module_params, " ", " ");
+               trim_char(module_params, ' ', ' ');
        }
        
-       trim_string(module_name, " ", " ");
+       trim_char(module_name, ' ', ' ');
        
        entry = auth_find_backend_entry(module_name);
        
@@ -411,7 +429,6 @@ static NTSTATUS make_auth_context_text_list(struct auth_context **auth_context,
 {
        auth_methods *list = NULL;
        auth_methods *t = NULL;
-       auth_methods *tmp;
        NTSTATUS nt_status;
 
        if (!text_list) {
@@ -424,7 +441,7 @@ static NTSTATUS make_auth_context_text_list(struct auth_context **auth_context,
 
        for (;*text_list; text_list++) { 
                if (load_auth_module(*auth_context, *text_list, &t)) {
-                   DLIST_ADD_END(list, t, tmp);
+                   DLIST_ADD_END(list, t, auth_methods *);
                }
        }
        
@@ -512,7 +529,7 @@ NTSTATUS make_auth_context_fixed(struct auth_context **auth_context, uchar chal[
                return nt_status;
        }
        
-       (*auth_context)->challenge = data_blob(chal, 8);
+       (*auth_context)->challenge = data_blob_talloc((*auth_context)->mem_ctx, chal, 8);
        (*auth_context)->challenge_set_by = "fixed";
        return nt_status;
 }