r9602: Add support for reading share_info.tdb and smb.conf.
[sfrench/samba-autobuild/.git] / source4 / param / loadparm.c
index 341b039aad989165543cb5a6650a392ad77f3e78..5191d108257b2909fa8dbf8a44772be96407ed8e 100644 (file)
  */
 
 #include "includes.h"
+#include "version.h"
+#include "dynconfig.h"
+#include "system/time.h"
+#include "system/iconv.h"
+#include "system/network.h"
+#include "system/printing.h"
+#include "librpc/gen_ndr/ndr_svcctl.h"
+#include "librpc/gen_ndr/ndr_samr.h"
+#include "librpc/gen_ndr/ndr_nbt.h"
+#include "dlinklist.h"
+#include "param/loadparm.h"
 
-BOOL in_client = False;                /* Not in the client by default */
 static BOOL bLoaded = False;
 
 #ifndef GLOBAL_NAME
@@ -73,11 +83,12 @@ static BOOL bLoaded = False;
 #define LP_SNUM_OK(i) (((i) >= 0) && ((i) < iNumServices) && ServicePtrs[(i)]->valid)
 #define VALID(i) ServicePtrs[i]->valid
 
-static BOOL do_parameter(const char *, const char *);
+static BOOL do_parameter(const char *, const char *, void *);
 static BOOL do_parameter_var(const char *pszParmName, const char *fmt, ...);
 
 static BOOL defaults_saved = False;
 
