s3:loadparm: rename lp_local_ptr() to lp_local_ptr_by_snum()
[ira/wip.git] / source3 / web / swat.c
index d97278c485911f21cddfc1a47b5b193320a6c139..b729e3b24138e96d84fa9bff6c3919142442c217 100644 (file)
@@ -7,7 +7,7 @@
    
    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
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
    
    This program is distributed in the hope that it will be useful,
@@ -16,8 +16,7 @@
    GNU General Public License for more details.
    
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 /**
  **/
 
 #include "includes.h"
-#include "../web/swat_proto.h"
+#include "web/swat_proto.h"
 
-static BOOL demo_mode = False;
-static BOOL have_write_access = False;
-static BOOL have_read_access = False;
+static int demo_mode = False;
+static int passwd_only = False;
+static bool have_write_access = False;
+static bool have_read_access = False;
 static int iNumNonAutoPrintServices = 0;
 
 /*
@@ -51,9 +51,7 @@ static int iNumNonAutoPrintServices = 0;
 #define ENABLE_USER_FLAG "enable_user_flag"
 #define RHOST "remote_host"
 
-/* we need these because we link to locking*.o */
- void become_root(void) {}
- void unbecome_root(void) {}
+#define _(x) lang_msg_rotate(talloc_tos(),x)
 
 /****************************************************************************
 ****************************************************************************/
@@ -79,13 +77,47 @@ static char *fix_backslash(const char *str)
        return newstring;
 }
 
+static const char *fix_quotes(TALLOC_CTX *ctx, const char *str)
+{
+       char *newstring = NULL;
+       char *p = NULL;
+       size_t newstring_len;
+       int quote_len = strlen("&quot;");
+
+       /* Count the number of quotes. */
+       newstring_len = 1;
+       p = (char *) str;
+       while (*p) {
+               if ( *p == '\"') {
+                       newstring_len += quote_len;
+               } else {
+                       newstring_len++;
+               }
+               ++p;
+       }
+       newstring = TALLOC_ARRAY(ctx, char, newstring_len);
+       if (!newstring) {
+               return "";
+       }
+       for (p = newstring; *str; str++) {
+               if ( *str == '\"') {
+                       strncpy( p, "&quot;", quote_len);
+                       p += quote_len;
+               } else {
+                       *p++ = *str;
+               }
+       }
+       *p = '\0';
+       return newstring;
+}
+
 static char *stripspaceupper(const char *str)
 {
        static char newstring[1024];
        char *p = newstring;
 
        while (*str) {
-               if (*str != ' ') *p++ = toupper(*str);
+               if (*str != ' ') *p++ = toupper_ascii(*str);
                ++str;
        }
        *p = '\0';
@@ -118,7 +150,8 @@ static int include_html(const char *fname)
        fd = web_open(fname, O_RDONLY, 0);
 
        if (fd == -1) {
-               d_printf("ERROR: Can't open %s\n", fname);
+               printf(_("ERROR: Can't open %s"), fname);
+               printf("\n");
                return 0;
        }
 
@@ -136,13 +169,13 @@ static int include_html(const char *fname)
 static void print_header(void)
 {
        if (!cgi_waspost()) {
-               d_printf("Expires: 0\r\n");
+               printf("Expires: 0\r\n");
        }
-       d_printf("Content-type: text/html\r\n\r\n");
+       printf("Content-type: text/html\r\n\r\n");
 
        if (!include_html("include/header.html")) {
-               d_printf("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 3.2//EN\">\n");
-               d_printf("<HTML>\n<HEAD>\n<TITLE>Samba Web Administration Tool</TITLE>\n</HEAD>\n<BODY background=\"/swat/images/background.jpg\">\n\n");
+               printf("<!DOCTYPE HTML PUBLIC \"-//W3C//DTD HTML 3.2//EN\">\n");
+               printf("<HTML>\n<HEAD>\n<TITLE>Samba Web Administration Tool</TITLE>\n</HEAD>\n<BODY background=\"/swat/images/background.jpg\">\n\n");
        }
 }
 
@@ -161,137 +194,157 @@ static void print_header(void)
    "i18n_translated_parm" class is used to change the color of the
    translated parameter with CSS.
    **************************************************************** */
-static const char* get_parm_translated(
+static const char *get_parm_translated(TALLOC_CTX *ctx,
        const char* pAnchor, const char* pHelp, const char* pLabel)
 {
-       const char* pTranslated = _(pLabel);
-       static pstring output;
-       if(strcmp(pLabel, pTranslated) != 0)
-       {
-               snprintf(output, sizeof(output),
-                 "<A HREF=\"/swat/help/smb.conf.5.html#%s\" target=\"docs\"> %s</A>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; %s <br><span class=\"i18n_translated_parm\">%s</span>",
+       const char *pTranslated = _(pLabel);
+       char *output;
+       if(strcmp(pLabel, pTranslated) != 0) {
+               output = talloc_asprintf(ctx,
+                 "<A HREF=\"/swat/help/manpages/smb.conf.5.html#%s\" target=\"docs\"> %s</A>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; %s <br><span class=\"i18n_translated_parm\">%s</span>",
                   pAnchor, pHelp, pLabel, pTranslated);
                return output;
        }
-       snprintf(output, sizeof(output), 
-         "<A HREF=\"/swat/help/smb.conf.5.html#%s\" target=\"docs\"> %s</A>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; %s",
+       output = talloc_asprintf(ctx,
+         "<A HREF=\"/swat/help/manpages/smb.conf.5.html#%s\" target=\"docs\"> %s</A>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; %s",
          pAnchor, pHelp, pLabel);
        return output;
 }
 /****************************************************************************
- finish off the page 
+ finish off the page
 ****************************************************************************/
 static void print_footer(void)
 {
        if (!include_html("include/footer.html")) {
-               d_printf("\n</BODY>\n</HTML>\n");
+               printf("\n</BODY>\n</HTML>\n");
        }
 }
 
 /****************************************************************************
-  display one editable parameter in a form 
+  display one editable parameter in a form
 ****************************************************************************/
 static void show_parameter(int snum, struct parm_struct *parm)
 {
        int i;
        void *ptr = parm->ptr;
+       char *utf8_s1, *utf8_s2;
+       size_t converted_size;
+       TALLOC_CTX *ctx = talloc_stackframe();
 
-       if (parm->class == P_LOCAL && snum >= 0) {
-               ptr = lp_local_ptr(snum, ptr);
+       if (parm->p_class == P_LOCAL && snum >= 0) {
+               ptr = lp_local_ptr_by_snum(snum, ptr);
        }
 
-       printf("<tr><td>%s</td><td>", get_parm_translated(stripspaceupper(parm->label), _("Help"), parm->label));
+       printf("<tr><td>%s</td><td>", get_parm_translated(ctx,
+                               stripspaceupper(parm->label), _("Help"), parm->label));
        switch (parm->type) {
        case P_CHAR:
-               d_printf("<input type=text size=2 name=\"parm_%s\" value=\"%c\">",
+               printf("<input type=text size=2 name=\"parm_%s\" value=\"%c\">",
                       make_parm_name(parm->label), *(char *)ptr);
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%c\'\">",
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%c\'\">",
                        _("Set Default"), make_parm_name(parm->label),(char)(parm->def.cvalue));
                break;
 
        case P_LIST:
-               d_printf("<input type=text size=40 name=\"parm_%s\" value=\"",
+               printf("<input type=text size=40 name=\"parm_%s\" value=\"",
                        make_parm_name(parm->label));
                if ((char ***)ptr && *(char ***)ptr && **(char ***)ptr) {
                        char **list = *(char ***)ptr;
                        for (;*list;list++) {
-                               d_printf("%s%s", *list, ((*(list+1))?" ":""));
+                               /* enclose in HTML encoded quotes if the string contains a space */
+                               if ( strchr_m(*list, ' ') ) {
+                                       push_utf8_allocate(&utf8_s1, *list, &converted_size);
+                                       push_utf8_allocate(&utf8_s2, ((*(list+1))?", ":""), &converted_size);
+                                       printf("&quot;%s&quot;%s", utf8_s1, utf8_s2);
+                               } else {
+                                       push_utf8_allocate(&utf8_s1, *list, &converted_size);
+                                       push_utf8_allocate(&utf8_s2, ((*(list+1))?", ":""), &converted_size);
+                                       printf("%s%s", utf8_s1, utf8_s2);
+                               }
+                               SAFE_FREE(utf8_s1);
+                               SAFE_FREE(utf8_s2);
                        }
                }
-               d_printf("\">");
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'",
+               printf("\">");
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'",
                        _("Set Default"), make_parm_name(parm->label));
                if (parm->def.lvalue) {
                        char **list = (char **)(parm->def.lvalue);
                        for (; *list; list++) {
-                               d_printf("%s%s", *list, ((*(list+1))?" ":""));
+                               /* enclose in HTML encoded quotes if the string contains a space */
+                               if ( strchr_m(*list, ' ') )
+                                       printf("&quot;%s&quot;%s", *list, ((*(list+1))?", ":""));
+                               else
+                                       printf("%s%s", *list, ((*(list+1))?", ":""));
                        }
                }
-               d_printf("\'\">");
+               printf("\'\">");
                break;
 
        case P_STRING:
        case P_USTRING:
-               d_printf("<input type=text size=40 name=\"parm_%s\" value=\"%s\">",
-                      make_parm_name(parm->label), *(char **)ptr);
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%s\'\">",
-                       _("Set Default"), make_parm_name(parm->label),fix_backslash((char *)(parm->def.svalue)));
-               break;
-
-       case P_GSTRING:
-       case P_UGSTRING:
-               d_printf("<input type=text size=40 name=\"parm_%s\" value=\"%s\">",
-                      make_parm_name(parm->label), (char *)ptr);
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%s\'\">",
+               push_utf8_allocate(&utf8_s1, *(char **)ptr, &converted_size);
+               printf("<input type=text size=40 name=\"parm_%s\" value=\"%s\">",
+                      make_parm_name(parm->label), fix_quotes(ctx, utf8_s1));
+               SAFE_FREE(utf8_s1);
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%s\'\">",
                        _("Set Default"), make_parm_name(parm->label),fix_backslash((char *)(parm->def.svalue)));
                break;
 
        case P_BOOL:
-               d_printf("<select name=\"parm_%s\">",make_parm_name(parm->label)); 
-               d_printf("<option %s>Yes", (*(BOOL *)ptr)?"selected":"");
-               d_printf("<option %s>No", (*(BOOL *)ptr)?"":"selected");
-               d_printf("</select>");
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.selectedIndex=\'%d\'\">",
-                       _("Set Default"), make_parm_name(parm->label),(BOOL)(parm->def.bvalue)?0:1);
+               printf("<select name=\"parm_%s\">",make_parm_name(parm->label)); 
+               printf("<option %s>Yes", (*(bool *)ptr)?"selected":"");
+               printf("<option %s>No", (*(bool *)ptr)?"":"selected");
+               printf("</select>");
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.selectedIndex=\'%d\'\">",
+                       _("Set Default"), make_parm_name(parm->label),(bool)(parm->def.bvalue)?0:1);
                break;
 
        case P_BOOLREV:
-               d_printf("<select name=\"parm_%s\">",make_parm_name(parm->label)); 
-               d_printf("<option %s>Yes", (*(BOOL *)ptr)?"":"selected");
-               d_printf("<option %s>No", (*(BOOL *)ptr)?"selected":"");
-               d_printf("</select>");
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.selectedIndex=\'%d\'\">",
-                       _("Set Default"), make_parm_name(parm->label),(BOOL)(parm->def.bvalue)?1:0);
+               printf("<select name=\"parm_%s\">",make_parm_name(parm->label)); 
+               printf("<option %s>Yes", (*(bool *)ptr)?"":"selected");
+               printf("<option %s>No", (*(bool *)ptr)?"selected":"");
+               printf("</select>");
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.selectedIndex=\'%d\'\">",
+                       _("Set Default"), make_parm_name(parm->label),(bool)(parm->def.bvalue)?1:0);
                break;
 
        case P_INTEGER:
-               d_printf("<input type=text size=8 name=\"parm_%s\" value=%d>", make_parm_name(parm->label), *(int *)ptr);
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%d\'\">",
+               printf("<input type=text size=8 name=\"parm_%s\" value=\"%d\">", make_parm_name(parm->label), *(int *)ptr);
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%d\'\">",
                        _("Set Default"), make_parm_name(parm->label),(int)(parm->def.ivalue));
                break;
 
-       case P_OCTAL:
-               d_printf("<input type=text size=8 name=\"parm_%s\" value=%s>", make_parm_name(parm->label), octal_string(*(int *)ptr));
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.value=\'%s\'\">",
-                      _("Set Default"), make_parm_name(parm->label),
-                      octal_string((int)(parm->def.ivalue)));
+       case P_OCTAL: {
+               char *o;
+               o = octal_string(*(int *)ptr);
+               printf("<input type=text size=8 name=\"parm_%s\" value=%s>",
+                      make_parm_name(parm->label), o);
+               TALLOC_FREE(o);
+               o = octal_string((int)(parm->def.ivalue));
+               printf("<input type=button value=\"%s\" "
+                      "onClick=\"swatform.parm_%s.value=\'%s\'\">",
+                      _("Set Default"), make_parm_name(parm->label), o);
+               TALLOC_FREE(o);
                break;
+       }
 
        case P_ENUM:
