diff options
author | Jelmer Vernooij <jelmer@samba.org> | 2003-05-15 08:43:52 +0000 |
---|---|---|
committer | Jelmer Vernooij <jelmer@samba.org> | 2003-05-15 08:43:52 +0000 |
commit | 3d804efe76335cd09640a82c0326dbf3eb83fe04 (patch) | |
tree | 33c84ef7f05e96cbc67e4b6d1f3c56568edc934c | |
parent | 1cabd4ba043447eaf15b498775a393e714099d40 (diff) | |
download | samba-3d804efe76335cd09640a82c0326dbf3eb83fe04.tar.gz samba-3d804efe76335cd09640a82c0326dbf3eb83fe04.tar.xz samba-3d804efe76335cd09640a82c0326dbf3eb83fe04.zip |
Patch from Luke Howard <lukeh@PADL.COM> that fixes some weird handling of
modules in auth/
-rw-r--r-- | source/auth/auth.c | 26 |
1 files changed, 17 insertions, 9 deletions
diff --git a/source/auth/auth.c b/source/auth/auth.c index cec15da1f66..0c4fe768307 100644 --- a/source/auth/auth.c +++ b/source/auth/auth.c @@ -372,17 +372,25 @@ BOOL load_auth_module(struct auth_context *auth_context, entry = auth_find_backend_entry(module_name); - if(!(entry = auth_find_backend_entry(module_name)) && NT_STATUS_IS_ERR(smb_probe_module("auth", module_name)) && - !(entry = auth_find_backend_entry(module_name))) { - DEBUG(0,("load_auth_module: can't find auth method %s!\n", module_name)); - } else if (!NT_STATUS_IS_OK(entry->init(auth_context, module_params, ret))) { - DEBUG(0,("load_auth_module: auth method %s did not correctly init\n", - module)); + if (entry == NULL) { + if (NT_STATUS_IS_OK(smb_probe_module("auth", module_name))) { + entry = auth_find_backend_entry(module_name); + } + } + + if (entry != NULL) { + if (!NT_STATUS_IS_OK(entry->init(auth_context, module_params, ret))) { + DEBUG(0,("load_auth_module: auth method %s did not correctly init\n", + module_name)); + } else { + DEBUG(5,("load_auth_module: auth method %s has a valid init\n", + module_name)); + good = True; + } } else { - DEBUG(5,("load_auth_module: auth method %s has a valid init\n", - module)); - good = True; + DEBUG(0,("load_auth_module: can't find auth method %s!\n", module_name)); } + SAFE_FREE(module_name); return good; } |