+
 struct param_opt {
        struct param_opt *prev, *next;
        char *key;
@@ -99,6 +110,7 @@ typedef struct
        char *szLockDir;
        char *szPidDir;
        char *szRootdir;
+       char *szSetupDir;
        char *szDefaultService;
        char *szHostsEquiv;
        char *szServerString;
@@ -110,9 +122,11 @@ typedef struct
        char *szSMBPasswdFile;
        char *szSAM_URL;
        char *szSPOOLSS_URL;
+       char *szWINS_URL;
        char *szPrivateDir;
+       char **jsInclude;
        char **szPreloadModules;
-       char *szPasswordServer;
+       char **szPasswordServers;
        char *szSocketOptions;
        char *szRealm;
        char *szADSserver;
@@ -131,7 +145,7 @@ typedef struct
        char **szNetbiosAliases;
        char *szNetbiosScope;
        char *szDomainOtherSIDs;
-       char *szNameResolveOrder;
+       char **szNameResolveOrder;
        char *szPanicAction;
        char *szAddUserScript; 
        char *szAddMachineScript;
@@ -139,6 +153,7 @@ typedef struct
        char *szWINSPartners;
        char **dcerpc_ep_servers;
        char **server_services;
+       char *ntptr_providor;
        char *szWinbindUID;
        char *szWinbindGID;
        char *szNonUnixAccountRange;
@@ -150,6 +165,12 @@ typedef struct
        BOOL bWinbindUseDefaultDomain;
        char *szIDMapBackend;
        char *szGuestaccount;
+       char *swat_directory;
+       BOOL tls_enabled;
+       char *tls_keyfile;
+       char *tls_certfile;
+       char *tls_cafile;
+       char *tls_crlfile;
        int max_mux;
        int max_xmit;
        int pwordlevel;
@@ -174,6 +195,11 @@ typedef struct
        int winbind_cache_time;
        int iLockSpinCount;
        int iLockSpinTime;
+       int nbt_port;
+       int dgram_port;
+       int cldap_port;
+       int krb5_port;
+       int web_port;
        char *socket_options;
        BOOL bDNSproxy;
        BOOL bWINSsupport;
@@ -195,11 +221,13 @@ typedef struct
        BOOL bNTSmbSupport;
        BOOL bNTStatusSupport;
        BOOL bAllowTrustedDomains;
+       BOOL bPlaintextAuth;
        BOOL bLanmanAuth;
        BOOL bNTLMAuth;
        BOOL bUseSpnego;
-       BOOL server_signing;
-       BOOL client_signing;
+       int  server_signing;
+       int  client_signing;
+       BOOL bClientPlaintextAuth;
        BOOL bClientLanManAuth;
        BOOL bClientNTLMv2Auth;
        BOOL bHostMSDfs;
@@ -342,7 +370,6 @@ static service **ServicePtrs = NULL;
 static int iNumServices = 0;
 static int iServiceIndex = 0;
 static BOOL bInGlobalSection = True;
-static BOOL bGlobalOnly = False;
 static int server_role;
 static int default_server_announce;
 
@@ -413,12 +440,6 @@ static const struct enum_list enum_announce_as[] = {
        {-1, NULL}
 };
 
-static const struct enum_list enum_case[] = {
-       {CASE_LOWER, "lower"},
-       {CASE_UPPER, "upper"},
-       {-1, NULL}
-};
-
 static const struct enum_list enum_bool_auto[] = {
        {False, "No"},
        {False, "False"},
@@ -456,42 +477,15 @@ static const struct enum_list enum_smb_signing_vals[] = {
        {SMB_SIGNING_SUPPORTED, "1"},
        {SMB_SIGNING_SUPPORTED, "On"},
        {SMB_SIGNING_SUPPORTED, "enabled"},
-       {SMB_SIGNING_SUPPORTED, "auto"},
        {SMB_SIGNING_REQUIRED, "required"},
        {SMB_SIGNING_REQUIRED, "mandatory"},
        {SMB_SIGNING_REQUIRED, "force"},
        {SMB_SIGNING_REQUIRED, "forced"},
        {SMB_SIGNING_REQUIRED, "enforced"},
+       {SMB_SIGNING_AUTO, "auto"},
        {-1, NULL}
 };
 
-/* 
-   Do you want session setups at user level security with a invalid
-   password to be rejected or allowed in as guest? WinNT rejects them
-   but it can be a pain as it means "net view" needs to use a password
-
-   You have 3 choices in the setting of map_to_guest:
-
-   "Never" means session setups with an invalid password
-   are rejected. This is the default.
-
-   "Bad User" means session setups with an invalid password
-   are rejected, unless the username does not exist, in which case it
-   is treated as a guest login
-
-   "Bad Password" means session setups with an invalid password
-   are treated as a guest login
-
-   Note that map_to_guest only has an effect in user or server
-   level security.
-*/
-
-static const struct enum_list enum_map_to_guest[] = {
-       {NEVER_MAP_TO_GUEST, "Never"},
-       {MAP_TO_GUEST_ON_BAD_USER, "Bad User"},
-       {MAP_TO_GUEST_ON_BAD_PASSWORD, "Bad Password"},
-       {-1, NULL}
-};
 
 /* Note: We do not initialise the defaults union - it is not allowed in ANSI C
  *
@@ -514,7 +508,7 @@ static struct parm_struct parm_table[] = {
        {"path", P_STRING, P_LOCAL, &sDefault.szPath, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_SHARE | FLAG_PRINT | FLAG_DEVELOPER},
        {"directory", P_STRING, P_LOCAL, &sDefault.szPath, NULL, NULL, FLAG_HIDE},
        {"workgroup", P_USTRING, P_GLOBAL, &Globals.szWorkgroup, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
-       {"realm", P_USTRING, P_GLOBAL, &Globals.szRealm, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
+       {"realm", P_STRING, P_GLOBAL, &Globals.szRealm, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"ADS server", P_STRING, P_GLOBAL, &Globals.szADSserver, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"netbios name", P_USTRING, P_GLOBAL, &Globals.szNetbiosName, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"netbios aliases", P_LIST, P_GLOBAL, &Globals.szNetbiosAliases, NULL, NULL, FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
@@ -523,6 +517,7 @@ static struct parm_struct parm_table[] = {
        {"interfaces", P_LIST, P_GLOBAL, &Globals.szInterfaces, NULL, NULL, FLAG_BASIC | FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"bind interfaces only", P_BOOL, P_GLOBAL, &Globals.bBindInterfacesOnly, NULL, NULL, FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"ntvfs handler", P_LIST, P_LOCAL, &sDefault.ntvfs_handler, NULL, NULL, FLAG_ADVANCED},
+       {"ntptr providor", P_STRING, P_GLOBAL, &Globals.ntptr_providor, NULL, NULL, FLAG_ADVANCED},
        {"dcerpc endpoint servers", P_LIST, P_GLOBAL, &Globals.dcerpc_ep_servers, NULL, NULL, FLAG_ADVANCED},
        {"server services", P_LIST, P_GLOBAL, &Globals.server_services, NULL, NULL, FLAG_ADVANCED},
 
@@ -536,10 +531,11 @@ static struct parm_struct parm_table[] = {
        {"idmap backend", P_STRING, P_GLOBAL, &Globals.szIDMapBackend, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"null passwords", P_BOOL, P_GLOBAL, &Globals.bNullPasswords, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"obey pam restrictions", P_BOOL, P_GLOBAL, &Globals.bObeyPamRestrictions, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
-       {"password server", P_STRING, P_GLOBAL, &Globals.szPasswordServer, NULL, NULL, FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
+       {"password server", P_LIST, P_GLOBAL, &Globals.szPasswordServers, NULL, NULL, FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"smb passwd file", P_STRING, P_GLOBAL, &Globals.szSMBPasswdFile, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"sam database", P_STRING, P_GLOBAL, &Globals.szSAM_URL, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"spoolss database", P_STRING, P_GLOBAL, &Globals.szSPOOLSS_URL, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"wins database", P_STRING, P_GLOBAL, &Globals.szWINS_URL, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"private dir", P_STRING, P_GLOBAL, &Globals.szPrivateDir, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"non unix account range", P_STRING, P_GLOBAL, &Globals.szNonUnixAccountRange, handle_non_unix_account_range, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"root directory", P_STRING, P_GLOBAL, &Globals.szRootdir, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
@@ -553,10 +549,12 @@ static struct parm_struct parm_table[] = {
        {"password level", P_INTEGER, P_GLOBAL, &Globals.pwordlevel, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"username level", P_INTEGER, P_GLOBAL, &Globals.unamelevel, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"restrict anonymous", P_INTEGER, P_GLOBAL, &Globals.restrict_anonymous, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"plaintext auth", P_BOOL, P_GLOBAL, &Globals.bPlaintextAuth, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"lanman auth", P_BOOL, P_GLOBAL, &Globals.bLanmanAuth, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"ntlm auth", P_BOOL, P_GLOBAL, &Globals.bNTLMAuth, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"client NTLMv2 auth", P_BOOL, P_GLOBAL, &Globals.bClientNTLMv2Auth, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"client lanman auth", P_BOOL, P_GLOBAL, &Globals.bClientLanManAuth, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"client plaintext auth", P_BOOL, P_GLOBAL, &Globals.bClientPlaintextAuth, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        
        {"username", P_STRING, P_LOCAL, &sDefault.szUsername, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
        {"user", P_STRING, P_LOCAL, &sDefault.szUsername, NULL, NULL, FLAG_HIDE},
@@ -586,6 +584,17 @@ static struct parm_struct parm_table[] = {
        {"Protocol Options", P_SEP, P_SEPARATOR},
        
        {"smb ports", P_LIST, P_GLOBAL, &Globals.smb_ports, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"nbt port", P_INTEGER, P_GLOBAL, &Globals.nbt_port, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"dgram port", P_INTEGER, P_GLOBAL, &Globals.dgram_port, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"cldap port", P_INTEGER, P_GLOBAL, &Globals.cldap_port, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"krb5 port", P_INTEGER, P_GLOBAL, &Globals.krb5_port, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"web port", P_INTEGER, P_GLOBAL, &Globals.web_port, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"tls enabled", P_BOOL, P_GLOBAL, &Globals.tls_enabled, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"tls keyfile", P_STRING, P_GLOBAL, &Globals.tls_keyfile, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"tls certfile", P_STRING, P_GLOBAL, &Globals.tls_certfile, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"tls cafile", P_STRING, P_GLOBAL, &Globals.tls_cafile, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"tls crlfile", P_STRING, P_GLOBAL, &Globals.tls_crlfile, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"swat directory", P_STRING, P_GLOBAL, &Globals.swat_directory, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"large readwrite", P_BOOL, P_GLOBAL, &Globals.bLargeReadwrite, NULL, NULL, FLAG_DEVELOPER},
        {"max protocol", P_ENUM, P_GLOBAL, &Globals.maxprotocol, NULL, enum_protocol, FLAG_DEVELOPER},
        {"min protocol", P_ENUM, P_GLOBAL, &Globals.minprotocol, NULL, enum_protocol, FLAG_DEVELOPER},
@@ -601,7 +610,7 @@ static struct parm_struct parm_table[] = {
        {"max mux", P_INTEGER, P_GLOBAL, &Globals.max_mux, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"max xmit", P_INTEGER, P_GLOBAL, &Globals.max_xmit, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
 
-       {"name resolve order", P_STRING, P_GLOBAL, &Globals.szNameResolveOrder, NULL, NULL, FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
+       {"name resolve order", P_LIST, P_GLOBAL, &Globals.szNameResolveOrder, NULL, NULL, FLAG_ADVANCED | FLAG_WIZARD | FLAG_DEVELOPER},
        {"max ttl", P_INTEGER, P_GLOBAL, &Globals.max_ttl, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER}, 
        {"max wins ttl", P_INTEGER, P_GLOBAL, &Globals.max_wins_ttl, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"min wins ttl", P_INTEGER, P_GLOBAL, &Globals.min_wins_ttl, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
@@ -712,6 +721,8 @@ static struct parm_struct parm_table[] = {
        {"lock dir", P_STRING, P_GLOBAL, &Globals.szLockDir, NULL, NULL, FLAG_HIDE}, 
        {"lock directory", P_STRING, P_GLOBAL, &Globals.szLockDir, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"pid directory", P_STRING, P_GLOBAL, &Globals.szPidDir, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER}, 
+       {"js include", P_LIST, P_GLOBAL, &Globals.jsInclude, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+       {"setup directory", P_STRING, P_GLOBAL, &Globals.szSetupDir, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        
        {"default service", P_STRING, P_GLOBAL, &Globals.szDefaultService, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
        {"default", P_STRING, P_GLOBAL, &Globals.szDefaultService, NULL, NULL,  FLAG_DEVELOPER},
@@ -734,6 +745,7 @@ static struct parm_struct parm_table[] = {
        {"msdfs root", P_BOOL, P_LOCAL, &sDefault.bMSDfsRoot, NULL, NULL, FLAG_SHARE},
        {"msdfs proxy", P_STRING, P_LOCAL, &sDefault.szMSDfsProxy, NULL, NULL, FLAG_SHARE},
        {"host msdfs", P_BOOL, P_GLOBAL, &Globals.bHostMSDfs, NULL, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
+
        {"Winbind options", P_SEP, P_SEPARATOR},
 
        {"winbind uid", P_STRING, P_GLOBAL, &Globals.szWinbindUID, handle_winbind_uid, NULL, FLAG_ADVANCED | FLAG_DEVELOPER},
@@ -749,6 +761,16 @@ static struct parm_struct parm_table[] = {
        {NULL, P_BOOL, P_NONE, NULL, NULL, NULL, 0}
 };
 
+
+/*
+  return the parameter table
+*/
+struct parm_struct *lp_parm_table(void)
+{
+       return parm_table;
+}
+
+
 /***************************************************************************
  Initialise the sDefault parameter structure for the printer values.
 ***************************************************************************/
@@ -761,103 +783,103 @@ static void init_printer_values(void)
                case PRINT_AIX:
                case PRINT_LPRNT:
                case PRINT_LPROS2:
-                       do_parameter("Lpqcommand", "lpq -P'%p'");
-                       do_parameter("Lprmcommand", "lprm -P'%p' %j");
+                       do_parameter("Lpqcommand", "lpq -P'%p'", NULL);
+                       do_parameter("Lprmcommand", "lprm -P'%p' %j", NULL);
                        do_parameter("Printcommand",
-                                  "lpr -r -P'%p' %s");
+                                  "lpr -r -P'%p' %s", NULL);
                        break;
 
                case PRINT_LPRNG:
                case PRINT_PLP:
-                       do_parameter("Lpqcommand", "lpq -P'%p'");
-                       do_parameter("Lprmcommand", "lprm -P'%p' %j");
+                       do_parameter("Lpqcommand", "lpq -P'%p'", NULL);
+                       do_parameter("Lprmcommand", "lprm -P'%p' %j", NULL);
                        do_parameter("Printcommand",
-                                  "lpr -r -P'%p' %s");
+                                  "lpr -r -P'%p' %s", NULL);
                        do_parameter("Queuepausecommand",
-                                  "lpc stop '%p'");
+                                  "lpc stop '%p'", NULL);
                        do_parameter("Queueresumecommand",
-                                  "lpc start '%p'");
+                                  "lpc start '%p'", NULL);
                        do_parameter("Lppausecommand",
-                                  "lpc hold '%p' %j");
+                                  "lpc hold '%p' %j", NULL);
                        do_parameter("Lpresumecommand",
-                                  "lpc release '%p' %j");
+                                  "lpc release '%p' %j", NULL);
                        break;
 
                case PRINT_CUPS:
 #ifdef HAVE_CUPS