-               d_printf("<select name=\"parm_%s\">",make_parm_name(parm->label)); 
+               printf("<select name=\"parm_%s\">",make_parm_name(parm->label)); 
                for (i=0;parm->enum_list[i].name;i++) {
                        if (i == 0 || parm->enum_list[i].value != parm->enum_list[i-1].value) {
-                               d_printf("<option %s>%s",(*(int *)ptr)==parm->enum_list[i].value?"selected":"",parm->enum_list[i].name);
+                               printf("<option %s>%s",(*(int *)ptr)==parm->enum_list[i].value?"selected":"",parm->enum_list[i].name);
                        }
                }
-               d_printf("</select>");
-               d_printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.selectedIndex=\'%d\'\">",
+               printf("</select>");
+               printf("<input type=button value=\"%s\" onClick=\"swatform.parm_%s.selectedIndex=\'%d\'\">",
                        _("Set Default"), make_parm_name(parm->label),enum_index((int)(parm->def.ivalue),parm->enum_list));
                break;
        case P_SEP:
                break;
        }
-       d_printf("</td></tr>\n");
+       printf("</td></tr>\n");
+       TALLOC_FREE(ctx);
 }
 
 /****************************************************************************
@@ -305,9 +358,9 @@ static void show_parameters(int snum, int allparameters, unsigned int parm_filte
        const char *last_heading = NULL;
 
        while ((parm = lp_next_parameter(snum, &i, allparameters))) {
-               if (snum < 0 && parm->class == P_LOCAL && !(parm->flags & FLAG_GLOBAL))
+               if (snum < 0 && parm->p_class == P_LOCAL && !(parm->flags & FLAG_GLOBAL))
                        continue;
-               if (parm->class == P_SEPARATOR) {
+               if (parm->p_class == P_SEPARATOR) {
                        heading = parm->label;
                        continue;
                }
@@ -316,12 +369,13 @@ static void show_parameters(int snum, int allparameters, unsigned int parm_filte
                        if (printers & !(parm->flags & FLAG_PRINT)) continue;
                        if (!printers & !(parm->flags & FLAG_SHARE)) continue;
                }
-               if (parm_filter == FLAG_BASIC) {
+
+               if (!( parm_filter & FLAG_ADVANCED )) {
                        if (!(parm->flags & FLAG_BASIC)) {
-                               void *ptr = parm->ptr;
+                                       void *ptr = parm->ptr;
 
-                               if (parm->class == P_LOCAL && snum >= 0) {
-                                       ptr = lp_local_ptr(snum, ptr);
+                               if (parm->p_class == P_LOCAL && snum >= 0) {
+                                       ptr = lp_local_ptr_by_snum(snum, ptr);
                                }
 
                                switch (parm->type) {
@@ -330,7 +384,8 @@ static void show_parameters(int snum, int allparameters, unsigned int parm_filte
                                        break;
 
                                case P_LIST:
-                                       if (!str_list_compare(*(char ***)ptr, (char **)(parm->def.lvalue))) continue;
+                                       if (!str_list_equal(*(const char ***)ptr, 
+                                                           (const char **)(parm->def.lvalue))) continue;
                                        break;
 
                                case P_STRING:
@@ -338,14 +393,9 @@ static void show_parameters(int snum, int allparameters, unsigned int parm_filte
                                        if (!strcmp(*(char **)ptr,(char *)(parm->def.svalue))) continue;
                                        break;
 
-                               case P_GSTRING:
-                               case P_UGSTRING:
-                                       if (!strcmp((char *)ptr,(char *)(parm->def.svalue))) continue;
-                                       break;
-
                                case P_BOOL:
                                case P_BOOLREV:
-                                       if (*(BOOL *)ptr == (BOOL)(parm->def.bvalue)) continue;
+                                       if (*(bool *)ptr == (bool)(parm->def.bvalue)) continue;
                                        break;
 
                                case P_INTEGER:
@@ -359,18 +409,17 @@ static void show_parameters(int snum, int allparameters, unsigned int parm_filte
                                        break;
                                case P_SEP:
                                        continue;
-                               }
+                                       }
                        }
                        if (printers && !(parm->flags & FLAG_PRINT)) continue;
                }
-               if (parm_filter == FLAG_WIZARD) {
-                       if (!((parm->flags & FLAG_WIZARD))) continue;
-               }
-               if (parm_filter == FLAG_ADVANCED) {
-                       if (!((parm->flags & FLAG_ADVANCED))) continue;
-               }
+
+               if ((parm_filter & FLAG_WIZARD) && !(parm->flags & FLAG_WIZARD)) continue;
+               
+               if ((parm_filter & FLAG_ADVANCED) && !(parm->flags & FLAG_ADVANCED)) continue;
+               
                if (heading && heading != last_heading) {
-                       d_printf("<tr><td></td></tr><tr><td><b><u>%s</u></b></td></tr>\n", _(heading));
+                       printf("<tr><td></td></tr><tr><td><b><u>%s</u></b></td></tr>\n", _(heading));
                        last_heading = heading;
                }
                show_parameter(snum, parm);
@@ -380,22 +429,25 @@ static void show_parameters(int snum, int allparameters, unsigned int parm_filte
 /****************************************************************************
   load the smb.conf file into loadparm.
 ****************************************************************************/
