From 17d5fbde4341d82c8a20dfa0bd94a6db12bebe41 Mon Sep 17 00:00:00 2001 From: Matthieu Patou Date: Mon, 11 Jun 2012 10:23:51 -0700 Subject: s3-winbindd: add comments about dump_core_setup This patch is related to change 0ea7152ff479e32e25817e4ddb1c2a1569bad81d Comments are added to explain why we call dump_core_setup 3 times. Autobuild-User(master): Matthieu Patou Autobuild-Date(master): Mon Jun 11 21:21:11 CEST 2012 on sn-devel-104 --- source3/winbindd/winbindd.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/source3/winbindd/winbindd.c b/source3/winbindd/winbindd.c index 20ee2fc060..1b6f4251b9 100644 --- a/source3/winbindd/winbindd.c +++ b/source3/winbindd/winbindd.c @@ -1350,6 +1350,13 @@ int main(int argc, char **argv, char **envp) } } + /* We call dump_core_setup one more time because the command line can + * set the log file or the log-basename and this will influence where + * cores are stored. Without this call get_dyn_LOGFILEBASE will be + * the default value derived from build's prefix. For EOM this value + * is often not related to the path where winbindd is actually run + * in production. + */ dump_core_setup("winbindd", lp_logfile()); if (is_daemon && interactive) { d_fprintf(stderr,"\nERROR: " @@ -1390,6 +1397,10 @@ int main(int argc, char **argv, char **envp) DEBUG(0, ("error opening config file '%s'\n", get_dyn_CONFIGFILE())); exit(1); } + /* After parsing the configuration file we setup the core path one more time + * as the log file might have been set in the configuration and cores's + * path is by default basename(lp_logfile()). + */ dump_core_setup("winbindd", lp_logfile()); /* Initialise messaging system */ -- cgit