s3:smbd: rename api_RNetServerEnum => api_RNetServerEnum2
[ira/wip.git] / source3 / smbd / lanman.c
index 03411b8dd9fcba32d88dca7aad63bcaa1bae6cc6..b969cd82c11ac41b8e40650776f527872bdb2780 100644 (file)
@@ -6,20 +6,19 @@
 
    SMB Version handling
    Copyright (C) John H Terpstra 1995-1998
-   
+
    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,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    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/>.
    */
 /*
    This file handles the named pipe and mailslot calls
@@ -27,9 +26,9 @@
    */
 
 #include "includes.h"
-
-extern struct current_user current_user;
-extern userdom_struct current_user_info;
+#include "smbd/globals.h"
+#include "../librpc/gen_ndr/cli_samr.h"
+#include "../librpc/gen_ndr/srv_samr.h"
 
 #ifdef CHECK_TYPES
 #undef CHECK_TYPES
@@ -51,39 +50,73 @@ extern userdom_struct current_user_info;
 
 #define SHPWLEN 8              /* share password length */
 
-static BOOL api_Unsupported(connection_struct *conn, uint16 vuid,
+/* Limit size of ipc replies */
+
+static char *smb_realloc_limit(void *ptr, size_t size)
+{
+       char *val;
+
+       size = MAX((size),4*1024);
+       val = (char *)SMB_REALLOC(ptr,size);
+       if (val) {
+               memset(val,'\0',size);
+       }
+       return val;
+}
+
+static bool api_Unsupported(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt, int mprcnt,
                                char **rdata, char **rparam,
                                int *rdata_len, int *rparam_len);
 
-static BOOL api_TooSmall(connection_struct *conn, uint16 vuid, char *param, char *data,
+static bool api_TooSmall(connection_struct *conn, uint16 vuid, char *param, char *data,
                         int mdrcnt, int mprcnt,
                         char **rdata, char **rparam,
                         int *rdata_len, int *rparam_len);
 
 
-static int CopyExpanded(connection_struct *conn, 
-                       int snum, char **dst, char *src, int *n)
+static int CopyExpanded(connection_struct *conn,
+                       int snum, char **dst, char *src, int *p_space_remaining)
 {
-       pstring buf;
+       TALLOC_CTX *ctx = talloc_tos();
+       char *buf = NULL;
        int l;
 
-       if (!src || !dst || !n || !(*dst)) {
+       if (!src || !dst || !p_space_remaining || !(*dst) ||
+                       *p_space_remaining <= 0) {
                return 0;
        }
 
-       StrnCpy(buf,src,sizeof(buf)/2);
-       pstring_sub(buf,"%S",lp_servicename(snum));
-       standard_sub_advanced(lp_servicename(SNUM(conn)), conn->user,
-                             conn->connectpath, conn->gid,
-                             get_current_username(),
-                             current_user_info.domain,
-                             buf, sizeof(buf));
-       l = push_ascii(*dst,buf,*n, STR_TERMINATE);
+       buf = talloc_strdup(ctx, src);
+       if (!buf) {
+               *p_space_remaining = 0;
+               return 0;
+       }
+       buf = talloc_string_sub(ctx, buf,"%S",lp_servicename(snum));
+       if (!buf) {
+               *p_space_remaining = 0;
+               return 0;
+       }
+       buf = talloc_sub_advanced(ctx,
+                               lp_servicename(SNUM(conn)),
+                               conn->server_info->unix_name,
+                               conn->connectpath,
+                               conn->server_info->utok.gid,
+                               conn->server_info->sanitized_username,
+                               pdb_get_domain(conn->server_info->sam_account),
+                               buf);
+       if (!buf) {
+               *p_space_remaining = 0;
+               return 0;
+       }
+       l = push_ascii(*dst,buf,*p_space_remaining, STR_TERMINATE);
+       if (l == -1) {
+               return 0;
+       }
        (*dst) += l;
-       (*n) -= l;
+       (*p_space_remaining) -= l;
        return l;
 }
 
@@ -94,6 +127,9 @@ static int CopyAndAdvance(char **dst, char *src, int *n)
                return 0;
        }
        l = push_ascii(*dst,src,*n, STR_TERMINATE);
+       if (l == -1) {
+               return 0;
+       }
        (*dst) += l;
        (*n) -= l;
        return l;
@@ -101,41 +137,64 @@ static int CopyAndAdvance(char **dst, char *src, int *n)
 
 static int StrlenExpanded(connection_struct *conn, int snum, char *s)
 {
-       pstring buf;
+       TALLOC_CTX *ctx = talloc_tos();
+       char *buf = NULL;
        if (!s) {
                return 0;
        }
-       StrnCpy(buf,s,sizeof(buf)/2);
-       pstring_sub(buf,"%S",lp_servicename(snum));
-       standard_sub_advanced(lp_servicename(SNUM(conn)), conn->user,
-                             conn->connectpath, conn->gid,
-                             get_current_username(),
-                             current_user_info.domain,
-                             buf, sizeof(buf));
+       buf = talloc_strdup(ctx,s);
+       if (!buf) {
+               return 0;
+       }
+       buf = talloc_string_sub(ctx,buf,"%S",lp_servicename(snum));
+       if (!buf) {
+               return 0;
+       }
+       buf = talloc_sub_advanced(ctx,
+                               lp_servicename(SNUM(conn)),
+                               conn->server_info->unix_name,
+                               conn->connectpath,
+                               conn->server_info->utok.gid,
+                               conn->server_info->sanitized_username,
+                               pdb_get_domain(conn->server_info->sam_account),
+                               buf);
+       if (!buf) {
+               return 0;
+       }
        return strlen(buf) + 1;
 }
 
 static char *Expand(connection_struct *conn, int snum, char *s)
 {
-       static pstring buf;
+       TALLOC_CTX *ctx = talloc_tos();
+       char *buf = NULL;
+
        if (!s) {
                return NULL;
        }
-       StrnCpy(buf,s,sizeof(buf)/2);
-       pstring_sub(buf,"%S",lp_servicename(snum));
-       standard_sub_advanced(lp_servicename(SNUM(conn)), conn->user,
-                             conn->connectpath, conn->gid,
-                             get_current_username(),
-                             current_user_info.domain,
-                             buf, sizeof(buf));
-       return &buf[0];
+       buf = talloc_strdup(ctx,s);
+       if (!buf) {
+               return 0;
+       }
+       buf = talloc_string_sub(ctx,buf,"%S",lp_servicename(snum));
+       if (!buf) {
+               return 0;
+       }
+       return talloc_sub_advanced(ctx,
+                               lp_servicename(SNUM(conn)),
+                               conn->server_info->unix_name,
+                               conn->connectpath,
+                               conn->server_info->utok.gid,
+                               conn->server_info->sanitized_username,
+                               pdb_get_domain(conn->server_info->sam_account),
+                               buf);
 }
 
 /*******************************************************************
  Check a API string for validity when we only need to check the prefix.
 ******************************************************************/
 
-static BOOL prefix_ok(const char *str, const char *prefix)
+static bool prefix_ok(const char *str, const char *prefix)
 {
        return(strncmp(str,prefix,strlen(prefix)) == 0);
 }
@@ -210,7 +269,7 @@ static int getlen(const char *p)
        return n;
 }
 
-static BOOL init_package(struct pack_desc *p, int count, int subcount)
+static bool init_package(struct pack_desc *p, int count, int subcount)
 {
        int n = p->buflen;
        int i;
@@ -404,7 +463,7 @@ static void PackDriverData(struct pack_desc* desc)
        SIVAL(drivdata,0,sizeof drivdata); /* cb */
        SIVAL(drivdata,4,1000); /* lVersion */
        memset(drivdata+8,0,32);        /* szDeviceName */
-       push_ascii(drivdata+8,"NULL",-1, STR_TERMINATE);
+       push_ascii(drivdata+8,"NULL",32, STR_TERMINATE);
        PACKl(desc,"l",drivdata,sizeof drivdata); /* pDriverData */
 }
 
@@ -441,12 +500,18 @@ static int check_printq_info(struct pack_desc* desc,
                        desc->subformat = "z";
                        break;
                default:
+                       DEBUG(0,("check_printq_info: invalid level %d\n",
+                               uLevel ));
                        return False;
        }
-       if (strcmp(desc->format,id1) != 0) {
+       if (id1 == NULL || strcmp(desc->format,id1) != 0) {
+               DEBUG(0,("check_printq_info: invalid format %s\n",
+                       id1 ? id1 : "<NULL>" ));
                return False;
        }
-       if (desc->subformat && strcmp(desc->subformat,id2) != 0) {
+       if (desc->subformat && (id2 == NULL || strcmp(desc->subformat,id2) != 0)) {
+               DEBUG(0,("check_printq_info: invalid subformat %s\n",
+                       id2 ? id2 : "<NULL>" ));
                return False;
        }
        return True;
@@ -555,16 +620,20 @@ static void fill_printjob_info(connection_struct *conn, int snum, int uLevel,
  Returns True if from tdb, False otherwise.
  ********************************************************************/
 
-static BOOL get_driver_name(int snum, pstring drivername)
+static bool get_driver_name(int snum, char **pp_drivername)
 {
        NT_PRINTER_INFO_LEVEL *info = NULL;
-       BOOL in_tdb = False;
+       bool in_tdb = false;
 
        get_a_printer (NULL, &info, 2, lp_servicename(snum));
        if (info != NULL) {
-               pstrcpy( drivername, info->info_2->drivername);
-               in_tdb = True;
+               *pp_drivername = talloc_strdup(talloc_tos(),
+                                       info->info_2->drivername);
+               in_tdb = true;
                free_a_printer(&info, 2);
+               if (!*pp_drivername) {
+                       return false;
+               }
        }
 
        return in_tdb;
@@ -579,18 +648,16 @@ static void fill_printq_info_52(connection_struct *conn, int snum,
 {
        int                             i;
        fstring                         location;
-       NT_PRINTER_DRIVER_INFO_LEVEL    driver;
+       struct spoolss_DriverInfo8 *driver = NULL;
        NT_PRINTER_INFO_LEVEL           *printer = NULL;
 
-       ZERO_STRUCT(driver);
-
        if ( !W_ERROR_IS_OK(get_a_printer( NULL, &printer, 2, lp_servicename(snum))) ) {
                DEBUG(3,("fill_printq_info_52: Failed to lookup printer [%s]\n", 
                        lp_servicename(snum)));
                goto err;
        }
 
-       if ( !W_ERROR_IS_OK(get_a_printer_driver(&driver, 3, printer->info_2->drivername, 
+       if (!W_ERROR_IS_OK(get_a_printer_driver(talloc_tos(), &driver, printer->info_2->drivername,
                "Windows 4.0", 0)) )
        {
                DEBUG(3,("fill_printq_info_52: Failed to lookup driver [%s]\n", 
@@ -598,38 +665,38 @@ static void fill_printq_info_52(connection_struct *conn, int snum,
                goto err;
        }
 
-       trim_string(driver.info_3->driverpath, "\\print$\\WIN40\\0\\", 0);
-       trim_string(driver.info_3->datafile, "\\print$\\WIN40\\0\\", 0);
-       trim_string(driver.info_3->helpfile, "\\print$\\WIN40\\0\\", 0);
+       trim_string((char *)driver->driver_path, "\\print$\\WIN40\\0\\", 0);
+       trim_string((char *)driver->data_file, "\\print$\\WIN40\\0\\", 0);
+       trim_string((char *)driver->help_file, "\\print$\\WIN40\\0\\", 0);
 
        PACKI(desc, "W", 0x0400);                     /* don't know */
-       PACKS(desc, "z", driver.info_3->name);        /* long printer name */
-       PACKS(desc, "z", driver.info_3->driverpath);  /* Driverfile Name */
-       PACKS(desc, "z", driver.info_3->datafile);    /* Datafile name */
-       PACKS(desc, "z", driver.info_3->monitorname); /* language monitor */
+       PACKS(desc, "z", driver->driver_name);        /* long printer name */
+       PACKS(desc, "z", driver->driver_path);  /* Driverfile Name */
+       PACKS(desc, "z", driver->data_file);    /* Datafile name */
+       PACKS(desc, "z", driver->monitor_name); /* language monitor */
 
        fstrcpy(location, "\\\\%L\\print$\\WIN40\\0");
        standard_sub_basic( "", "", location, sizeof(location)-1 );
        PACKS(desc,"z", location);                          /* share to retrieve files */
 
-       PACKS(desc,"z", driver.info_3->defaultdatatype);    /* default data type */
-       PACKS(desc,"z", driver.info_3->helpfile);           /* helpfile name */
-       PACKS(desc,"z", driver.info_3->driverpath);               /* driver name */
+       PACKS(desc,"z", driver->default_datatype);    /* default data type */
+       PACKS(desc,"z", driver->help_file);           /* helpfile name */
+       PACKS(desc,"z", driver->driver_path);               /* driver name */
 
-       DEBUG(3,("Printer Driver Name: %s:\n",driver.info_3->name));
-       DEBUG(3,("Driver: %s:\n",driver.info_3->driverpath));
-       DEBUG(3,("Data File: %s:\n",driver.info_3->datafile));
-       DEBUG(3,("Language Monitor: %s:\n",driver.info_3->monitorname));
+       DEBUG(3,("Printer Driver Name: %s:\n",driver->driver_name));
+       DEBUG(3,("Driver: %s:\n",driver->driver_path));
+       DEBUG(3,("Data File: %s:\n",driver->data_file));
+       DEBUG(3,("Language Monitor: %s:\n",driver->monitor_name));
        DEBUG(3,("Driver Location: %s:\n",location));
-       DEBUG(3,("Data Type: %s:\n",driver.info_3->defaultdatatype));
-       DEBUG(3,("Help File: %s:\n",driver.info_3->helpfile));
+       DEBUG(3,("Data Type: %s:\n",driver->default_datatype));
+       DEBUG(3,("Help File: %s:\n",driver->help_file));
        PACKI(desc,"N",count);                     /* number of files to copy */
 
-       for ( i=0; i<count && driver.info_3->dependentfiles && *driver.info_3->dependentfiles[i]; i++) 
+       for ( i=0; i<count && driver->dependent_files && *driver->dependent_files[i]; i++)
        {
-               trim_string(driver.info_3->dependentfiles[i], "\\print$\\WIN40\\0\\", 0);
-               PACKS(desc,"z",driver.info_3->dependentfiles[i]);         /* driver files to copy */
-               DEBUG(3,("Dependent File: %s:\n",driver.info_3->dependentfiles[i]));
+               trim_string((char *)driver->dependent_files[i], "\\print$\\WIN40\\0\\", 0);
+               PACKS(desc,"z",driver->dependent_files[i]);         /* driver files to copy */
+               DEBUG(3,("Dependent File: %s:\n", driver->dependent_files[i]));
        }
 
        /* sanity check */
@@ -650,8 +717,7 @@ done:
        if ( printer )
                free_a_printer( &printer, 2 );
 
-       if ( driver.info_3 )
-               free_a_printer_driver( driver, 3 );
+       free_a_printer_driver(driver);
 }
 
 
@@ -699,7 +765,7 @@ static void fill_printq_info(connection_struct *conn, int snum, int uLevel,
        }
 
        if (uLevel == 3 || uLevel == 4) {
-               pstring drivername;
+               char *drivername = NULL;
 
                PACKI(desc,"W",5);              /* uPriority */
                PACKI(desc,"W",0);              /* uStarttime */
@@ -709,7 +775,7 @@ static void fill_printq_info(connection_struct *conn, int snum, int uLevel,
                PACKS(desc,"z","WinPrint");     /* pszPrProc */
                PACKS(desc,"z",NULL);           /* pszParms */
                PACKS(desc,"z",NULL);           /* pszComment - don't ask.... JRA */
-               /* "don't ask" that it's done this way to fix corrupted 
+               /* "don't ask" that it's done this way to fix corrupted
                   Win9X/ME printer comments. */
                if (!status) {
                        PACKI(desc,"W",LPSTAT_OK); /* fsStatus */
@@ -718,7 +784,10 @@ static void fill_printq_info(connection_struct *conn, int snum, int uLevel,
                }
                PACKI(desc,(uLevel == 3 ? "W" : "N"),count);    /* cJobs */
                PACKS(desc,"z",SERVICE(snum)); /* pszPrinters */
-               get_driver_name(snum,drivername);
+               get_driver_name(snum,&drivername);
+               if (!drivername) {
+                       return;
+               }
                PACKS(desc,"z",drivername);             /* pszDriverName */
                PackDriverData(desc);   /* pDriverData */
        }
@@ -737,7 +806,7 @@ static void fill_printq_info(connection_struct *conn, int snum, int uLevel,
 static int get_printerdrivernumber(int snum)
 {
        int                             result = 0;
-       NT_PRINTER_DRIVER_INFO_LEVEL    driver;
+       struct spoolss_DriverInfo8 *driver;
        NT_PRINTER_INFO_LEVEL           *printer = NULL;
 
        ZERO_STRUCT(driver);
@@ -748,7 +817,7 @@ static int get_printerdrivernumber(int snum)
                goto done;
        }
 
-       if ( !W_ERROR_IS_OK(get_a_printer_driver(&driver, 3, printer->info_2->drivername, 
+       if (!W_ERROR_IS_OK(get_a_printer_driver(talloc_tos(), &driver, printer->info_2->drivername,
                "Windows 4.0", 0)) )
        {
                DEBUG(3,("get_printerdrivernumber: Failed to lookup driver [%s]\n", 
@@ -757,20 +826,18 @@ static int get_printerdrivernumber(int snum)
        }
 
        /* count the number of files */
-       while ( driver.info_3->dependentfiles && *driver.info_3->dependentfiles[result] )
-                       result++;
-                       \
+       while (driver->dependent_files && *driver->dependent_files[result])
+               result++;
  done:
        if ( printer )
                free_a_printer( &printer, 2 );
 
-       if ( driver.info_3 )
-               free_a_printer_driver( driver, 3 );
+       free_a_printer_driver(driver);
 
        return result;
 }
 
-static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
+static bool api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -778,8 +845,8 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        char *QueueName = p;
        unsigned int uLevel;
        int count=0;
@@ -796,22 +863,20 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
        memset((char *)&status,'\0',sizeof(status));
        memset((char *)&desc,'\0',sizeof(desc));
 
-       p = skip_string(param,tpscnt,p,1);
+       p = skip_string(param,tpscnt,p);
        if (!p) {
                return False;
        }
        uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        str3 = get_safe_str_ptr(param,tpscnt,p,4);
-       if (!str3) {
-               return False;
-       }
+       /* str3 may be null here and is checked in check_printq_info(). */
 
        /* remove any trailing username */
        if ((p = strchr_m(QueueName,'%')))
                *p = 0;
+
        DEBUG(3,("api_DosPrintQGetInfo uLevel=%d name=%s\n",uLevel,QueueName));
+
        /* check it's a supported varient */
        if (!prefix_ok(str1,"zWrLh"))
                return False;
@@ -823,7 +888,7 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
                 */
                *rdata_len = 0;
                *rparam_len = 6;
-               *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+               *rparam = smb_realloc_limit(*rparam,*rparam_len);
                if (!*rparam) {
                        return False;
                }
@@ -832,11 +897,11 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
                SSVAL(*rparam,4,0);
                return(True);
        }
+
        snum = find_service(QueueName);
        if ( !(lp_snum_ok(snum) && lp_print_ok(snum)) )
                return False;
-               
+
        if (uLevel==52) {
                count = get_printerdrivernumber(snum);
                DEBUG(3,("api_DosPrintQGetInfo: Driver files count: %d\n",count));
@@ -845,8 +910,9 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
+                       SAFE_FREE(queue);
                        return False;
                }
                desc.base = *rdata;
@@ -866,7 +932,7 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        *rdata_len = desc.usedlen;
-  
+
        /*
         * We must set the return code to ERRbuftoosmall
         * in order to support lanman style printing with Win NT/2k
@@ -874,17 +940,19 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
         */
        if (!mdrcnt && lp_disable_spoolss())
                desc.errcode = ERRbuftoosmall;
+
        *rdata_len = desc.usedlen;
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
+               SAFE_FREE(queue);
+               SAFE_FREE(tmpdata);
                return False;
        }
        SSVALS(*rparam,0,desc.errcode);
        SSVAL(*rparam,2,0);
        SSVAL(*rparam,4,desc.neededlen);
-  
+
        DEBUG(4,("printqgetinfo: errorcode %d\n",desc.errcode));
 
        SAFE_FREE(queue);
@@ -897,7 +965,7 @@ static BOOL api_DosPrintQGetInfo(connection_struct *conn, uint16 vuid,
  View list of all print jobs on all queues.
 ****************************************************************************/
 
-static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
+static bool api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt, int mprcnt,
@@ -905,8 +973,8 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
                                int *rdata_len, int *rparam_len)
 {
        char *param_format = get_safe_str_ptr(param,tpscnt,param,2);
-       char *output_format1 = skip_string(param,tpscnt,param_format,1);
-       char *p = skip_string(param,tpscnt,output_format1,1);
+       char *output_format1 = skip_string(param,tpscnt,param_format);
+       char *p = skip_string(param,tpscnt,output_format1);
        unsigned int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        char *output_format2 = get_safe_str_ptr(param,tpscnt,p,4);
        int services = lp_numservices();
@@ -916,18 +984,15 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
        print_status_struct *status = NULL;
        int *subcntarr = NULL;
        int queuecnt = 0, subcnt = 0, succnt = 0;
-       if (!param_format || !output_format1 || !p || !output_format2) {
+
+       if (!param_format || !output_format1 || !p) {
                return False;
        }
 
-       uLevel = SVAL(p,0);
-       output_format2 = p + 4;
-
        memset((char *)&desc,'\0',sizeof(desc));
 
        DEBUG(3,("DosPrintQEnum uLevel=%d\n",uLevel));
+
        if (!prefix_ok(param_format,"WrLeh")) {
                return False;
        }
@@ -939,7 +1004,7 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
                 */
                *rdata_len = 0;
                *rparam_len = 6;
-               *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+               *rparam = smb_realloc_limit(*rparam,*rparam_len);
                if (!*rparam) {
                        return False;
                }
@@ -981,7 +1046,7 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        goto err;
                }
@@ -1004,10 +1069,10 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
        }
 
        SAFE_FREE(subcntarr);
+
        *rdata_len = desc.usedlen;
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                goto err;
        }
@@ -1015,7 +1080,7 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
        SSVAL(*rparam,2,0);
        SSVAL(*rparam,4,succnt);
        SSVAL(*rparam,6,queuecnt);
-  
+
        for (i = 0; i < queuecnt; i++) {
                if (queue) {
                        SAFE_FREE(queue[i]);
@@ -1024,7 +1089,7 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
 
        SAFE_FREE(queue);
        SAFE_FREE(status);
-  
+
        return True;
 
   err:
@@ -1045,7 +1110,7 @@ static BOOL api_DosPrintQEnum(connection_struct *conn, uint16 vuid,
  Get info level for a server list query.
 ****************************************************************************/
 
-static BOOL check_server_info(int uLevel, char* id)
+static bool check_server_info(int uLevel, char* id)
 {
        switch( uLevel ) {
                case 0:
@@ -1069,7 +1134,7 @@ struct srv_info_struct {
        uint32 type;
        fstring comment;
        fstring domain;
-       BOOL server_added;
+       bool server_added;
 };
 
 /*******************************************************************
@@ -1084,12 +1149,12 @@ static int get_server_info(uint32 servertype,
        int count=0;
        int alloced=0;
        char **lines;
-       BOOL local_list_only;
+       bool local_list_only;
        int i;
 
-       lines = file_lines_load(lock_path(SERVER_LIST), NULL, 0);
+       lines = file_lines_load(cache_path(SERVER_LIST), NULL, 0, NULL);
        if (!lines) {
-               DEBUG(4,("Can't open %s - %s\n",lock_path(SERVER_LIST),strerror(errno)));
+               DEBUG(4,("Can't open %s - %s\n",cache_path(SERVER_LIST),strerror(errno)));
                return 0;
        }
 
@@ -1106,43 +1171,63 @@ static int get_server_info(uint32 servertype,
                fstring stype;
                struct srv_info_struct *s;
                const char *ptr = lines[i];
-               BOOL ok = True;
+               bool ok = True;
+               TALLOC_CTX *frame = NULL;
+               char *p;
 
                if (!*ptr) {
                        continue;
                }
-    
+
                if (count == alloced) {
                        alloced += 10;
                        *servers = SMB_REALLOC_ARRAY(*servers,struct srv_info_struct, alloced);
                        if (!*servers) {
                                DEBUG(0,("get_server_info: failed to enlarge servers info struct!\n"));
-                               file_lines_free(lines);
+                               TALLOC_FREE(lines);
                                return 0;
                        }
                        memset((char *)((*servers)+count),'\0',sizeof(**servers)*(alloced-count));
                }
                s = &(*servers)[count];
-    
-               if (!next_token(&ptr,s->name, NULL, sizeof(s->name))) {
+
+               frame = talloc_stackframe();
+               s->name[0] = '\0';
+               if (!next_token_talloc(frame,&ptr,&p, NULL)) {
+                       TALLOC_FREE(frame);
                        continue;
                }
-               if (!next_token(&ptr,stype, NULL, sizeof(stype))) {
+               fstrcpy(s->name, p);
+
+               stype[0] = '\0';
+               if (!next_token_talloc(frame,&ptr, &p, NULL)) {
+                       TALLOC_FREE(frame);
                        continue;
                }
-               if (!next_token(&ptr,s->comment, NULL, sizeof(s->comment))) {
+               fstrcpy(stype, p);
+
+               s->comment[0] = '\0';
+               if (!next_token_talloc(frame,&ptr, &p, NULL)) {
+                       TALLOC_FREE(frame);
                        continue;
                }
-               if (!next_token(&ptr,s->domain, NULL, sizeof(s->domain))) {
+               fstrcpy(s->comment, p);
+               string_truncate(s->comment, MAX_SERVER_STRING_LENGTH);
+
+               s->domain[0] = '\0';
+               if (!next_token_talloc(frame,&ptr,&p, NULL)) {
                        /* this allows us to cope with an old nmbd */
-                       fstrcpy(s->domain,lp_workgroup()); 
+                       fstrcpy(s->domain,lp_workgroup());
+               } else {
+                       fstrcpy(s->domain, p);
                }
-    
-               if (sscanf(stype,"%X",&s->type) != 1) { 
-                       DEBUG(4,("r:host file ")); 
-                       ok = False; 
+               TALLOC_FREE(frame);
+
+               if (sscanf(stype,"%X",&s->type) != 1) {
+                       DEBUG(4,("r:host file "));
+                       ok = False;
                }
-    
+
                /* Filter the servers/domains we return based on what was asked for. */
 
                /* Check to see if we are being asked for a local list only. */
@@ -1152,21 +1237,21 @@ static int get_server_info(uint32 servertype,
                }
 
                /* doesn't match up: don't want it */
-               if (!(servertype & s->type)) { 
-                       DEBUG(4,("r:serv type ")); 
-                       ok = False; 
+               if (!(servertype & s->type)) {
+                       DEBUG(4,("r:serv type "));
+                       ok = False;
                }
-    
+
                if ((servertype & SV_TYPE_DOMAIN_ENUM) != 
                                (s->type & SV_TYPE_DOMAIN_ENUM)) {
                        DEBUG(4,("s: dom mismatch "));
                        ok = False;
                }
-    
+
                if (!strequal(domain, s->domain) && !(servertype & SV_TYPE_DOMAIN_ENUM)) {
                        ok = False;
                }
-    
+
                /* We should never return a server type with a SV_TYPE_LOCAL_LIST_ONLY set. */
                s->type &= ~SV_TYPE_LOCAL_LIST_ONLY;
 
@@ -1180,8 +1265,8 @@ static int get_server_info(uint32 servertype,
                                s->name, s->type, s->comment, s->domain));
                }
        }
-  
-       file_lines_free(lines);
+
+       TALLOC_FREE(lines);
        return count;
 }
 
@@ -1198,7 +1283,7 @@ static int fill_srv_info(struct srv_info_struct *service,
        char* p2;
        int l2;
        int len;
+
        switch (uLevel) {
                case 0:
                        struct_len = 16;
@@ -1209,7 +1294,7 @@ static int fill_srv_info(struct srv_info_struct *service,
                default:
                        return -1;
        }
+
        if (!buf) {
                len = 0;
                switch (uLevel) {
@@ -1222,7 +1307,7 @@ static int fill_srv_info(struct srv_info_struct *service,
                *stringspace = len;
                return struct_len + len;
        }
-  
+
        len = struct_len;
        p = *buf;
        if (*buflen < struct_len) {
@@ -1238,7 +1323,7 @@ static int fill_srv_info(struct srv_info_struct *service,
        if (!baseaddr) {
                baseaddr = p;
        }
-  
+
        switch (uLevel) {
                case 0:
                        push_ascii(p,service->name, MAX_NETBIOSNAME_LEN, STR_TERMINATE);
@@ -1265,7 +1350,7 @@ static int fill_srv_info(struct srv_info_struct *service,
 }
 
 
-static BOOL srv_comp(struct srv_info_struct *s1,struct srv_info_struct *s2)
+static bool srv_comp(struct srv_info_struct *s1,struct srv_info_struct *s2)
 {
        return(strcmp(s1->name,s2->name));
 }
@@ -1275,15 +1360,15 @@ static BOOL srv_comp(struct srv_info_struct *s1,struct srv_info_struct *s2)
  extracted from lists saved by nmbd on the local host.
 ****************************************************************************/
 
-static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
+static bool api_RNetServerEnum2(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt, int mprcnt, char **rdata, 
                                char **rparam, int *rdata_len, int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param, tpscnt, param, 2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel = get_safe_SVAL(param, tpscnt, p, 0, -1);
        int buf_len = get_safe_SVAL(param,tpscnt, p, 2, 0);
        uint32 servertype = get_safe_IVAL(param,tpscnt,p,4, 0);
@@ -1294,8 +1379,8 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
        int counted=0,total=0;
        int i,missed;
        fstring domain;
-       BOOL domain_request;
-       BOOL local_request;
+       bool domain_request;
+       bool local_request;
 
        if (!str1 || !str2 || !p) {
                return False;
@@ -1310,7 +1395,7 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
        }
 
        /* If someone sets SV_TYPE_LOCAL_LIST_ONLY but hasn't set
-          any other bit (they may just set this bit on it's own) they 
+          any other bit (they may just set this bit on its own) they 
           want all the locally seen servers. However this bit can be 
           set on its own so set the requested servers to be 
           ALL - DOMAIN_ENUM. */
@@ -1330,13 +1415,13 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
        if (!check_server_info(uLevel,str2)) {
                return False;
        }
-  
+
        DEBUG(4, ("server request level: %s %8x ", str2, servertype));
        DEBUG(4, ("domains_req:%s ", BOOLSTR(domain_request)));
        DEBUG(4, ("local_only:%s\n", BOOLSTR(local_request)));
 
        if (strcmp(str1, "WrLehDz") == 0) {
-               if (skip_string(param,tpscnt,p,1) == NULL) {
+               if (skip_string(param,tpscnt,p) == NULL) {
                        return False;
                }
                pull_ascii_fstring(domain, p);
@@ -1368,7 +1453,7 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
                        data_len += fill_srv_info(s,uLevel,0,&f_len,0,&s_len,0);
                        DEBUG(4,("fill_srv_info %20s %8x %25s %15s\n",
                                s->name, s->type, s->comment, s->domain));
-      
+
                        if (data_len <= buf_len) {
                                counted++;
                                fixed_len += f_len;
@@ -1380,12 +1465,11 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
        }
 
        *rdata_len = fixed_len + string_len;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
-       memset(*rdata,'\0',*rdata_len);
-  
+
        p2 = (*rdata) + fixed_len;      /* auxilliary data (strings) will go here */
        p = *rdata;
        f_len = fixed_len;
@@ -1408,9 +1492,9 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
                        count2--;
                }
        }
-  
+
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -1421,7 +1505,7 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
 
        SAFE_FREE(servers);
 
-       DEBUG(3,("NetServerEnum domain = %s uLevel=%d counted=%d total=%d\n",
+       DEBUG(3,("NetServerEnum2 domain = %s uLevel=%d counted=%d total=%d\n",
                domain,uLevel,counted,counted+missed));
 
        return True;
@@ -1431,15 +1515,15 @@ static BOOL api_RNetServerEnum(connection_struct *conn, uint16 vuid,
   command 0x34 - suspected of being a "Lookup Names" stub api
   ****************************************************************************/
 
-static BOOL api_RNetGroupGetUsers(connection_struct *conn, uint16 vuid,
+static bool api_RNetGroupGetUsers(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt, int mprcnt, char **rdata, 
                                char **rparam, int *rdata_len, int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        int buf_len = get_safe_SVAL(param,tpscnt,p,2,0);
        int counted=0;
@@ -1455,11 +1539,11 @@ static BOOL api_RNetGroupGetUsers(connection_struct *conn, uint16 vuid,
        if (!prefix_ok(str1,"zWrLeh")) {
                return False;
        }
-  
+
        *rdata_len = 0;
-  
+
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -1476,7 +1560,7 @@ static BOOL api_RNetGroupGetUsers(connection_struct *conn, uint16 vuid,
   get info about a share
   ****************************************************************************/
 
-static BOOL check_share_info(int uLevel, char* id)
+static bool check_share_info(int uLevel, char* id)
 {
        switch( uLevel ) {
                case 0:
@@ -1485,7 +1569,9 @@ static BOOL check_share_info(int uLevel, char* id)
                        }
                        break;
                case 1:
-                       if (strcmp(id,"B13BWz") != 0) {
+                       /* Level-2 descriptor is allowed (and ignored) */
+                       if (strcmp(id,"B13BWz") != 0 &&
+                           strcmp(id,"B13BWzWWWzB9B") != 0) {
                                return False;
                        }
                        break;
@@ -1514,7 +1600,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
        char* p2;
        int l2;
        int len;
+
        switch( uLevel ) {
                case 0:
                        struct_len = 13;
@@ -1531,8 +1617,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
                default:
                        return -1;
        }
-  
+
        if (!buf) {
                len = 0;
 
@@ -1550,7 +1635,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
                }
                return struct_len + len;
        }
-  
+
        len = struct_len;
        p = *buf;
        if ((*buflen) < struct_len) {
@@ -1568,9 +1653,9 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
        if (!baseaddr) {
                baseaddr = p;
        }
-  
+
        push_ascii(p,lp_servicename(snum),13, STR_TERMINATE);
-  
+
        if (uLevel > 0) {
                int type;
 
@@ -1586,7 +1671,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
                SIVAL(p,16,PTR_DIFF(p2,baseaddr));
                len += CopyExpanded(conn,snum,&p2,lp_comment(snum),&l2);
        }
-  
+
        if (uLevel > 1) {
                SSVAL(p,20,ACCESS_READ|ACCESS_WRITE|ACCESS_CREATE); /* permissions */
                SSVALS(p,22,-1);                /* max uses */
@@ -1595,7 +1680,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
                len += CopyAndAdvance(&p2,lp_pathname(snum),&l2);
                memset(p+30,0,SHPWLEN+2); /* passwd (reserved), pad field */
        }
-  
+
        if (uLevel > 2) {
                memset(p+40,0,SHPWLEN+2);
                SSVAL(p,50,0);
@@ -1606,7 +1691,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
                SSVAL(p,64,0);
                SSVAL(p,66,0);
        }
-       
+
        if (stringbuf) {
                (*buf) = p + struct_len;
                (*buflen) -= struct_len;
@@ -1620,7 +1705,7 @@ static int fill_share_info(connection_struct *conn, int snum, int uLevel,
        return len;
 }
 
-static BOOL api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
+static bool api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -1628,12 +1713,12 @@ static BOOL api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *netname = skip_string(param,tpscnt,str2,1);
-       char *p = skip_string(param,tpscnt,netname,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *netname = skip_string(param,tpscnt,str2);
+       char *p = skip_string(param,tpscnt,netname);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        int snum;
-  
+
        if (!str1 || !str2 || !netname || !p) {
                return False;
        }
@@ -1642,7 +1727,7 @@ static BOOL api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
        if (snum < 0) {
                return False;
        }
-  
+
        /* check it's a supported varient */
        if (!prefix_ok(str1,"zWrLh")) {
                return False;
@@ -1650,8 +1735,8 @@ static BOOL api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
        if (!check_share_info(uLevel,str2)) {
                return False;
        }
-       *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+
+       *rdata = smb_realloc_limit(*rdata,mdrcnt);
        if (!*rdata) {
                return False;
        }
@@ -1660,16 +1745,16 @@ static BOOL api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
        if (*rdata_len < 0) {
                return False;
        }
+
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
        SSVAL(*rparam,0,NERR_Success);
        SSVAL(*rparam,2,0);             /* converter word */
        SSVAL(*rparam,4,*rdata_len);
+
        return True;
 }
 
@@ -1683,7 +1768,7 @@ static BOOL api_RNetShareGetInfo(connection_struct *conn,uint16 vuid,
   Share names longer than 12 bytes must be skipped.
  ****************************************************************************/
 
-static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
+static bool api_RNetShareEnum( connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int                mdrcnt,
@@ -1694,18 +1779,18 @@ static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
                                int               *rparam_len )
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        int buf_len = get_safe_SVAL(param,tpscnt,p,2,0);
        char *p2;
        int count = 0;
        int total=0,counted=0;
-       BOOL missed = False;
+       bool missed = False;
        int i;
        int data_len, fixed_len, string_len;
        int f_len = 0, s_len = 0;
+
        if (!str1 || !str2 || !p) {
                return False;
        }
@@ -1716,7 +1801,7 @@ static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
        if (!check_share_info(uLevel,str2)) {
                return False;
        }
-  
+
        /* Ensure all the usershares are loaded. */
        become_root();
        load_registry_shares();
@@ -1745,12 +1830,11 @@ static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
        }
 
        *rdata_len = fixed_len + string_len;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
-       memset(*rdata,0,*rdata_len);
-  
+
        p2 = (*rdata) + fixed_len;      /* auxiliary data (strings) will go here */
        p = *rdata;
        f_len = fixed_len;
@@ -1769,9 +1853,9 @@ static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
                        }
                }
        }
-  
+
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -1779,7 +1863,7 @@ static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
        SSVAL(*rparam,2,0);
        SSVAL(*rparam,4,counted);
        SSVAL(*rparam,6,total);
-  
+
        DEBUG(3,("RNetShareEnum gave %d entries of %d (%d %d %d %d)\n",
                counted,total,uLevel,
                buf_len,*rdata_len,mdrcnt));
@@ -1791,7 +1875,7 @@ static BOOL api_RNetShareEnum( connection_struct *conn, uint16 vuid,
   Add a share
   ****************************************************************************/
 
-static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
+static bool api_RNetShareAdd(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -1799,17 +1883,18 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        fstring sharename;
        fstring comment;
-       pstring pathname;
+       char *pathname = NULL;
        char *command, *cmdname;
        unsigned int offset;
        int snum;
        int res = ERRunsup;
-  
+       size_t converted_size;
+
        if (!str1 || !str2 || !p) {
                return False;
        }
@@ -1826,7 +1911,7 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
        }
 
        /* Do we have a string ? */
-       if (skip_string(data,mdrcnt,data,1) == NULL) {
+       if (skip_string(data,mdrcnt,data) == NULL) {
                return False;
        }
        pull_ascii_fstring(sharename,data);
@@ -1852,7 +1937,7 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
        }
 
        /* Do we have a string ? */
-       if (skip_string(data,mdrcnt,data+offset,1) == NULL) {
+       if (skip_string(data,mdrcnt,data+offset) == NULL) {
                return False;
        }
        pull_ascii_fstring(comment, offset? (data+offset) : "");
@@ -1865,10 +1950,20 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
        }
 
        /* Do we have a string ? */
-       if (skip_string(data,mdrcnt,data+offset,1) == NULL) {
+       if (skip_string(data,mdrcnt,data+offset) == NULL) {
                return False;
        }
-       pull_ascii_pstring(pathname, offset? (data+offset) : "");
+
+       if (!pull_ascii_talloc(talloc_tos(), &pathname,
+                              offset ? (data+offset) : "", &converted_size))
+       {
+               DEBUG(0,("api_RNetShareAdd: pull_ascii_talloc failed: %s",
+                        strerror(errno)));
+       }
+
+       if (!pathname) {
+               return false;
+       }
 
        string_replace(sharename, '"', ' ');
        string_replace(pathname, '"', ' ');
@@ -1880,27 +1975,28 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
                return False;
        }
 
-       asprintf(&command, "%s \"%s\" \"%s\" \"%s\" \"%s\"",
-               lp_add_share_cmd(), dyn_CONFIGFILE, sharename, pathname, comment);
+       if (asprintf(&command, "%s \"%s\" \"%s\" \"%s\" \"%s\"",
+                    lp_add_share_cmd(), get_dyn_CONFIGFILE(), sharename,
+                    pathname, comment) == -1) {
+               return false;
+       }
 
-       if (command) {
-               DEBUG(10,("api_RNetShareAdd: Running [%s]\n", command ));
+       DEBUG(10,("api_RNetShareAdd: Running [%s]\n", command ));
 
-               if ((res = smbrun(command, NULL)) != 0) {
-                       DEBUG(1,("api_RNetShareAdd: Running [%s] returned (%d)\n", command, res ));
-                       SAFE_FREE(command);
-                       res = ERRnoaccess;
-                       goto error_exit;
-               } else {
-                       SAFE_FREE(command);
-                       message_send_all(conn_tdb_ctx(), MSG_SMB_CONF_UPDATED, NULL, 0, False, NULL);
-               }
+       if ((res = smbrun(command, NULL)) != 0) {
+               DEBUG(1,("api_RNetShareAdd: Running [%s] returned (%d)\n",
+                        command, res ));
+               SAFE_FREE(command);
+               res = ERRnoaccess;
+               goto error_exit;
        } else {
-               return False;
+               SAFE_FREE(command);
+               message_send_all(smbd_messaging_context(),
+                                MSG_SMB_CONF_UPDATED, NULL, 0, NULL);
        }
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -1908,13 +2004,13 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
        SSVAL(*rparam,2,0);             /* converter word */
        SSVAL(*rparam,4,*rdata_len);
        *rdata_len = 0;
-  
+
        return True;
 
   error_exit:
 
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -1928,7 +2024,7 @@ static BOOL api_RNetShareAdd(connection_struct *conn,uint16 vuid,
   view list of groups available
   ****************************************************************************/
 
-static BOOL api_RNetGroupEnum(connection_struct *conn,uint16 vuid,
+static bool api_RNetGroupEnum(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -1939,14 +2035,15 @@ static BOOL api_RNetGroupEnum(connection_struct *conn,uint16 vuid,
        int errflags=0;
        int resume_context, cli_buf_size;
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
 
-       struct pdb_search *search;
-       struct samr_displayentry *entries;
+       uint32_t num_groups;
+       uint32_t resume_handle;
+       struct rpc_pipe_client *samr_pipe;
+       struct policy_handle samr_handle, domain_handle;
+       NTSTATUS status;
 
-       int num_entries;
        if (!str1 || !str2 || !p) {
                return False;
        }
@@ -1967,14 +2064,31 @@ static BOOL api_RNetGroupEnum(connection_struct *conn,uint16 vuid,
                return False;
        }
 
-       /* get list of domain groups SID_DOMAIN_GRP=2 */
-       become_root();
-       search = pdb_search_groups();
-       unbecome_root();
+       status = rpc_pipe_open_internal(
+               talloc_tos(), &ndr_table_samr.syntax_id, rpc_samr_dispatch,
+               conn->server_info, &samr_pipe);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: Could not connect to samr: %s\n",
+                         nt_errstr(status)));
+               return false;
+       }
 
-       if (search == NULL) {
-               DEBUG(3,("api_RNetGroupEnum:failed to get group list"));
-               return False;
+       status = rpccli_samr_Connect2(samr_pipe, talloc_tos(), global_myname(),
+                                     SAMR_ACCESS_LOOKUP_DOMAIN, &samr_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_Connect2 failed: %s\n",
+                         nt_errstr(status)));
+               return false;
+       }
+
+       status = rpccli_samr_OpenDomain(samr_pipe, talloc_tos(), &samr_handle,
+                                       SAMR_DOMAIN_ACCESS_ENUM_ACCOUNTS,
+                                       get_global_sam_sid(), &domain_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_OpenDomain failed: %s\n",
+                         nt_errstr(status)));
+               rpccli_samr_Close(samr_pipe, talloc_tos(), &samr_handle);
+               return false;
        }
 
        resume_context = get_safe_SVAL(param,tpscnt,p,0,-1);
@@ -1982,50 +2096,83 @@ static BOOL api_RNetGroupEnum(connection_struct *conn,uint16 vuid,
        DEBUG(10,("api_RNetGroupEnum:resume context: %d, client buffer size: "
                  "%d\n", resume_context, cli_buf_size));
 
-       become_root();
-       num_entries = pdb_search_entries(search, resume_context, 0xffffffff,
-                                        &entries);
-       unbecome_root();
-
        *rdata_len = cli_buf_size;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
 
        p = *rdata;
 
-       for(i=0; i<num_entries; i++) {
-               fstring name;
-               fstrcpy(name, entries[i].account_name);
-               if( ((PTR_DIFF(p,*rdata)+21) <= *rdata_len) ) {
+       errflags = NERR_Success;
+       num_groups = 0;
+       resume_handle = 0;
+
+       while (true) {
+               struct samr_SamArray *sam_entries;
+               uint32_t num_entries;
+
+               status = rpccli_samr_EnumDomainGroups(samr_pipe, talloc_tos(),
+                                                     &domain_handle,
+                                                     &resume_handle,
+                                                     &sam_entries, 1,
+                                                     &num_entries);
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(10, ("rpccli_samr_EnumDomainGroups returned "
+                                  "%s\n", nt_errstr(status)));
+                       break;
+               }
+
+               if (num_entries == 0) {
+                       DEBUG(10, ("rpccli_samr_EnumDomainGroups returned "
+                                  "no entries -- done\n"));
+                       break;
+               }
+
+               for(i=0; i<num_entries; i++) {
+                       const char *name;
+
+                       name = sam_entries->entries[i].name.string;
+
+                       if( ((PTR_DIFF(p,*rdata)+21) > *rdata_len) ) {
+                               /* set overflow error */
+                               DEBUG(3,("overflow on entry %d group %s\n", i,
+                                        name));
+                               errflags=234;
+                               break;
+                       }
+
                        /* truncate the name at 21 chars. */
-                       memcpy(p, name, 21); 
+                       memset(p, 0, 21);
+                       strlcpy(p, name, 21);
                        DEBUG(10,("adding entry %d group %s\n", i, p));
                        p += 21;
-                       p += 5; /* Both NT4 and W2k3SP1 do padding here.
-                                  No idea why... */
-               } else {
-                       /* set overflow error */
-                       DEBUG(3,("overflow on entry %d group %s\n", i, name));
-                       errflags=234;
+                       p += 5; /* Both NT4 and W2k3SP1 do padding here.  No
+                                * idea why... */
+                       num_groups += 1;
+               }
+
+               if (errflags != NERR_Success) {
                        break;
                }
+
+               TALLOC_FREE(sam_entries);
        }
 
-       pdb_search_destroy(search);
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &domain_handle);
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &samr_handle);
 
        *rdata_len = PTR_DIFF(p,*rdata);
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
        SSVAL(*rparam, 0, errflags);
        SSVAL(*rparam, 2, 0);           /* converter word */
-       SSVAL(*rparam, 4, i);   /* is this right?? */
-       SSVAL(*rparam, 6, resume_context+num_entries);  /* is this right?? */
+       SSVAL(*rparam, 4, num_groups);  /* is this right?? */
+       SSVAL(*rparam, 6, resume_context+num_groups);   /* is this right?? */
 
        return(True);
 }
@@ -2034,7 +2181,7 @@ static BOOL api_RNetGroupEnum(connection_struct *conn,uint16 vuid,
  Get groups that a user is a member of.
 ******************************************************************/
 
-static BOOL api_NetUserGetGroups(connection_struct *conn,uint16 vuid,
+static bool api_NetUserGetGroups(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2042,38 +2189,39 @@ static BOOL api_NetUserGetGroups(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *UserName = skip_string(param,tpscnt,str2,1);
-       char *p = skip_string(param,tpscnt,UserName,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *UserName = skip_string(param,tpscnt,str2);
+       char *p = skip_string(param,tpscnt,UserName);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        const char *level_string;
        int count=0;
-       struct samu *sampw = NULL;
-       BOOL ret = False;
-       DOM_SID *sids;
-       gid_t *gids;
-       size_t num_groups;
-       size_t i;
-       NTSTATUS result;
-       DOM_SID user_sid;
-       enum lsa_SidType type;
-       TALLOC_CTX *mem_ctx;
+       bool ret = False;
+       uint32_t i;
+       char *endp = NULL;
+
+       struct rpc_pipe_client *samr_pipe;
+       struct policy_handle samr_handle, domain_handle, user_handle;
+       struct lsa_String name;
+       struct lsa_Strings names;
+       struct samr_Ids type, rid;
+       struct samr_RidWithAttributeArray *rids;
+       NTSTATUS status;
 
        if (!str1 || !str2 || !UserName || !p) {
                return False;
        }
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
-  
+
        /* check it's a supported varient */
-       
+
        if ( strcmp(str1,"zWrLeh") != 0 )
                return False;
-               
+
        switch( uLevel ) {
                case 0:
                        level_string = "B21";
@@ -2086,70 +2234,87 @@ static BOOL api_NetUserGetGroups(connection_struct *conn,uint16 vuid,
                return False;
 
        *rdata_len = mdrcnt + 1024;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
+
        SSVAL(*rparam,0,NERR_Success);
        SSVAL(*rparam,2,0);             /* converter word */
 
        p = *rdata;
+       endp = *rdata + *rdata_len;
 
-       mem_ctx = talloc_new(NULL);
-       if (mem_ctx == NULL) {
-               DEBUG(0, ("talloc_new failed\n"));
-               return False;
+       status = rpc_pipe_open_internal(
+               talloc_tos(), &ndr_table_samr.syntax_id, rpc_samr_dispatch,
+               conn->server_info, &samr_pipe);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: Could not connect to samr: %s\n",
+                         nt_errstr(status)));
+               return false;
        }
 
-       if ( !(sampw = samu_new(mem_ctx)) ) {
-               DEBUG(0, ("samu_new() failed!\n"));
-               TALLOC_FREE(mem_ctx);
-               return False;
+       status = rpccli_samr_Connect2(samr_pipe, talloc_tos(), global_myname(),
+                                     SAMR_ACCESS_LOOKUP_DOMAIN, &samr_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_Connect2 failed: %s\n",
+                         nt_errstr(status)));
+               return false;
        }
 
-       /* Lookup the user information; This should only be one of 
-          our accounts (not remote domains) */
+       status = rpccli_samr_OpenDomain(samr_pipe, talloc_tos(), &samr_handle,
+                                       SAMR_DOMAIN_ACCESS_OPEN_ACCOUNT,
+                                       get_global_sam_sid(), &domain_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_OpenDomain failed: %s\n",
+                         nt_errstr(status)));
+               goto close_sam;
+       }
 
-       become_root();                                  /* ROOT BLOCK */
+       name.string = UserName;
 
-       if (!lookup_name(mem_ctx, UserName, LOOKUP_NAME_ALL,
-                        NULL, NULL, &user_sid, &type)) {
-               DEBUG(10, ("lookup_name(%s) failed\n", UserName));
-               goto done;
+       status = rpccli_samr_LookupNames(samr_pipe, talloc_tos(),
+                                        &domain_handle, 1, &name,
+                                        &rid, &type);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_LookupNames failed: %s\n",
+                         nt_errstr(status)));
+               goto close_domain;
        }
 
-       if (type != SID_NAME_USER) {
+       if (type.ids[0] != SID_NAME_USER) {
                DEBUG(10, ("%s is a %s, not a user\n", UserName,
-                          sid_type_lookup(type)));
-               goto done;
+                          sid_type_lookup(type.ids[0])));
+               goto close_domain;
        }
 
-       if ( !pdb_getsampwsid(sampw, &user_sid) ) {
-               DEBUG(10, ("pdb_getsampwsid(%s) failed for user %s\n",
-                          sid_string_static(&user_sid), UserName));
-               goto done;
+       status = rpccli_samr_OpenUser(samr_pipe, talloc_tos(),
+                                     &domain_handle,
+                                     SAMR_USER_ACCESS_GET_GROUPS,
+                                     rid.ids[0], &user_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_LookupNames failed: %s\n",
+                         nt_errstr(status)));
+               goto close_domain;
        }
 
-       gids = NULL;
-       sids = NULL;
-       num_groups = 0;
-
-       result = pdb_enum_group_memberships(mem_ctx, sampw,
-                                           &sids, &gids, &num_groups);
-
-       if (!NT_STATUS_IS_OK(result)) {
-               DEBUG(10, ("pdb_enum_group_memberships failed for %s\n",
-                          UserName));
-               goto done;
+       status = rpccli_samr_GetGroupsForUser(samr_pipe, talloc_tos(),
+                                             &user_handle, &rids);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_LookupNames failed: %s\n",
+                         nt_errstr(status)));
+               goto close_user;
        }
 
-       for (i=0; i<num_groups; i++) {
+       for (i=0; i<rids->count; i++) {
 
-               const char *grp_name;
-       
-               if ( lookup_sid(mem_ctx, &sids[i], NULL, &grp_name, NULL) ) {
-                       pstrcpy(p, grp_name);
-                       p += 21; 
+               status = rpccli_samr_LookupRids(samr_pipe, talloc_tos(),
+                                               &domain_handle,
+                                               1, &rids->rids[i].rid,
+                                               &names, &type);
+               if (NT_STATUS_IS_OK(status) && (names.count == 1)) {
+                       strlcpy(p, names.names[0].string, PTR_DIFF(endp,p));
+                       p += 21;
                        count++;
                }
        }
@@ -2161,10 +2326,12 @@ static BOOL api_NetUserGetGroups(connection_struct *conn,uint16 vuid,
 
        ret = True;
 
-done:
-       unbecome_root();                                /* END ROOT BLOCK */
-
-       TALLOC_FREE(mem_ctx);
+ close_user:
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &user_handle);
+ close_domain:
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &domain_handle);
+ close_sam:
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &samr_handle);
 
        return ret;
 }
@@ -2173,7 +2340,7 @@ done:
  Get all users.
 ******************************************************************/
 
-static BOOL api_RNetUserEnum(connection_struct *conn, uint16 vuid,
+static bool api_RNetUserEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2184,12 +2351,16 @@ static BOOL api_RNetUserEnum(connection_struct *conn, uint16 vuid,
        int num_users=0;
        int errflags=0;
        int i, resume_context, cli_buf_size;
-       struct pdb_search *search;
-       struct samr_displayentry *users;
+       uint32_t resume_handle;
+
+       struct rpc_pipe_client *samr_pipe;
+       struct policy_handle samr_handle, domain_handle;
+       NTSTATUS status;
 
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
+       char *endp = NULL;
 
        if (!str1 || !str2 || !p) {
                return False;
@@ -2204,14 +2375,14 @@ static BOOL api_RNetUserEnum(connection_struct *conn, uint16 vuid,
          * e -> return parameter number of entries
          * h -> return parameter total number of users
          */
-  
+
        resume_context = get_safe_SVAL(param,tpscnt,p,0,-1);
        cli_buf_size= get_safe_SVAL(param,tpscnt,p,2,0);
        DEBUG(10,("api_RNetUserEnum:resume context: %d, client buffer size: %d\n",
                        resume_context, cli_buf_size));
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2221,47 +2392,97 @@ static BOOL api_RNetUserEnum(connection_struct *conn, uint16 vuid,
                return False;
 
        *rdata_len = cli_buf_size;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
 
        p = *rdata;
+       endp = *rdata + *rdata_len;
 
-       become_root();
-       search = pdb_search_users(ACB_NORMAL);
-       unbecome_root();
-       if (search == NULL) {
-               DEBUG(0, ("api_RNetUserEnum:unable to open sam database.\n"));
-               return False;
+       status = rpc_pipe_open_internal(
+               talloc_tos(), &ndr_table_samr.syntax_id, rpc_samr_dispatch,
+               conn->server_info, &samr_pipe);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: Could not connect to samr: %s\n",
+                         nt_errstr(status)));
+               return false;
        }
 
-       become_root();
-       num_users = pdb_search_entries(search, resume_context, 0xffffffff,
-                                      &users);
-       unbecome_root();
+       status = rpccli_samr_Connect2(samr_pipe, talloc_tos(), global_myname(),
+                                     SAMR_ACCESS_LOOKUP_DOMAIN, &samr_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_Connect2 failed: %s\n",
+                         nt_errstr(status)));
+               return false;
+       }
+
+       status = rpccli_samr_OpenDomain(samr_pipe, talloc_tos(), &samr_handle,
+                                       SAMR_DOMAIN_ACCESS_ENUM_ACCOUNTS,
+                                       get_global_sam_sid(), &domain_handle);
+       if (!NT_STATUS_IS_OK(status)) {
+               DEBUG(0, ("api_RNetUserEnum: samr_OpenDomain failed: %s\n",
+                         nt_errstr(status)));
+               rpccli_samr_Close(samr_pipe, talloc_tos(), &samr_handle);
+               return false;
+       }
 
        errflags=NERR_Success;
 
-       for (i=0; i<num_users; i++) {
-               const char *name = users[i].account_name;
-               
-               if(((PTR_DIFF(p,*rdata)+21)<=*rdata_len)&&(strlen(name)<=21)) {
-                       pstrcpy(p,name); 
-                       DEBUG(10,("api_RNetUserEnum:adding entry %d username "
-                                 "%s\n",count_sent,p));
-                       p += 21; 
-                       count_sent++; 
-               } else {
-                       /* set overflow error */
-                       DEBUG(10,("api_RNetUserEnum:overflow on entry %d "
-                                 "username %s\n",count_sent,name));
-                       errflags=234;
+       resume_handle = 0;
+
+       while (true) {
+               struct samr_SamArray *sam_entries;
+               uint32_t num_entries;
+
+               status = rpccli_samr_EnumDomainUsers(samr_pipe, talloc_tos(),
+                                                    &domain_handle,
+                                                    &resume_handle,
+                                                    0, &sam_entries, 1,
+                                                    &num_entries);
+
+               if (!NT_STATUS_IS_OK(status)) {
+                       DEBUG(10, ("rpccli_samr_EnumDomainUsers returned "
+                                  "%s\n", nt_errstr(status)));
+                       break;
+               }
+
+               if (num_entries == 0) {
+                       DEBUG(10, ("rpccli_samr_EnumDomainUsers returned "
+                                  "no entries -- done\n"));
+                       break;
+               }
+
+               for (i=0; i<num_entries; i++) {
+                       const char *name;
+
+                       name = sam_entries->entries[i].name.string;
+
+                       if(((PTR_DIFF(p,*rdata)+21)<=*rdata_len)
+                          &&(strlen(name)<=21)) {
+                               strlcpy(p,name,PTR_DIFF(endp,p));
+                               DEBUG(10,("api_RNetUserEnum:adding entry %d "
+                                         "username %s\n",count_sent,p));
+                               p += 21;
+                               count_sent++;
+                       } else {
+                               /* set overflow error */
+                               DEBUG(10,("api_RNetUserEnum:overflow on entry %d "
+                                         "username %s\n",count_sent,name));
+                               errflags=234;
+                               break;
+                       }
+               }
+
+               if (errflags != NERR_Success) {
                        break;
                }
+
+               TALLOC_FREE(sam_entries);
        }
 
-       pdb_search_destroy(search);
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &domain_handle);
+       rpccli_samr_Close(samr_pipe, talloc_tos(), &samr_handle);
 
        *rdata_len = PTR_DIFF(p,*rdata);
 
@@ -2277,7 +2498,7 @@ static BOOL api_RNetUserEnum(connection_struct *conn, uint16 vuid,
  Get the time of day info.
 ****************************************************************************/
 
-static BOOL api_NetRemoteTOD(connection_struct *conn,uint16 vuid,
+static bool api_NetRemoteTOD(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2289,13 +2510,13 @@ static BOOL api_NetRemoteTOD(connection_struct *conn,uint16 vuid,
        char *p;
 
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
 
        *rdata_len = 21;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
@@ -2335,7 +2556,7 @@ static BOOL api_NetRemoteTOD(connection_struct *conn,uint16 vuid,
  Set the user password.
 *****************************************************************************/
 
-static BOOL api_SetUserPassword(connection_struct *conn,uint16 vuid,
+static bool api_SetUserPassword(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2343,35 +2564,43 @@ static BOOL api_SetUserPassword(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *np = get_safe_str_ptr(param,tpscnt,param,2);
-       char *p = skip_string(param,tpscnt,np,2);
+       char *p = NULL;
        fstring user;
        fstring pass1,pass2;
 
+       /* Skip 2 strings. */
+       p = skip_string(param,tpscnt,np);
+       p = skip_string(param,tpscnt,p);
+
        if (!np || !p) {
                return False;
        }
 
        /* Do we have a string ? */
-       if (skip_string(param,tpscnt,p,1) == NULL) {
+       if (skip_string(param,tpscnt,p) == NULL) {
                return False;
        }
        pull_ascii_fstring(user,p);
 
-       p = skip_string(param,tpscnt,p,1);
+       p = skip_string(param,tpscnt,p);
        if (!p) {
                return False;
        }
 
        memset(pass1,'\0',sizeof(pass1));
        memset(pass2,'\0',sizeof(pass2));
-       if (!is_offset_safe(param,tpscnt,p,32)) {
+       /*
+        * We use 31 here not 32 as we're checking
+        * the last byte we want to access is safe.
+        */
+       if (!is_offset_safe(param,tpscnt,p,31)) {
                return False;
        }
        memcpy(pass1,p,16);
        memcpy(pass2,p+16,16);
 
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2389,7 +2618,7 @@ static BOOL api_SetUserPassword(connection_struct *conn,uint16 vuid,
         */
 
        {
-               auth_serversupplied_info *server_info = NULL;
+               struct auth_serversupplied_info *server_info = NULL;
                DATA_BLOB password = data_blob(pass1, strlen(pass1)+1);
 
                if (NT_STATUS_IS_OK(check_plaintext_password(user,password,&server_info))) {
@@ -2430,7 +2659,7 @@ static BOOL api_SetUserPassword(connection_struct *conn,uint16 vuid,
 
        memset((char *)pass1,'\0',sizeof(fstring));
        memset((char *)pass2,'\0',sizeof(fstring));      
-        
+
        return(True);
 }
 
@@ -2438,17 +2667,18 @@ static BOOL api_SetUserPassword(connection_struct *conn,uint16 vuid,
   Set the user password (SamOEM version - gets plaintext).
 ****************************************************************************/
 
-static BOOL api_SamOEMChangePassword(connection_struct *conn,uint16 vuid,
+static bool api_SamOEMChangePassword(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
                                char **rdata,char **rparam,
                                int *rdata_len,int *rparam_len)
 {
+       struct smbd_server_connection *sconn = smbd_server_conn;
        fstring user;
        char *p = get_safe_str_ptr(param,tpscnt,param,2);
        *rparam_len = 2;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2465,32 +2695,32 @@ static BOOL api_SamOEMChangePassword(connection_struct *conn,uint16 vuid,
         */
 
        /* Do we have a string ? */
-       if (skip_string(param,tpscnt,p,1) == 0) {
+       if (skip_string(param,tpscnt,p) == 0) {
                return False;
        }
        if(!strequal(p, "zsT")) {
                DEBUG(0,("api_SamOEMChangePassword: Invalid parameter string %s\n", p));
                return False;
        }
-       p = skip_string(param, tpscnt, p, 1);
+       p = skip_string(param, tpscnt, p);
        if (!p) {
                return False;
        }
 
        /* Do we have a string ? */
-       if (skip_string(param,tpscnt,p,1) == 0) {
+       if (skip_string(param,tpscnt,p) == 0) {
                return False;
        }
        if(!strequal(p, "B516B16")) {
                DEBUG(0,("api_SamOEMChangePassword: Invalid data parameter string %s\n", p));
                return False;
        }
-       p = skip_string(param,tpscnt,p,1);
+       p = skip_string(param,tpscnt,p);
        if (!p) {
                return False;
        }
        /* Do we have a string ? */
-       if (skip_string(param,tpscnt,p,1) == 0) {
+       if (skip_string(param,tpscnt,p) == 0) {
                return False;
        }
        p += pull_ascii_fstring(user,p);
@@ -2502,7 +2732,7 @@ static BOOL api_SamOEMChangePassword(connection_struct *conn,uint16 vuid,
         * function.
         */
 
-       (void)map_username(user);
+       (void)map_username(sconn, user);
 
        if (NT_STATUS_IS_OK(pass_oem_change(user, (uchar*) data, (uchar *)&data[516], NULL, NULL, NULL))) {
                SSVAL(*rparam,0,NERR_Success);
@@ -2516,7 +2746,7 @@ static BOOL api_SamOEMChangePassword(connection_struct *conn,uint16 vuid,
   Form: <W> <> 
   ****************************************************************************/
 
-static BOOL api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
+static bool api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2525,8 +2755,8 @@ static BOOL api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
 {
        int function = get_safe_SVAL(param,tpscnt,param,0,0);
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        uint32 jobid;
        int snum;
        fstring sharename;
@@ -2536,7 +2766,11 @@ static BOOL api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
        if (!str1 || !str2 || !p) {
                return False;
        }
-       if (!is_offset_safe(param,tpscnt,p,2)) {
+       /*
+        * We use 1 here not 2 as we're checking
+        * the last byte we want to access is safe.
+        */
+       if (!is_offset_safe(param,tpscnt,p,1)) {
                return False;
        }
        if(!rap_to_pjobid(SVAL(p,0), sharename, &jobid))
@@ -2547,7 +2781,7 @@ static BOOL api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
                return(False);
 
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);       
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2565,25 +2799,25 @@ static BOOL api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
        }
 
        errcode = NERR_notsupported;
-       
+
        switch (function) {
        case 81:                /* delete */ 
-               if (print_job_delete(&current_user, snum, jobid, &werr)) 
+               if (print_job_delete(conn->server_info, snum, jobid, &werr))
                        errcode = NERR_Success;
                break;
        case 82:                /* pause */
-               if (print_job_pause(&current_user, snum, jobid, &werr)) 
+               if (print_job_pause(conn->server_info, snum, jobid, &werr))
                        errcode = NERR_Success;
                break;
        case 83:                /* resume */
-               if (print_job_resume(&current_user, snum, jobid, &werr)) 
+               if (print_job_resume(conn->server_info, snum, jobid, &werr))
                        errcode = NERR_Success;
                break;
        }
 
        if (!W_ERROR_IS_OK(werr))
                errcode = W_ERROR_V(werr);
-       
+
  out:
        SSVAL(*rparam,0,errcode);       
        SSVAL(*rparam,2,0);             /* converter word */
@@ -2595,7 +2829,7 @@ static BOOL api_RDosPrintJobDel(connection_struct *conn,uint16 vuid,
   Purge a print queue - or pause or resume it.
   ****************************************************************************/
 
-static BOOL api_WPrintQueueCtrl(connection_struct *conn,uint16 vuid,
+static bool api_WPrintQueueCtrl(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2604,8 +2838,8 @@ static BOOL api_WPrintQueueCtrl(connection_struct *conn,uint16 vuid,
 {
        int function = get_safe_SVAL(param,tpscnt,param,0,0);
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *QueueName = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *QueueName = skip_string(param,tpscnt,str2);
        int errcode = NERR_notsupported;
        int snum;
        WERROR werr = WERR_OK;
@@ -2619,13 +2853,13 @@ static BOOL api_WPrintQueueCtrl(connection_struct *conn,uint16 vuid,
                return(False);
 
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
        *rdata_len = 0;
 
-       if (skip_string(param,tpscnt,QueueName,1) == NULL) {
+       if (skip_string(param,tpscnt,QueueName) == NULL) {
                return False;
        }
        snum = print_queue_snum(QueueName);
@@ -2637,17 +2871,20 @@ static BOOL api_WPrintQueueCtrl(connection_struct *conn,uint16 vuid,
 
        switch (function) {
        case 74: /* Pause queue */
-               if (print_queue_pause(&current_user, snum, &werr)) errcode = NERR_Success;
+               werr = print_queue_pause(conn->server_info, snum);
                break;
        case 75: /* Resume queue */
-               if (print_queue_resume(&current_user, snum, &werr)) errcode = NERR_Success;
+               werr = print_queue_resume(conn->server_info, snum);
                break;
        case 103: /* Purge */
-               if (print_queue_purge(&current_user, snum, &werr)) errcode = NERR_Success;
+               werr = print_queue_purge(conn->server_info, snum);
+               break;
+       default:
+               werr = WERR_NOT_SUPPORTED;
                break;
        }
 
-       if (!W_ERROR_IS_OK(werr)) errcode = W_ERROR_V(werr);
+       errcode = W_ERROR_V(werr);
 
  out:
        SSVAL(*rparam,0,errcode);
@@ -2674,13 +2911,20 @@ static int check_printjob_info(struct pack_desc* desc,
        case 2: desc->format = "WWzWWDDzz"; break;
        case 3: desc->format = "WWzWWDDzzzzzzzzzzlz"; break;
        case 4: desc->format = "WWzWWDDzzzzzDDDDDDD"; break;
-       default: return False;
+       default:
+               DEBUG(0,("check_printjob_info: invalid level %d\n",
+                       uLevel ));
+               return False;
+       }
+       if (id == NULL || strcmp(desc->format,id) != 0) {
+               DEBUG(0,("check_printjob_info: invalid format %s\n",
+                       id ? id : "<NULL>" ));
+               return False;
        }
-       if (strcmp(desc->format,id) != 0) return False;
        return True;
 }
 
-static BOOL api_PrintJobInfo(connection_struct *conn, uint16 vuid,
+static bool api_PrintJobInfo(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2689,8 +2933,8 @@ static BOOL api_PrintJobInfo(connection_struct *conn, uint16 vuid,
 {
        struct pack_desc desc;
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        uint32 jobid;
        fstring sharename;
        int uLevel = get_safe_SVAL(param,tpscnt,p,2,-1);
@@ -2700,13 +2944,17 @@ static BOOL api_PrintJobInfo(connection_struct *conn, uint16 vuid,
        if (!str1 || !str2 || !p) {
                return False;
        }
-       if (!is_offset_safe(param,tpscnt,p,2)) {
+       /*
+        * We use 1 here not 2 as we're checking
+        * the last byte we want to access is safe.
+        */
+       if (!is_offset_safe(param,tpscnt,p,1)) {
                return False;
        }
        if(!rap_to_pjobid(SVAL(p,0), sharename, &jobid))
                return False;
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2716,9 +2964,9 @@ static BOOL api_PrintJobInfo(connection_struct *conn, uint16 vuid,
                         sharename));
                return False;
        }
-  
+
        *rdata_len = 0;
-       
+
        /* check it's a supported varient */
        if ((strcmp(str1,"WWsTP")) || 
            (!check_printjob_info(&desc,uLevel,str2)))
@@ -2755,7 +3003,7 @@ static BOOL api_PrintJobInfo(connection_struct *conn, uint16 vuid,
  out:
        SSVALS(*rparam,0,errcode);
        SSVAL(*rparam,2,0);             /* converter word */
-       
+
        return(True);
 }
 
@@ -2764,7 +3012,7 @@ static BOOL api_PrintJobInfo(connection_struct *conn, uint16 vuid,
  Get info about the server.
 ****************************************************************************/
 
-static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
+static bool api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2772,8 +3020,8 @@ static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        char *p2;
        int struct_len;
@@ -2831,7 +3079,7 @@ static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
        }
 
        *rdata_len = mdrcnt;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
@@ -2839,23 +3087,32 @@ static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
        p = *rdata;
        p2 = p + struct_len;
        if (uLevel != 20) {
-               srvstr_push(NULL, p,global_myname(),16, 
+               srvstr_push(NULL, 0, p,global_myname(),16,
                        STR_ASCII|STR_UPPER|STR_TERMINATE);
        }
        p += 16;
        if (uLevel > 0) {
                struct srv_info_struct *servers=NULL;
                int i,count;
-               pstring comment;
+               char *comment = NULL;
+               TALLOC_CTX *ctx = talloc_tos();
                uint32 servertype= lp_default_server_announce();
 
-               push_ascii(comment,lp_serverstring(), MAX_SERVER_STRING_LENGTH,STR_TERMINATE);
+               comment = talloc_strdup(ctx,lp_serverstring());
+               if (!comment) {
+                       return false;
+               }
 
                if ((count=get_server_info(SV_TYPE_ALL,&servers,lp_workgroup()))>0) {
                        for (i=0;i<count;i++) {
                                if (strequal(servers[i].name,global_myname())) {
                                        servertype = servers[i].type;
-                                       push_ascii(comment,servers[i].comment,sizeof(pstring),STR_TERMINATE);
+                                       TALLOC_FREE(comment);
+                                       comment = talloc_strdup(ctx,
+                                                       servers[i].comment);
+                                       if (comment) {
+                                               return false;
+                                       }
                                }
                        }
                }
@@ -2870,13 +3127,27 @@ static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
                        SIVAL(p,6,0);
                } else {
                        SIVAL(p,6,PTR_DIFF(p2,*rdata));
-                       standard_sub_advanced(lp_servicename(SNUM(conn)), conn->user,
-                                             conn->connectpath, conn->gid,
-                                             get_current_username(),
-                                             current_user_info.domain,
-                                             comment, sizeof(comment));
-                       StrnCpy(p2,comment,MAX(mdrcnt - struct_len,0));
-                       p2 = skip_string(*rdata,*rdata_len,p2,1);
+                       comment = talloc_sub_advanced(
+                               ctx,
+                               lp_servicename(SNUM(conn)),
+                               conn->server_info->unix_name,
+                               conn->connectpath,
+                               conn->server_info->utok.gid,
+                               conn->server_info->sanitized_username,
+                               pdb_get_domain(conn->server_info->sam_account),
+                               comment);
+                       if (comment) {
+                               return false;
+                       }
+                       if (mdrcnt - struct_len <= 0) {
+                               return false;
+                       }
+                       push_ascii(p2,
+                               comment,
+                               MIN(mdrcnt - struct_len,
+                                       MAX_SERVER_STRING_LENGTH),
+                               STR_TERMINATE);
+                       p2 = skip_string(*rdata,*rdata_len,p2);
                        if (!p2) {
                                return False;
                        }
@@ -2890,7 +3161,7 @@ static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
        *rdata_len = PTR_DIFF(p2,*rdata);
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2905,7 +3176,7 @@ static BOOL api_RNetServerGetInfo(connection_struct *conn,uint16 vuid,
  Get info about the server.
 ****************************************************************************/
 
-static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
+static bool api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -2913,9 +3184,10 @@ static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        char *p2;
+       char *endp;
        int level = get_safe_SVAL(param,tpscnt,p,0,-1);
 
        if (!str1 || !str2 || !p) {
@@ -2925,7 +3197,7 @@ static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
        DEBUG(4,("NetWkstaGetInfo level %d\n",level));
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -2936,7 +3208,7 @@ static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
        }
 
        *rdata_len = mdrcnt + 1024;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
@@ -2945,32 +3217,34 @@ static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
        SSVAL(*rparam,2,0);             /* converter word */
 
        p = *rdata;
+       endp = *rdata + *rdata_len;
+
        p2 = get_safe_ptr(*rdata,*rdata_len,p,22);
        if (!p2) {
                return False;
        }
 
        SIVAL(p,0,PTR_DIFF(p2,*rdata)); /* host name */
-       pstrcpy(p2,get_local_machine_name());
+       strlcpy(p2,get_local_machine_name(),PTR_DIFF(endp,p2));
        strupper_m(p2);
-       p2 = skip_string(*rdata,*rdata_len,p2,1);
+       p2 = skip_string(*rdata,*rdata_len,p2);
        if (!p2) {
                return False;
        }
        p += 4;
 
        SIVAL(p,0,PTR_DIFF(p2,*rdata));
-       pstrcpy(p2,current_user_info.smb_name);
-       p2 = skip_string(*rdata,*rdata_len,p2,1);
+       strlcpy(p2,conn->server_info->sanitized_username,PTR_DIFF(endp,p2));
+       p2 = skip_string(*rdata,*rdata_len,p2);
        if (!p2) {
                return False;
        }
        p += 4;
 
        SIVAL(p,0,PTR_DIFF(p2,*rdata)); /* login domain */
-       pstrcpy(p2,lp_workgroup());
+       strlcpy(p2,lp_workgroup(),PTR_DIFF(endp,p2));
        strupper_m(p2);
-       p2 = skip_string(*rdata,*rdata_len,p2,1);
+       p2 = skip_string(*rdata,*rdata_len,p2);
        if (!p2) {
                return False;
        }
@@ -2981,16 +3255,16 @@ static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
        p += 2;
 
        SIVAL(p,0,PTR_DIFF(p2,*rdata));
-       pstrcpy(p2,lp_workgroup());     /* don't know.  login domain?? */
-       p2 = skip_string(*rdata,*rdata_len,p2,1);
+       strlcpy(p2,lp_workgroup(),PTR_DIFF(endp,p2));   /* don't know.  login domain?? */
+       p2 = skip_string(*rdata,*rdata_len,p2);
        if (!p2) {
                return False;
        }
        p += 4;
 
        SIVAL(p,0,PTR_DIFF(p2,*rdata)); /* don't know */
-       pstrcpy(p2,"");
-       p2 = skip_string(*rdata,*rdata_len,p2,1);
+       strlcpy(p2,"",PTR_DIFF(endp,p2));
+       p2 = skip_string(*rdata,*rdata_len,p2);
        if (!p2) {
                return False;
        }
@@ -3031,7 +3305,7 @@ static BOOL api_NetWkstaGetInfo(connection_struct *conn,uint16 vuid,
 
 where:
 
-  usri11_name specifies the user name for which information is retireved
+  usri11_name specifies the user name for which information is retrieved
 
   usri11_pad aligns the next data structure element to a word boundary
 
@@ -3173,28 +3447,31 @@ There is no auxiliary data in the response.
 #define AF_OP_ACCOUNTS  3
 
 
-static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
+static bool api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
                                char **rdata,char **rparam,
                                int *rdata_len,int *rparam_len)
 {
+       struct smbd_server_connection *sconn = smbd_server_conn;
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *UserName = skip_string(param,tpscnt,str2,1);
-       char *p = skip_string(param,tpscnt,UserName,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *UserName = skip_string(param,tpscnt,str2);
+       char *p = skip_string(param,tpscnt,UserName);
        int uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
        char *p2;
+       char *endp;
        const char *level_string;
 
        /* get NIS home of a previously validated user - simeon */
        /* With share level security vuid will always be zero.
           Don't depend on vuser being non-null !!. JRA */
-       user_struct *vuser = get_valid_user_struct(vuid);
+       user_struct *vuser = get_valid_user_struct(sconn, vuid);
        if(vuser != NULL) {
-               DEBUG(3,("  Username of UID %d is %s\n", (int)vuser->uid, 
-                       vuser->user.unix_name));
+               DEBUG(3,("  Username of UID %d is %s\n",
+                        (int)vuser->server_info->utok.uid,
+                        vuser->server_info->unix_name));
        }
 
        if (!str1 || !str2 || !UserName || !p) {
@@ -3202,13 +3479,13 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
 
        DEBUG(4,("RNetUserGetInfo level=%d\n", uLevel));
-  
+
        /* check it's a supported variant */
        if (strcmp(str1,"zWrLh") != 0) {
                return False;
@@ -3227,7 +3504,7 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        *rdata_len = mdrcnt + 1024;
-       *rdata = SMB_REALLOC_LIMIT(*rdata,*rdata_len);
+       *rdata = smb_realloc_limit(*rdata,*rdata_len);
        if (!*rdata) {
                return False;
        }
@@ -3236,12 +3513,13 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
        SSVAL(*rparam,2,0);             /* converter word */
 
        p = *rdata;
+       endp = *rdata + *rdata_len;
        p2 = get_safe_ptr(*rdata,*rdata_len,p,usri11_end);
        if (!p2) {
                return False;
        }
 
-       memset(p,0,21); 
+       memset(p,0,21);
        fstrcpy(p+usri11_name,UserName); /* 21 bytes - user name */
 
        if (uLevel > 0) {
@@ -3251,42 +3529,49 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
 
        if (uLevel >= 10) {
                SIVAL(p,usri11_comment,PTR_DIFF(p2,p)); /* comment */
-               pstrcpy(p2,"Comment");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2,"Comment",PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
 
                SIVAL(p,usri11_usr_comment,PTR_DIFF(p2,p)); /* user_comment */
-               pstrcpy(p2,"UserComment");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2,"UserComment",PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
 
                /* EEK! the cifsrap.txt doesn't have this in!!!! */
                SIVAL(p,usri11_full_name,PTR_DIFF(p2,p)); /* full name */
-               pstrcpy(p2,((vuser != NULL) ? vuser->user.full_name : UserName));
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2,((vuser != NULL)
+                           ? pdb_get_fullname(vuser->server_info->sam_account)
+                           : UserName),PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
        }
 
        if (uLevel == 11) {
+               const char *homedir = "";
+               if (vuser != NULL) {
+                       homedir = pdb_get_homedir(
+                               vuser->server_info->sam_account);
+               }
                /* modelled after NTAS 3.51 reply */
                SSVAL(p,usri11_priv,conn->admin_user?USER_PRIV_ADMIN:USER_PRIV_USER); 
                SIVAL(p,usri11_auth_flags,AF_OP_PRINT);         /* auth flags */
                SIVALS(p,usri11_password_age,-1);               /* password age */
                SIVAL(p,usri11_homedir,PTR_DIFF(p2,p)); /* home dir */
-               pstrcpy(p2, vuser && vuser->homedir ? vuser->homedir : "");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2, homedir, PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
                SIVAL(p,usri11_parms,PTR_DIFF(p2,p)); /* parms */
-               pstrcpy(p2,"");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2,"",PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
@@ -3295,16 +3580,16 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
                SSVALS(p,usri11_bad_pw_count,-1);       /* bad pw counts */
                SSVALS(p,usri11_num_logons,-1);         /* num logons */
                SIVAL(p,usri11_logon_server,PTR_DIFF(p2,p)); /* logon server */
-               pstrcpy(p2,"\\\\*");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2,"\\\\*",PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
                SSVAL(p,usri11_country_code,0);         /* country code */
 
                SIVAL(p,usri11_workstations,PTR_DIFF(p2,p)); /* workstations */
-               pstrcpy(p2,"");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2,"",PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
@@ -3316,7 +3601,7 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
                /* a simple way to get logon hours at all times. */
                memset(p2,0xff,21);
                SCVAL(p2,21,0);           /* fix zero termination */
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
@@ -3330,8 +3615,10 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
                SSVAL(p,42,
                conn->admin_user?USER_PRIV_ADMIN:USER_PRIV_USER);
                SIVAL(p,44,PTR_DIFF(p2,*rdata)); /* home dir */
-               pstrcpy(p2, vuser && vuser->homedir ? vuser->homedir : "");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2, vuser ? pdb_get_homedir(
+                               vuser->server_info->sam_account) : "",
+                       PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
@@ -3339,23 +3626,27 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
                *p2++ = 0;
                SSVAL(p,52,0);          /* flags */
                SIVAL(p,54,PTR_DIFF(p2,*rdata));                /* script_path */
-               pstrcpy(p2,vuser && vuser->logon_script ? vuser->logon_script : "");
-               p2 = skip_string(*rdata,*rdata_len,p2,1);
+               strlcpy(p2, vuser ? pdb_get_logon_script(
+                               vuser->server_info->sam_account) : "",
+                       PTR_DIFF(endp,p2));
+               p2 = skip_string(*rdata,*rdata_len,p2);
                if (!p2) {
                        return False;
                }
                if (uLevel == 2) {
                        SIVAL(p,60,0);          /* auth_flags */
                        SIVAL(p,64,PTR_DIFF(p2,*rdata)); /* full_name */
-                       pstrcpy(p2,((vuser != NULL) ? vuser->user.full_name : UserName));
-                       p2 = skip_string(*rdata,*rdata_len,p2,1);
+                       strlcpy(p2,((vuser != NULL)
+                                   ? pdb_get_fullname(vuser->server_info->sam_account)
+                                   : UserName),PTR_DIFF(endp,p2));
+                       p2 = skip_string(*rdata,*rdata_len,p2);
                        if (!p2) {
                                return False;
                        }
                        SIVAL(p,68,0);          /* urs_comment */
                        SIVAL(p,72,PTR_DIFF(p2,*rdata)); /* parms */
-                       pstrcpy(p2,"");
-                       p2 = skip_string(*rdata,*rdata_len,p2,1);
+                       strlcpy(p2,"",PTR_DIFF(endp,p2));
+                       p2 = skip_string(*rdata,*rdata_len,p2);
                        if (!p2) {
                                return False;
                        }
@@ -3372,12 +3663,31 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
                        SSVALS(p,104,-1);       /* num_logons */
                        SIVAL(p,106,PTR_DIFF(p2,*rdata)); /* logon_server */
                        {
-                               pstring tmp;
-                               pstrcpy(tmp, "\\\\%L");
-                               standard_sub_basic("", "", tmp, sizeof(tmp));
-                               pstrcpy(p2, tmp);
+                               TALLOC_CTX *ctx = talloc_tos();
+                               int space_rem = *rdata_len - (p2 - *rdata);
+                               char *tmp;
+
+                               if (space_rem <= 0) {
+                                       return false;
+                               }
+                               tmp = talloc_strdup(ctx, "\\\\%L");
+                               if (!tmp) {
+                                       return false;
+                               }
+                               tmp = talloc_sub_basic(ctx,
+                                               "",
+                                               "",
+                                               tmp);
+                               if (!tmp) {
+                                       return false;
+                               }
+
+                               push_ascii(p2,
+                                       tmp,
+                                       space_rem,
+                                       STR_TERMINATE);
                        }
-                       p2 = skip_string(*rdata,*rdata_len,p2,1);
+                       p2 = skip_string(*rdata,*rdata_len,p2);
                        if (!p2) {
                                return False;
                        }
@@ -3393,30 +3703,32 @@ static BOOL api_RNetUserGetInfo(connection_struct *conn, uint16 vuid,
        return(True);
 }
 
-static BOOL api_WWkstaUserLogon(connection_struct *conn,uint16 vuid,
+static bool api_WWkstaUserLogon(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
                                char **rdata,char **rparam,
                                int *rdata_len,int *rparam_len)
 {
+       struct smbd_server_connection *sconn = smbd_server_conn;
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        struct pack_desc desc;
        char* name;
                /* With share level security vuid will always be zero.
                   Don't depend on vuser being non-null !!. JRA */
-       user_struct *vuser = get_valid_user_struct(vuid);
+       user_struct *vuser = get_valid_user_struct(sconn, vuid);
 
        if (!str1 || !str2 || !p) {
                return False;
        }
 
        if(vuser != NULL) {
-               DEBUG(3,("  Username of UID %d is %s\n", (int)vuser->uid, 
-                       vuser->user.unix_name));
+               DEBUG(3,("  Username of UID %d is %s\n",
+                        (int)vuser->server_info->utok.uid,
+                        vuser->server_info->unix_name));
        }
 
        uLevel = get_safe_SVAL(param,tpscnt,p,0,-1);
@@ -3437,7 +3749,7 @@ static BOOL api_WWkstaUserLogon(connection_struct *conn,uint16 vuid,
                return False;
        }
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -3447,7 +3759,7 @@ static BOOL api_WWkstaUserLogon(connection_struct *conn,uint16 vuid,
        desc.buflen = mdrcnt;
        desc.subformat = NULL;
        desc.format = str2;
-  
+
        if (init_package(&desc,1,0)) {
                PACKI(&desc,"W",0);             /* code */
                PACKS(&desc,"B21",name);        /* eff. name */
@@ -3473,13 +3785,14 @@ static BOOL api_WWkstaUserLogon(connection_struct *conn,uint16 vuid,
                }
 
                PACKS(&desc,"z",lp_workgroup());/* domain */
-               PACKS(&desc,"z", vuser && vuser->logon_script ? vuser->logon_script :""); /* script path */
+               PACKS(&desc,"z", vuser ? pdb_get_logon_script(
+                             vuser->server_info->sam_account) : ""); /* script path */
                PACKI(&desc,"D",0x00000000);            /* reserved */
        }
 
        *rdata_len = desc.usedlen;
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -3496,7 +3809,7 @@ static BOOL api_WWkstaUserLogon(connection_struct *conn,uint16 vuid,
  api_WAccessGetUserPerms
 ****************************************************************************/
 
-static BOOL api_WAccessGetUserPerms(connection_struct *conn,uint16 vuid,
+static bool api_WAccessGetUserPerms(connection_struct *conn,uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -3504,15 +3817,15 @@ static BOOL api_WAccessGetUserPerms(connection_struct *conn,uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *user = skip_string(param,tpscnt,str2,1);
-       char *resource = skip_string(param,tpscnt,user,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *user = skip_string(param,tpscnt,str2);
+       char *resource = skip_string(param,tpscnt,user);
 
        if (!str1 || !str2 || !user || !resource) {
                return False;
        }
 
-       if (skip_string(param,tpscnt,resource,1) == NULL) {
+       if (skip_string(param,tpscnt,resource) == NULL) {
                return False;
        }
        DEBUG(3,("WAccessGetUserPerms user=%s resource=%s\n",user,resource));
@@ -3526,7 +3839,7 @@ static BOOL api_WAccessGetUserPerms(connection_struct *conn,uint16 vuid,
        }
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -3541,7 +3854,7 @@ static BOOL api_WAccessGetUserPerms(connection_struct *conn,uint16 vuid,
   api_WPrintJobEnumerate
   ****************************************************************************/
 
-static BOOL api_WPrintJobGetInfo(connection_struct *conn, uint16 vuid,
+static bool api_WPrintJobGetInfo(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -3549,8 +3862,8 @@ static BOOL api_WPrintJobGetInfo(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        int count;
        int i;
@@ -3598,7 +3911,7 @@ static BOOL api_WPrintJobGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -3624,7 +3937,7 @@ static BOOL api_WPrintJobGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -3640,7 +3953,7 @@ static BOOL api_WPrintJobGetInfo(connection_struct *conn, uint16 vuid,
        return True;
 }
 
-static BOOL api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
+static bool api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -3648,8 +3961,8 @@ static BOOL api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        char *name = p;
        int uLevel;
        int count;
@@ -3666,7 +3979,7 @@ static BOOL api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
        memset((char *)&desc,'\0',sizeof(desc));
        memset((char *)&status,'\0',sizeof(status));
 
-       p = skip_string(param,tpscnt,p,1);
+       p = skip_string(param,tpscnt,p);
        if (!p) {
                return False;
        }
@@ -3678,11 +3991,11 @@ static BOOL api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
        if (strcmp(str1,"zWrLeh") != 0) {
                return False;
        }
-    
+
        if (uLevel > 2) {
                return False;   /* defined only for uLevel 0,1,2 */
        }
-    
+
        if (!check_printjob_info(&desc,uLevel,str2)) { 
                return False;
        }
@@ -3694,7 +4007,7 @@ static BOOL api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
 
        count = print_queue_status(snum,&queue,&status);
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -3715,7 +4028,7 @@ static BOOL api_WPrintJobEnumerate(connection_struct *conn, uint16 vuid,
        *rdata_len = desc.usedlen;
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -3749,9 +4062,13 @@ static int check_printdest_info(struct pack_desc* desc,
                        desc->format = "zzzWWzzzWW";
                        break;
                default:
+                       DEBUG(0,("check_printdest_info: invalid level %d\n",
+                               uLevel));
                        return False;
        }
-       if (strcmp(desc->format,id) != 0) {
+       if (id == NULL || strcmp(desc->format,id) != 0) {
+               DEBUG(0,("check_printdest_info: invalid string %s\n", 
+                       id ? id : "<NULL>" ));
                return False;
        }
        return True;
@@ -3793,7 +4110,7 @@ static void fill_printdest_info(connection_struct *conn, int snum, int uLevel,
        }
 }
 
-static BOOL api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
+static bool api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -3801,8 +4118,8 @@ static BOOL api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        char* PrinterName = p;
        int uLevel;
        struct pack_desc desc;
@@ -3815,7 +4132,7 @@ static BOOL api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
 
        memset((char *)&desc,'\0',sizeof(desc));
 
-       p = skip_string(param,tpscnt,p,1);
+       p = skip_string(param,tpscnt,p);
        if (!p) {
                return False;
        }
@@ -3838,7 +4155,7 @@ static BOOL api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
                desc.neededlen = 0;
        } else {
                if (mdrcnt > 0) {
-                       *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+                       *rdata = smb_realloc_limit(*rdata,mdrcnt);
                        if (!*rdata) {
                                return False;
                        }
@@ -3859,7 +4176,7 @@ static BOOL api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
        }
 
        *rparam_len = 6;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -3873,7 +4190,7 @@ static BOOL api_WPrintDestGetInfo(connection_struct *conn, uint16 vuid,
        return True;
 }
 
-static BOOL api_WPrintDestEnum(connection_struct *conn, uint16 vuid,
+static bool api_WPrintDestEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -3881,8 +4198,8 @@ static BOOL api_WPrintDestEnum(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        int queuecnt;
        int i, n, succnt=0;
@@ -3915,7 +4232,7 @@ static BOOL api_WPrintDestEnum(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -3940,7 +4257,7 @@ static BOOL api_WPrintDestEnum(connection_struct *conn, uint16 vuid,
        *rdata_len = desc.usedlen;
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -3954,7 +4271,7 @@ static BOOL api_WPrintDestEnum(connection_struct *conn, uint16 vuid,
        return True;
 }
 
-static BOOL api_WPrintDriverEnum(connection_struct *conn, uint16 vuid,
+static bool api_WPrintDriverEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -3962,8 +4279,8 @@ static BOOL api_WPrintDriverEnum(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        int succnt;
        struct pack_desc desc;
@@ -3987,7 +4304,7 @@ static BOOL api_WPrintDriverEnum(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -4003,7 +4320,7 @@ static BOOL api_WPrintDriverEnum(connection_struct *conn, uint16 vuid,
        *rdata_len = desc.usedlen;
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -4017,7 +4334,7 @@ static BOOL api_WPrintDriverEnum(connection_struct *conn, uint16 vuid,
        return True;
 }
 
-static BOOL api_WPrintQProcEnum(connection_struct *conn, uint16 vuid,
+static bool api_WPrintQProcEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -4025,8 +4342,8 @@ static BOOL api_WPrintQProcEnum(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        int succnt;
        struct pack_desc desc;
@@ -4049,7 +4366,7 @@ static BOOL api_WPrintQProcEnum(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -4066,7 +4383,7 @@ static BOOL api_WPrintQProcEnum(connection_struct *conn, uint16 vuid,
        *rdata_len = desc.usedlen;
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -4080,7 +4397,7 @@ static BOOL api_WPrintQProcEnum(connection_struct *conn, uint16 vuid,
        return True;
 }
 
-static BOOL api_WPrintPortEnum(connection_struct *conn, uint16 vuid,
+static bool api_WPrintPortEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -4088,8 +4405,8 @@ static BOOL api_WPrintPortEnum(connection_struct *conn, uint16 vuid,
                                int *rdata_len,int *rparam_len)
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        int succnt;
        struct pack_desc desc;
@@ -4113,7 +4430,7 @@ static BOOL api_WPrintPortEnum(connection_struct *conn, uint16 vuid,
        }
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -4131,7 +4448,7 @@ static BOOL api_WPrintPortEnum(connection_struct *conn, uint16 vuid,
        *rdata_len = desc.usedlen;
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -4149,7 +4466,7 @@ static BOOL api_WPrintPortEnum(connection_struct *conn, uint16 vuid,
  List open sessions
  ****************************************************************************/
 
-static BOOL api_RNetSessionEnum(connection_struct *conn, uint16 vuid,
+static bool api_RNetSessionEnum(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt,int mprcnt,
@@ -4158,8 +4475,8 @@ static BOOL api_RNetSessionEnum(connection_struct *conn, uint16 vuid,
 
 {
        char *str1 = get_safe_str_ptr(param,tpscnt,param,2);
-       char *str2 = skip_string(param,tpscnt,str1,1);
-       char *p = skip_string(param,tpscnt,str2,1);
+       char *str2 = skip_string(param,tpscnt,str1);
+       char *p = skip_string(param,tpscnt,str2);
        int uLevel;
        struct pack_desc desc;
        struct sessionid *session_list;
@@ -4185,10 +4502,10 @@ static BOOL api_RNetSessionEnum(connection_struct *conn, uint16 vuid,
                return False;
        }
 
-       num_sessions = list_sessions(&session_list);
+       num_sessions = list_sessions(talloc_tos(), &session_list);
 
        if (mdrcnt > 0) {
-               *rdata = SMB_REALLOC_LIMIT(*rdata,mdrcnt);
+               *rdata = smb_realloc_limit(*rdata,mdrcnt);
                if (!*rdata) {
                        return False;
                }
@@ -4216,7 +4533,7 @@ static BOOL api_RNetSessionEnum(connection_struct *conn, uint16 vuid,
        *rdata_len = desc.usedlen;
 
        *rparam_len = 8;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -4234,13 +4551,13 @@ static BOOL api_RNetSessionEnum(connection_struct *conn, uint16 vuid,
  The buffer was too small.
  ****************************************************************************/
 
-static BOOL api_TooSmall(connection_struct *conn,uint16 vuid, char *param, char *data,
+static bool api_TooSmall(connection_struct *conn,uint16 vuid, char *param, char *data,
                         int mdrcnt, int mprcnt,
                         char **rdata, char **rparam,
                         int *rdata_len, int *rparam_len)
 {
        *rparam_len = MIN(*rparam_len,mprcnt);
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -4258,7 +4575,7 @@ static BOOL api_TooSmall(connection_struct *conn,uint16 vuid, char *param, char
  The request is not supported.
  ****************************************************************************/
 
-static BOOL api_Unsupported(connection_struct *conn, uint16 vuid,
+static bool api_Unsupported(connection_struct *conn, uint16 vuid,
                                char *param, int tpscnt,
                                char *data, int tdscnt,
                                int mdrcnt, int mprcnt,
@@ -4266,7 +4583,7 @@ static BOOL api_Unsupported(connection_struct *conn, uint16 vuid,
                                int *rdata_len, int *rparam_len)
 {
        *rparam_len = 4;
-       *rparam = SMB_REALLOC_LIMIT(*rparam,*rparam_len);
+       *rparam = smb_realloc_limit(*rparam,*rparam_len);
        if (!*rparam) {
                return False;
        }
@@ -4284,11 +4601,11 @@ static BOOL api_Unsupported(connection_struct *conn, uint16 vuid,
 static const struct {
        const char *name;
        int id;
-       BOOL (*fn)(connection_struct *, uint16,
+       bool (*fn)(connection_struct *, uint16,
                        char *, int,
                        char *, int,
                        int,int,char **,char **,int *,int *);
-       BOOL auth_user;         /* Deny anonymous access? */
+       bool auth_user;         /* Deny anonymous access? */
 } api_commands[] = {
        {"RNetShareEnum",       RAP_WshareEnum,         api_RNetShareEnum, True},
        {"RNetShareGetInfo",    RAP_WshareGetInfo,      api_RNetShareGetInfo},
@@ -4314,7 +4631,7 @@ static const struct {
        {"WPrintDestGetInfo",   RAP_WPrintDestGetInfo,  api_WPrintDestGetInfo},
        {"NetRemoteTOD",        RAP_NetRemoteTOD,       api_NetRemoteTOD},
        {"WPrintQueuePurge",    RAP_WPrintQPurge,       api_WPrintQueueCtrl},
-       {"NetServerEnum",       RAP_NetServerEnum2,     api_RNetServerEnum}, /* anon OK */
+       {"NetServerEnum2",      RAP_NetServerEnum2,     api_RNetServerEnum2}, /* anon OK */
        {"WAccessGetUserPerms",RAP_WAccessGetUserPerms,api_WAccessGetUserPerms},
        {"SetUserPassword",     RAP_WUserPasswordSet2,  api_SetUserPassword},
        {"WWkstaUserLogon",     RAP_WWkstaUserLogon,    api_WWkstaUserLogon},
@@ -4332,12 +4649,16 @@ static const struct {
 
 
 /****************************************************************************
- Handle remote api calls
- ****************************************************************************/
+ Handle remote api calls.
+****************************************************************************/
 
-int api_reply(connection_struct *conn,uint16 vuid,char *outbuf,char *data,char *params,
-                    int tdscnt,int tpscnt,int mdrcnt,int mprcnt)
+void api_reply(connection_struct *conn, uint16 vuid,
+              struct smb_request *req,
+              char *data, char *params,
+              int tdscnt, int tpscnt,
+              int mdrcnt, int mprcnt)
 {
+       struct smbd_server_connection *sconn = smbd_server_conn;
        int api_command;
        char *rdata = NULL;
        char *rparam = NULL;
@@ -4345,25 +4666,27 @@ int api_reply(connection_struct *conn,uint16 vuid,char *outbuf,char *data,char *
        const char *name2 = NULL;
        int rdata_len = 0;
        int rparam_len = 0;
-       BOOL reply=False;
+       bool reply=False;
        int i;
 
        if (!params) {
                DEBUG(0,("ERROR: NULL params in api_reply()\n"));
-               return 0;
+               reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
+               return;
        }
 
        if (tpscnt < 2) {
-               return 0;
+               reply_nterror(req, NT_STATUS_INVALID_PARAMETER);
+               return;
        }
        api_command = SVAL(params,0);
        /* Is there a string at position params+2 ? */
-       if (skip_string(params,tpscnt,params+2,1)) {
+       if (skip_string(params,tpscnt,params+2)) {
                name1 = params + 2;
        } else {
                name1 = "";
        }
-       name2 = skip_string(params,tpscnt,params+2,1);
+       name2 = skip_string(params,tpscnt,params+2);
        if (!name2) {
                name2 = "";
        }
@@ -4384,10 +4707,11 @@ int api_reply(connection_struct *conn,uint16 vuid,char *outbuf,char *data,char *
        /* Check whether this api call can be done anonymously */
 
        if (api_commands[i].auth_user && lp_restrict_anonymous()) {
-               user_struct *user = get_valid_user_struct(vuid);
+               user_struct *user = get_valid_user_struct(sconn, vuid);
 
-               if (!user || user->guest) {
-                       return ERROR_NT(NT_STATUS_ACCESS_DENIED);
+               if (!user || user->server_info->guest) {
+                       reply_nterror(req, NT_STATUS_ACCESS_DENIED);
+                       return;
                }
        }
 
@@ -4405,7 +4729,8 @@ int api_reply(connection_struct *conn,uint16 vuid,char *outbuf,char *data,char *
                DEBUG(0,("api_reply: malloc fail !\n"));
                SAFE_FREE(rdata);
                SAFE_FREE(rparam);
-               return -1;
+               reply_nterror(req, NT_STATUS_NO_MEMORY);
+               return;
        }
 
        reply = api_commands[i].fn(conn,
@@ -4429,10 +4754,11 @@ int api_reply(connection_struct *conn,uint16 vuid,char *outbuf,char *data,char *
 
        /* If api_Unsupported returns false we can't return anything. */
        if (reply) {
-               send_trans_reply(outbuf, rparam, rparam_len, rdata, rdata_len, False);
+               send_trans_reply(conn, req, rparam, rparam_len,
+                                rdata, rdata_len, False);
        }
 
        SAFE_FREE(rdata);
        SAFE_FREE(rparam);
-       return -1;
+       return;
 }