-static BOOL load_config(BOOL save_def)
+static bool load_config(bool save_def)
 {
-       lp_resetnumservices();
-       return lp_load(dyn_CONFIGFILE,False,save_def,False);
+       return lp_load(get_dyn_CONFIGFILE(),False,save_def,False,True);
 }
 
 /****************************************************************************
   write a config file 
 ****************************************************************************/
-static void write_config(FILE *f, BOOL show_defaults)
+static void write_config(FILE *f, bool show_defaults)
 {
+       TALLOC_CTX *ctx = talloc_stackframe();
+
        fprintf(f, "# Samba config file created using SWAT\n");
        fprintf(f, "# from %s (%s)\n", cgi_remote_host(), cgi_remote_addr());
-       fprintf(f, "# Date: %s\n\n", timestring(False));
+       fprintf(f, "# Date: %s\n\n", current_timestring(ctx, False));
        
        lp_dump(f, show_defaults, iNumNonAutoPrintServices);
+
+       TALLOC_FREE(ctx);
 }
 
 /****************************************************************************
@@ -406,9 +458,10 @@ static int save_reload(int snum)
        FILE *f;
        struct stat st;
 
-       f = sys_fopen(dyn_CONFIGFILE,"w");
+       f = sys_fopen(get_dyn_CONFIGFILE(),"w");
        if (!f) {
-               d_printf("failed to open %s for writing\n", dyn_CONFIGFILE);
+               printf(_("failed to open %s for writing"), get_dyn_CONFIGFILE());
+               printf("\n");
                return 0;
        }
 
@@ -418,7 +471,7 @@ static int save_reload(int snum)
 #if defined HAVE_FCHMOD
                fchmod(fileno(f), S_IWUSR | S_IRUSR | S_IRGRP | S_IROTH);
 #else
-               chmod(dyn_CONFIGFILE, S_IWUSR | S_IRUSR | S_IRGRP | S_IROTH);
+               chmod(get_dyn_CONFIGFILE(), S_IWUSR | S_IRUSR | S_IRGRP | S_IROTH);
 #endif
        }
 
@@ -427,10 +480,11 @@ static int save_reload(int snum)
                lp_dump_one(f, False, snum);
        fclose(f);
 
-       lp_killunused(NULL);
+       lp_kill_all_services();
 
        if (!load_config(False)) {
-                d_printf("Can't reload %s\n", dyn_CONFIGFILE);
+                printf(_("Can't reload %s"), get_dyn_CONFIGFILE());
+               printf("\n");
                 return 0;
         }
        iNumNonAutoPrintServices = lp_numservices();
@@ -447,7 +501,7 @@ static void commit_parameter(int snum, struct parm_struct *parm, const char *v)
        int i;
        char *s;
 
-       if (snum < 0 && parm->class == P_LOCAL) {
+       if (snum < 0 && parm->p_class == P_LOCAL) {
                /* this handles the case where we are changing a local
                   variable globally. We need to change the parameter in 
                   all shares where it is currently set to the default */
@@ -469,14 +523,17 @@ static void commit_parameters(int snum)
 {
        int i = 0;
        struct parm_struct *parm;
-       pstring label;
+       char *label;
        const char *v;
 
        while ((parm = lp_next_parameter(snum, &i, 1))) {
-               slprintf(label, sizeof(label)-1, "parm_%s", make_parm_name(parm->label));
-               if ((v = cgi_variable(label))) {
-                       if (parm->flags & FLAG_HIDE) continue;
-                       commit_parameter(snum, parm, v); 
+               if (asprintf(&label, "parm_%s", make_parm_name(parm->label)) > 0) {
+                       if ((v = cgi_variable(label)) != NULL) {
+                               if (parm->flags & FLAG_HIDE)
+                                       continue;
+                               commit_parameter(snum, parm, v);
+                       }
+                       SAFE_FREE(label);
                }
        }
 }
@@ -486,7 +543,7 @@ static void commit_parameters(int snum)
 ****************************************************************************/
 static void image_link(const char *name, const char *hlink, const char *src)
 {
-       d_printf("<A HREF=\"%s/%s\"><img border=\"0\" src=\"/swat/%s\" alt=\"%s\"></A>\n", 
+       printf("<A HREF=\"%s/%s\"><img border=\"0\" src=\"/swat/%s\" alt=\"%s\"></A>\n", 
               cgi_baseurl(), hlink, src, name);
 }
 
@@ -499,7 +556,8 @@ static void show_main_buttons(void)
        char *p;
        
        if ((p = cgi_user_name()) && strcmp(p, "root")) {
-               d_printf(_("Logged in as <b>%s</b><p>\n"), p);
+               printf(_("Logged in as <b>%s</b>"), p);
+               printf("<p>\n");
        }
 
        image_link(_("Home"), "", "images/home.gif");
@@ -509,13 +567,14 @@ static void show_main_buttons(void)
                image_link(_("Printers"), "printers", "images/printers.gif");
                image_link(_("Wizard"), "wizard", "images/wizard.gif");
        }
-       if (have_read_access) {
+   /* root always gets all buttons, otherwise look for -P */
+       if ( have_write_access || (!passwd_only && have_read_access) ) {
                image_link(_("Status"), "status", "images/status.gif");
                image_link(_("View Config"), "viewconfig", "images/viewconfig.gif");
        }
        image_link(_("Password Management"), "passwd", "images/passwd.gif");
 
-       d_printf("<HR>\n");
+       printf("<HR>\n");
 }
 
 /****************************************************************************
@@ -523,11 +582,13 @@ static void show_main_buttons(void)
  ****************************************************************************/
 static void ViewModeBoxes(int mode)
 {
-       d_printf("<p>%s\n", _("Configuration View:&nbsp"));
-       d_printf("<input type=radio name=\"ViewMode\" value=0 %s>Basic\n", (mode == 0) ? "checked" : "");
-       d_printf("<input type=radio name=\"ViewMode\" value=1 %s>Advanced\n", (mode == 1) ? "checked" : "");
-       d_printf("<input type=radio name=\"ViewMode\" value=2 %s>Developer\n", (mode == 2) ? "checked" : "");
-       d_printf("</p><br>\n");
+       printf("<p>%s:&nbsp;\n", _("Current View Is"));
+       printf("<input type=radio name=\"ViewMode\" value=0 %s>%s\n", ((mode == 0) ? "checked" : ""), _("Basic"));
+       printf("<input type=radio name=\"ViewMode\" value=1 %s>%s\n", ((mode == 1) ? "checked" : ""), _("Advanced"));
+       printf("<br>%s:&nbsp;\n", _("Change View To"));
+       printf("<input type=submit name=\"BasicMode\" value=\"%s\">\n", _("Basic"));
+       printf("<input type=submit name=\"AdvMode\" value=\"%s\">\n", _("Advanced"));
+       printf("</p><br>\n");
 }
 
 /****************************************************************************
@@ -535,7 +596,11 @@ static void ViewModeBoxes(int mode)
 ****************************************************************************/
 static void welcome_page(void)
 {
-       include_html("help/welcome.html");
+       if (file_exist("help/welcome.html")) {
+               include_html("help/welcome.html");
+       } else {
+               include_html("help/welcome-no-samba-doc.html");
+       }
 }
 
 /****************************************************************************
@@ -549,19 +614,19 @@ static void viewconfig_page(void)
                full_view = 1;
        }
 
-       d_printf("<H2>%s</H2>\n", _("Current Config"));
-       d_printf("<form method=post>\n");
+       printf("<H2>%s</H2>\n", _("Current Config"));
+       printf("<form method=post>\n");
 
        if (full_view) {
-               d_printf("<input type=submit name=\"normal_view\" value=\"%s\">\n", _("Normal View"));
+               printf("<input type=submit name=\"normal_view\" value=\"%s\">\n", _("Normal View"));
        } else {
-               d_printf("<input type=submit name=\"full_view\" value=\"%s\">\n", _("Full View"));
+               printf("<input type=submit name=\"full_view\" value=\"%s\">\n", _("Full View"));
        }
 
-       d_printf("<p><pre>");
+       printf("<p><pre>");
        write_config(stdout, full_view);
-       d_printf("</pre>");
-       d_printf("</form>\n");
+       printf("</pre>");
+       printf("</form>\n");
 }
 
 /****************************************************************************
@@ -574,26 +639,26 @@ static void wizard_params_page(void)
        /* Here we first set and commit all the parameters that were selected
           in the previous screen. */
 
-       d_printf("<H2>Wizard Parameter Edit Page</H2>\n");
+       printf("<H2>%s</H2>\n", _("Wizard Parameter Edit Page"));
 
        if (cgi_variable("Commit")) {
                commit_parameters(GLOBAL_SECTION_SNUM);
                save_reload(0);
        }
 
-       d_printf("<form name=\"swatform\" method=post action=wizard_params>\n");
+       printf("<form name=\"swatform\" method=post action=wizard_params>\n");
 
        if (have_write_access) {
-               d_printf("<input type=submit name=\"Commit\" value=\"Commit Changes\">\n");
+               printf("<input type=submit name=\"Commit\" value=\"Commit Changes\">\n");
        }
 
-       d_printf("<input type=reset name=\"Reset Values\" value=\"Reset\">\n");
-       d_printf("<p>\n");
+       printf("<input type=reset name=\"Reset Values\" value=\"Reset\">\n");
+       printf("<p>\n");
        
-       d_printf("<table>\n");
+       printf("<table>\n");
        show_parameters(GLOBAL_SECTION_SNUM, 1, parm_filter, 0);
-       d_printf("</table>\n");
-       d_printf("</form>\n");
+       printf("</table>\n");
+       printf("</form>\n");
 }
 
 /****************************************************************************
@@ -603,7 +668,7 @@ static void rewritecfg_file(void)
 {
        commit_parameters(GLOBAL_SECTION_SNUM);
        save_reload(0);
-       d_printf("<H2>Note: smb.conf %s</H2>\n", _("file has been read and rewritten"));
+       printf("<H2>%s</H2>\n", _("Note: smb.conf file has been read and rewritten"));
 }
 
 /****************************************************************************
@@ -629,10 +694,10 @@ static void wizard_page(void)
        }
 
        if (cgi_variable("Commit")){
-               SerType = atoi(cgi_variable("ServerType"));
-               winstype = atoi(cgi_variable("WINSType"));
+               SerType = atoi(cgi_variable_nonull("ServerType"));
+               winstype = atoi(cgi_variable_nonull("WINSType"));
                have_home = lp_servicenumber(HOMES_NAME);
-               HomeExpo = atoi(cgi_variable("HomeExpo"));
+               HomeExpo = atoi(cgi_variable_nonull("HomeExpo"));
 
                /* Plain text passwords are too badly broken - use encrypted passwords only */
                lp_do_parameter( GLOBAL_SECTION_SNUM, "encrypt passwords", "Yes");
