s3:loadparm: reinit_globals in lp_load_with_registry_shares()

This was set to false in 0e0d77519c27038b30fec92d542198e97be767d9 based
on the assumption that callers would have no need to call
lp_load_initial_only() with a later call to lp_load_something().

This is not quite correct, since for accessing registry config on a
cluster with include=registry, we need messaging up and running which
*itself* requires loadparm to be initialized to get the statedir,
lockdir asf. directories.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13465

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>

diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index 291ba57e0bb2a84be441813e503a77b7cdd3c936..322934c55f09f03f2bbec7a818e6f9a2f3ad2130 100644 (file)
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -4120,7 +4120,7 @@ bool lp_load_with_registry_shares(const char *pszFname)
                          false, /* global_only */
                          true,  /* save_defaults */
                          false, /* add_ipc */
-                         false, /* reinit_globals */
+                         true, /* reinit_globals */
                          true,  /* allow_include_registry */
                          true); /* load_all_shares*/
 }