The branch, v3-2-test has been updated via aabe9b33fcaed8af98b1ed6b736253e196d87d48 (commit) from ce7fe8acf41e90553431c7cda6823700701835c7 (commit)
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test - Log ----------------------------------------------------------------- commit aabe9b33fcaed8af98b1ed6b736253e196d87d48 Author: Andreas Schneider <[EMAIL PROTECTED]> Date: Fri Nov 23 10:54:48 2007 +0100 Don't restart winbind if a corrupted tdb is found during initialization. The tdb is validated before it gets initialized. Since then sighandlers changed a restart isn't needed anymore. ----------------------------------------------------------------------- Summary of changes: source/winbindd/winbindd.c | 24 +++++++++--------------- source/winbindd/winbindd_cache.c | 3 +-- 2 files changed, 10 insertions(+), 17 deletions(-) Changeset truncated at 500 lines: diff --git a/source/winbindd/winbindd.c b/source/winbindd/winbindd.c index b695c2e..07cf226 100644 --- a/source/winbindd/winbindd.c +++ b/source/winbindd/winbindd.c @@ -1157,20 +1157,6 @@ int main(int argc, char **argv, char **envp) pidfile_create("winbindd"); - /* Ensure all cache and idmap caches are consistent - before we startup. */ - - if (winbindd_validate_cache()) { - /* We have a bad cache, but luckily we - just deleted it. Restart ourselves */ - int i; - /* Ensure we have no open low fd's. */ - for (i = 3; i < 100; i++) { - close(i); - } - return execve(argv[0], argv, envp); - } - #if HAVE_SETPGID /* * If we're interactive we want to set our own process group for @@ -1188,7 +1174,15 @@ int main(int argc, char **argv, char **envp) DEBUG(0, ("unable to initialize messaging system\n")); exit(1); } - + + /* + * Ensure all cache and idmap caches are consistent + * before we startup. + */ + if (winbindd_validate_cache() < 0) { + DEBUG(0, ("corrupted tdb found, trying to restore backup\n")); + } + /* Initialize cache (ensure version is correct). */ if (!initialize_winbindd_cache()) { exit(1); diff --git a/source/winbindd/winbindd_cache.c b/source/winbindd/winbindd_cache.c index f3aa0fc..7ec8208 100644 --- a/source/winbindd/winbindd_cache.c +++ b/source/winbindd/winbindd_cache.c @@ -3338,8 +3338,7 @@ static void validate_panic(const char *const why) /*********************************************************************** Try and validate every entry in the winbindd cache. If we fail here, - delete the cache tdb and return non-zero - the caller (main winbindd - function) will restart us as we don't know if we crashed or not. + delete the cache tdb and return non-zero. ***********************************************************************/ int winbindd_validate_cache(void) -- Samba Shared Repository