@@ -665,23 +730,22 @@ static void wizard_page(void)
                                break;
                        case 2:
                                lp_do_parameter( GLOBAL_SECTION_SNUM, "wins support", "No" );
-                               lp_do_parameter( GLOBAL_SECTION_SNUM, "wins server", cgi_variable("WINSAddr"));
+                               lp_do_parameter( GLOBAL_SECTION_SNUM, "wins server", cgi_variable_nonull("WINSAddr"));
                                break;
                }
 
                /* Have to create Homes share? */
                if ((HomeExpo == 1) && (have_home == -1)) {
-                       pstring unix_share;
-                       
-                       pstrcpy(unix_share,HOMES_NAME);
+                       const char *unix_share = HOMES_NAME;
+
                        load_config(False);
                        lp_copy_service(GLOBAL_SECTION_SNUM, unix_share);
-                       iNumNonAutoPrintServices = lp_numservices();
                        have_home = lp_servicenumber(HOMES_NAME);
                        lp_do_parameter( have_home, "read only", "No");
                        lp_do_parameter( have_home, "valid users", "%S");
                        lp_do_parameter( have_home, "browseable", "No");
                        commit_parameters(have_home);
+                       save_reload(have_home);
                }
 
                /* Need to Delete Homes share? */
@@ -700,7 +764,6 @@ static void wizard_page(void)
                        winstype = 1;
                if (lp_wins_server_list() && strlen(*lp_wins_server_list()))
                        winstype = 2;
-               
 
                /* Do we have a homes share? */
                have_home = lp_servicenumber(HOMES_NAME);
@@ -711,57 +774,66 @@ static void wizard_page(void)
        role = lp_server_role();
        
        /* Here we go ... */
-       d_printf("<H2>Samba Configuration Wizard</H2>\n");
-       d_printf("<form method=post action=wizard>\n");
+       printf("<H2>%s</H2>\n", _("Samba Configuration Wizard"));
+       printf("<form method=post action=wizard>\n");
 
        if (have_write_access) {
-               d_printf(_("The \"Rewrite smb.conf file\" button will clear the smb.conf file of all default values and of comments.\n"));
-               d_printf(_("The same will happen if you press the commit button."));
-               d_printf("<br><br>");
-               d_printf("<center>");
-               d_printf("<input type=submit name=\"Rewrite\" value=%s> &nbsp;&nbsp;",_("Rewrite smb.conf file"));
-               d_printf("<input type=submit name=\"Commit\" value=%s> &nbsp;&nbsp;",_("Commit"));
-               d_printf("<input type=submit name=\"GetWizardParams\" value=%s>", _("Edit Parameter Values"));
-               d_printf("</center>");
-       }
-
-       d_printf("<hr>");
-       d_printf("<center><table border=0>");
-       d_printf("<tr><td><b>%s</b></td>\n", "Server Type:&nbsp;");
-       d_printf("<td><input type=radio name=\"ServerType\" value=0 %s> Stand Alone&nbsp;</td>", (role == ROLE_STANDALONE) ? "checked" : "");
-       d_printf("<td><input type=radio name=\"ServerType\" value=1 %s> Domain Member&nbsp;</td>", (role == ROLE_DOMAIN_MEMBER) ? "checked" : ""); 
-       d_printf("<td><input type=radio name=\"ServerType\" value=2 %s> Domain Controller&nbsp;</td>", (role == ROLE_DOMAIN_PDC) ? "checked" : "");
-       d_printf("</tr>");
+               printf("%s\n", _("The \"Rewrite smb.conf file\" button will clear the smb.conf file of all default values and of comments."));
+               printf("%s", _("The same will happen if you press the commit button."));
+               printf("<br><br>\n");
+               printf("<center>");
+               printf("<input type=submit name=\"Rewrite\" value=\"%s\"> &nbsp;&nbsp;",_("Rewrite smb.conf file"));
+               printf("<input type=submit name=\"Commit\" value=\"%s\"> &nbsp;&nbsp;",_("Commit"));
+               printf("<input type=submit name=\"GetWizardParams\" value=\"%s\">", _("Edit Parameter Values"));
+               printf("</center>\n");
+       }
+
+       printf("<hr>");
+       printf("<center><table border=0>");
+       printf("<tr><td><b>%s:&nbsp;</b></td>\n", _("Server Type"));
+       printf("<td><input type=radio name=\"ServerType\" value=\"0\" %s> %s&nbsp;</td>", ((role == ROLE_STANDALONE) ? "checked" : ""), _("Stand Alone"));
+       printf("<td><input type=radio name=\"ServerType\" value=\"1\" %s> %s&nbsp;</td>", ((role == ROLE_DOMAIN_MEMBER) ? "checked" : ""), _("Domain Member")); 
+       printf("<td><input type=radio name=\"ServerType\" value=\"2\" %s> %s&nbsp;</td>", ((role == ROLE_DOMAIN_PDC) ? "checked" : ""), _("Domain Controller"));
+       printf("</tr>\n");
        if (role == ROLE_DOMAIN_BDC) {
-               d_printf("<tr><td></td><td colspan=3><font color=\"#ff0000\">Unusual Type in smb.conf - Please Select New Mode</font></td></tr>");
+               printf("<tr><td></td><td colspan=3><font color=\"#ff0000\">%s</font></td></tr>\n", _("Unusual Type in smb.conf - Please Select New Mode"));
        }