-                       do_parameter("Lpqcommand", "");
-                       do_parameter("Lprmcommand", "");
-                       do_parameter("Printcommand", "");
-                       do_parameter("Lppausecommand", "");
-                       do_parameter("Lpresumecommand", "");
-                       do_parameter("Queuepausecommand", "");
-                       do_parameter("Queueresumecommand", "");
-
-                       do_parameter("Printcapname", "cups");
+                       do_parameter("Lpqcommand", "", NULL);
+                       do_parameter("Lprmcommand", "", NULL);
+                       do_parameter("Printcommand", "", NULL);
+                       do_parameter("Lppausecommand", "", NULL);
+                       do_parameter("Lpresumecommand", "", NULL);
+                       do_parameter("Queuepausecommand", "", NULL);
+                       do_parameter("Queueresumecommand", "", NULL);
+
+                       do_parameter("Printcapname", "cups", NULL);
 #else
                        do_parameter("Lpqcommand",
-                                  "/usr/bin/lpstat -o '%p'");
+                                  "/usr/bin/lpstat -o '%p'", NULL);
                        do_parameter("Lprmcommand",
-                                  "/usr/bin/cancel '%p-%j'");
+                                  "/usr/bin/cancel '%p-%j'", NULL);
                        do_parameter("Printcommand",
-                                  "/usr/bin/lp -d '%p' %s; rm %s");
+                                  "/usr/bin/lp -d '%p' %s; rm %s", NULL);
                        do_parameter("Lppausecommand",
-                                  "lp -i '%p-%j' -H hold");
+                                  "lp -i '%p-%j' -H hold", NULL);
                        do_parameter("Lpresumecommand",
-                                  "lp -i '%p-%j' -H resume");
+                                  "lp -i '%p-%j' -H resume", NULL);
                        do_parameter("Queuepausecommand",
-                                  "/usr/bin/disable '%p'");
+                                  "/usr/bin/disable '%p'", NULL);
                        do_parameter("Queueresumecommand",
-                                  "/usr/bin/enable '%p'");
-                       do_parameter("Printcapname", "lpstat");
+                                  "/usr/bin/enable '%p'", NULL);
+                       do_parameter("Printcapname", "lpstat", NULL);
 #endif /* HAVE_CUPS */
                        break;
 
                case PRINT_SYSV:
                case PRINT_HPUX:
-                       do_parameter("Lpqcommand", "lpstat -o%p");
-                       do_parameter("Lprmcommand", "cancel %p-%j");
+                       do_parameter("Lpqcommand", "lpstat -o%p", NULL);
+                       do_parameter("Lprmcommand", "cancel %p-%j", NULL);
                        do_parameter("Printcommand",
-                                  "lp -c -d%p %s; rm %s");
+                                  "lp -c -d%p %s; rm %s", NULL);
                        do_parameter("Queuepausecommand",
-                                  "disable %p");
+                                  "disable %p", NULL);
                        do_parameter("Queueresumecommand",
-                                  "enable %p");
+                                  "enable %p", NULL);
 #ifndef HPUX
                        do_parameter("Lppausecommand",
-                                  "lp -i %p-%j -H hold");
+                                  "lp -i %p-%j -H hold", NULL);
                        do_parameter("Lpresumecommand",
-                                  "lp -i %p-%j -H resume");
+                                  "lp -i %p-%j -H resume", NULL);
 #endif /* HPUX */
                        break;
 
                case PRINT_QNX:
-                       do_parameter("Lpqcommand", "lpq -P%p");
-                       do_parameter("Lprmcommand", "lprm -P%p %j");
-                       do_parameter("Printcommand", "lp -r -P%p %s");
+                       do_parameter("Lpqcommand", "lpq -P%p", NULL);
+                       do_parameter("Lprmcommand", "lprm -P%p %j", NULL);
+                       do_parameter("Printcommand", "lp -r -P%p %s", NULL);
                        break;
 
                case PRINT_SOFTQ:
-                       do_parameter("Lpqcommand", "qstat -l -d%p");
+                       do_parameter("Lpqcommand", "qstat -l -d%p", NULL);
                        do_parameter("Lprmcommand",
-                                  "qstat -s -j%j -c");
+                                  "qstat -s -j%j -c", NULL);
                        do_parameter("Printcommand",
-                                  "lp -d%p -s %s; rm %s");
+                                  "lp -d%p -s %s; rm %s", NULL);
                        do_parameter("Lppausecommand",
-                                  "qstat -s -j%j -h");
+                                  "qstat -s -j%j -h", NULL);
                        do_parameter("Lpresumecommand",
-                                  "qstat -s -j%j -r");
+                                  "qstat -s -j%j -r", NULL);
                        break;
 #ifdef DEVELOPER
        case PRINT_TEST:
        case PRINT_VLP:
-               do_parameter("Printcommand", "vlp print %p %s");
-               do_parameter("Lpqcommand", "vlp lpq %p");
-               do_parameter("Lprmcommand", "vlp lprm %p %j");
-               do_parameter("Lppausecommand", "vlp lppause %p %j");
-               do_parameter("Lpresumecommand", "vlp lpresum %p %j");
-               do_parameter("Queuepausecommand", "vlp queuepause %p");
-               do_parameter("Queueresumecommand", "vlp queueresume %p");
+               do_parameter("Printcommand", "vlp print %p %s", NULL);
+               do_parameter("Lpqcommand", "vlp lpq %p", NULL);
+               do_parameter("Lprmcommand", "vlp lprm %p %j", NULL);
+               do_parameter("Lppausecommand", "vlp lppause %p %j", NULL);
+               do_parameter("Lpresumecommand", "vlp lpresum %p %j", NULL);
+               do_parameter("Queuepausecommand", "vlp queuepause %p", NULL);
+               do_parameter("Queueresumecommand", "vlp queueresume %p", NULL);
                break;
 #endif /* DEVELOPER */
 
@@ -871,6 +893,7 @@ static void init_printer_values(void)
 static void init_globals(void)
 {
        int i;
+       char *myname;
 
        DEBUG(3, ("Initialising global parameters\n"));
 
@@ -883,136 +906,168 @@ static void init_globals(void)
                }
        }
 
+       do_parameter("config file", dyn_CONFIGFILE, NULL);
+
        /* options that can be set on the command line must be initialised via
           the slower do_parameter() to ensure that FLAG_CMDLINE is obeyed */
-       do_parameter("socket options", DEFAULT_SOCKET_OPTIONS);
-       do_parameter("workgroup", DEFAULT_WORKGROUP);
-       do_parameter("netbios name", get_myname());
-       do_parameter("max protocol", "NT1");
-       do_parameter("name resolve order", "lmhosts wins host bcast");
+#ifdef TCP_NODELAY
+       do_parameter("socket options", "TCP_NODELAY", NULL);
+#endif
+       do_parameter("workgroup", DEFAULT_WORKGROUP, NULL);
+       myname = get_myname();
+       do_parameter("netbios name", myname, NULL);
+       SAFE_FREE(myname);
+       do_parameter("max protocol", "NT1", NULL);
+       do_parameter("name resolve order", "lmhosts wins host bcast", NULL);
 
        init_printer_values();
 
-       do_parameter("fstype", FSTYPE_STRING);
-       do_parameter("ntvfs handler", "unixuid default");
-
-       do_parameter("dcerpc endpoint servers", "epmapper srvsvc wkssvc rpcecho samr netlogon lsarpc spoolss drsuapi winreg");
-       do_parameter("server services", "smb rpc");
-       do_parameter("auth methods", "guest sam_ignoredomain");
-       do_parameter("smb passwd file", dyn_SMB_PASSWD_FILE);
-       do_parameter("private dir", dyn_PRIVATE_DIR);
-       do_parameter_var("sam database", "tdb://%s/sam.ldb", dyn_PRIVATE_DIR);
-       do_parameter_var("spoolss database", "tdb://%s/spoolss.ldb", dyn_PRIVATE_DIR);
-       do_parameter("guest account", GUEST_ACCOUNT);
+       do_parameter("fstype", FSTYPE_STRING, NULL);
+       do_parameter("ntvfs handler", "unixuid default", NULL);
+       do_parameter("max connections", "-1", NULL);
+
+       do_parameter("dcerpc endpoint servers", "epmapper srvsvc wkssvc rpcecho samr netlogon lsarpc spoolss drsuapi winreg dssetup", NULL);
+       do_parameter("server services", "smb rpc nbt ldap cldap web kdc", NULL);
+       do_parameter("ntptr providor", "simple_ldb", NULL);
+       do_parameter("auth methods", "anonymous sam_ignoredomain", NULL);
+       do_parameter("smb passwd file", dyn_SMB_PASSWD_FILE, NULL);
+       do_parameter("private dir", dyn_PRIVATE_DIR, NULL);
+       do_parameter("sam database", "sam.ldb", NULL);
+       do_parameter("spoolss database", "spoolss.ldb", NULL);
+       do_parameter("wins database", "wins.ldb", NULL);
+       do_parameter("registry:HKEY_LOCAL_MACHINE", "hklm.ldb", NULL);
+
+       /* This hive should be dynamically generated by Samba using
+          data from the sam, but for the moment leave it in a tdb to
+          keep regedt32 from popping up an annoying dialog. */
+       do_parameter("registry:HKEY_USERS", "hku.ldb", NULL);
+       
+       do_parameter("guest account", GUEST_ACCOUNT, NULL);
 
        /* using UTF8 by default allows us to support all chars */
-       do_parameter("unix charset", "UTF8");
+       do_parameter("unix charset", "UTF8", NULL);
 
        /* Use codepage 850 as a default for the dos character set */
-       do_parameter("dos charset", "CP850");
+       do_parameter("dos charset", "CP850", NULL);
 
        /*
         * Allow the default PASSWD_CHAT to be overridden in local.h.
         */
-       do_parameter("passwd chat", DEFAULT_PASSWD_CHAT);
+       do_parameter("passwd chat", DEFAULT_PASSWD_CHAT, NULL);
 
-       do_parameter("passwd program", "");
-       do_parameter("printcap name", PRINTCAP_NAME);
+       do_parameter("passwd program", "", NULL);
+       do_parameter("printcap name", PRINTCAP_NAME, NULL);
        
