Added "use mmap" for HPUX.
[kai/samba.git] / source / param / loadparm.c
index 1527dd1e695ed89ad1d914aaac1c522d17f10537..a1871437ac30458482d546e77ab015766bb06e88 100644 (file)
@@ -5,6 +5,8 @@
    Copyright (C) Karl Auer 1993-1998
 
    Largely re-written by Andrew Tridgell, September 1994
+
+   Copyright (C) Simo Sorce 2001
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 
 #include "includes.h"
 
-/* Set default coding system for KANJI if none specified in Makefile. */
-/* 
- * We treat KANJI specially due to historical precedent (it was the
- * first non-english codepage added to Samba). With the new dynamic
- * codepage support this is not needed anymore.
- *
- * The define 'KANJI' is being overloaded to mean 'use kanji codepage
- * by default' and also 'this is the filename-to-disk conversion 
- * method to use'. This really should be removed and all control
- * over this left in the smb.conf parameters 'client codepage'
- * and 'coding system'.
- */
-#ifndef KANJI
-#define KANJI "sbcs"
-#endif /* KANJI */
-
 BOOL in_client = False;                /* Not in the client by default */
 BOOL bLoaded = False;
 
@@ -101,6 +87,8 @@ static BOOL defaults_saved = False;
  */
 typedef struct
 {
+       char *dos_charset;
+       char *unix_charset;
        char *szPrintcapname;
        char *szEnumPortsCommand;
        char *szAddPrinterCommand;
@@ -123,33 +111,25 @@ typedef struct
        char *szPassdbModulePath;
        char *szPasswordServer;
        char *szSocketOptions;
-       char *szValidChars;
        char *szWorkGroup;
-       char *szDomainAdminGroup;
-       char *szDomainGuestGroup;
-       char *szDomainAdminUsers;
-       char *szDomainGuestUsers;
-       char *szDomainHostsallow;
-       char *szDomainHostsdeny;
+       char **szDomainAdminGroup;
+       char **szDomainGuestGroup;
        char *szUsernameMap;
 #ifdef USING_GROUPNAME_MAP
        char *szGroupnameMap;
 #endif                         /* USING_GROUPNAME_MAP */
-       char *szCharacterSet;
-       char *szCodePageDir;
        char *szLogonScript;
        char *szLogonPath;
        char *szLogonDrive;
        char *szLogonHome;
        char *szWINSserver;
-       char *szCodingSystem;
-       char *szInterfaces;
+       char **szInterfaces;
        char *szRemoteAnnounce;
        char *szRemoteBrowseSync;
        char *szSocketAddress;
        char *szNISHomeMapName;
        char *szAnnounceVersion;        /* This is initialised in init_globals */
-       char *szNetbiosAliases;
+       char **szNetbiosAliases;
        char *szDomainOtherSIDs;
        char *szNameResolveOrder;
        char *szLdapServer;
@@ -164,6 +144,7 @@ typedef struct
        char *szDelGroupScript;
        char *szAddUserToGroupScript;
        char *szDelUserToGroupScript;
+       char *szAddMachineScript;
        char *szWINSHook;
 #ifdef WITH_UTMP
        char *szUtmpDir;
@@ -207,7 +188,6 @@ typedef struct
        int ReadSize;
        int lm_announce;
        int lm_interval;
-       int client_code_page;
        int announce_as;        /* This is initialised in init_globals */
        int machine_password_timeout;
        int change_notify_timeout;
@@ -278,6 +258,7 @@ typedef struct
        BOOL bHostMSDfs;
        BOOL bHideLocalUsers;
        BOOL bUnicode;
+       BOOL bUseMmap;
 }
 global;
 
@@ -296,9 +277,9 @@ typedef struct
        char *szPath;
        char *szUsername;
        char *szGuestaccount;
-       char *szInvalidUsers;
-       char *szValidUsers;
-       char *szAdminUsers;
+       char **szInvalidUsers;
+       char **szValidUsers;
+       char **szAdminUsers;
        char *szCopy;
        char *szInclude;
        char *szPreExec;
@@ -328,9 +309,9 @@ typedef struct
        char *comment;
        char *force_user;
        char *force_group;
-       char *readlist;
-       char *writelist;
-       char *printer_admin;
+       char **readlist;
+       char **writelist;
+       char **printer_admin;
        char *volume;
        char *fstype;
        char *szVfsObjectFile;
@@ -526,12 +507,8 @@ static int default_server_announce;
 #define NUMPARAMETERS (sizeof(parm_table) / sizeof(struct parm_struct))
 
 /* prototypes for the special type handlers */
-static BOOL handle_valid_chars(char *pszParmValue, char **ptr);
 static BOOL handle_include(char *pszParmValue, char **ptr);
 static BOOL handle_copy(char *pszParmValue, char **ptr);
-static BOOL handle_character_set(char *pszParmValue, char **ptr);
-static BOOL handle_coding_system(char *pszParmValue, char **ptr);
-static BOOL handle_client_code_page(char *pszParmValue, char **ptr);
 static BOOL handle_vfs_object(char *pszParmValue, char **ptr);
 static BOOL handle_source_env(char *pszParmValue, char **ptr);
 static BOOL handle_netbios_name(char *pszParmValue, char **ptr);