-       d_printf("<tr><td><b>%s</b></td>\n", "Configure WINS As:&nbsp;");
-       d_printf("<td><input type=radio name=\"WINSType\" value=0 %s> Not Used&nbsp;</td>", (winstype == 0) ? "checked" : "");
-       d_printf("<td><input type=radio name=\"WINSType\" value=1 %s> Server for client use&nbsp;</td>", (winstype == 1) ? "checked" : "");
-       d_printf("<td><input type=radio name=\"WINSType\" value=2 %s> Client of another WINS server&nbsp;</td>", (winstype == 2) ? "checked" : "");
-       d_printf("<tr><td></td><td></td><td></td><td>Remote WINS Server&nbsp;<input type=text size=\"16\" name=\"WINSAddr\" value=\"%s\"></td></tr>",lp_wins_server_list());
+       printf("<tr><td><b>%s:&nbsp;</b></td>\n", _("Configure WINS As"));
+       printf("<td><input type=radio name=\"WINSType\" value=\"0\" %s> %s&nbsp;</td>", ((winstype == 0) ? "checked" : ""), _("Not Used"));
+       printf("<td><input type=radio name=\"WINSType\" value=\"1\" %s> %s&nbsp;</td>", ((winstype == 1) ? "checked" : ""), _("Server for client use"));
+       printf("<td><input type=radio name=\"WINSType\" value=\"2\" %s> %s&nbsp;</td>", ((winstype == 2) ? "checked" : ""), _("Client of another WINS server"));
+       printf("</tr>\n");
+       printf("<tr><td></td><td></td><td></td><td>%s&nbsp;<input type=text size=\"16\" name=\"WINSAddr\" value=\"", _("Remote WINS Server"));
+
+       /* Print out the list of wins servers */
+       if(lp_wins_server_list()) {
+               int i;
+               const char **wins_servers = lp_wins_server_list();
+               for(i = 0; wins_servers[i]; i++) printf("%s ", wins_servers[i]);
+       }
+       
+       printf("\"></td></tr>\n");
        if (winstype == 3) {
-               d_printf("<tr><td></td><td colspan=3><font color=\"#ff0000\">Error: WINS Server Mode and WINS Support both set in smb.conf</font></td></tr>");
-               d_printf("<tr><td></td><td colspan=3><font color=\"#ff0000\">Please Select desired WINS mode above.</font></td></tr>");
-       }
-       d_printf("</tr>");
-       d_printf("<tr><td><b>%s</b></td>\n","Expose Home Directories:&nbsp;");
-       d_printf("<td><input type=radio name=\"HomeExpo\" value=1 %s> Yes</td>", (have_home == -1) ? "" : "checked ");
-       d_printf("<td><input type=radio name=\"HomeExpo\" value=0 %s> No</td>", (have_home == -1 ) ? "checked" : "");
-       d_printf("<td></td></tr>");
+               printf("<tr><td></td><td colspan=3><font color=\"#ff0000\">%s</font></td></tr>\n", _("Error: WINS Server Mode and WINS Support both set in smb.conf"));
+               printf("<tr><td></td><td colspan=3><font color=\"#ff0000\">%s</font></td></tr>\n", _("Please Select desired WINS mode above."));
+       }
+       printf("<tr><td><b>%s:&nbsp;</b></td>\n", _("Expose Home Directories"));
+       printf("<td><input type=radio name=\"HomeExpo\" value=\"1\" %s> Yes</td>", (have_home == -1) ? "" : "checked ");
+       printf("<td><input type=radio name=\"HomeExpo\" value=\"0\" %s> No</td>", (have_home == -1 ) ? "checked" : "");
+       printf("<td></td></tr>\n");
        
        /* Enable this when we are ready ....
-        * d_printf("<tr><td><b>%s</b></td>\n","Is Print Server:&nbsp;");
-        * d_printf("<td><input type=radio name=\"PtrSvr\" value=1 %s> Yes</td>");
-        * d_printf("<td><input type=radio name=\"PtrSvr\" value=0 %s> No</td>");
-        * d_printf("<td></td></tr>");
+        * printf("<tr><td><b>%s:&nbsp;</b></td>\n", _("Is Print Server"));
+        * printf("<td><input type=radio name=\"PtrSvr\" value=\"1\" %s> Yes</td>");
+        * printf("<td><input type=radio name=\"PtrSvr\" value=\"0\" %s> No</td>");
+        * printf("<td></td></tr>\n");
         */
        
-       d_printf("</table></center>");
-       d_printf("<hr>");
+       printf("</table></center>");
+       printf("<hr>");
 
-       d_printf(_("The above configuration options will set multiple parameters and will generally assist with rapid Samba deployment.\n"));
-       d_printf("</form>\n");
+       printf("%s\n", _("The above configuration options will set multiple parameters and will generally assist with rapid Samba deployment."));
+       printf("</form>\n");
 }
 
 
@@ -773,7 +845,7 @@ static void globals_page(void)
        unsigned int parm_filter = FLAG_BASIC;
        int mode = 0;
 
-       d_printf("<H2>%s</H2>\n", _("Global Variables"));
+       printf("<H2>%s</H2>\n", _("Global Parameters"));
 
        if (cgi_variable("Commit")) {
                commit_parameters(GLOBAL_SECTION_SNUM);
@@ -781,9 +853,13 @@ static void globals_page(void)
        }
 
        if ( cgi_variable("ViewMode") )
-               mode = atoi(cgi_variable("ViewMode"));
+               mode = atoi(cgi_variable_nonull("ViewMode"));
+       if ( cgi_variable("BasicMode"))
+               mode = 0;
+       if ( cgi_variable("AdvMode"))
+               mode = 1;
 
-       d_printf("<form name=\"swatform\" method=post action=globals>\n");
+       printf("<form name=\"swatform\" method=post action=globals>\n");
 
        ViewModeBoxes( mode );
        switch ( mode ) {
@@ -793,47 +869,46 @@ static void globals_page(void)
                case 1:
                        parm_filter = FLAG_ADVANCED;
                        break;
-               case 2:
-                       parm_filter = FLAG_DEVELOPER;
-                       break;
        }