-       do_parameter("pid directory", dyn_PIDDIR);
-       do_parameter("lock dir", dyn_LOCKDIR);
-       do_parameter("ncalrpc dir", dyn_NCALRPCDIR);
+       do_parameter("pid directory", dyn_PIDDIR, NULL);
+       do_parameter("lock dir", dyn_LOCKDIR, NULL);
+       do_parameter("ncalrpc dir", dyn_NCALRPCDIR, NULL);
 
-       do_parameter("socket address", "0.0.0.0");
+       do_parameter("socket address", "0.0.0.0", NULL);
        do_parameter_var("server string", "Samba %s", SAMBA_VERSION_STRING);
 
        do_parameter_var("announce version", "%d.%d", 
                         DEFAULT_MAJOR_VERSION,
                         DEFAULT_MINOR_VERSION);
 
-       do_parameter("logon drive", "");
-
-       do_parameter("logon home", "\\\\%N\\%U");
-       do_parameter("logon path", "\\\\%N\\%U\\profile");
-       do_parameter("password server", "*");
-
-       do_parameter("load printers", "True");
-
-       do_parameter("max mux", "50");
-       do_parameter("max xmit", "65535");
-       do_parameter("lpqcachetime", "10");
-       do_parameter("DisableSpoolss", "False");
-       do_parameter("password level", "0");
-       do_parameter("username level", "0");
-       do_parameter("LargeReadwrite", "True");
-       do_parameter("minprotocol", "CORE");
-       do_parameter("security", "USER");
-       do_parameter("paranoid server security", "True");
-       do_parameter("EncryptPasswords", "True");
-       do_parameter("ReadRaw", "True");
-       do_parameter("WriteRaw", "True");
-       do_parameter("NullPasswords", "False");
-       do_parameter("ObeyPamRestrictions", "False");
-       do_parameter("lm announce", "Auto");    
-       do_parameter("lm interval", "60");
-       do_parameter("announce as", "NT SERVER");
-
-       do_parameter("TimeServer", "False");
-       do_parameter("BindInterfacesOnly", "False");
-       do_parameter("PamPasswordChange", "False");
-       do_parameter("Unicode", "True");
-       do_parameter("restrict anonymous", "0");
-       do_parameter("ClientLanManAuth", "True");
-       do_parameter("LanmanAuth", "True");
-       do_parameter("NTLMAuth", "True");
+       do_parameter("logon drive", "", NULL);
+
+       do_parameter("logon home", "\\\\%N\\%U", NULL);
+       do_parameter("logon path", "\\\\%N\\%U\\profile", NULL);
+       do_parameter("password server", "*", NULL);
+
+       do_parameter("load printers", "True", NULL);
+
+       do_parameter("max mux", "50", NULL);
+       do_parameter("max xmit", "12288", NULL);
+       do_parameter("lpqcachetime", "10", NULL);
+       do_parameter("DisableSpoolss", "False", NULL);
+       do_parameter("password level", "0", NULL);
+       do_parameter("username level", "0", NULL);
+       do_parameter("LargeReadwrite", "True", NULL);
+       do_parameter("minprotocol", "CORE", NULL);
+       do_parameter("security", "USER", NULL);
+       do_parameter("paranoid server security", "True", NULL);
+       do_parameter("EncryptPasswords", "True", NULL);
+       do_parameter("ReadRaw", "True", NULL);
+       do_parameter("WriteRaw", "True", NULL);
+       do_parameter("NullPasswords", "False", NULL);
+       do_parameter("ObeyPamRestrictions", "False", NULL);
+       do_parameter("lm announce", "Auto", NULL);      
+       do_parameter("lm interval", "60", NULL);
+       do_parameter("announce as", "NT SERVER", NULL);
+
+       do_parameter("TimeServer", "False", NULL);
+       do_parameter("BindInterfacesOnly", "False", NULL);
+       do_parameter("PamPasswordChange", "False", NULL);
+       do_parameter("Unicode", "True", NULL);
+       do_parameter("restrict anonymous", "0", NULL);
+       do_parameter("ClientLanManAuth", "True", NULL);
+       do_parameter("LanmanAuth", "True", NULL);
+       do_parameter("NTLMAuth", "True", NULL);
        
-       do_parameter("enhanced browsing", "True"); 
-       do_parameter("LockSpinCount", "3");
-       do_parameter("LockSpinTime", "10");
+       do_parameter("enhanced browsing", "True", NULL); 
+       do_parameter("LockSpinCount", "3", NULL);
+       do_parameter("LockSpinTime", "10", NULL);
 #ifdef MMAP_BLACKLIST
-       do_parameter("UseMmap", "False");
+       do_parameter("UseMmap", "False", NULL);
 #else
-       do_parameter("UseMmap", "True");
+       do_parameter("UseMmap", "True", NULL);
 #endif
-       do_parameter("UnixExtensions", "False");
+       do_parameter("UnixExtensions", "False", NULL);
 
        /* hostname lookups can be very expensive and are broken on
           a large number of sites (tridge) */
-       do_parameter("HostnameLookups", "False");
+       do_parameter("HostnameLookups", "False", NULL);
+
+       do_parameter("PreferredMaster", "Auto", NULL);
+       do_parameter("os level", "20", NULL);
+       do_parameter("LocalMaster", "True", NULL);
+       do_parameter("DomainMaster", "Auto", NULL);     /* depending on bDomainLogons */
+       do_parameter("DomainLogons", "False", NULL);
+       do_parameter("WINSsupport", "False", NULL);
+       do_parameter("WINSproxy", "False", NULL);
+
+       do_parameter("DNSproxy", "True", NULL);
 
-       do_parameter("PreferredMaster", "Auto");
-       do_parameter("os level", "20");
-       do_parameter("LocalMaster", "True");
-       do_parameter("DomainMaster", "Auto");   /* depending on bDomainLogons */
-       do_parameter("DomainLogons", "False");
-       do_parameter("WINSsupport", "False");
-       do_parameter("WINSproxy", "False");
+       do_parameter("AllowTrustedDomains", "True", NULL);
 
-       do_parameter("DNSproxy", "True");
+       do_parameter("TemplateShell", "/bin/false", NULL);
+       do_parameter("TemplateHomedir", "/home/%D/%U", NULL);
+       do_parameter("WinbindSeparator", "\\", NULL);
 
-       do_parameter("AllowTrustedDomains", "True");
+       do_parameter("winbind cache time", "15", NULL);
+       do_parameter("WinbindEnumUsers", "True", NULL);
+       do_parameter("WinbindEnumGroups", "True", NULL);
+       do_parameter("WinbindUseDefaultDomain", "False", NULL);
 
-       do_parameter("TemplateShell", "/bin/false");
-       do_parameter("TemplateHomedir", "/home/%D/%U");
-       do_parameter("WinbindSeparator", "\\");
+       do_parameter("IDMapBackend", "tdb", NULL);
 
-       do_parameter("winbind cache time", "15");
-       do_parameter("WinbindEnumUsers", "True");
-       do_parameter("WinbindEnumGroups", "True");
-       do_parameter("WinbindUseDefaultDomain", "False");
+       do_parameter("name cache timeout", "660", NULL); /* In seconds */
 
-       do_parameter("IDMapBackend", "tdb");
+       do_parameter("client signing", "Yes", NULL);
+       do_parameter("server signing", "auto", NULL);
 
-       do_parameter("name cache timeout", "660"); /* In seconds */
+       do_parameter("use spnego", "True", NULL);
 
-       do_parameter("client signing", "Yes");
-       do_parameter("server signing", "Yes");
+       do_parameter("smb ports", SMB_PORTS, NULL);
+       do_parameter("nbt port", "137", NULL);
+       do_parameter("dgram port", "138", NULL);
+       do_parameter("cldap port", "389", NULL);
+       do_parameter("krb5 port", "88", NULL);
+       do_parameter("web port", "901", NULL);
+       do_parameter("swat directory", dyn_SWATDIR, NULL);
 
-       do_parameter("use spnego", "True");
+       do_parameter("nt status support", "True", NULL);
 
-       do_parameter("smb ports", SMB_PORTS);
+       do_parameter("max wins ttl", "432000", NULL);
+       do_parameter("min wins ttl", "10", NULL);
 
-       do_parameter("nt status support", "True");
+       do_parameter("tls enabled", "True", NULL);
+       do_parameter("tls keyfile", "tls/key.pem", NULL);
+       do_parameter("tls certfile", "tls/cert.pem", NULL);
+       do_parameter("tls cafile", "tls/ca.pem", NULL);
+       do_parameter_var("js include", "%s/js", dyn_LIBDIR);
+       do_parameter_var("setup directory", "%s/setup", dyn_LIBDIR);
 }
 
 static TALLOC_CTX *lp_talloc;
@@ -1055,7 +1110,7 @@ static const char *lp_string(const char *s)
        if (!lp_talloc)
                lp_talloc = talloc_init("lp_talloc");
 
-       ret = (char *)talloc(lp_talloc, len + 100);     /* leave room for substitution */
+       ret = talloc_array(lp_talloc, char, len + 100); /* leave room for substitution */
 
        if (!ret)
                return NULL;