@@ -652,19 +629,18 @@ static struct enum_list enum_ssl_version[] = {
 /* note that we do not initialise the defaults union - it is not allowed in ANSI C */
 static struct parm_struct parm_table[] = {
        {"Base Options", P_SEP, P_SEPARATOR},
-       
-       {"coding system", P_STRING, P_GLOBAL, &Globals.szCodingSystem, handle_coding_system, NULL, 0},
-       {"client code page", P_INTEGER, P_GLOBAL, &Globals.client_code_page, handle_client_code_page, NULL, 0},
-       {"code page directory", P_STRING, P_GLOBAL, &Globals.szCodePageDir,   NULL,   NULL,  0},
-       {"comment", P_STRING, P_LOCAL, &sDefault.comment, NULL, NULL, FLAG_BASIC | FLAG_SHARE | FLAG_PRINT | FLAG_DOS_STRING},
-       {"path", P_STRING, P_LOCAL, &sDefault.szPath, NULL, NULL, FLAG_BASIC | FLAG_SHARE | FLAG_PRINT | FLAG_DOS_STRING},
-       {"directory", P_STRING, P_LOCAL, &sDefault.szPath, NULL, NULL, FLAG_DOS_STRING},
-       {"workgroup", P_USTRING, P_GLOBAL, &Globals.szWorkGroup, NULL, NULL, FLAG_BASIC | FLAG_DOS_STRING},
-       {"netbios name", P_UGSTRING, P_GLOBAL, global_myname, handle_netbios_name, NULL, FLAG_BASIC | FLAG_DOS_STRING},
-       {"netbios aliases", P_STRING, P_GLOBAL, &Globals.szNetbiosAliases, NULL, NULL, FLAG_DOS_STRING},
-       {"netbios scope", P_UGSTRING, P_GLOBAL, global_scope, NULL, NULL, FLAG_DOS_STRING},
-       {"server string", P_STRING, P_GLOBAL, &Globals.szServerString, NULL, NULL, FLAG_BASIC | FLAG_DOS_STRING},
-       {"interfaces", P_STRING, P_GLOBAL, &Globals.szInterfaces, NULL, NULL, FLAG_BASIC},
+
+       {"dos charset", P_STRING, P_GLOBAL, &Globals.dos_charset, NULL, NULL, 0},
+       {"unix charset", P_STRING, P_GLOBAL, &Globals.unix_charset, NULL, NULL, 0},
+       {"comment", P_STRING, P_LOCAL, &sDefault.comment, NULL, NULL, FLAG_BASIC | FLAG_SHARE | FLAG_PRINT},
+       {"path", P_STRING, P_LOCAL, &sDefault.szPath, NULL, NULL, FLAG_BASIC | FLAG_SHARE | FLAG_PRINT},
+       {"directory", P_STRING, P_LOCAL, &sDefault.szPath, NULL, NULL, 0},
+       {"workgroup", P_USTRING, P_GLOBAL, &Globals.szWorkGroup, NULL, NULL, FLAG_BASIC},
+       {"netbios name", P_UGSTRING, P_GLOBAL, global_myname, handle_netbios_name, NULL, FLAG_BASIC},
+       {"netbios aliases", P_LIST, P_GLOBAL, &Globals.szNetbiosAliases, NULL, NULL, 0},
+       {"netbios scope", P_UGSTRING, P_GLOBAL, global_scope, NULL, NULL, 0},
+       {"server string", P_STRING, P_GLOBAL, &Globals.szServerString, NULL, NULL, FLAG_BASIC },
+       {"interfaces", P_LIST, P_GLOBAL, &Globals.szInterfaces, NULL, NULL, FLAG_BASIC},
        {"bind interfaces only", P_BOOL, P_GLOBAL, &Globals.bBindInterfacesOnly, NULL, NULL, 0},
 
        {"Security Options", P_SEP, P_SEPARATOR},
@@ -705,12 +681,12 @@ static struct parm_struct parm_table[] = {
        {"users", P_STRING, P_LOCAL, &sDefault.szUsername, NULL, NULL, 0},
        
        {"guest account", P_STRING, P_LOCAL, &sDefault.szGuestaccount, NULL, NULL, FLAG_BASIC | FLAG_SHARE | FLAG_PRINT | FLAG_GLOBAL},
-       {"invalid users", P_STRING, P_LOCAL, &sDefault.szInvalidUsers, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
-       {"valid users", P_STRING, P_LOCAL, &sDefault.szValidUsers, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
-       {"admin users", P_STRING, P_LOCAL, &sDefault.szAdminUsers, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
-       {"read list", P_STRING, P_LOCAL, &sDefault.readlist, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
-       {"write list", P_STRING, P_LOCAL, &sDefault.writelist, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
-       {"printer admin", P_STRING, P_LOCAL, &sDefault.printer_admin, NULL, NULL, FLAG_GLOBAL | FLAG_PRINT},
+       {"invalid users", P_LIST, P_LOCAL, &sDefault.szInvalidUsers, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
+       {"valid users", P_LIST, P_LOCAL, &sDefault.szValidUsers, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
+       {"admin users", P_LIST, P_LOCAL, &sDefault.szAdminUsers, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
+       {"read list", P_LIST, P_LOCAL, &sDefault.readlist, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
+       {"write list", P_LIST, P_LOCAL, &sDefault.writelist, NULL, NULL, FLAG_GLOBAL | FLAG_SHARE},
+       {"printer admin", P_LIST, P_LOCAL, &sDefault.printer_admin, NULL, NULL, FLAG_GLOBAL | FLAG_PRINT},
        {"force user", P_STRING, P_LOCAL, &sDefault.force_user, NULL, NULL, FLAG_SHARE},
        {"force group", P_STRING, P_LOCAL, &sDefault.force_group, NULL, NULL, FLAG_SHARE},
        {"group", P_STRING, P_LOCAL, &sDefault.force_group, NULL, NULL, 0},
@@ -824,6 +800,7 @@ static struct parm_struct parm_table[] = {
        {"stat cache size", P_INTEGER, P_GLOBAL, &Globals.stat_cache_size, NULL, NULL, 0},
        {"strict sync", P_BOOL, P_LOCAL, &sDefault.bStrictSync, NULL, NULL, FLAG_SHARE},
        {"sync always", P_BOOL, P_LOCAL, &sDefault.bSyncAlways, NULL, NULL, FLAG_SHARE},
+       {"use mmap", P_BOOL, P_GLOBAL, &Globals.bUseMmap, NULL, NULL, 0},
        {"write cache size", P_INTEGER, P_LOCAL, &sDefault.iWriteCacheSize, NULL, NULL, FLAG_SHARE},
 
        {"Printing Options", P_SEP, P_SEPARATOR},
@@ -851,8 +828,8 @@ static struct parm_struct parm_table[] = {
        {"show add printer wizard", P_BOOL, P_GLOBAL, &Globals.bMsAddPrinterWizard, NULL, NULL, 0},
     {"os2 driver map", P_STRING, P_GLOBAL, &Globals.szOs2DriverMap, NULL, NULL, 0},
        
-       {"printer name", P_STRING, P_LOCAL, &sDefault.szPrintername, NULL, NULL, FLAG_PRINT|FLAG_DOS_STRING},
-       {"printer", P_STRING, P_LOCAL, &sDefault.szPrintername, NULL, NULL, FLAG_DOS_STRING},
+       {"printer name", P_STRING, P_LOCAL, &sDefault.szPrintername, NULL, NULL, FLAG_PRINT},
+       {"printer", P_STRING, P_LOCAL, &sDefault.szPrintername, NULL, NULL, 0},
        {"printer driver", P_STRING, P_LOCAL, &sDefault.szPrinterDriver, NULL, NULL, FLAG_PRINT},
        {"printer driver file", P_STRING, P_LOCAL, &sDefault.szDriverFile, NULL, NULL, FLAG_PRINT},
        {"printer driver location", P_STRING, P_LOCAL, &sDefault.szPrinterDriverLocation, NULL, NULL, FLAG_PRINT | FLAG_GLOBAL},
@@ -860,7 +837,6 @@ static struct parm_struct parm_table[] = {
        {"Filename Handling", P_SEP, P_SEPARATOR},
        {"strip dot", P_BOOL, P_GLOBAL, &Globals.bStripDot, NULL, NULL, 0},
        
-       {"character set", P_STRING, P_GLOBAL, &Globals.szCharacterSet, handle_character_set, NULL, 0},
        {"mangled stack", P_INTEGER, P_GLOBAL, &Globals.mangled_stack, NULL, NULL, 0},
        {"default case", P_ENUM, P_LOCAL, &sDefault.iDefaultCase, NULL, enum_case, FLAG_SHARE},
        {"case sensitive", P_BOOL, P_LOCAL, &sDefault.bCaseSensitive, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
@@ -872,9 +848,9 @@ static struct parm_struct parm_table[] = {
        {"hide dot files", P_BOOL, P_LOCAL, &sDefault.bHideDotFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
        {"hide unreadable", P_BOOL, P_LOCAL, &sDefault.bHideUnReadable, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
        {"delete veto files", P_BOOL, P_LOCAL, &sDefault.bDeleteVetoFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
-       {"veto files", P_STRING, P_LOCAL, &sDefault.szVetoFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL | FLAG_DOS_STRING},
-       {"hide files", P_STRING, P_LOCAL, &sDefault.szHideFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL | FLAG_DOS_STRING},
-       {"veto oplock files", P_STRING, P_LOCAL, &sDefault.szVetoOplockFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL | FLAG_DOS_STRING},
+       {"veto files", P_STRING, P_LOCAL, &sDefault.szVetoFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL },
+       {"hide files", P_STRING, P_LOCAL, &sDefault.szHideFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL },
+       {"veto oplock files", P_STRING, P_LOCAL, &sDefault.szVetoOplockFiles, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL },
        {"map system", P_BOOL, P_LOCAL, &sDefault.bMap_system, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
        {"map hidden", P_BOOL, P_LOCAL, &sDefault.bMap_hidden, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
        {"map archive", P_BOOL, P_LOCAL, &sDefault.bMap_archive, NULL, NULL, FLAG_SHARE | FLAG_GLOBAL},
@@ -884,8 +860,8 @@ static struct parm_struct parm_table[] = {
 
        {"Domain Options", P_SEP, P_SEPARATOR},
        
-       {"domain admin group", P_STRING, P_GLOBAL, &Globals.szDomainAdminGroup, NULL, NULL, 0},
-       {"domain guest group", P_STRING, P_GLOBAL, &Globals.szDomainGuestGroup, NULL, NULL, 0},
+       {"domain admin group", P_LIST, P_GLOBAL, &Globals.szDomainAdminGroup, NULL, NULL, 0},
+       {"domain guest group", P_LIST, P_GLOBAL, &Globals.szDomainGuestGroup, NULL, NULL, 0},
 
 #ifdef USING_GROUPNAME_MAP
        {"groupname map", P_STRING, P_GLOBAL, &Globals.szGroupnameMap, NULL, NULL, 0},
@@ -901,11 +877,12 @@ static struct parm_struct parm_table[] = {
        {"delete group script", P_STRING, P_GLOBAL, &Globals.szDelGroupScript, NULL, NULL, 0},
        {"add user to group script", P_STRING, P_GLOBAL, &Globals.szAddUserToGroupScript, NULL, NULL, 0},
        {"delete user from group script", P_STRING, P_GLOBAL, &Globals.szDelUserToGroupScript, NULL, NULL, 0},
+       {"add machine script", P_STRING, P_GLOBAL, &Globals.szAddMachineScript, NULL, NULL, 0},
 
-       {"logon script", P_STRING, P_GLOBAL, &Globals.szLogonScript, NULL, NULL, FLAG_DOS_STRING},
-       {"logon path", P_STRING, P_GLOBAL, &Globals.szLogonPath, NULL, NULL, FLAG_DOS_STRING},
+       {"logon script", P_STRING, P_GLOBAL, &Globals.szLogonScript, NULL, NULL, 0},
+       {"logon path", P_STRING, P_GLOBAL, &Globals.szLogonPath, NULL, NULL, 0},
        {"logon drive", P_STRING, P_GLOBAL, &Globals.szLogonDrive, NULL, NULL, 0},
-       {"logon home", P_STRING, P_GLOBAL, &Globals.szLogonHome, NULL, NULL, FLAG_DOS_STRING},
+       {"logon home", P_STRING, P_GLOBAL, &Globals.szLogonHome, NULL, NULL, 0},
        {"domain logons", P_BOOL, P_GLOBAL, &Globals.bDomainLogons, NULL, NULL, 0},
 
        {"Browse Options", P_SEP, P_SEPARATOR},
@@ -962,8 +939,8 @@ static struct parm_struct parm_table[] = {
        {"delete share command", P_STRING, P_GLOBAL, &Globals.szDeleteShareCommand, NULL, NULL, 0},
        
        {"config file", P_STRING, P_GLOBAL, &Globals.szConfigFile, NULL, NULL, FLAG_HIDE},
-       {"preload", P_STRING, P_GLOBAL, &Globals.szAutoServices, NULL, NULL, FLAG_DOS_STRING},
-       {"auto services", P_STRING, P_GLOBAL, &Globals.szAutoServices, NULL, NULL, FLAG_DOS_STRING},
+       {"preload", P_STRING, P_GLOBAL, &Globals.szAutoServices, NULL, NULL, 0},
+       {"auto services", P_STRING, P_GLOBAL, &Globals.szAutoServices, NULL, NULL, 0},
        {"lock dir", P_STRING, P_GLOBAL, &Globals.szLockDir, NULL, NULL, 0}, 
        {"lock directory", P_STRING, P_GLOBAL, &Globals.szLockDir, NULL, NULL, 0},
 #ifdef WITH_UTMP
@@ -977,7 +954,6 @@ static struct parm_struct parm_table[] = {
        {"default", P_STRING, P_GLOBAL, &Globals.szDefaultService, NULL, NULL, 0},
        {"message command", P_STRING, P_GLOBAL, &Globals.szMsgCommand, NULL, NULL, 0},
        {"dfree command", P_STRING, P_GLOBAL, &Globals.szDfree, NULL, NULL, 0},
-       {"valid chars", P_STRING, P_GLOBAL, &Globals.szValidChars, handle_valid_chars, NULL, 0},
        {"remote announce", P_STRING, P_GLOBAL, &Globals.szRemoteAnnounce, NULL, NULL, 0},
        {"remote browse sync", P_STRING, P_GLOBAL, &Globals.szRemoteBrowseSync, NULL, NULL, 0},
        {"socket address", P_STRING, P_GLOBAL, &Globals.szSocketAddress, NULL, NULL, 0},
@@ -997,7 +973,7 @@ static struct parm_struct parm_table[] = {
        {"root preexec close", P_BOOL, P_LOCAL, &sDefault.bRootpreexecClose, NULL, NULL, FLAG_SHARE},
        {"root postexec", P_STRING, P_LOCAL, &sDefault.szRootPostExec, NULL, NULL, FLAG_SHARE | FLAG_PRINT},
        {"available", P_BOOL, P_LOCAL, &sDefault.bAvailable, NULL, NULL, FLAG_BASIC | FLAG_SHARE | FLAG_PRINT},
-       {"volume", P_STRING, P_LOCAL, &sDefault.volume, NULL, NULL, FLAG_SHARE},
+       {"volume", P_STRING, P_LOCAL, &sDefault.volume, NULL, NULL, FLAG_SHARE },
        {"fstype", P_STRING, P_LOCAL, &sDefault.fstype, NULL, NULL, FLAG_SHARE},
        {"set directory", P_BOOLREV, P_LOCAL, &sDefault.bNo_set_dir, NULL, NULL, FLAG_SHARE},
        {"source environment", P_STRING, P_GLOBAL, &Globals.szSourceEnv, handle_source_env, NULL, 0},
@@ -1222,7 +1198,6 @@ static void init_globals(void)
        string_set(&Globals.szLogonPath, "\\\\%N\\%U\\profile");
 
        string_set(&Globals.szNameResolveOrder, "lmhosts host wins bcast");
-       string_set(&Globals.szCodePageDir, CODEPAGEDIR);
 
        Globals.bLoadPrinters = True;
        Globals.bUseRhosts = False;
@@ -1275,7 +1250,6 @@ static void init_globals(void)
        string_set(&Globals.szNISHomeMapName, "auto.home");
 #endif
 #endif
-       Globals.client_code_page = DEFAULT_CLIENT_CODE_PAGE;
        Globals.bTimeServer = False;
        Globals.bBindInterfacesOnly = False;
        Globals.bUnixPasswdSync = False;
@@ -1292,6 +1266,7 @@ static void init_globals(void)
        Globals.min_passwd_length = MINPASSWDLENGTH;    /* By Default, 5. */
        Globals.oplock_break_wait_time = 0;     /* By Default, 0 msecs. */
        Globals.enhanced_browsing = True; 
+       Globals.bUseMmap = True;
 
 #ifdef WITH_LDAP
        /* default values for ldap */
@@ -1352,13 +1327,6 @@ static void init_globals(void)
        Globals.winbind_cache_time = 15;
        Globals.bWinbindEnumUsers = True;
        Globals.bWinbindEnumGroups = True;
-
-       /*
-        * This must be done last as it checks the value in 
-        * client_code_page.
-        */
-
-       interpret_coding_system(KANJI);
 }
 
 static TALLOC_CTX *lp_talloc;
@@ -1472,11 +1440,11 @@ FN_GLOBAL_STRING(lp_logon_home, &Globals.szLogonHome)
 FN_GLOBAL_STRING(lp_remote_announce, &Globals.szRemoteAnnounce)
 FN_GLOBAL_STRING(lp_remote_browse_sync, &Globals.szRemoteBrowseSync)
 FN_GLOBAL_STRING(lp_wins_server, &Globals.szWINSserver)
-FN_GLOBAL_STRING(lp_interfaces, &Globals.szInterfaces)
+FN_GLOBAL_LIST(lp_interfaces, &Globals.szInterfaces)
 FN_GLOBAL_STRING(lp_socket_address, &Globals.szSocketAddress)
 FN_GLOBAL_STRING(lp_nis_home_map_name, &Globals.szNISHomeMapName)
 static FN_GLOBAL_STRING(lp_announce_version, &Globals.szAnnounceVersion)
-FN_GLOBAL_STRING(lp_netbios_aliases, &Globals.szNetbiosAliases)
+FN_GLOBAL_LIST(lp_netbios_aliases, &Globals.szNetbiosAliases)
 FN_GLOBAL_STRING(lp_panic_action, &Globals.szPanicAction)
 FN_GLOBAL_STRING(lp_adduser_script, &Globals.szAddUserScript)
 FN_GLOBAL_STRING(lp_deluser_script, &Globals.szDelUserScript)
@@ -1486,9 +1454,11 @@ FN_GLOBAL_STRING(lp_delgroup_script, &Globals.szDelGroupScript)
 FN_GLOBAL_STRING(lp_addusertogroup_script, &Globals.szAddUserToGroupScript)
 FN_GLOBAL_STRING(lp_deluserfromgroup_script, &Globals.szDelUserToGroupScript)
 
+FN_GLOBAL_STRING(lp_addmachine_script, &Globals.szAddMachineScript)
+
 FN_GLOBAL_STRING(lp_wins_hook, &Globals.szWINSHook)
-FN_GLOBAL_STRING(lp_domain_admin_group, &Globals.szDomainAdminGroup)
-FN_GLOBAL_STRING(lp_domain_guest_group, &Globals.szDomainGuestGroup)
+FN_GLOBAL_LIST(lp_domain_admin_group, &Globals.szDomainAdminGroup)
+FN_GLOBAL_LIST(lp_domain_guest_group, &Globals.szDomainGuestGroup)
 FN_GLOBAL_STRING(lp_winbind_uid, &Globals.szWinbindUID)
 FN_GLOBAL_STRING(lp_winbind_gid, &Globals.szWinbindGID)
 FN_GLOBAL_STRING(lp_template_homedir, &Globals.szTemplateHomedir)
@@ -1496,7 +1466,6 @@ FN_GLOBAL_STRING(lp_template_shell, &Globals.szTemplateShell)
 FN_GLOBAL_STRING(lp_winbind_separator, &Globals.szWinbindSeparator)
 FN_GLOBAL_BOOL(lp_winbind_enum_users, &Globals.bWinbindEnumUsers)
 FN_GLOBAL_BOOL(lp_winbind_enum_groups, &Globals.bWinbindEnumGroups)
-FN_GLOBAL_STRING(lp_codepagedir,&Globals.szCodePageDir)
 #ifdef WITH_LDAP
 FN_GLOBAL_STRING(lp_ldap_server, &Globals.szLdapServer)
 FN_GLOBAL_STRING(lp_ldap_suffix, &Globals.szLdapSuffix)
@@ -1567,6 +1536,7 @@ FN_GLOBAL_BOOL(lp_lanman_auth, &Globals.bLanmanAuth)
 FN_GLOBAL_BOOL(lp_host_msdfs, &Globals.bHostMSDfs)
 FN_GLOBAL_BOOL(lp_kernel_oplocks, &Globals.bKernelOplocks)
 FN_GLOBAL_BOOL(lp_enhanced_browsing, &Globals.enhanced_browsing)
+FN_GLOBAL_BOOL(lp_use_mmap, &Globals.bUseMmap)
 FN_GLOBAL_INTEGER(lp_os_level, &Globals.os_level)
 FN_GLOBAL_INTEGER(lp_max_ttl, &Globals.max_ttl)
 FN_GLOBAL_INTEGER(lp_max_wins_ttl, &Globals.max_wins_ttl)
@@ -1587,7 +1557,6 @@ FN_GLOBAL_INTEGER(lp_lpqcachetime, &Globals.lpqcachetime)
 FN_GLOBAL_INTEGER(lp_max_smbd_processes, &Globals.iMaxSmbdProcesses)
 FN_GLOBAL_INTEGER(lp_totalprintjobs, &Globals.iTotalPrintJobs)
 FN_GLOBAL_INTEGER(lp_syslog, &Globals.syslog)
-FN_GLOBAL_INTEGER(lp_client_code_page, &Globals.client_code_page)
 static FN_GLOBAL_INTEGER(lp_announce_as, &Globals.announce_as)
 FN_GLOBAL_INTEGER(lp_lm_announce, &Globals.lm_announce)
 FN_GLOBAL_INTEGER(lp_lm_interval, &Globals.lm_interval)
@@ -1609,9 +1578,9 @@ FN_LOCAL_STRING(lp_pathname, szPath)
 FN_LOCAL_STRING(lp_dontdescend, szDontdescend)
 FN_LOCAL_STRING(lp_username, szUsername)
 FN_LOCAL_STRING(lp_guestaccount, szGuestaccount)
-FN_LOCAL_STRING(lp_invalid_users, szInvalidUsers)
-FN_LOCAL_STRING(lp_valid_users, szValidUsers)
-FN_LOCAL_STRING(lp_admin_users, szAdminUsers)
+FN_LOCAL_LIST(lp_invalid_users, szInvalidUsers)
+FN_LOCAL_LIST(lp_valid_users, szValidUsers)
+FN_LOCAL_LIST(lp_admin_users, szAdminUsers)
 FN_LOCAL_STRING(lp_printcommand, szPrintcommand)
 FN_LOCAL_STRING(lp_lpqcommand, szLpqcommand)
 FN_LOCAL_STRING(lp_lprmcommand, szLprmcommand)
@@ -1629,9 +1598,9 @@ FN_LOCAL_STRING(lp_magicoutput, szMagicOutput)
 FN_LOCAL_STRING(lp_comment, comment)
 FN_LOCAL_STRING(lp_force_user, force_user)
 FN_LOCAL_STRING(lp_force_group, force_group)
-FN_LOCAL_STRING(lp_readlist, readlist)
-FN_LOCAL_STRING(lp_writelist, writelist)
-FN_LOCAL_STRING(lp_printer_admin, printer_admin)
+FN_LOCAL_LIST(lp_readlist, readlist)
+FN_LOCAL_LIST(lp_writelist, writelist)
+FN_LOCAL_LIST(lp_printer_admin, printer_admin)
 FN_LOCAL_STRING(lp_fstype, fstype)
 FN_LOCAL_STRING(lp_vfsobj, szVfsObjectFile)
 static FN_LOCAL_STRING(lp_volume, volume)
@@ -1756,7 +1725,7 @@ static void free_service(service * pservice)
                                     PTR_DIFF(parm_table[i].ptr, &sDefault)));
                else if (parm_table[i].type == P_LIST &&
                         parm_table[i].class == P_LOCAL)
-                            lp_list_free(*(char ***)
+                            lp_list_free((char ***)
                                            (((char *)pservice) +
                                             PTR_DIFF(parm_table[i].ptr, &sDefault)));
        }
@@ -1767,7 +1736,7 @@ static void free_service(service * pservice)
 
 /***************************************************************************
 add a new service to the services array initialising it with the given 
-service. name must be in DOS codepage.
+service. 
 ***************************************************************************/
 static int add_a_service(service * pservice, char *name)
 {
@@ -1822,7 +1791,7 @@ static int add_a_service(service * pservice, char *name)
 
 /***************************************************************************
 add a new home service, with the specified home directory, defaults coming 
-from service ifrom. homename must be in DOS codepage.
+from service ifrom.
 ***************************************************************************/
 BOOL lp_add_home(char *pszHomename, int iDefaultService, char *pszHomedir)
 {
@@ -1851,7 +1820,7 @@ BOOL lp_add_home(char *pszHomename, int iDefaultService, char *pszHomedir)
 }
 
 /***************************************************************************
-add a new service, based on an old one. pszService must be in DOS codepage.
+add a new service, based on an old one.
 ***************************************************************************/
 int lp_add_service(char *pszService, int iDefaultService)
 {
@@ -1894,7 +1863,6 @@ static BOOL lp_add_ipc(char *ipc_name, BOOL guest_ok)
 
 /***************************************************************************
 add a new printer service, with defaults coming from service iFrom.
-printername must be in DOS codepage.
 ***************************************************************************/
 BOOL lp_add_printer(char *pszPrintername, int iDefaultService)
 {
@@ -2117,6 +2085,7 @@ static struct file_lists
 {
        struct file_lists *next;
        char *name;
+       char *subfname;
        time_t modtime;
 }
  *file_lists = NULL;
@@ -2125,7 +2094,7 @@ static struct file_lists
 keep a linked list of all config files so we know when one has changed 
 it's date and needs to be reloaded
 ********************************************************************/
-static void add_to_file_list(char *fname)
+static void add_to_file_list(char *fname, char *subfname)
 {
        struct file_lists *f = file_lists;
 
@@ -2148,15 +2117,16 @@ static void add_to_file_list(char *fname)
                        free(f);
                        return;
                }
+               f->subfname = strdup(subfname);
+               if (!f->subfname)
+               {
+                       free(f);
+                       return;
+               }
                file_lists = f;
        }
 
-       {
-               pstring n2;
-               pstrcpy(n2, fname);
-               standard_sub_basic(n2);
-               f->modtime = file_modtime(n2);
-       }
+       f->modtime = file_modtime(subfname);
 
 }
 
@@ -2181,12 +2151,14 @@ BOOL lp_file_list_changed(void)
 
                mod_time = file_modtime(n2);
 
-               if (f->modtime != mod_time)
+               if ((f->modtime != mod_time) || (f->subfname == NULL) || (strcmp(n2, f->subfname) != 0))
                {
                        DEBUGADD(6,
                                 ("file %s modified: %s\n", n2,
                                  ctime(&mod_time)));
                        f->modtime = mod_time;
+                       free(f->subfname);
+                       f->subfname = strdup(n2);
                        return (True);
                }
                f = f->next;
@@ -2209,11 +2181,6 @@ static BOOL handle_netbios_name(char *pszParmValue, char **ptr)
        standard_sub_basic(netbios_name);
        strupper(netbios_name);
 
-       /*
-        * Convert from UNIX to DOS string - the UNIX to DOS converter
-        * isn't called on the special handlers.
-        */
-       unix_to_dos(netbios_name, True);
        pstrcpy(global_myname, netbios_name);
 
        DEBUG(4,
@@ -2254,7 +2221,7 @@ static BOOL source_env(char **lines)
                strncpy(varval, line, len);
                varval[len] = '\0';
 
-               p = strchr(line, (int)'=');
+               p = strchr_m(line, (int)'=');
                if (p == NULL)
                {
                        DEBUG(4, ("source_env: missing '=': %s\n", line));
@@ -2305,11 +2272,11 @@ static BOOL handle_source_env(char *pszParmValue, char **ptr)
 
        if (*p == '|')
        {
-               lines = file_lines_pload(p + 1, NULL, True);
+               lines = file_lines_pload(p + 1, NULL);
        }
        else
        {
-               lines = file_lines_load(fname, NULL, True);
+               lines = file_lines_load(fname, NULL);
        }
 
        if (!lines)
@@ -2341,67 +2308,6 @@ static BOOL handle_vfs_object(char *pszParmValue, char **ptr)
        return True;
 }
 
-/***************************************************************************
-  handle the interpretation of the coding system parameter
-  *************************************************************************/
-static BOOL handle_coding_system(char *pszParmValue, char **ptr)
-{
-       string_set(ptr, pszParmValue);
-       interpret_coding_system(pszParmValue);
-       return (True);
-}
-
-/***************************************************************************
- Handle the interpretation of the character set system parameter.
-***************************************************************************/
-
-static char *saved_character_set = NULL;
-
-static BOOL handle_character_set(char *pszParmValue, char **ptr)
-{
-       /* A dependency here is that the parameter client code page should be
-          set before this is called.
-        */
-       string_set(ptr, pszParmValue);
-       strupper(*ptr);
-       saved_character_set = strdup(*ptr);
-       interpret_character_set(*ptr, lp_client_code_page());
-       return (True);
-}
-
-/***************************************************************************
- Handle the interpretation of the client code page parameter.
- We handle this separately so that we can reset the character set
- parameter in case this came before 'client code page' in the smb.conf.
-***************************************************************************/
-
-static BOOL handle_client_code_page(char *pszParmValue, char **ptr)
-{
-       Globals.client_code_page = atoi(pszParmValue);
-       if (saved_character_set != NULL)
-               interpret_character_set(saved_character_set,
-                                       lp_client_code_page());
-       codepage_initialise(lp_client_code_page());
-       return (True);
-}
-
-/***************************************************************************
-handle the valid chars lines
-***************************************************************************/
-
-static BOOL handle_valid_chars(char *pszParmValue, char **ptr)
-{
-       string_set(ptr, pszParmValue);
-
-       /* A dependency here is that the parameter client code page must be
-          set before this is called - as calling codepage_initialise()
-          would overwrite the valid char lines.
-        */
-       codepage_initialise(lp_client_code_page());
-
-       add_char_string(pszParmValue);
-       return (True);
-}
 
 /***************************************************************************
 handle the include operation
@@ -2412,10 +2318,10 @@ static BOOL handle_include(char *pszParmValue, char **ptr)
        pstring fname;
        pstrcpy(fname, pszParmValue);
 
-       add_to_file_list(fname);
-
        standard_sub_basic(fname);
 
+       add_to_file_list(pszParmValue, fname);
+
        string_set(ptr, fname);
 
        if (file_exist(fname, NULL))
@@ -2651,27 +2557,19 @@ BOOL lp_do_parameter(int snum, char *pszParmName, char *pszParmValue)
 
                case P_STRING:
                        string_set(parm_ptr, pszParmValue);
-                       if (parm_table[parmnum].flags & FLAG_DOS_STRING)
-                               unix_to_dos(*(char **)parm_ptr, True);
                        break;
 
                case P_USTRING:
                        string_set(parm_ptr, pszParmValue);
-                       if (parm_table[parmnum].flags & FLAG_DOS_STRING)
-                               unix_to_dos(*(char **)parm_ptr, True);
                        strupper(*(char **)parm_ptr);
                        break;
 
                case P_GSTRING:
                        pstrcpy((char *)parm_ptr, pszParmValue);
-                       if (parm_table[parmnum].flags & FLAG_DOS_STRING)
-                               unix_to_dos((char *)parm_ptr, True);
                        break;
 
                case P_UGSTRING:
                        pstrcpy((char *)parm_ptr, pszParmValue);
-                       if (parm_table[parmnum].flags & FLAG_DOS_STRING)
-                               unix_to_dos((char *)parm_ptr, True);
                        strupper((char *)parm_ptr);
                        break;
 
@@ -2715,7 +2613,7 @@ static BOOL do_parameter(char *pszParmName, char *pszParmValue)
 /***************************************************************************
 print a parameter of the specified type
 ***************************************************************************/
-static void print_parameter(struct parm_struct *p, void *ptr, FILE * f,  char *(*dos_to_ext)(char *, BOOL))
+static void print_parameter(struct parm_struct *p, void *ptr, FILE * f)
 {
        int i;
        switch (p->type)
@@ -2756,34 +2654,23 @@ static void print_parameter(struct parm_struct *p, void *ptr, FILE * f,  char *(
                        if ((char ***)ptr && *(char ***)ptr) {
                                char **list = *(char ***)ptr;
                                
-                               if (p->flags & FLAG_DOS_STRING)
-                                       for (; *list; list++)
-                                               fprintf(f, "%s%s", dos_to_ext(*list, False),
-                                                                  ((*(list+1))?", ":""));
-                               else
-                                       for (; *list; list++)
-                                               fprintf(f, "%s%s", *list,
-                                                                  ((*(list+1))?", ":""));
+                               for (; *list; list++)
+                                       fprintf(f, "%s%s", *list,
+                                               ((*(list+1))?", ":""));
                        }
                        break;
 
                case P_GSTRING:
                case P_UGSTRING:
                        if ((char *)ptr) {
-                               if (p->flags & FLAG_DOS_STRING)
-                                       fprintf(f, "%s", dos_to_ext((char *)ptr, False));
-                               else
-                                       fprintf(f, "%s", (char *)ptr);
+                               fprintf(f, "%s", (char *)ptr);
                        }
                        break;
 
                case P_STRING:
                case P_USTRING:
                        if (*(char **)ptr) {
-                               if(p->flags & FLAG_DOS_STRING)
-                                       fprintf(f,"%s",dos_to_ext(*(char **)ptr, False));
-                               else
-                                       fprintf(f, "%s", *(char **)ptr);
+                               fprintf(f, "%s", *(char **)ptr);
                        }
                        break;
                case P_SEP:
@@ -2852,7 +2739,7 @@ void init_locals(void)
 /***************************************************************************
 Process a new section (service). At this stage all sections are services.
 Later we'll have special sections that permit server parameters to be set.
-Returns True on success, False on failure. SectionName must be in DOS codepage.
+Returns True on success, False on failure. 
 ***************************************************************************/
 static BOOL do_section(char *pszSectionName)
 {
@@ -2904,7 +2791,7 @@ static BOOL do_section(char *pszSectionName)
 
 
 /***************************************************************************
-determine if a partcular base parameter is currently set to the default value.
+determine if a partcular base parameter is currentl set to the default value.
 ***************************************************************************/
 static BOOL is_default(int i)
 {
@@ -2945,7 +2832,7 @@ static BOOL is_default(int i)
 /***************************************************************************
 Display the contents of the global structure.
 ***************************************************************************/
-static void dump_globals(FILE *f, char *(*dos_to_ext)(char *, BOOL))
+static void dump_globals(FILE *f)
 {
        int i;
        fprintf(f, "# Global parameters\n[global]\n");
@@ -2958,7 +2845,7 @@ static void dump_globals(FILE *f, char *(*dos_to_ext)(char *, BOOL))
                        if (defaults_saved && is_default(i))
                                continue;
                        fprintf(f, "\t%s = ", parm_table[i].label);
-                       print_parameter(&parm_table[i], parm_table[i].ptr, f, dos_to_ext);
+                       print_parameter(&parm_table[i], parm_table[i].ptr, f);
                        fprintf(f, "\n");
                }
 }
@@ -2979,7 +2866,7 @@ BOOL lp_is_default(int snum, struct parm_struct *parm)
 /***************************************************************************
 Display the contents of a single services record.
 ***************************************************************************/
-static void dump_a_service(service * pService, FILE * f, char *(*dos_to_ext)(char *, BOOL))
+static void dump_a_service(service * pService, FILE * f)
 {
        int i;
        if (pService != &sDefault)
@@ -3010,7 +2897,7 @@ static void dump_a_service(service * pService, FILE * f, char *(*dos_to_ext)(cha
 
                        fprintf(f, "\t%s = ", parm_table[i].label);
                        print_parameter(&parm_table[i],
-                                       ((char *)pService) + pdiff, f, dos_to_ext);
+                                       ((char *)pService) + pdiff, f);
                        fprintf(f, "\n");
                }
 }
@@ -3158,7 +3045,6 @@ void lp_add_one_printer(char *name, char *comment)
                if ((i = lp_servicenumber(name)) >= 0)
                {
                        string_set(&ServicePtrs[i]->comment, comment);
-            unix_to_dos(ServicePtrs[i]->comment, True);
                        ServicePtrs[i]->autoloaded = True;
                }
        }
@@ -3311,7 +3197,10 @@ BOOL lp_load(char *pszFname, BOOL global_only, BOOL save_defaults,
        pstring n2;
        BOOL bRetval;
 
-       add_to_file_list(pszFname);
+       pstrcpy(n2, pszFname);
+       standard_sub_basic(n2);
+
+       add_to_file_list(pszFname, n2);
 
        bRetval = False;
 
@@ -3326,9 +3215,6 @@ BOOL lp_load(char *pszFname, BOOL global_only, BOOL save_defaults,
                lp_save_defaults();
        }
 
-       pstrcpy(n2, pszFname);
-       standard_sub_basic(n2);
-
        /* We get sections first, so have to start 'behind' to make up */
        iServiceIndex = -1;
        bRetval = pm_process(n2, do_section, do_parameter);
@@ -3353,14 +3239,12 @@ BOOL lp_load(char *pszFname, BOOL global_only, BOOL save_defaults,
 
        /* 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 (in_client && Globals.bWINSsupport) {
                string_set(&Globals.szWINSserver, "127.0.0.1");
-
        }
 
+       init_iconv(Globals.unix_charset, Globals.dos_charset);
+
        return (bRetval);
 }
 
@@ -3384,7 +3268,7 @@ int lp_numservices(void)
 /***************************************************************************
 Display the contents of the services array in human-readable form.
 ***************************************************************************/
-void lp_dump(FILE *f, BOOL show_defaults, int maxtoprint, char *(*dos_to_ext)(char *, BOOL))
+void lp_dump(FILE *f, BOOL show_defaults, int maxtoprint)
 {
        int iService;
 
@@ -3393,24 +3277,24 @@ void lp_dump(FILE *f, BOOL show_defaults, int maxtoprint, char *(*dos_to_ext)(ch
                defaults_saved = False;
        }
 
-       dump_globals(f, dos_to_ext);
+       dump_globals(f);
 
-       dump_a_service(&sDefault, f, dos_to_ext);
+       dump_a_service(&sDefault, f);
 
        for (iService = 0; iService < maxtoprint; iService++)
-               lp_dump_one(f, show_defaults, iService, dos_to_ext);
+               lp_dump_one(f, show_defaults, iService);
 }
 
 /***************************************************************************
 Display the contents of one service in human-readable form.
 ***************************************************************************/
-void lp_dump_one(FILE * f, BOOL show_defaults, int snum, char *(*dos_to_ext)(char *, BOOL))
+void lp_dump_one(FILE * f, BOOL show_defaults, int snum)
 {
        if (VALID(snum))
        {
                if (ServicePtrs[snum]->szService[0] == '\0')
                        return;
-               dump_a_service(ServicePtrs[snum], f, dos_to_ext);
+               dump_a_service(ServicePtrs[snum], f);
        }
 }
 
@@ -3429,7 +3313,6 @@ int lp_servicenumber(char *pszServiceName)
                if (VALID(iService) &&
                    strequal(lp_servicename(iService), pszServiceName))
                        break;
-
        if (iService < 0)
                DEBUG(7,
                      ("lp_servicenumber: couldn't find %s\n",
@@ -3439,20 +3322,14 @@ int lp_servicenumber(char *pszServiceName)
 }
 
 /*******************************************************************
-  a useful volume label function
-  ******************************************************************/
+ A useful volume label function. Returns a string in DOS codepage.
+********************************************************************/
+
 char *volume_label(int snum)
 {
        char *ret = lp_volume(snum);
-       if (!*ret) {
-               /* lp_volume returns a unix charset - lp_servicename returns a
-                  dos codepage - convert so volume_label() always returns UNIX.
-               */
-               char *p = dos_to_unix(lp_servicename(snum), False);
-               int len = strlen(p)+1;
-               ret = (char *)talloc(lp_talloc, len);
-               memcpy(ret, p, len);
-       }
+       if (!*ret)
+               return lp_servicename(snum);
        return (ret);
 }
 
@@ -3579,7 +3456,7 @@ void lp_remove_service(int snum)
 }
 
 /*******************************************************************
-copy a service. new_name must be in dos codepage
+copy a service. 
 ********************************************************************/
 void lp_copy_service(int snum, char *new_name)
 {
@@ -3619,7 +3496,7 @@ int lp_major_announce_version(void)
        if ((vers = lp_announce_version()) == NULL)
                return major_version;
 
-       if ((p = strchr(vers, '.')) == 0)
+       if ((p = strchr_m(vers, '.')) == 0)
                return major_version;
 
        *p = '\0';
@@ -3641,7 +3518,7 @@ int lp_minor_announce_version(void)
        if ((vers = lp_announce_version()) == NULL)
                return minor_version;
 
-       if ((p = strchr(vers, '.')) == 0)
+       if ((p = strchr_m(vers, '.')) == 0)
                return minor_version;
 
        p++;
@@ -3683,12 +3560,15 @@ char **lp_list_make(char *string)
        
        if (!string || !*string) return NULL;
        s = strdup(string);
-       if (!str || !*str) return NULL;
-       str = s;
+       if (!s || !*s) {
+               DEBUG(0,("ERROR: Unable to allocate memory"));
+               return NULL;
+       }
        
        list = (char**)malloc(((sizeof(char**)) * P_LIST_ABS));
        if (!list) {
-               free (str);
+               DEBUG(0,("ERROR: Unable to allocate memory"));
+               free (s);
                return NULL;
        }
        memset (list, 0, ((sizeof(char**)) * P_LIST_ABS));
@@ -3696,6 +3576,7 @@ char **lp_list_make(char *string)
        
        num = 0;
        
+       str = s;
        while (*str)
        {
                if (!next_token(&str, tok, LIST_SEP, sizeof(pstring))) continue;
@@ -3704,8 +3585,9 @@ char **lp_list_make(char *string)
                        lsize += P_LIST_ABS;
                        rlist = (char **)realloc(list, ((sizeof(char **)) * lsize));
                        if (!rlist) {
-                               lp_list_free (list);
-                               free (str);
+                               DEBUG(0,("ERROR: Unable to allocate memory"));
+                               lp_list_free (&list);
+                               free (s);
                                return NULL;
                        }
                        else list = rlist;
@@ -3714,8 +3596,9 @@ char **lp_list_make(char *string)
                
                list[num] = strdup(tok);
                if (!list[num]) {
-                       lp_list_free (list);
-                       free (str);
+                       DEBUG(0,("ERROR: Unable to allocate memory"));
+                       lp_list_free (&list);
+                       free (s);
                        return NULL;
                }
        
@@ -3729,14 +3612,16 @@ char **lp_list_make(char *string)
 BOOL lp_list_copy(char ***dest, char **src)
 {
        char **list, **rlist;
-       char *tok;
        int num, lsize;
        
        *dest = NULL;
        if (!src) return False;
        
        list = (char**)malloc(((sizeof(char**)) * P_LIST_ABS));
-       if (!list) return False;
+       if (!list) {
+               DEBUG(0,("ERROR: Unable to allocate memory"));
+               return False;
+       }
        memset (list, 0, ((sizeof(char**)) * P_LIST_ABS));
        lsize = P_LIST_ABS;
                
@@ -3746,7 +3631,8 @@ BOOL lp_list_copy(char ***dest, char **src)
                        lsize += P_LIST_ABS;
                        rlist = (char **)realloc(list, ((sizeof(char **)) * lsize));
                        if (!rlist) {
-                               lp_list_free (list);
+                               DEBUG(0,("ERROR: Unable to allocate memory"));
+                               lp_list_free (&list);
                                return False;
                        }
                        else list = rlist;
@@ -3755,7 +3641,8 @@ BOOL lp_list_copy(char ***dest, char **src)
                
                list[num] = strdup(src[num]);
                if (!list[num]) {
-                       lp_list_free (list);
+                       DEBUG(0,("ERROR: Unable to allocate memory"));
+                       lp_list_free (&list);
                        return False;
                }
        }
@@ -3780,12 +3667,75 @@ BOOL lp_list_compare(char **list1, char **list2)
        return True;
 }
 
-void lp_list_free(char **list)
+void lp_list_free(char ***list)
 {
-       char **tlist = list;
+       char **tlist;
        
-       if (!list) return;
+       if (!list || !*list) return;
+       tlist = *list;
        for(; *tlist; tlist++) free(*tlist);
-       free (list);
+       free (*list);
+       *list = NULL;
+}
+
+BOOL lp_list_substitute(char **list, const char *pattern, const char *insert)
+{
+       char *p, *s, *t;
+       ssize_t ls, lp, li, ld, i, d;
+
+       if (!list || !*list) return False;
+       if (!pattern) return False;
+       if (!insert) return False;
+
+       lp = (ssize_t)strlen(pattern);
+       li = (ssize_t)strlen(insert);
+       ld = li -lp;
+                       
+       while (*list)
+       {
+               s = *list;
+               ls = (ssize_t)strlen(s);
+
+               while ((p = strstr(s, pattern)))
+               {
+                       t = *list;
+                       d = p -t;
+                       if (ld)
+                       {
+                               t = (char *) malloc(ls +ld +1);
+                               if (!t) {
+                                       DEBUG(0,("ERROR: Unable to allocate memory"));
+                                       return False;
+                               }
+                               memcpy(t, *list, d);
+                               memcpy(t +d +li, p +lp, ls -d -lp +1);
+                               free (*list);
+                               *list = t;
+                               ls += ld;
+                               s = t +d +li;
+                       }
+                       
+                       for (i = 0; i < li; i++) {
+                               switch (insert[i]) {
+                                       case '`':
+                                       case '"':
+                                       case '\'':
+                                       case ';':
+                                       case '$':
+                                       case '%':
+                                       case '\r':
+                                       case '\n':
+                                               t[d +i] = '_';
+                                               break;
+                                       default:
+                                               t[d +i] = insert[i];
+                               }
+                       }       
+               }
+               
+               list++;
+       }
+       
+       return True;
 }