loadarm: fix testparm with "config backend = registry".

Set the iServiceIndex to "-1" for starup with either config
backend (originally only for text backend). Otherwise,
process_registry_shares() will fail.

Currently, the only user of lp_load_with_registry_shares() is testparm.

Michael
(This used to be commit 5e2b925367241c41793b2eb7a628e9fc9f3ac8ff)

diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index e45c8b89b753ad0155c805bb2d6950e6a2b51424..c2722748377cff8ce12121fca8beb8c272c34de8 100644 (file)
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -8746,6 +8746,9 @@ bool lp_load_ex(const char *pszFname,
                lp_save_defaults();
        }
 
+       /* We get sections first, so have to start 'behind' to make up */
+       iServiceIndex = -1;
+
        if (Globals.param_opt != NULL) {
                data = Globals.param_opt;
                while (data) {
@@ -8769,8 +8772,6 @@ bool lp_load_ex(const char *pszFname,
 
                add_to_file_list(pszFname, n2);
 
-               /* We get sections first, so have to start 'behind' to make up */
-               iServiceIndex = -1;
                bRetval = pm_process(n2, do_section, do_parameter, NULL);
                SAFE_FREE(n2);