@@ -1108,7 +1163,18 @@ static const char *lp_string(const char *s)
  int fn_name(int i) {return(LP_SNUM_OK(i)? ServicePtrs[(i)]->val : sDefault.val);}
 
 FN_GLOBAL_LIST(lp_smb_ports, &Globals.smb_ports)
+FN_GLOBAL_INTEGER(lp_nbt_port, &Globals.nbt_port)
+FN_GLOBAL_INTEGER(lp_dgram_port, &Globals.dgram_port)
+FN_GLOBAL_INTEGER(lp_cldap_port, &Globals.cldap_port)
+FN_GLOBAL_INTEGER(lp_krb5_port, &Globals.krb5_port)
+FN_GLOBAL_INTEGER(lp_web_port, &Globals.web_port)
 FN_GLOBAL_STRING(lp_dos_charset, &Globals.dos_charset)
+FN_GLOBAL_STRING(lp_swat_directory, &Globals.swat_directory)
+FN_GLOBAL_BOOL(lp_tls_enabled, &Globals.tls_enabled)
+FN_GLOBAL_STRING(lp_tls_keyfile, &Globals.tls_keyfile)
+FN_GLOBAL_STRING(lp_tls_certfile, &Globals.tls_certfile)
+FN_GLOBAL_STRING(lp_tls_cafile, &Globals.tls_cafile)
+FN_GLOBAL_STRING(lp_tls_crlfile, &Globals.tls_crlfile)
 FN_GLOBAL_STRING(lp_unix_charset, &Globals.unix_charset)
 FN_GLOBAL_STRING(lp_display_charset, &Globals.display_charset)
 FN_GLOBAL_STRING(lp_logfile, &Globals.szLogFile)
@@ -1116,22 +1182,25 @@ FN_GLOBAL_STRING(lp_configfile, &Globals.szConfigFile)
 FN_GLOBAL_STRING(lp_smb_passwd_file, &Globals.szSMBPasswdFile)
 FN_GLOBAL_STRING(lp_sam_url, &Globals.szSAM_URL)
 FN_GLOBAL_STRING(lp_spoolss_url, &Globals.szSPOOLSS_URL)
+FN_GLOBAL_STRING(lp_wins_url, &Globals.szWINS_URL)
 FN_GLOBAL_STRING(lp_private_dir, &Globals.szPrivateDir)
 FN_GLOBAL_STRING(lp_serverstring, &Globals.szServerString)
 FN_GLOBAL_STRING(lp_printcapname, &Globals.szPrintcapname)
 FN_GLOBAL_STRING(lp_lockdir, &Globals.szLockDir)
+FN_GLOBAL_STRING(lp_setupdir, &Globals.szSetupDir)
 FN_GLOBAL_STRING(lp_ncalrpc_dir, &Globals.ncalrpc_dir)
 FN_GLOBAL_STRING(lp_piddir, &Globals.szPidDir)
 FN_GLOBAL_LIST(lp_dcerpc_endpoint_servers, &Globals.dcerpc_ep_servers)
 FN_GLOBAL_LIST(lp_server_services, &Globals.server_services)
+FN_GLOBAL_STRING(lp_ntptr_providor, &Globals.ntptr_providor)
 FN_GLOBAL_STRING(lp_rootdir, &Globals.szRootdir)
 FN_GLOBAL_STRING(lp_defaultservice, &Globals.szDefaultService)
 FN_GLOBAL_STRING(lp_hosts_equiv, &Globals.szHostsEquiv)
 FN_GLOBAL_STRING(lp_auto_services, &Globals.szAutoServices)
 FN_GLOBAL_STRING(lp_passwd_program, &Globals.szPasswdProgram)
 FN_GLOBAL_STRING(lp_passwd_chat, &Globals.szPasswdChat)
-FN_GLOBAL_STRING(lp_passwordserver, &Globals.szPasswordServer)
-FN_GLOBAL_STRING(lp_name_resolve_order, &Globals.szNameResolveOrder)
+FN_GLOBAL_LIST(lp_passwordserver, &Globals.szPasswordServers)
+FN_GLOBAL_LIST(lp_name_resolve_order, &Globals.szNameResolveOrder)
 FN_GLOBAL_STRING(lp_realm, &Globals.szRealm)
 FN_GLOBAL_STRING(lp_ads_server, &Globals.szADSserver)
 FN_GLOBAL_STRING(lp_socket_options, &Globals.socket_options)
@@ -1147,7 +1216,6 @@ FN_GLOBAL_STRING(lp_remote_browse_sync, &Globals.szRemoteBrowseSync)
 FN_GLOBAL_LIST(lp_wins_server_list, &Globals.szWINSservers)
 FN_GLOBAL_LIST(lp_interfaces, &Globals.szInterfaces)
 FN_GLOBAL_STRING(lp_socket_address, &Globals.szSocketAddress)
-static FN_GLOBAL_STRING(lp_announce_version, &Globals.szAnnounceVersion)
 FN_GLOBAL_LIST(lp_netbios_aliases, &Globals.szNetbiosAliases)
 FN_GLOBAL_LIST(lp_preload_modules, &Globals.szPreloadModules)
 FN_GLOBAL_STRING(lp_panic_action, &Globals.szPanicAction)
@@ -1188,8 +1256,10 @@ FN_GLOBAL_BOOL(lp_unicode, &Globals.bUnicode)
 FN_GLOBAL_BOOL(lp_nt_status_support, &Globals.bNTStatusSupport)
 FN_GLOBAL_BOOL(lp_allow_trusted_domains, &Globals.bAllowTrustedDomains)
 FN_GLOBAL_INTEGER(lp_restrict_anonymous, &Globals.restrict_anonymous)
+FN_GLOBAL_BOOL(lp_plaintext_auth, &Globals.bPlaintextAuth)
 FN_GLOBAL_BOOL(lp_lanman_auth, &Globals.bLanmanAuth)
 FN_GLOBAL_BOOL(lp_ntlm_auth, &Globals.bNTLMAuth)
+FN_GLOBAL_BOOL(lp_client_plaintext_auth, &Globals.bClientPlaintextAuth)
 FN_GLOBAL_BOOL(lp_client_lanman_auth, &Globals.bClientLanManAuth)
 FN_GLOBAL_BOOL(lp_client_ntlmv2_auth, &Globals.bClientNTLMv2Auth)
 FN_GLOBAL_BOOL(lp_host_msdfs, &Globals.bHostMSDfs)
@@ -1221,6 +1291,9 @@ FN_GLOBAL_INTEGER(lp_lm_interval, &Globals.lm_interval)
 FN_GLOBAL_INTEGER(lp_machine_password_timeout, &Globals.machine_password_timeout)
 FN_GLOBAL_INTEGER(lp_lock_spin_count, &Globals.iLockSpinCount)
 FN_GLOBAL_INTEGER(lp_lock_sleep_time, &Globals.iLockSpinTime)
+FN_GLOBAL_LIST(lp_js_include, &Globals.jsInclude)
+
+
 FN_LOCAL_STRING(lp_servicename, szService)
 FN_LOCAL_CONST_STRING(lp_const_servicename, szService)
 FN_LOCAL_STRING(lp_pathname, szPath)
@@ -1281,13 +1354,13 @@ static int getservicebyname(const char *pszServiceName,
 static void copy_service(service * pserviceDest,
                         service * pserviceSource, BOOL *pcopymapDest);
 static BOOL service_ok(int iService);
-static BOOL do_section(const char *pszSectionName);
+static BOOL do_section(const char *pszSectionName, void *);
 static void init_copymap(service * pservice);
 
 /* This is a helper function for parametrical options support. */
 /* It returns a pointer to parametrical option value if it exists or NULL otherwise */
 /* Actual parametrical functions are quite simple */
-static const char *get_parametrics(int lookup_service, const char *type, const char *option)
+const char *lp_get_parametric(int lookup_service, const char *type, const char *option)
 {
        char *vfskey;
         struct param_opt *data;
@@ -1338,7 +1411,7 @@ static int lp_int(const char *s)
                return (-1);
        }
 
-       return atoi(s); 
+       return strtol(s, NULL, 0); 
 }
 
 /*******************************************************************
@@ -1352,7 +1425,7 @@ static int lp_ulong(const char *s)
                return (-1);
        }
 
-       return strtoul(s, NULL, 10);
+       return strtoul(s, NULL, 0);
 }
 
 /*******************************************************************
@@ -1375,26 +1448,6 @@ static BOOL lp_bool(const char *s)
        return ret;
 }
 
-/*******************************************************************
-convenience routine to return enum parameters.
-********************************************************************/
-static int lp_enum(const char *s,const struct enum_list *_enum)
-{
-       int i;
-
-       if (!s || !_enum) {
-               DEBUG(0,("lp_enum(%s,enum): is called with NULL!\n",s));
-               return False;
-       }
-       
-       for (i=0; _enum[i].name; i++) {
-               if (strcasecmp(_enum[i].name,s)==0)
-                       return _enum[i].value;
-       }
-
-       DEBUG(0,("lp_enum(%s,enum): value is not in enum_list!\n",s));
-       return (-1);
-}
 
 /* Return parametric option from a given service. Type is a part of option before ':' */
 /* Parametric option has following syntax: 'Type: option = value' */