-       d_printf("<br>\n");
+       printf("<br>\n");
        if (have_write_access) {
-               d_printf("<input type=submit name=\"Commit\" value=\"%s\">\n",
+               printf("<input type=submit name=\"Commit\" value=\"%s\">\n",
                        _("Commit Changes"));
        }
 
-       d_printf("<input type=reset name=\"Reset Values\" value=\"%s\">\n", 
+       printf("<input type=reset name=\"Reset Values\" value=\"%s\">\n", 
                 _("Reset Values"));
 
-       d_printf("<p>\n");
-       d_printf("<table>\n");
+       printf("<p>\n");
+       printf("<table>\n");
        show_parameters(GLOBAL_SECTION_SNUM, 1, parm_filter, 0);
-       d_printf("</table>\n");
-       d_printf("</form>\n");
+       printf("</table>\n");
+       printf("</form>\n");
 }
 
 /****************************************************************************
-  display a shares editing page. share is in unix codepage, and must be in
-  dos codepage. FIXME !!! JRA.
+  display a shares editing page. share is in unix codepage, 
 ****************************************************************************/
 static void shares_page(void)
 {
        const char *share = cgi_variable("share");
        char *s;
+       char *utf8_s;
        int snum = -1;
        int i;
        int mode = 0;
        unsigned int parm_filter = FLAG_BASIC;
+       size_t converted_size;
 
        if (share)
                snum = lp_servicenumber(share);
 
-       d_printf("<H2>%s</H2>\n", _("Share Parameters"));
+       printf("<H2>%s</H2>\n", _("Share Parameters"));
 
        if (cgi_variable("Commit") && snum >= 0) {
                commit_parameters(snum);
                save_reload(0);
+               snum = lp_servicenumber(share);
        }
 
        if (cgi_variable("Delete") && snum >= 0) {
@@ -844,18 +919,27 @@ static void shares_page(void)
        }
 
        if (cgi_variable("createshare") && (share=cgi_variable("newshare"))) {
-               load_config(False);
-               lp_copy_service(GLOBAL_SECTION_SNUM, share);
-               iNumNonAutoPrintServices = lp_numservices();
-               save_reload(0);
                snum = lp_servicenumber(share);
+               if (snum < 0) {
+                       load_config(False);
+                       lp_copy_service(GLOBAL_SECTION_SNUM, share);
+                       snum = lp_servicenumber(share);
+                       save_reload(snum);
+                       snum = lp_servicenumber(share);
+               }
        }
 
-       d_printf("<FORM name=\"swatform\" method=post>\n");
+       printf("<FORM name=\"swatform\" method=post>\n");
+
+       printf("<table>\n");
 
-       d_printf("<table>\n");
        if ( cgi_variable("ViewMode") )
-               mode = atoi(cgi_variable("ViewMode"));
+               mode = atoi(cgi_variable_nonull("ViewMode"));
+       if ( cgi_variable("BasicMode"))
+               mode = 0;
+       if ( cgi_variable("AdvMode"))
+               mode = 1;
+
        ViewModeBoxes( mode );
        switch ( mode ) {
                case 0:
@@ -864,94 +948,96 @@ static void shares_page(void)
                case 1:
                        parm_filter = FLAG_ADVANCED;
                        break;
-               case 2:
-                       parm_filter = FLAG_DEVELOPER;
-                       break;
        }
-       d_printf("<br><tr>\n");
-       d_printf("<td><input type=submit name=selectshare value=\"%s\"></td>\n", _("Choose Share"));
-       d_printf("<td><select name=share>\n");
+       printf("<br><tr>\n");
+       printf("<td><input type=submit name=selectshare value=\"%s\"></td>\n", _("Choose Share"));
+       printf("<td><select name=share>\n");
        if (snum < 0)
-               d_printf("<option value=\" \"> \n");
+               printf("<option value=\" \"> \n");
        for (i=0;i<lp_numservices();i++) {
                s = lp_servicename(i);
                if (s && (*s) && strcmp(s,"IPC$") && !lp_print_ok(i)) {
-                       d_printf("<option %s value=\"%s\">%s\n", 
+                       push_utf8_allocate(&utf8_s, s, &converted_size);
+                       printf("<option %s value=\"%s\">%s\n", 
                               (share && strcmp(share,s)==0)?"SELECTED":"",
-                              s, s);
+                              utf8_s, utf8_s);
+                       SAFE_FREE(utf8_s);
                }
        }
-       d_printf("</select></td>\n");
+       printf("</select></td>\n");
        if (have_write_access) {
-               d_printf("<td><input type=submit name=\"Delete\" value=\"%s\"></td>\n", _("Delete Share"));
+               printf("<td><input type=submit name=\"Delete\" value=\"%s\"></td>\n", _("Delete Share"));
        }
-       d_printf("</tr>\n");
-       d_printf("</table>");
-       d_printf("<table>");
+       printf("</tr>\n");
+       printf("</table>");
+       printf("<table>");
        if (have_write_access) {
-               d_printf("<tr>\n");
-               d_printf("<td><input type=submit name=createshare value=\"%s\"></td>\n", _("Create Share"));
-               d_printf("<td><input type=text size=30 name=newshare></td></tr>\n");
+               printf("<tr>\n");
+               printf("<td><input type=submit name=createshare value=\"%s\"></td>\n", _("Create Share"));
+               printf("<td><input type=text size=30 name=newshare></td></tr>\n");
        }
-       d_printf("</table>");
+       printf("</table>");
 
 
        if (snum >= 0) {
                if (have_write_access) {
-                       d_printf("<input type=submit name=\"Commit\" value=\"%s\">\n", _("Commit Changes"));
+                       printf("<input type=submit name=\"Commit\" value=\"%s\">\n", _("Commit Changes"));
                }
 
-               d_printf("<input type=reset name=\"Reset Values\" value=\"%s\">\n", _("Reset Values"));
-               d_printf("<p>\n");
+               printf("<input type=reset name=\"Reset Values\" value=\"%s\">\n", _("Reset Values"));
+               printf("<p>\n");
        }
 
        if (snum >= 0) {
-               d_printf("<table>\n");
+               printf("<table>\n");
                show_parameters(snum, 1, parm_filter, 0);
-               d_printf("</table>\n");
+               printf("</table>\n");
        }
 
-       d_printf("</FORM>\n");
+       printf("</FORM>\n");
 }
 
 /*************************************************************
 change a password either locally or remotely
 *************************************************************/
-static BOOL change_password(const char *remote_machine, const char *user_name, 
+static bool change_password(const char *remote_machine, const char *user_name, 
                            const char *old_passwd, const char *new_passwd, 
                                int local_flags)
 {
-       BOOL ret = False;
-       pstring err_str;
-       pstring msg_str;
+       NTSTATUS ret;
+       char *err_str = NULL;
+       char *msg_str = NULL;
 
        if (demo_mode) {
-               d_printf("%s<p>", _("password change in demo mode rejected\n"));
+               printf("%s\n<p>", _("password change in demo mode rejected"));
                return False;
        }
        
        if (remote_machine != NULL) {
-               ret = remote_password_change(remote_machine, user_name, old_passwd, 
-                                                                        new_passwd, err_str, sizeof(err_str));
-               if(*err_str)
-                       d_printf("%s\n<p>", err_str);
-               return ret;
+               ret = remote_password_change(remote_machine, user_name,
+                                            old_passwd, new_passwd, &err_str);
+               if (err_str != NULL)
+                       printf("%s\n<p>", err_str);
+               SAFE_FREE(err_str);
+               return NT_STATUS_IS_OK(ret);
        }
 
-       if(!initialize_password_db(True)) {
-               d_printf("Can't setup password database vectors.\n<p>");
+       if(!initialize_password_db(True, NULL)) {
+               printf("%s\n<p>", _("Can't setup password database vectors."));
                return False;
        }
        
-       ret = local_password_change(user_name, local_flags, new_passwd, err_str, sizeof(err_str),
-                                        msg_str, sizeof(msg_str));
+       ret = local_password_change(user_name, local_flags, new_passwd,
+                                       &err_str, &msg_str);
 
-       if(*msg_str)
-               d_printf("%s\n<p>", msg_str);
-       if(*err_str)
-               d_printf("%s\n<p>", err_str);
+       if(msg_str)
+               printf("%s\n<p>", msg_str);
+       if(err_str)
+               printf("%s\n<p>", err_str);
 
-       return ret;
+       SAFE_FREE(msg_str);
+       SAFE_FREE(err_str);
+       return NT_STATUS_IS_OK(ret);
 }
 
 /****************************************************************************
@@ -960,12 +1046,12 @@ static BOOL change_password(const char *remote_machine, const char *user_name,
 static void chg_passwd(void)
 {
        const char *host;
-       BOOL rslt;
+       bool rslt;
        int local_flags = 0;
 
        /* Make sure users name has been specified */
-       if (strlen(cgi_variable(SWAT_USER)) == 0) {
-               d_printf("<p>%s", _(" Must specify \"User Name\" \n"));
+       if (strlen(cgi_variable_nonull(SWAT_USER)) == 0) {
+               printf("<p>%s\n", _(" Must specify \"User Name\" "));
                return;
        }
 
@@ -979,28 +1065,28 @@ static void chg_passwd(void)
                 * If current user is not root, make sure old password has been specified 
                 * If REMOTE change, even root must provide old password 
                 */
-               if (((!am_root()) && (strlen( cgi_variable(OLD_PSWD)) <= 0)) ||
-                   ((cgi_variable(CHG_R_PASSWD_FLAG)) &&  (strlen( cgi_variable(OLD_PSWD)) <= 0))) {
-                       d_printf("<p>%s", _(" Must specify \"Old Password\" \n"));
+               if (((!am_root()) && (strlen( cgi_variable_nonull(OLD_PSWD)) <= 0)) ||
+                   ((cgi_variable(CHG_R_PASSWD_FLAG)) &&  (strlen( cgi_variable_nonull(OLD_PSWD)) <= 0))) {
+                       printf("<p>%s\n", _(" Must specify \"Old Password\" "));
                        return;
                }
 
                /* If changing a users password on a remote hosts we have to know what host */
-               if ((cgi_variable(CHG_R_PASSWD_FLAG)) && (strlen( cgi_variable(RHOST)) <= 0)) {
-                       d_printf("<p>%s", _(" Must specify \"Remote Machine\" \n"));
+               if ((cgi_variable(CHG_R_PASSWD_FLAG)) && (strlen( cgi_variable_nonull(RHOST)) <= 0)) {
+                       printf("<p>%s\n", _(" Must specify \"Remote Machine\" "));
                        return;
                }
 
                /* Make sure new passwords have been specified */
-               if ((strlen( cgi_variable(NEW_PSWD)) <= 0) ||
-                   (strlen( cgi_variable(NEW2_PSWD)) <= 0)) {
-                       d_printf("<p>%s", _(" Must specify \"New, and Re-typed Passwords\" \n"));
+               if ((strlen( cgi_variable_nonull(NEW_PSWD)) <= 0) ||
+                   (strlen( cgi_variable_nonull(NEW2_PSWD)) <= 0)) {
+                       printf("<p>%s\n", _(" Must specify \"New, and Re-typed Passwords\" "));
                        return;
                }
 
                /* Make sure new passwords was typed correctly twice */
-               if (strcmp(cgi_variable(NEW_PSWD), cgi_variable(NEW2_PSWD)) != 0) {
-                       d_printf("<p>%s", _(" Re-typed password didn't match new password\n"));
+               if (strcmp(cgi_variable_nonull(NEW_PSWD), cgi_variable_nonull(NEW2_PSWD)) != 0) {
+                       printf("<p>%s\n", _(" Re-typed password didn't match new password "));
                        return;
                }
        }
@@ -1018,21 +1104,26 @@ static void chg_passwd(void)
         */
 
        local_flags |= (cgi_variable(ADD_USER_FLAG) ? LOCAL_ADD_USER : 0);
+       local_flags |= (cgi_variable(ADD_USER_FLAG) ?  LOCAL_SET_PASSWORD : 0);
+       local_flags |= (cgi_variable(CHG_S_PASSWD_FLAG) ? LOCAL_SET_PASSWORD : 0);
        local_flags |= (cgi_variable(DELETE_USER_FLAG) ? LOCAL_DELETE_USER : 0);
        local_flags |= (cgi_variable(ENABLE_USER_FLAG) ? LOCAL_ENABLE_USER : 0);
        local_flags |= (cgi_variable(DISABLE_USER_FLAG) ? LOCAL_DISABLE_USER : 0);
+       
 
        rslt = change_password(host,
-                              cgi_variable(SWAT_USER),
-                              cgi_variable(OLD_PSWD), cgi_variable(NEW_PSWD),
+                              cgi_variable_nonull(SWAT_USER),
+                              cgi_variable_nonull(OLD_PSWD), cgi_variable_nonull(NEW_PSWD),
                                   local_flags);
 
-       if(local_flags == 0) {
-               d_printf("<p>");
+       if(cgi_variable(CHG_S_PASSWD_FLAG)) {
+               printf("<p>");
                if (rslt == True) {
-                       d_printf(_(" The passwd for '%s' has been changed. \n"), cgi_variable(SWAT_USER));
+                       printf(_(" The passwd for '%s' has been changed."), cgi_variable_nonull(SWAT_USER));
+                       printf("\n");
                } else {
-                       d_printf(_(" The passwd for '%s' has NOT been changed. \n"), cgi_variable(SWAT_USER));
+                       printf(_(" The passwd for '%s' has NOT been changed."), cgi_variable_nonull(SWAT_USER));
+                       printf("\n");
                }
        }
        
@@ -1051,48 +1142,48 @@ static void passwd_page(void)
         * changed the User box text to another users name, remember it.
         */
        if (cgi_variable(SWAT_USER)) {
-               new_name = cgi_variable(SWAT_USER);
+               new_name = cgi_variable_nonull(SWAT_USER);
        } 
 
        if (!new_name) new_name = "";
 
-       d_printf("<H2>%s</H2>\n", _("Server Password Management"));
+       printf("<H2>%s</H2>\n", _("Server Password Management"));
 
-       d_printf("<FORM name=\"swatform\" method=post>\n");
+       printf("<FORM name=\"swatform\" method=post>\n");
 
-       d_printf("<table>\n");
+       printf("<table>\n");
 
        /* 
         * Create all the dialog boxes for data collection
         */
-       d_printf("<tr><td>%s</td>\n", _(" User Name : "));
-       d_printf("<td><input type=text size=30 name=%s value=%s></td></tr> \n", SWAT_USER, new_name);
+       printf("<tr><td> %s : </td>\n", _("User Name"));
+       printf("<td><input type=text size=30 name=%s value=%s></td></tr> \n", SWAT_USER, new_name);
        if (!am_root()) {
-               d_printf("<tr><td>%s</td>\n", _(" Old Password : "));
-               d_printf("<td><input type=password size=30 name=%s></td></tr> \n",OLD_PSWD);
+               printf("<tr><td> %s : </td>\n", _("Old Password"));
+               printf("<td><input type=password size=30 name=%s></td></tr> \n",OLD_PSWD);
        }
-       d_printf("<tr><td>%s</td>\n", _(" New Password : "));
-       d_printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW_PSWD);
-       d_printf("<tr><td>%s</td>\n", _(" Re-type New Password : "));
-       d_printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW2_PSWD);
-       d_printf("</table>\n");
+       printf("<tr><td> %s : </td>\n", _("New Password"));
+       printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW_PSWD);
+       printf("<tr><td> %s : </td>\n", _("Re-type New Password"));
+       printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW2_PSWD);
+       printf("</table>\n");
 
        /*
         * Create all the control buttons for requesting action
         */
-       d_printf("<input type=submit name=%s value=\"%s\">\n", 
+       printf("<input type=submit name=%s value=\"%s\">\n", 
               CHG_S_PASSWD_FLAG, _("Change Password"));
        if (demo_mode || am_root()) {
-               d_printf("<input type=submit name=%s value=\"%s\">\n",
+               printf("<input type=submit name=%s value=\"%s\">\n",
                       ADD_USER_FLAG, _("Add New User"));
-               d_printf("<input type=submit name=%s value=\"%s\">\n",
+               printf("<input type=submit name=%s value=\"%s\">\n",
                       DELETE_USER_FLAG, _("Delete User"));
-               d_printf("<input type=submit name=%s value=\"%s\">\n", 
+               printf("<input type=submit name=%s value=\"%s\">\n", 
                       DISABLE_USER_FLAG, _("Disable User"));
-               d_printf("<input type=submit name=%s value=\"%s\">\n", 
+               printf("<input type=submit name=%s value=\"%s\">\n", 
                       ENABLE_USER_FLAG, _("Enable User"));
        }
-       d_printf("<p></FORM>\n");
+       printf("<p></FORM>\n");
 
        /*
         * Do some work if change, add, disable or enable was
@@ -1103,35 +1194,35 @@ static void passwd_page(void)
                chg_passwd();           
        }
 
-       d_printf("<H2>%s</H2>\n", _("Client/Server Password Management"));
+       printf("<H2>%s</H2>\n", _("Client/Server Password Management"));
 
-       d_printf("<FORM name=\"swatform\" method=post>\n");
+       printf("<FORM name=\"swatform\" method=post>\n");
 
-       d_printf("<table>\n");
+       printf("<table>\n");
 
        /* 
         * Create all the dialog boxes for data collection
         */
-       d_printf("<tr><td>%s</td>\n", _(" User Name : "));
-       d_printf("<td><input type=text size=30 name=%s value=%s></td></tr>\n",SWAT_USER, new_name);
-       d_printf("<tr><td>%s</td>\n", _(" Old Password : "));
-       d_printf("<td><input type=password size=30 name=%s></td></tr>\n",OLD_PSWD);
-       d_printf("<tr><td>%s</td>\n", _(" New Password : "));
-       d_printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW_PSWD);
-       d_printf("<tr><td>%s</td>\n", _(" Re-type New Password : "));
-       d_printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW2_PSWD);
-       d_printf("<tr><td>%s</td>\n", _(" Remote Machine : "));
-       d_printf("<td><input type=text size=30 name=%s></td></tr>\n",RHOST);
-
-       d_printf("</table>");
+       printf("<tr><td> %s : </td>\n", _("User Name"));
+       printf("<td><input type=text size=30 name=%s value=%s></td></tr>\n",SWAT_USER, new_name);
+       printf("<tr><td> %s : </td>\n", _("Old Password"));
+       printf("<td><input type=password size=30 name=%s></td></tr>\n",OLD_PSWD);
+       printf("<tr><td> %s : </td>\n", _("New Password"));
+       printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW_PSWD);
+       printf("<tr><td> %s : </td>\n", _("Re-type New Password"));
+       printf("<td><input type=password size=30 name=%s></td></tr>\n",NEW2_PSWD);
+       printf("<tr><td> %s : </td>\n", _("Remote Machine"));
+       printf("<td><input type=text size=30 name=%s></td></tr>\n",RHOST);
+
+       printf("</table>");
 
        /*
         * Create all the control buttons for requesting action
         */
-       d_printf("<input type=submit name=%s value=\"%s\">", 
+       printf("<input type=submit name=%s value=\"%s\">", 
               CHG_R_PASSWD_FLAG, _("Change Password"));
 
-       d_printf("<p></FORM>\n");
+       printf("<p></FORM>\n");
 
        /*
         * Do some work if a request has been made to change the
@@ -1159,13 +1250,13 @@ static void printers_page(void)
        if (share)
                snum = lp_servicenumber(share);
 
-        d_printf("<H2>%s</H2>\n", _("Printer Parameters"));
+        printf("<H2>%s</H2>\n", _("Printer Parameters"));
  
-        d_printf("<H3>%s</H3>\n", _("Important Note:"));
-        d_printf(_("Printer names marked with [*] in the Choose Printer drop-down box "));
-        d_printf(_("are autoloaded printers from "));
-        d_printf("<A HREF=\"/swat/help/smb.conf.5.html#printcapname\" target=\"docs\">%s</A>\n", _("Printcap Name"));
-        d_printf(_("Attempting to delete these printers from SWAT will have no effect.\n"));
+        printf("<H3>%s</H3>\n", _("Important Note:"));
+        printf(_("Printer names marked with [*] in the Choose Printer drop-down box "));
+        printf(_("are autoloaded printers from "));
+        printf("<A HREF=\"/swat/help/smb.conf.5.html#printcapname\" target=\"docs\">%s</A>\n", _("Printcap Name"));
+        printf("%s\n", _("Attempting to delete these printers from SWAT will have no effect."));
 
        if (cgi_variable("Commit") && snum >= 0) {
                commit_parameters(snum);
@@ -1173,6 +1264,7 @@ static void printers_page(void)
                    save_reload(snum);
                else
                    save_reload(0);
+               snum = lp_servicenumber(share);
        }
 
        if (cgi_variable("Delete") && snum >= 0) {
@@ -1183,19 +1275,26 @@ static void printers_page(void)
        }
 
        if (cgi_variable("createshare") && (share=cgi_variable("newshare"))) {
-               load_config(False);
-               lp_copy_service(GLOBAL_SECTION_SNUM, share);
-               iNumNonAutoPrintServices = lp_numservices();
-               snum = lp_servicenumber(share);
-               lp_do_parameter(snum, "print ok", "Yes");
-               save_reload(0);
                snum = lp_servicenumber(share);
+               if (snum < 0 || snum >= iNumNonAutoPrintServices) {
+                       load_config(False);
+                       lp_copy_service(GLOBAL_SECTION_SNUM, share);
+                       snum = lp_servicenumber(share);
+                       lp_do_parameter(snum, "print ok", "Yes");
+                       save_reload(snum);
+                       snum = lp_servicenumber(share);
+               }
        }
 
-       d_printf("<FORM name=\"swatform\" method=post>\n");
+       printf("<FORM name=\"swatform\" method=post>\n");
 
        if ( cgi_variable("ViewMode") )
-               mode = atoi(cgi_variable("ViewMode"));
+               mode = atoi(cgi_variable_nonull("ViewMode"));
+        if ( cgi_variable("BasicMode"))
+                mode = 0;
+        if ( cgi_variable("AdvMode"))
+                mode = 1;
+
        ViewModeBoxes( mode );
        switch ( mode ) {
                case 0:
@@ -1204,74 +1303,98 @@ static void printers_page(void)
                case 1:
                        parm_filter = FLAG_ADVANCED;
                        break;
-               case 2:
-                       parm_filter = FLAG_DEVELOPER;
-                       break;
        }
-       d_printf("<table>\n");
-       d_printf("<tr><td><input type=submit name=selectshare value=\"%s\"></td>\n", _("Choose Printer"));
-       d_printf("<td><select name=share>\n");
+       printf("<table>\n");
+       printf("<tr><td><input type=submit name=\"selectshare\" value=\"%s\"></td>\n", _("Choose Printer"));
+       printf("<td><select name=\"share\">\n");
        if (snum < 0 || !lp_print_ok(snum))
-               d_printf("<option value=\" \"> \n");
+               printf("<option value=\" \"> \n");
        for (i=0;i<lp_numservices();i++) {
                s = lp_servicename(i);
                if (s && (*s) && strcmp(s,"IPC$") && lp_print_ok(i)) {
                     if (i >= iNumNonAutoPrintServices)
-                        d_printf("<option %s value=\"%s\">[*]%s\n",
+                        printf("<option %s value=\"%s\">[*]%s\n",
                                (share && strcmp(share,s)==0)?"SELECTED":"",
                                s, s);
                     else
-                       d_printf("<option %s value=\"%s\">%s\n", 
+                       printf("<option %s value=\"%s\">%s\n", 
                               (share && strcmp(share,s)==0)?"SELECTED":"",
                               s, s);
                }
        }
-       d_printf("</select></td>");
+       printf("</select></td>");
        if (have_write_access) {
-               d_printf("<td><input type=submit name=\"Delete\" value=\"%s\"></td>\n", _("Delete Printer"));
+               printf("<td><input type=submit name=\"Delete\" value=\"%s\"></td>\n", _("Delete Printer"));
        }
-       d_printf("</tr>");
-       d_printf("</table>\n");
+       printf("</tr>");
+       printf("</table>\n");
 
        if (have_write_access) {
-               d_printf("<table>\n");
-               d_printf("<tr><td><input type=submit name=createshare value=\"%s\"></td>\n", _("Create Printer"));
-               d_printf("<td><input type=text size=30 name=newshare></td></tr>\n");
-               d_printf("</table>");
+               printf("<table>\n");
+               printf("<tr><td><input type=submit name=\"createshare\" value=\"%s\"></td>\n", _("Create Printer"));
+               printf("<td><input type=text size=30 name=\"newshare\"></td></tr>\n");
+               printf("</table>");
        }
 
 
        if (snum >= 0) {
                if (have_write_access) {
-                       d_printf("<input type=submit name=\"Commit\" value=\"%s\">\n", _("Commit Changes"));
+                       printf("<input type=submit name=\"Commit\" value=\"%s\">\n", _("Commit Changes"));
                }
-               d_printf("<input type=reset name=\"Reset Values\" value=\"%s\">\n", _("Reset Values"));
-               d_printf("<p>\n");
+               printf("<input type=reset name=\"Reset Values\" value=\"%s\">\n", _("Reset Values"));
+               printf("<p>\n");
        }
 
        if (snum >= 0) {
-               d_printf("<table>\n");
+               printf("<table>\n");
                show_parameters(snum, 1, parm_filter, 1);
-               d_printf("</table>\n");
+               printf("</table>\n");
        }
-       d_printf("</FORM>\n");
+       printf("</FORM>\n");
 }
 
+/*
+  when the _() translation macro is used there is no obvious place to free
+  the resulting string and there is no easy way to give a static pointer.
+  All we can do is rotate between some static buffers and hope a single d_printf()
+  doesn't have more calls to _() than the number of buffers
+*/
+
+const char *lang_msg_rotate(TALLOC_CTX *ctx, const char *msgid)
+{
+       const char *msgstr;
+       const char *ret;
+
+       msgstr = lang_msg(msgid);
+       if (!msgstr) {
+               return msgid;
+       }
+
+       ret = talloc_strdup(ctx, msgstr);
+
+       lang_msg_free(msgstr);
+       if (!ret) {
+               return msgid;
+       }
+
+       return ret;
+}
 
 /**
  * main function for SWAT.
  **/
  int main(int argc, char *argv[])
 {
-       int opt;
        const char *page;
        poptContext pc;
        struct poptOption long_options[] = {
                POPT_AUTOHELP
                { "disable-authentication", 'a', POPT_ARG_VAL, &demo_mode, True, "Disable authentication (demo mode)" },
+               { "password-menu-only", 'P', POPT_ARG_VAL, &passwd_only, True, "Show only change password menu" }, 
                POPT_COMMON_SAMBA
                POPT_TABLEEND
        };
+       TALLOC_CTX *frame = talloc_stackframe();
 
        fault_setup(NULL);
        umask(S_IWGRP | S_IWOTH);
@@ -1299,32 +1422,35 @@ static void printers_page(void)
 
        /* Parse command line options */
 
-       while((opt = poptGetNextOpt(pc)) != -1) { }
+       while(poptGetNextOpt(pc) != -1) { }
 
        poptFreeContext(pc);
 
+       load_case_tables();
+
        setup_logging(argv[0],False);
        load_config(True);
+       load_interfaces();
        iNumNonAutoPrintServices = lp_numservices();
        load_printers();
 
-       cgi_setup(dyn_SWATDIR, !demo_mode);
+       cgi_setup(get_dyn_SWATDIR(), !demo_mode);
 
        print_header();
 
        cgi_load_variables();
 
-       if (!file_exist(dyn_CONFIGFILE, NULL)) {
+       if (!file_exist(get_dyn_CONFIGFILE())) {
                have_read_access = True;
                have_write_access = True;
        } else {
                /* check if the authenticated user has write access - if not then
                   don't show write options */
-               have_write_access = (access(dyn_CONFIGFILE,W_OK) == 0);
+               have_write_access = (access(get_dyn_CONFIGFILE(),W_OK) == 0);
 
                /* if the user doesn't have read access to smb.conf then
                   don't let them view it */
-               have_read_access = (access(dyn_CONFIGFILE,R_OK) == 0);
+               have_read_access = (access(get_dyn_CONFIGFILE(),R_OK) == 0);
        }
 
        show_main_buttons();
@@ -1355,6 +1481,8 @@ static void printers_page(void)
        }
 
        print_footer();
+
+       TALLOC_FREE(frame);
        return 0;
 }