@@ -1402,8 +1455,8 @@ static int lp_enum(const char *s,const struct enum_list *_enum)
 
 const char *lp_parm_string(int lookup_service, const char *type, const char *option)
 {
-       const char *value = get_parametrics(lookup_service, type, option);
-       
+       const char *value = lp_get_parametric(lookup_service, type, option);
+
        if (value)
                return lp_string(value);
 
@@ -1414,13 +1467,13 @@ const char *lp_parm_string(int lookup_service, const char *type, const char *opt
 /* Parametric option has following syntax: 'Type: option = value' */
 /* Returned value is allocated in 'lp_talloc' context */
 
-char **lp_parm_string_list(int lookup_service, const char *type, const char *option,
-                          const char *separator)
+const char **lp_parm_string_list(int lookup_service, const char *type, const char *option,
+                                const char *separator)
 {
-       const char *value = get_parametrics(lookup_service, type, option);
+       const char *value = lp_get_parametric(lookup_service, type, option);
        
        if (value)
-               return str_list_make(value, separator);
+               return str_list_make(talloc_autofree_context(), value, separator);
 
        return NULL;
 }
@@ -1428,27 +1481,27 @@ char **lp_parm_string_list(int lookup_service, const char *type, const char *opt
 /* Return parametric option from a given service. Type is a part of option before ':' */
 /* Parametric option has following syntax: 'Type: option = value' */
 
-int lp_parm_int(int lookup_service, const char *type, const char *option)
+int lp_parm_int(int lookup_service, const char *type, const char *option, int default_v)
 {
-       const char *value = get_parametrics(lookup_service, type, option);
+       const char *value = lp_get_parametric(lookup_service, type, option);
        
        if (value)
                return lp_int(value);
 
-       return (-1);
+       return default_v;
 }
 
 /* Return parametric option from a given service. Type is a part of option before ':' */
 /* Parametric option has following syntax: 'Type: option = value' */
 
-unsigned long lp_parm_ulong(int lookup_service, const char *type, const char *option)
+unsigned long lp_parm_ulong(int lookup_service, const char *type, const char *option, unsigned long default_v)
 {
-       const char *value = get_parametrics(lookup_service, type, option);
+       const char *value = lp_get_parametric(lookup_service, type, option);
        
        if (value)
                return lp_ulong(value);
 
-       return (0);
+       return default_v;
 }
 
 /* Return parametric option from a given service. Type is a part of option before ':' */
@@ -1456,7 +1509,7 @@ unsigned long lp_parm_ulong(int lookup_service, const char *type, const char *op
 
 BOOL lp_parm_bool(int lookup_service, const char *type, const char *option, BOOL default_v)
 {
-       const char *value = get_parametrics(lookup_service, type, option);
+       const char *value = lp_get_parametric(lookup_service, type, option);
        
        if (value)
                return lp_bool(value);
@@ -1464,20 +1517,6 @@ BOOL lp_parm_bool(int lookup_service, const char *type, const char *option, BOOL
        return default_v;
 }
 
-/* Return parametric option from a given service. Type is a part of option before ':' */
-/* Parametric option has following syntax: 'Type: option = value' */
-
-int lp_parm_enum(int lookup_service, const char *type, const char *option,
-                const struct enum_list *_enum)
-{
-       const char *value = get_parametrics(lookup_service, type, option);
-       
-       if (value)
-               return lp_enum(value, _enum);
-
-       return (-1);
-}
-
 
 /***************************************************************************
  Initialise a service to the defaults.
@@ -1510,15 +1549,17 @@ static void free_service(service *pservice)
        for (i = 0; parm_table[i].label; i++) {
                if ((parm_table[i].type == P_STRING ||
                     parm_table[i].type == P_USTRING) &&
-                   parm_table[i].class == P_LOCAL)
+                   parm_table[i].class == P_LOCAL) {
                        string_free((char **)
                                    (((char *)pservice) +
                                     PTR_DIFF(parm_table[i].ptr, &sDefault)));
-               else if (parm_table[i].type == P_LIST &&
-                        parm_table[i].class == P_LOCAL)
-                            str_list_free((char ***)
-                                           (((char *)pservice) +
-                                            PTR_DIFF(parm_table[i].ptr, &sDefault)));
+               } else if (parm_table[i].type == P_LIST &&
+                          parm_table[i].class == P_LOCAL) {
+                       char ***listp = (char ***)(((char *)pservice) + 
+                                                  PTR_DIFF(parm_table[i].ptr, &sDefault));
+                       talloc_free(*listp);
+                       *listp = NULL;
+               }
        }
                                
        DEBUG(5,("Freeing parametrics:\n"));
@@ -1577,9 +1618,7 @@ static int add_a_service(const service *pservice, const char *name)
        if (i == iNumServices) {
                service **tsp;
                
-               tsp = (service **) Realloc(ServicePtrs,
-                                          sizeof(service *) *
-                                          num_to_alloc);
+               tsp = realloc_p(ServicePtrs, service *, num_to_alloc);
                                           
                if (!tsp) {
                        DEBUG(0,("add_a_service: failed to enlarge ServicePtrs!\n"));
@@ -1587,8 +1626,7 @@ static int add_a_service(const service *pservice, const char *name)
                }
                else {
                        ServicePtrs = tsp;
-                       ServicePtrs[iNumServices] =
-                               (service *) malloc(sizeof(service));
+                       ServicePtrs[iNumServices] = malloc_p(service);
                }
                if (!ServicePtrs[iNumServices]) {
                        DEBUG(0,("add_a_service: out of memory!\n"));
@@ -1662,30 +1700,34 @@ int lp_add_service(const char *pszService, int iDefaultService)
  Add the IPC service.
 ***************************************************************************/
 
-static BOOL lp_add_ipc(const char *ipc_name, BOOL guest_ok)
+static BOOL lp_add_hidden(const char *name, const char *fstype, BOOL guest_ok)
 {
        pstring comment;
-       int i = add_a_service(&sDefault, ipc_name);
+       int i = add_a_service(&sDefault, name);
 
        if (i < 0)
                return (False);
 
        slprintf(comment, sizeof(comment) - 1,
-                "IPC Service (%s)", Globals.szServerString);
+                "%s Service (%s)", fstype, Globals.szServerString);
 
        string_set(&ServicePtrs[i]->szPath, tmpdir());
        string_set(&ServicePtrs[i]->szUsername, "");
        string_set(&ServicePtrs[i]->comment, comment);
-       string_set(&ServicePtrs[i]->fstype, "IPC");
-       ServicePtrs[i]->iMaxConnections = 0;
+       string_set(&ServicePtrs[i]->fstype, fstype);
+       ServicePtrs[i]->iMaxConnections = -1;
        ServicePtrs[i]->bAvailable = True;
        ServicePtrs[i]->bRead_only = True;
        ServicePtrs[i]->bGuest_only = False;
        ServicePtrs[i]->bGuest_ok = guest_ok;
        ServicePtrs[i]->bPrint_ok = False;
-       ServicePtrs[i]->bBrowseable = sDefault.bBrowseable;
+       ServicePtrs[i]->bBrowseable = False;
+
+       if (strcasecmp(fstype, "IPC") == 0) {
+               lp_do_parameter(i, "ntvfs handler", "default");
+       }
 
-       DEBUG(3, ("adding IPC service\n"));
+       DEBUG(3, ("adding hidden service %s\n", name));
 
        return (True);
 }
@@ -1752,6 +1794,28 @@ static int map_parameter(const char *pszParmName)
        return (-1);
 }
 
+
+/*
+  return the parameter structure for a parameter
+*/
+struct parm_struct *lp_parm_struct(const char *name)
+{
+       int parmnum = map_parameter(name);
+       if (parmnum == -1) return NULL;
+       return &parm_table[parmnum];
+}
+
+/*
+  return the parameter pointer for a parameter
+*/
+void *lp_parm_ptr(int snum, struct parm_struct *parm)
+{
+       if (snum == -1) {
+               return parm->ptr;
+       }
+       return ((char *)ServicePtrs[snum]) + PTR_DIFF(parm->ptr, &sDefault);
+}
+
 /***************************************************************************
  Set a boolean variable from the text value stored in the passed string.
  Returns True in success, False if the passed string does not correctly 
@@ -1824,20 +1888,14 @@ static void copy_service(service * pserviceDest, service * pserviceSource, BOOL
 
                        switch (parm_table[i].type) {
                                case P_BOOL:
-                               case P_BOOLREV:
                                        *(BOOL *)dest_ptr = *(BOOL *)src_ptr;
                                        break;
 
                                case P_INTEGER:
                                case P_ENUM:
-                               case P_OCTAL:
                                        *(int *)dest_ptr = *(int *)src_ptr;
                                        break;
 
-                               case P_CHAR:
-                                       *(char *)dest_ptr = *(char *)src_ptr;
-                                       break;
-
                                case P_STRING:
                                        string_set(dest_ptr,
                                                   *(char **)src_ptr);
@@ -1849,7 +1907,8 @@ static void copy_service(service * pserviceDest, service * pserviceSource, BOOL
                                        strupper(*(char **)dest_ptr);
                                        break;
                                case P_LIST:
-                                       str_list_copy((char ***)dest_ptr, *(const char ***)src_ptr);
+                                       *(const char ***)dest_ptr = str_list_copy(talloc_autofree_context(), 
+                                                                                 *(const char ***)src_ptr);
                                        break;
                                default:
                                        break;
@@ -1880,7 +1939,7 @@ static void copy_service(service * pserviceDest, service * pserviceSource, BOOL
                        pdata = pdata->next;
                }
                if (not_added) {
-                       paramo = smb_xmalloc(sizeof(*paramo));
+                       paramo = smb_xmalloc_p(struct param_opt);
                        paramo->key = strdup(data->key);
                        paramo->value = strdup(data->value);
                        DLIST_ADD(pserviceDest->param_opt, paramo);
@@ -1912,6 +1971,7 @@ static BOOL service_ok(int iService)
                        DEBUG(0, ("WARNING: [%s] service MUST be printable!\n",
                               ServicePtrs[iService]->szService));
                        ServicePtrs[iService]->bPrint_ok = True;
+                       update_server_announce_as_printserver();
                }
                /* [printers] service must also be non-browsable. */
                if (ServicePtrs[iService]->bBrowseable)
@@ -1949,7 +2009,7 @@ static void add_to_file_list(const char *fname, const char *subfname)
        }
 
        if (!f) {
-               f = (struct file_lists *)malloc(sizeof(file_lists[0]));
+               f = malloc_p(struct file_lists);
                if (!f)
                        return;
                f->next = file_lists;
@@ -2022,8 +2082,8 @@ static BOOL handle_include(const char *pszParmValue, char **ptr)
 
        string_set(ptr, fname);
 
-       if (file_exist(fname, NULL))
-               return (pm_process(fname, do_section, do_parameter));
+       if (file_exist(fname))
+               return (pm_process(fname, do_section, do_parameter, NULL));
 
        DEBUG(2, ("Can't find include file %s\n", fname));
 
@@ -2194,7 +2254,7 @@ static void init_copymap(service * pservice)
 {
        int i;
        SAFE_FREE(pservice->copymap);
-       pservice->copymap = (BOOL *)malloc(sizeof(BOOL) * NUMPARAMETERS);
+       pservice->copymap = malloc_array_p(BOOL, NUMPARAMETERS);
        if (!pservice->copymap)
                DEBUG(0,
                      ("Couldn't allocate copymap!! (size %d)\n",
@@ -2223,7 +2283,7 @@ static BOOL lp_do_parameter_parametric(int snum, const char *pszParmName, const
        struct param_opt *paramo, *data;
        char *name;
 
-       while (isspace(*pszParmName)) {
+       while (isspace((unsigned char)*pszParmName)) {
                pszParmName++;
        }
 
@@ -2256,7 +2316,7 @@ static BOOL lp_do_parameter_parametric(int snum, const char *pszParmName, const
                }
        }
 
-       paramo = smb_xmalloc(sizeof(*paramo));
+       paramo = smb_xmalloc_p(struct param_opt);
        paramo->key = strdup(name);
        paramo->value = strdup(pszParmValue);
        paramo->flags = flags;
@@ -2343,25 +2403,13 @@ BOOL lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue
                        set_boolean(parm_ptr, pszParmValue);
                        break;
 
-               case P_BOOLREV:
-                       set_boolean(parm_ptr, pszParmValue);
-                       *(BOOL *)parm_ptr = !*(BOOL *)parm_ptr;
-                       break;
-
                case P_INTEGER:
                        *(int *)parm_ptr = atoi(pszParmValue);
                        break;
 
-               case P_CHAR:
-                       *(char *)parm_ptr = *pszParmValue;
-                       break;
-
-               case P_OCTAL:
-                       sscanf(pszParmValue, "%o", (int *)parm_ptr);
-                       break;
-
                case P_LIST:
-                       *(char ***)parm_ptr = str_list_make(pszParmValue, NULL);
+                       *(const char ***)parm_ptr = str_list_make(talloc_autofree_context(), 
+                                                                 pszParmValue, NULL);
                        break;
 
                case P_STRING:
@@ -2401,11 +2449,8 @@ BOOL lp_do_parameter(int snum, const char *pszParmName, const char *pszParmValue
  Process a parameter.
 ***************************************************************************/
 
-static BOOL do_parameter(const char *pszParmName, const char *pszParmValue)
+static BOOL do_parameter(const char *pszParmName, const char *pszParmValue, void *userdata)
 {
-       if (!bInGlobalSection && bGlobalOnly)
-               return (True);
-
        return (lp_do_parameter(bInGlobalSection ? -2 : iServiceIndex,
                                pszParmName, pszParmValue));
 }
@@ -2424,7 +2469,7 @@ static BOOL do_parameter_var(const char *pszParmName, const char *fmt, ...)
        va_start(ap, fmt);      
        s = talloc_vasprintf(NULL, fmt, ap);
        va_end(ap);
-       ret = do_parameter(pszParmName, s);
+       ret = do_parameter(pszParmName, s, NULL);
        talloc_free(s);
        return ret;
 }
@@ -2440,7 +2485,7 @@ BOOL lp_set_cmdline(const char *pszParmName, const char *pszParmValue)
        int parmnum = map_parameter(pszParmName);
        int i;
 
-       while (isspace(*pszParmValue)) pszParmValue++;
+       while (isspace((unsigned char)*pszParmValue)) pszParmValue++;
 
 
        if (parmnum < 0 && strchr(pszParmName, ':')) {
@@ -2500,6 +2545,8 @@ BOOL lp_set_option(const char *option)
 }
 
 
+#define BOOLSTR(b) ((b) ? "Yes" : "No")
+
 /***************************************************************************
  Print a parameter of the specified type.
 ***************************************************************************/
@@ -2523,26 +2570,10 @@ static void print_parameter(struct parm_struct *p, void *ptr, FILE * f)
                        fprintf(f, "%s", BOOLSTR(*(BOOL *)ptr));
                        break;
 
-               case P_BOOLREV:
-                       fprintf(f, "%s", BOOLSTR(!*(BOOL *)ptr));
-                       break;
-
                case P_INTEGER:
                        fprintf(f, "%d", *(int *)ptr);
                        break;
 
-               case P_CHAR:
-                       fprintf(f, "%c", *(char *)ptr);
-                       break;
-
-               case P_OCTAL:
-                       if (*(int *)ptr == -1) {
-                               fprintf(f, "-1");
-                       } else {
-                               fprintf(f, "0%o", *(int *)ptr);
-                       }
-                       break;
-
                case P_LIST:
                        if ((char ***)ptr && *(char ***)ptr) {
                                char **list = *(char ***)ptr;
@@ -2572,19 +2603,15 @@ static BOOL equal_parameter(parm_type type, void *ptr1, void *ptr2)
 {
        switch (type) {
                case P_BOOL:
-               case P_BOOLREV:
                        return (*((BOOL *)ptr1) == *((BOOL *)ptr2));
 
                case P_INTEGER:
                case P_ENUM:
-               case P_OCTAL:
                        return (*((int *)ptr1) == *((int *)ptr2));
 
-               case P_CHAR:
-                       return (*((char *)ptr1) == *((char *)ptr2));
-               
                case P_LIST:
-                       return str_list_compare(*(char ***)ptr1, *(char ***)ptr2);
+                       return str_list_equal((const char **)(*(char ***)ptr1), 
+                                             (const char **)(*(char ***)ptr2));
 
                case P_STRING:
                case P_USTRING:
@@ -2608,7 +2635,7 @@ static BOOL equal_parameter(parm_type type, void *ptr1, void *ptr2)
  Returns True on success, False on failure. 
 ***************************************************************************/
 
-static BOOL do_section(const char *pszSectionName)
+static BOOL do_section(const char *pszSectionName, void *userdata)
 {
        BOOL bRetval;
        BOOL isglobal = ((strwicmp(pszSectionName, GLOBAL_NAME) == 0) ||
@@ -2624,9 +2651,6 @@ static BOOL do_section(const char *pszSectionName)
                return (True);
        }
 
-       if (!bInGlobalSection && bGlobalOnly)
-               return (True);
-
        /* if we have a current service, tidy it up before moving on */
        bRetval = True;
 
@@ -2660,21 +2684,16 @@ static BOOL is_default(int i)
                return False;
        switch (parm_table[i].type) {
                case P_LIST:
-                       return str_list_compare (parm_table[i].def.lvalue, 
-                                               *(char ***)parm_table[i].ptr);
+                       return str_list_equal((const char **)parm_table[i].def.lvalue, 
+                                             (const char **)(*(char ***)parm_table[i].ptr));
                case P_STRING:
                case P_USTRING:
                        return strequal(parm_table[i].def.svalue,
                                        *(char **)parm_table[i].ptr);
                case P_BOOL:
-               case P_BOOLREV:
                        return parm_table[i].def.bvalue ==
                                *(BOOL *)parm_table[i].ptr;
-               case P_CHAR:
-                       return parm_table[i].def.cvalue ==
-                               *(char *)parm_table[i].ptr;
                case P_INTEGER:
-               case P_OCTAL:
                case P_ENUM:
                        return parm_table[i].def.ivalue ==
                                *(int *)parm_table[i].ptr;
@@ -2715,19 +2734,6 @@ static void dump_globals(FILE *f)
 
 }
 
-/***************************************************************************
- Return True if a local parameter is currently set to the global default.
-***************************************************************************/
-
-BOOL lp_is_default(int snum, struct parm_struct *parm)
-{
-       int pdiff = PTR_DIFF(parm->ptr, &sDefault);
-
-       return equal_parameter(parm->type,
-                              ((char *)ServicePtrs[snum]) + pdiff,
-                              ((char *)&sDefault) + pdiff);
-}
-
 /***************************************************************************
  Display the contents of a single services record.
 ***************************************************************************/
@@ -2833,28 +2839,6 @@ struct parm_struct *lp_next_parameter(int snum, int *i, int allparameters)
 }
 
 
-#if 0
-/***************************************************************************
- Display the contents of a single copy structure.
-***************************************************************************/
-static void dump_copy_map(BOOL *pcopymap)
-{
-       int i;
-       if (!pcopymap)
-               return;
-
-       printf("\n\tNon-Copied parameters:\n");
-
-       for (i = 0; parm_table[i].label; i++)
-               if (parm_table[i].class == P_LOCAL &&
-                   parm_table[i].ptr && !pcopymap[i] &&
-                   (i == 0 || (parm_table[i].ptr != parm_table[i - 1].ptr)))
-               {
-                       printf("\t\t%s\n", parm_table[i].label);
-               }
-}
-#endif
-
 /***************************************************************************
  Return TRUE if the passed service number is within range.
 ***************************************************************************/
@@ -2939,53 +2923,6 @@ void lp_killservice(int iServiceIn)
        }
 }
 
-/***************************************************************************
- Save the curent values of all global and sDefault parameters into the 
- defaults union. This allows swat and testparm to show only the
- changed (ie. non-default) parameters.
-***************************************************************************/
-
-static void lp_save_defaults(void)
-{
-       int i;
-       for (i = 0; parm_table[i].label; i++) {
-               if (i > 0 && parm_table[i].ptr == parm_table[i - 1].ptr)
-                       continue;
-               switch (parm_table[i].type) {
-                       case P_LIST:
-                               str_list_copy(&(parm_table[i].def.lvalue),
-                                           *(const char ***)parm_table[i].ptr);
-                               break;
-                       case P_STRING:
-                       case P_USTRING:
-                               if (parm_table[i].ptr) {
-                                       parm_table[i].def.svalue = strdup(*(char **)parm_table[i].ptr);
-                               } else {
-                                       parm_table[i].def.svalue = NULL;
-                               }
-                               break;
-                       case P_BOOL:
-                       case P_BOOLREV:
-                               parm_table[i].def.bvalue =
-                                       *(BOOL *)parm_table[i].ptr;
-                               break;
-                       case P_CHAR:
-                               parm_table[i].def.cvalue =
-                                       *(char *)parm_table[i].ptr;
-                               break;
-                       case P_INTEGER:
-                       case P_OCTAL:
-                       case P_ENUM:
-                               parm_table[i].def.ivalue =
-                                       *(int *)parm_table[i].ptr;
-                               break;
-                       case P_SEP:
-                               break;
-               }
-       }
-       defaults_saved = True;
-}
-
 /*******************************************************************
  Set the server type we will announce as via nmbd.
 ********************************************************************/
@@ -3003,7 +2940,10 @@ static void set_server_role(void)
                case SEC_DOMAIN:
                case SEC_ADS:
                        if (lp_domain_logons()) {
-                               server_role = ROLE_DOMAIN_PDC;
+                               if (Globals.bDomainMaster) /* auto or yes */ 
+                                       server_role = ROLE_DOMAIN_PDC;
+                               else
+                                       server_role = ROLE_DOMAIN_BDC;
                                break;
                        }
                        server_role = ROLE_DOMAIN_MEMBER;
@@ -3045,31 +2985,15 @@ static void set_server_role(void)
  False on failure.
 ***************************************************************************/
 
-BOOL lp_load(const char *pszFname, BOOL global_only, BOOL save_defaults,
-            BOOL add_ipc)
+BOOL lp_load(void)
 {
        pstring n2;
        BOOL bRetval;
        struct param_opt *data;
 
-       pstrcpy(n2, pszFname);
-       standard_sub_basic(n2,sizeof(n2));
-
-       add_to_file_list(pszFname, n2);
-
        bRetval = False;
 
-       DEBUG(2, ("lp_load: refreshing parameters from %s\n", pszFname));
-       
        bInGlobalSection = True;
-       bGlobalOnly = global_only;
-
-       init_globals();
-
-       if (save_defaults)
-       {
-               lp_save_defaults();
-       }
 
        if (Globals.param_opt != NULL) {
                struct param_opt *next;
@@ -3083,9 +3007,17 @@ BOOL lp_load(const char *pszFname, BOOL global_only, BOOL save_defaults,
                }
        }
        
+       init_globals();
+
+       pstrcpy(n2, lp_configfile());
+       standard_sub_basic(n2,sizeof(n2));
+       DEBUG(2, ("lp_load: refreshing parameters from %s\n", n2));
+       
+       add_to_file_list(lp_configfile(), n2);
+
        /* We get sections first, so have to start 'behind' to make up */
        iServiceIndex = -1;
-       bRetval = pm_process(n2, do_section, do_parameter);
+       bRetval = pm_process(n2, do_section, do_parameter, NULL);
 
        /* finish up the last section */
        DEBUG(4, ("pm_process() returned %s\n", BOOLSTR(bRetval)));
@@ -3095,21 +3027,17 @@ BOOL lp_load(const char *pszFname, BOOL global_only, BOOL save_defaults,
 
        lp_add_auto_services(lp_auto_services());
 
-       if (add_ipc) {
-               /* When 'restrict anonymous = 2' guest connections to ipc$
-                  are denied */
-               lp_add_ipc("IPC$", (lp_restrict_anonymous() < 2));
-               lp_add_ipc("ADMIN$", False);
-       }
+       /* When 'restrict anonymous = 2' guest connections to ipc$
+          are denied */
+       lp_add_hidden("IPC$", "IPC", (lp_restrict_anonymous() < 2));
+       lp_add_hidden("ADMIN$", "DISK", False);
 
        set_server_role();
        set_default_server_announce_type();
 
        bLoaded = True;
 
-       /* Now we check bWINSsupport and set szWINSserver to 127.0.0.1 */
-       /* if bWINSsupport is true and we are in the client            */
-       if (in_client && Globals.bWINSsupport) {
+       if (Globals.bWINSsupport) {
                lp_do_parameter(-1, "wins server", "127.0.0.1");
        }
 
@@ -3259,6 +3187,9 @@ static void set_default_server_announce_type(void)
 
        if (lp_host_msdfs())
                default_server_announce |= SV_TYPE_DFS_SERVER;
+
+       /* TODO: only announce us as print server when we are a print server */
+       default_server_announce |= SV_TYPE_PRINTQ_SERVER;
 }
 
 /***********************************************************
@@ -3310,7 +3241,7 @@ void lp_remove_service(int snum)
 void lp_copy_service(int snum, const char *new_name)
 {
        const char *oldname = lp_servicename(snum);
-       do_section(new_name);
+       do_section(new_name, NULL);
        if (snum >= 0) {
                snum = lp_servicenumber(new_name);
                if (snum >= 0)
@@ -3322,60 +3253,11 @@ void lp_copy_service(int snum, const char *new_name)
 /*******************************************************************
  Get the default server type we will announce as via nmbd.
 ********************************************************************/
-
 int lp_default_server_announce(void)
 {
        return default_server_announce;
 }
 
-/*******************************************************************
- Split the announce version into major and minor numbers.
-********************************************************************/
-
-int lp_major_announce_version(void)
-{
-       static BOOL got_major = False;
-       static int major_version = DEFAULT_MAJOR_VERSION;
-       const char *vers;
-       char *p;
-
-       if (got_major)
-               return major_version;
-
-       got_major = True;
-       if ((vers = lp_announce_version()) == NULL)
-               return major_version;
-
-       if ((p = strchr_m(vers, '.')) == 0)
-               return major_version;
-
-       *p = '\0';
-       major_version = atoi(vers);
-       return major_version;
-}
-
-int lp_minor_announce_version(void)
-{
-       static BOOL got_minor = False;
-       static int minor_version = DEFAULT_MINOR_VERSION;
-       const char *vers;
-       char *p;
-
-       if (got_minor)
-               return minor_version;
-
-       got_minor = True;
-       if ((vers = lp_announce_version()) == NULL)
-               return minor_version;
-
-       if ((p = strchr_m(vers, '.')) == 0)
-               return minor_version;
-
-       p++;
-       minor_version = atoi(p);
-       return minor_version;
-}
-
 const char *lp_printername(int snum)
 {
        const char *ret = _lp_printername(snum);