Remove smbldap_get_single_pstring() and all pstrings
[samba.git] / source / lib / util_str.c
index 0248ad63ad86441175194a815646a56d29767454..68b06a6d9013274f3ecdbac97259effbbf854261 100644 (file)
@@ -1,24 +1,25 @@
-/* 
+/*
    Unix SMB/CIFS implementation.
    Samba utility functions
-   
+
    Copyright (C) Andrew Tridgell 1992-2001
    Copyright (C) Simo Sorce      2001-2002
    Copyright (C) Martin Pool     2003
-   
+   Copyright (C) James Peach    2006
+   Copyright (C) Jeremy Allison  1992-2007
+
    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/>.
 */
 
 #include "includes.h"
  **/
 
 /**
- * Internal function to get the next token from a string, return False if none
+ * Internal function to get the next token from a string, return false if none
  * found.  Handles double-quotes.  This is the work horse function called by
  * next_token() and next_token_no_ltrim().
  *
- * Based on a routine by GJC@VILLAGE.COM. 
+ * Based on a routine by GJC@VILLAGE.COM.
  * Extensively modified by Andrew.Tridgell@anu.edu.au
  */
-static BOOL next_token_internal(const char **ptr,
+static bool next_token_internal(const char **ptr,
                                 char *buff,
                                 const char *sep,
                                 size_t bufsize,
-                                int ltrim)
+                                bool ltrim)
 {
        char *s;
        char *pbuf;
-       BOOL quoted;
+       bool quoted;
        size_t len=1;
 
        if (!ptr)
-               return(False);
+               return(false);
 
        s = (char *)*ptr;
 
@@ -57,16 +58,19 @@ static BOOL next_token_internal(const char **ptr,
                sep = " \t\n\r";
 
        /* find the first non sep char, if left-trimming is requested */
-       while (ltrim && *s && strchr_m(sep,*s))
-               s++;
-       
+       if (ltrim) {
+               while (*s && strchr_m(sep,*s))
+                       s++;
+       }
+
        /* nothing left? */
        if (! *s)
-               return(False);
-       
+               return(false);
+
        /* copy over the token */
        pbuf = buff;
-       for (quoted = False; len < bufsize && *s && (quoted || !strchr_m(sep,*s)); s++) {
+       for (quoted = false; len < bufsize && *s &&
+                       (quoted || !strchr_m(sep,*s)); s++) {
                if ( *s == '\"' ) {
                        quoted = !quoted;
                } else {
@@ -74,57 +78,55 @@ static BOOL next_token_internal(const char **ptr,
                        *pbuf++ = *s;
                }
        }
-       
-       *ptr = (*s) ? s+1 : s;  
+
+       *ptr = (*s) ? s+1 : s;
        *pbuf = 0;
-       
-       return(True);
+
+       return(true);
 }
 
 /*
- * Get the next token from a string, return False if none found.  Handles
+ * Get the next token from a string, return false if none found.  Handles
  * double-quotes.  This version trims leading separator characters before
  * looking for a token.
  */
-BOOL next_token(const char **ptr, char *buff, const char *sep, size_t bufsize)
+bool next_token(const char **ptr, char *buff, const char *sep, size_t bufsize)
 {
-    return next_token_internal(ptr, buff, sep, bufsize, True);
+    return next_token_internal(ptr, buff, sep, bufsize, true);
 }
 
 /*
- * Get the next token from a string, return False if none found.  Handles
+ * Get the next token from a string, return false if none found.  Handles
  * double-quotes.  This version does not trim leading separator characters
  * before looking for a token.
  */
-BOOL next_token_no_ltrim(const char **ptr,
+bool next_token_no_ltrim(const char **ptr,
                          char *buff,
                          const char *sep,
                          size_t bufsize)
 {
-    return next_token_internal(ptr, buff, sep, bufsize, False);
+    return next_token_internal(ptr, buff, sep, bufsize, false);
 }
 
 /**
-This is like next_token but is not re-entrant and "remembers" the first 
+This is like next_token but is not re-entrant and "remembers" the first
 parameter so you can pass NULL. This is useful for user interface code
 but beware the fact that it is not re-entrant!
 **/
 
 static const char *last_ptr=NULL;
 
-BOOL next_token_nr(const char **ptr,char *buff, const char *sep, size_t bufsize)
+bool next_token_nr(const char **ptr,char *buff, const char *sep, size_t bufsize)
 {
-       BOOL ret;
+       bool ret;
        if (!ptr)
                ptr = &last_ptr;
 
        ret = next_token(ptr, buff, sep, bufsize);
        last_ptr = *ptr;
-       return ret;     
+       return ret;
 }
 
-static uint16 tmpbuf[sizeof(pstring)];
-
 void set_first_token(char *ptr)
 {
        last_ptr = ptr;
@@ -158,14 +160,14 @@ char **toktocliplist(int *ctok, const char *sep)
                while(*s && strchr_m(sep,*s))
                        *s++=0;
        } while(*s);
-       
+
        *ctok=ictok;
        s=(char *)last_ptr;
-       
+
        if (!(ret=iret=SMB_MALLOC_ARRAY(char *,ictok+1)))
                return NULL;
-       
-       while(ictok--) {    
+
+       while(ictok--) {
                *iret++=s;
                if (ictok > 0) {
                        while(*s++)
@@ -206,7 +208,7 @@ char **toktocliplist(int *ctok, const char *sep)
  * different, we'd need to restart the whole thing.
  *
  * Even better is to implement strcasecmp for each encoding and use a
- * function pointer. 
+ * function pointer.
  **/
 int StrCaseCmp(const char *s, const char *t)
 {
@@ -226,7 +228,8 @@ int StrCaseCmp(const char *s, const char *t)
                else if (!*pt)
                        return +1; /* t is a prefix */
                else if ((*ps & 0x80) || (*pt & 0x80))
-                       /* not ascii anymore, do it the hard way from here on in */
+                       /* not ascii anymore, do it the hard way
+                        * from here on in */
                        break;
 
                us = toupper_ascii(*ps);
@@ -241,19 +244,21 @@ int StrCaseCmp(const char *s, const char *t)
 
        size = push_ucs2_allocate(&buffer_s, ps);
        if (size == (size_t)-1) {
-               return strcmp(ps, pt); 
+               return strcmp(ps, pt);
                /* Not quite the right answer, but finding the right one
-                  under this failure case is expensive, and it's pretty close */
+                  under this failure case is expensive, and it's pretty
+                  close */
        }
-       
+
        size = push_ucs2_allocate(&buffer_t, pt);
        if (size == (size_t)-1) {
                SAFE_FREE(buffer_s);
-               return strcmp(ps, pt); 
+               return strcmp(ps, pt);
                /* Not quite the right answer, but finding the right one
-                  under this failure case is expensive, and it's pretty close */
+                  under this failure case is expensive, and it's pretty
+                  close */
        }
-       
+
        ret = strcasecmp_w(buffer_s, buffer_t);
        SAFE_FREE(buffer_s);
        SAFE_FREE(buffer_t);
@@ -264,12 +269,63 @@ int StrCaseCmp(const char *s, const char *t)
 /**
  Case insensitive string compararison, length limited.
 **/
-int StrnCaseCmp(const char *s, const char *t, size_t n)
+int StrnCaseCmp(const char *s, const char *t, size_t len)
 {
-       pstring buf1, buf2;
-       unix_strupper(s, strlen(s)+1, buf1, sizeof(buf1));
-       unix_strupper(t, strlen(t)+1, buf2, sizeof(buf2));
-       return strncmp(buf1,buf2,n);
+       size_t n = 0;
+       const char *ps, *pt;
+       size_t size;
+       smb_ucs2_t *buffer_s, *buffer_t;
+       int ret;
+
+       for (ps = s, pt = t; n < len ; ps++, pt++, n++) {
+               char us, ut;
+
+               if (!*ps && !*pt)
+                       return 0; /* both ended */
+               else if (!*ps)
+                       return -1; /* s is a prefix */
+               else if (!*pt)
+                       return +1; /* t is a prefix */
+               else if ((*ps & 0x80) || (*pt & 0x80))
+                       /* not ascii anymore, do it the
+                        * hard way from here on in */
+                       break;
+
+               us = toupper_ascii(*ps);
+               ut = toupper_ascii(*pt);
+               if (us == ut)
+                       continue;
+               else if (us < ut)
+                       return -1;
+               else if (us > ut)
+                       return +1;
+       }
+
+       if (n == len) {
+               return 0;
+       }
+
+       size = push_ucs2_allocate(&buffer_s, ps);
+       if (size == (size_t)-1) {
+               return strncmp(ps, pt, len-n);
+               /* Not quite the right answer, but finding the right one
+                  under this failure case is expensive,
+                  and it's pretty close */
+       }
+
+       size = push_ucs2_allocate(&buffer_t, pt);
+       if (size == (size_t)-1) {
+               SAFE_FREE(buffer_s);
+               return strncmp(ps, pt, len-n);
+               /* Not quite the right answer, but finding the right one
+                  under this failure case is expensive,
+                  and it's pretty close */
+       }
+
+       ret = strncasecmp_w(buffer_s, buffer_t, len-n);
+       SAFE_FREE(buffer_s);
+       SAFE_FREE(buffer_t);
+       return ret;
 }
 
 /**
@@ -277,13 +333,13 @@ int StrnCaseCmp(const char *s, const char *t, size_t n)
  *
  * @note The comparison is case-insensitive.
  **/
-BOOL strequal(const char *s1, const char *s2)
+bool strequal(const char *s1, const char *s2)
 {
        if (s1 == s2)
-               return(True);
+               return(true);
        if (!s1 || !s2)
-               return(False);
-  
+               return(false);
+
        return(StrCaseCmp(s1,s2)==0);
 }
 
@@ -292,13 +348,13 @@ BOOL strequal(const char *s1, const char *s2)
  *
  * @note The comparison is case-insensitive.
  **/
-BOOL strnequal(const char *s1,const char *s2,size_t n)
+bool strnequal(const char *s1,const char *s2,size_t n)
 {
        if (s1 == s2)
-               return(True);
+               return(true);
        if (!s1 || !s2 || !n)
-               return(False);
-  
+               return(false);
+
        return(StrnCaseCmp(s1,s2,n)==0);
 }
 
@@ -306,13 +362,13 @@ BOOL strnequal(const char *s1,const char *s2,size_t n)
  Compare 2 strings (case sensitive).
 **/
 
-BOOL strcsequal(const char *s1,const char *s2)
+bool strcsequal(const char *s1,const char *s2)
 {
        if (s1 == s2)
-               return(True);
+               return(true);
        if (!s1 || !s2)
-               return(False);
-  
+               return(false);
+
        return(strcmp(s1,s2)==0);
 }
 
@@ -337,8 +393,8 @@ int strwicmp(const char *psz1, const char *psz2)
                        psz1++;
                while (isspace((int)*psz2))
                        psz2++;
-               if (toupper_ascii(*psz1) != toupper_ascii(*psz2) || *psz1 == '\0'
-                   || *psz2 == '\0')
+               if (toupper_ascii(*psz1) != toupper_ascii(*psz2) ||
+                               *psz1 == '\0' || *psz2 == '\0')
                        break;
                psz1++;
                psz2++;
@@ -353,11 +409,11 @@ int strwicmp(const char *psz1, const char *psz2)
 
 char *strupper_static(const char *s)
 {
-       static pstring str;
+       static char *str = NULL;
 
-       pstrcpy(str, s);
+       SAFE_FREE(str);
+       str = SMB_STRDUP(s);
        strupper_m(str);
-
        return str;
 }
 
@@ -377,11 +433,11 @@ void strnorm(char *s, int case_default)
  Check if a string is in "normal" case.
 **/
 
-BOOL strisnormal(const char *s, int case_default)
+bool strisnormal(const char *s, int case_default)
 {
        if (case_default == CASE_UPPER)
                return(!strhaslower(s));
-       
+
        return(!strhasupper(s));
 }
 
@@ -390,8 +446,7 @@ BOOL strisnormal(const char *s, int case_default)
  String replace.
  NOTE: oldc and newc must be 7 bit characters
 **/
-
-void string_replace( pstring s, char oldc, char newc )
+void string_replace( char *s, char oldc, char newc )
 {
        char *p;
 
@@ -403,8 +458,9 @@ void string_replace( pstring s, char oldc, char newc )
        for (p = s; *p; p++) {
                if (*p & 0x80) /* mb string - slow path. */
                        break;
-               if (*p == oldc)
+               if (*p == oldc) {
                        *p = newc;
+               }
        }
 
        if (!*p)
@@ -415,22 +471,56 @@ void string_replace( pstring s, char oldc, char newc )
        /* With compose characters we must restart from the beginning. JRA. */
        p = s;
 #endif
-       push_ucs2(NULL, tmpbuf, p, sizeof(tmpbuf), STR_TERMINATE);
-       string_replace_w(tmpbuf, UCS2_CHAR(oldc), UCS2_CHAR(newc));
-       pull_ucs2(NULL, p, tmpbuf, -1, sizeof(tmpbuf), STR_TERMINATE);
+
+       while (*p) {
+               size_t c_size;
+               next_codepoint(p, &c_size);
+
+               if (c_size == 1) {
+                       if (*p == oldc) {
+                               *p = newc;
+                       }
+               }
+               p += c_size;
+       }
 }
 
 /**
Skip past some strings in a buffer.
-**/
*  Skip past some strings in a buffer - old version - no checks.
+ *  **/
 
-char *skip_string(char *buf,size_t n)
+char *push_skip_string(char *buf)
 {
-       while (n--)
-               buf += strlen(buf) + 1;
+       buf += strlen(buf) + 1;
        return(buf);
 }
 
+/**
+ Skip past a string in a buffer. Buffer may not be
+ null terminated. end_ptr points to the first byte after
+ then end of the buffer.
+**/
+
+char *skip_string(const char *base, size_t len, char *buf)
+{
+       const char *end_ptr = base + len;
+
+       if (end_ptr < base || !base || !buf || buf >= end_ptr) {
+               return NULL;
+       }
+
+       /* Skip the string */
+       while (*buf) {
+               buf++;
+               if (buf >= end_ptr) {
+                       return NULL;
+               }
+       }
+       /* Skip the '\0' */
+       buf++;
+       return buf;
+}
+
 /**
  Count the number of characters in a string. Normally this will
  be the same as the number of bytes in a string for single byte strings,
@@ -439,9 +529,14 @@ char *skip_string(char *buf,size_t n)
 
 size_t str_charnum(const char *s)
 {
-       uint16 tmpbuf2[sizeof(pstring)];
-       push_ucs2(NULL, tmpbuf2,s, sizeof(tmpbuf2), STR_TERMINATE);
-       return strlen_w(tmpbuf2);
+       size_t ret;
+       smb_ucs2_t *tmpbuf2 = NULL;
+       if (push_ucs2_allocate(&tmpbuf2, s) == (size_t)-1) {
+               return 0;
+       }
+       ret = strlen_w(tmpbuf2);
+       SAFE_FREE(tmpbuf2);
+       return ret;
 }
 
 /**
@@ -452,20 +547,25 @@ size_t str_charnum(const char *s)
 
 size_t str_ascii_charnum(const char *s)
 {
-       pstring tmpbuf2;
-       push_ascii(tmpbuf2, s, sizeof(tmpbuf2), STR_TERMINATE);
-       return strlen(tmpbuf2);
+       size_t ret;
+       char *tmpbuf2 = NULL;
+       if (push_ascii_allocate(&tmpbuf2, s) == (size_t)-1) {
+               return 0;
+       }
+       ret = strlen(tmpbuf2);
+       SAFE_FREE(tmpbuf2);
+       return ret;
 }
 
-BOOL trim_char(char *s,char cfront,char cback)
+bool trim_char(char *s,char cfront,char cback)
 {
-       BOOL ret = False;
+       bool ret = false;
        char *ep;
        char *fp = s;
 
        /* Ignore null or empty strings. */
        if (!s || (s[0] == '\0'))
-               return False;
+               return false;
 
        if (cfront) {
                while (*fp && *fp == cfront)
@@ -473,17 +573,17 @@ BOOL trim_char(char *s,char cfront,char cback)
                if (!*fp) {
                        /* We ate the string. */
                        s[0] = '\0';
-                       return True;
+                       return true;
                }
                if (fp != s)
-                       ret = True;
+                       ret = true;
        }
 
        ep = fp + strlen(fp) - 1;
        if (cback) {
                /* Attempt ascii only. Bail for mb strings. */
                while ((ep >= fp) && (*ep == cback)) {
-                       ret = True;
+                       ret = true;
                        if ((ep > fp) && (((unsigned char)ep[-1]) & 0x80)) {
                                /* Could be mb... bail back to tim_string. */
                                char fs[2], bs[2];
@@ -501,7 +601,7 @@ BOOL trim_char(char *s,char cfront,char cback)
                if (ep < fp) {
                        /* We ate the string. */
                        s[0] = '\0';
-                       return True;
+                       return true;
                }
        }
 
@@ -514,16 +614,16 @@ BOOL trim_char(char *s,char cfront,char cback)
  Trim the specified elements off the front and back of a string.
 **/
 
-BOOL trim_string(char *s,const char *front,const char *back)
+bool trim_string(char *s,const char *front,const char *back)
 {
-       BOOL ret = False;
+       bool ret = false;
        size_t front_len;
        size_t back_len;
        size_t len;
 
        /* Ignore null or empty strings. */
        if (!s || (s[0] == '\0'))
-               return False;
+               return false;
 
        front_len       = front? strlen(front) : 0;
        back_len        = back? strlen(back) : 0;
@@ -536,15 +636,16 @@ BOOL trim_string(char *s,const char *front,const char *back)
                         * easily overlap. Found by valgrind. JRA. */
                        memmove(s, s+front_len, (len-front_len)+1);
                        len -= front_len;
-                       ret=True;
+                       ret=true;
                }
        }
-       
+
        if (back_len) {
-               while ((len >= back_len) && strncmp(s+len-back_len,back,back_len)==0) {
+               while ((len >= back_len) &&
+                               strncmp(s+len-back_len,back,back_len)==0) {
                        s[len-back_len]='\0';
                        len -= back_len;
-                       ret=True;
+                       ret=true;
                }
        }
        return ret;
@@ -554,28 +655,48 @@ BOOL trim_string(char *s,const char *front,const char *back)
  Does a string have any uppercase chars in it?
 **/
 
-BOOL strhasupper(const char *s)
+bool strhasupper(const char *s)
 {
-       smb_ucs2_t *ptr;
-       push_ucs2(NULL, tmpbuf,s, sizeof(tmpbuf), STR_TERMINATE);
-       for(ptr=tmpbuf;*ptr;ptr++)
-               if(isupper_w(*ptr))
-                       return True;
-       return(False);
+       smb_ucs2_t *tmp, *p;
+       bool ret;
+
+       if (push_ucs2_allocate(&tmp, s) == -1) {
+               return false;
+       }
+
+       for(p = tmp; *p != 0; p++) {
+               if(isupper_w(*p)) {
+                       break;
+               }
+       }
+
+       ret = (*p != 0);
+       SAFE_FREE(tmp);
+       return ret;
 }
 
 /**
  Does a string have any lowercase chars in it?
 **/
 
-BOOL strhaslower(const char *s)
+bool strhaslower(const char *s)
 {
-       smb_ucs2_t *ptr;
-       push_ucs2(NULL, tmpbuf,s, sizeof(tmpbuf), STR_TERMINATE);
-       for(ptr=tmpbuf;*ptr;ptr++)
-               if(islower_w(*ptr))
-                       return True;
-       return(False);
+       smb_ucs2_t *tmp, *p;
+       bool ret;
+
+       if (push_ucs2_allocate(&tmp, s) == -1) {
+               return false;
+       }
+
+       for(p = tmp; *p != 0; p++) {
+               if(islower_w(*p)) {
+                       break;
+               }
+       }
+
+       ret = (*p != 0);
+       SAFE_FREE(tmp);
+       return ret;
 }
 
 /**
@@ -605,12 +726,17 @@ size_t count_chars(const char *s,char c)
  include the terminating zero.
 **/
 
-char *safe_strcpy_fn(const char *fn, int line, char *dest,const char *src, size_t maxlength)
+char *safe_strcpy_fn(const char *fn,
+               int line,
+               char *dest,
+               const char *src,
+               size_t maxlength)
 {
        size_t len;
 
        if (!dest) {
-               DEBUG(0,("ERROR: NULL dest in safe_strcpy, called from [%s][%d]\n", fn, line));
+               DEBUG(0,("ERROR: NULL dest in safe_strcpy, "
+                       "called from [%s][%d]\n", fn, line));
                return NULL;
        }
 
@@ -621,38 +747,44 @@ char *safe_strcpy_fn(const char *fn, int line, char *dest,const char *src, size_
        if (!src) {
                *dest = 0;
                return dest;
-       }  
+       }
 
        len = strnlen(src, maxlength+1);
 
        if (len > maxlength) {
-               DEBUG(0,("ERROR: string overflow by %lu (%lu - %lu) in safe_strcpy [%.50s]\n",
-                        (unsigned long)(len-maxlength), (unsigned long)len, 
+               DEBUG(0,("ERROR: string overflow by "
+                       "%lu (%lu - %lu) in safe_strcpy [%.50s]\n",
+                        (unsigned long)(len-maxlength), (unsigned long)len,
                         (unsigned long)maxlength, src));
                len = maxlength;
        }
-      
+
        memmove(dest, src, len);
        dest[len] = 0;
        return dest;
-}  
+}
 
 /**
  Safe string cat into a string. maxlength does not
  include the terminating zero.
 **/
-char *safe_strcat_fn(const char *fn, int line, char *dest, const char *src, size_t maxlength)
+char *safe_strcat_fn(const char *fn,
+               int line,
+               char *dest,
+               const char *src,
+               size_t maxlength)
 {
        size_t src_len, dest_len;
 
        if (!dest) {
-               DEBUG(0,("ERROR: NULL dest in safe_strcat, called from [%s][%d]\n", fn, line));
+               DEBUG(0,("ERROR: NULL dest in safe_strcat, "
+                       "called from [%s][%d]\n", fn, line));
                return NULL;
        }
 
        if (!src)
                return dest;
-       
+
        src_len = strnlen(src, maxlength + 1);
        dest_len = strnlen(dest, maxlength + 1);
 
@@ -661,7 +793,8 @@ char *safe_strcat_fn(const char *fn, int line, char *dest, const char *src, size
 #endif
 
        if (src_len + dest_len > maxlength) {
-               DEBUG(0,("ERROR: string overflow by %d in safe_strcat [%.50s]\n",
+               DEBUG(0,("ERROR: string overflow by %d "
+                       "in safe_strcat [%.50s]\n",
                         (int)(src_len + dest_len - maxlength), src));
                if (maxlength > dest_len) {
                        memcpy(&dest[dest_len], src, maxlength - dest_len);
@@ -681,7 +814,13 @@ char *safe_strcat_fn(const char *fn, int line, char *dest, const char *src, size
  and replaces with '_'. Deliberately does *NOT* check for multibyte
  characters. Don't change it !
 **/
-char *alpha_strcpy_fn(const char *fn, int line, char *dest, const char *src, const char *other_safe_chars, size_t maxlength)
+
+char *alpha_strcpy_fn(const char *fn,
+               int line,
+               char *dest,
+               const char *src,
+               const char *other_safe_chars,
+               size_t maxlength)
 {
        size_t len, i;
 
@@ -690,14 +829,15 @@ char *alpha_strcpy_fn(const char *fn, int line, char *dest, const char *src, con
 #endif
 
        if (!dest) {
-               DEBUG(0,("ERROR: NULL dest in alpha_strcpy, called from [%s][%d]\n", fn, line));
+               DEBUG(0,("ERROR: NULL dest in alpha_strcpy, "
+                       "called from [%s][%d]\n", fn, line));
                return NULL;
        }
 
        if (!src) {
                *dest = 0;
                return dest;
-       }  
+       }
 
        len = strlen(src);
        if (len >= maxlength)
@@ -708,7 +848,8 @@ char *alpha_strcpy_fn(const char *fn, int line, char *dest, const char *src, con
 
        for(i = 0; i < len; i++) {
                int val = (src[i] & 0xff);
-               if (isupper_ascii(val) || islower_ascii(val) || isdigit(val) || strchr_m(other_safe_chars, val))
+               if (isupper_ascii(val) || islower_ascii(val) ||
+                               isdigit(val) || strchr_m(other_safe_chars, val))
                        dest[i] = src[i];
                else
                        dest[i] = '_';
@@ -732,7 +873,8 @@ char *StrnCpy_fn(const char *fn, int line,char *dest,const char *src,size_t n)
 #endif
 
        if (!dest) {
-               DEBUG(0,("ERROR: NULL dest in StrnCpy, called from [%s][%d]\n", fn, line));
+               DEBUG(0,("ERROR: NULL dest in StrnCpy, "
+                       "called from [%s][%d]\n", fn, line));
                return(NULL);
        }
 
@@ -740,7 +882,7 @@ char *StrnCpy_fn(const char *fn, int line,char *dest,const char *src,size_t n)
                *dest = 0;
                return(dest);
        }
-       
+
        while (n-- && (*d = *src)) {
                d++;
                src++;
@@ -823,7 +965,7 @@ size_t strhex_to_str(char *p, size_t len, const char *strhex)
        return num_chars;
 }
 
-DATA_BLOB strhex_to_data_blob(TALLOC_CTX *mem_ctx, const char *strhex) 
+DATA_BLOB strhex_to_data_blob(TALLOC_CTX *mem_ctx, const char *strhex)
 {
        DATA_BLOB ret_blob;
 
@@ -832,8 +974,8 @@ DATA_BLOB strhex_to_data_blob(TALLOC_CTX *mem_ctx, const char *strhex)
        else
                ret_blob = data_blob(NULL, strlen(strhex)/2+1);
 
-       ret_blob.length = strhex_to_str((char*)ret_blob.data,   
-                                       strlen(strhex), 
+       ret_blob.length = strhex_to_str((char*)ret_blob.data,
+                                       strlen(strhex),
                                        strhex);
 
        return ret_blob;
@@ -860,24 +1002,40 @@ char *hex_encode(TALLOC_CTX *mem_ctx, const unsigned char *buff_in, size_t len)
  Check if a string is part of a list.
 **/
 
-BOOL in_list(const char *s, const char *list, BOOL casesensitive)
+bool in_list(const char *s, const char *list, bool casesensitive)
 {
-       pstring tok;
+       char *tok;
        const char *p=list;
+       size_t bufsize = strlen(list);
+       bool ret = false;
 
        if (!list)
-               return(False);
+               return(false);
+
+       /* We know a token can't be larger
+        * than the entire list. */
+
+       tok = SMB_MALLOC_ARRAY(char, bufsize+1);
+       if (!tok) {
+               return false;
+       }
 
-       while (next_token(&p,tok,LIST_SEP,sizeof(tok))) {
+       while (next_token(&p,tok,LIST_SEP,bufsize+1)) {
                if (casesensitive) {
-                       if (strcmp(tok,s) == 0)
-                               return(True);
+                       if (strcmp(tok,s) == 0) {
+                               ret = true;
+                               break;
+                       }
                } else {
-                       if (StrCaseCmp(tok,s) == 0)
-                               return(True);
+                       if (StrCaseCmp(tok,s) == 0) {
+                               ret = true;
+                               break;
+                       }
                }
        }
-       return(False);
+
+       SAFE_FREE(tok);
+       return ret;
 }
 
 /* this is used to prevent lots of mallocs of size 1 */
@@ -887,11 +1045,11 @@ static const char *null_string = "";
  Set a string value, allocing the space for the string
 **/
 
-static BOOL string_init(char **dest,const char *src)
+static bool string_init(char **dest,const char *src)
 {
        size_t l;
 
-       if (!src)     
+       if (!src)
                src = "";
 
        l = strlen(src);
@@ -902,10 +1060,10 @@ static BOOL string_init(char **dest,const char *src)
                (*dest) = SMB_STRDUP(src);
                if ((*dest) == NULL) {
                        DEBUG(0,("Out of memory in string_init\n"));
-                       return False;
+                       return false;
                }
        }
-       return(True);
+       return(true);
 }
 
 /**
@@ -926,17 +1084,17 @@ void string_free(char **s)
  for the string
 **/
 
-BOOL string_set(char **dest,const char *src)
+bool string_set(char **dest,const char *src)
 {
        string_free(dest);
        return(string_init(dest,src));
 }
 
 /**
- Substitute a string for a pattern in another string. Make sure there is 
+ Substitute a string for a pattern in another string. Make sure there is
  enough room!
 
- This routine looks for pattern in s and replaces it with 
+ This routine looks for pattern in s and replaces it with
  insert. It may do multiple replacements or just one.
 
  Any of " ; ' $ or ` in the insert string are replaced with _
@@ -944,8 +1102,9 @@ BOOL string_set(char **dest,const char *src)
  use of len==0 which was for no length checks to be done.
 **/
 
-void string_sub2(char *s,const char *pattern, const char *insert, size_t len, 
-                BOOL remove_unsafe_characters, BOOL replace_once, BOOL allow_trailing_dollar)
+void string_sub2(char *s,const char *pattern, const char *insert, size_t len,
+                bool remove_unsafe_characters, bool replace_once,
+                bool allow_trailing_dollar)
 {
        char *p;
        ssize_t ls,lp,li, i;
@@ -962,7 +1121,8 @@ void string_sub2(char *s,const char *pattern, const char *insert, size_t len,
 
        while (lp <= ls && (p = strstr_m(s,pattern))) {
                if (ls + (li-lp) >= len) {
-                       DEBUG(0,("ERROR: string overflow by %d in string_sub(%.50s, %d)\n", 
+                       DEBUG(0,("ERROR: string overflow by "
+                               "%d in string_sub(%.50s, %d)\n",
                                 (int)(ls + (li-lp) - len),
                                 pattern, (int)len));
                        break;
@@ -977,7 +1137,8 @@ void string_sub2(char *s,const char *pattern, const char *insert, size_t len,
                        case '\'':
                        case ';':
                        case '$':
-                               /* allow a trailing $ (as in machine accounts) */
+                               /* allow a trailing $
+                                * (as in machine accounts) */
                                if (allow_trailing_dollar && (i == li - 1 )) {
                                        p[i] = insert[i];
                                        break;
@@ -987,8 +1148,9 @@ void string_sub2(char *s,const char *pattern, const char *insert, size_t len,
                        case '\n':
                                if ( remove_unsafe_characters ) {
                                        p[i] = '_';
-                                       /* yes this break should be here since we want to 
-                                          fall throw if not replacing unsafe chars */
+                                       /* yes this break should be here
+                                        * since we want to fall throw if
+                                        * not replacing unsafe chars */
                                        break;
                                }
                        default:
@@ -1003,14 +1165,15 @@ void string_sub2(char *s,const char *pattern, const char *insert, size_t len,
        }
 }
 
-void string_sub_once(char *s, const char *pattern, const char *insert, size_t len)
+void string_sub_once(char *s, const char *pattern,
+               const char *insert, size_t len)
 {
-       string_sub2( s, pattern, insert, len, True, True, False );
+       string_sub2( s, pattern, insert, len, true, true, false );
 }
 
 void string_sub(char *s,const char *pattern, const char *insert, size_t len)
 {
-       string_sub2( s, pattern, insert, len, True, False, False );
+       string_sub2( s, pattern, insert, len, true, false, false );
 }
 
 void fstring_sub(char *s,const char *pattern,const char *insert)
@@ -1024,14 +1187,17 @@ void pstring_sub(char *s,const char *pattern,const char *insert)
 }
 
 /**
- Similar to string_sub, but it will accept only allocated strings
+ Similar to string_sub2, but it will accept only allocated strings
  and may realloc them so pay attention at what you pass on no
  pointers inside strings, no pstrings or const may be passed
  as string.
 **/
 
-char *realloc_string_sub(char *string, const char *pattern,
-                        const char *insert)
+char *realloc_string_sub2(char *string,
+                       const char *pattern,
+                       const char *insert,
+                       bool remove_unsafe_characters,
+                       bool allow_trailing_dollar)
 {
        char *p, *in;
        char *s;
@@ -1058,22 +1224,31 @@ char *realloc_string_sub(char *string, const char *pattern,
                        case '\'':
                        case ';':
                        case '$':
+                               /* allow a trailing $
+                                * (as in machine accounts) */
+                               if (allow_trailing_dollar && (i == li - 1 )) {
+                                       break;
+                               }
                        case '%':
                        case '\r':
                        case '\n':
-                               in[i] = '_';
+                               if ( remove_unsafe_characters ) {
+                                       in[i] = '_';
+                                       break;
+                               }
                        default:
                                /* ok */
                                break;
                }
        }
-       
+
        while ((p = strstr_m(s,pattern))) {
                if (ld > 0) {
                        int offset = PTR_DIFF(s,string);
-                       string = SMB_REALLOC(string, ls + ld + 1);
+                       string = (char *)SMB_REALLOC(string, ls + ld + 1);
                        if (!string) {
-                               DEBUG(0, ("realloc_string_sub: out of memory!\n"));
+                               DEBUG(0, ("realloc_string_sub: "
+                                       "out of memory!\n"));
                                SAFE_FREE(in);
                                return NULL;
                        }
@@ -1090,22 +1265,34 @@ char *realloc_string_sub(char *string, const char *pattern,
        return string;
 }
 
-/* Same as string_sub, but returns a talloc'ed string */
+char *realloc_string_sub(char *string,
+                       const char *pattern,
+                       const char *insert)
+{
+       return realloc_string_sub2(string, pattern, insert, true, false);
+}
 
-char *talloc_string_sub(TALLOC_CTX *mem_ctx, const char *src,
-                       const char *pattern, const char *insert)
+/*
+ * Internal guts of talloc_string_sub and talloc_all_string_sub.
+ * 'filter' differentiates between them.
+ */
+
+static char *talloc_string_sub_internal(TALLOC_CTX *mem_ctx, const char *src,
+                       const char *pattern, const char *insert, bool filter)
 {
        char *p, *in;
        char *s;
        char *string;
        ssize_t ls,lp,li,ld, i;
 
-       if (!insert || !pattern || !*pattern || !src || !*src)
+       if (!insert || !pattern || !*pattern || !src || !*src) {
                return NULL;
+       }
 
        string = talloc_strdup(mem_ctx, src);
        if (string == NULL) {
-               DEBUG(0, ("talloc_strdup failed\n"));
+               DEBUG(0, ("talloc_string_sub_internal: "
+                       "talloc_strdup failed\n"));
                return NULL;
        }
 
@@ -1113,34 +1300,38 @@ char *talloc_string_sub(TALLOC_CTX *mem_ctx, const char *src,
 
        in = SMB_STRDUP(insert);
        if (!in) {
-               DEBUG(0, ("talloc_string_sub: out of memory!\n"));
+               DEBUG(0, ("talloc_string_sub_internal: ENOMEM\n"));
                return NULL;
        }
        ls = (ssize_t)strlen(s);
        lp = (ssize_t)strlen(pattern);
        li = (ssize_t)strlen(insert);
        ld = li - lp;
-       for (i=0;i<li;i++) {
-               switch (in[i]) {
-                       case '`':
-                       case '"':
-                       case '\'':
-                       case ';':
-                       case '$':
-                       case '%':
-                       case '\r':
-                       case '\n':
-                               in[i] = '_';
-                       default:
-                               /* ok */
-                               break;
+
+       if (filter) {
+               for (i=0;i<li;i++) {
+                       switch (in[i]) {
+                               case '`':
+                               case '"':
+                               case '\'':
+                               case ';':
+                               case '$':
+                               case '%':
+                               case '\r':
+                               case '\n':
+                                       in[i] = '_';
+                               default:
+                                       /* ok */
+                                       break;
+                       }
                }
        }
-       
+
        while ((p = strstr_m(s,pattern))) {
                if (ld > 0) {
                        int offset = PTR_DIFF(s,string);
-                       string = TALLOC_REALLOC(mem_ctx, string, ls + ld + 1);
+                       string = (char *)TALLOC_REALLOC(mem_ctx, string,
+                                                       ls + ld + 1);
                        if (!string) {
                                DEBUG(0, ("talloc_string_sub: out of "
                                          "memory!\n"));
@@ -1160,8 +1351,16 @@ char *talloc_string_sub(TALLOC_CTX *mem_ctx, const char *src,
        return string;
 }
 
+/* Same as string_sub, but returns a talloc'ed string */
+
+char *talloc_string_sub(TALLOC_CTX *mem_ctx, const char *src,
+                       const char *pattern, const char *insert)
+{
+       return talloc_string_sub_internal(mem_ctx, src, pattern, insert, true);
+}
+
 /**
- Similar to string_sub() but allows for any character to be substituted. 
+ Similar to string_sub() but allows for any character to be substituted.
  Use with caution!
  if len==0 then the string cannot be extended. This is different from the old
  use of len==0 which was for no length checks to be done.
@@ -1181,13 +1380,14 @@ void all_string_sub(char *s,const char *pattern,const char *insert, size_t len)
 
        if (!*pattern)
                return;
-       
+
        if (len == 0)
                len = ls + 1; /* len is number of *bytes* */
-       
+
        while (lp <= ls && (p = strstr_m(s,pattern))) {
                if (ls + (li-lp) >= len) {
-                       DEBUG(0,("ERROR: string overflow by %d in all_string_sub(%.50s, %d)\n", 
+                       DEBUG(0,("ERROR: string overflow by "
+                               "%d in all_string_sub(%.50s, %d)\n",
                                 (int)(ls + (li-lp) - len),
                                 pattern, (int)len));
                        break;
@@ -1201,6 +1401,14 @@ void all_string_sub(char *s,const char *pattern,const char *insert, size_t len)
        }
 }
 
+char *talloc_all_string_sub(TALLOC_CTX *ctx,
+                               const char *src,
+                               const char *pattern,
+                               const char *insert)
+{
+       return talloc_string_sub_internal(ctx, src, pattern, insert, false);
+}
+
 /**
  Similar to all_string_sub but for unicode strings.
  Return a new allocated unicode string.
@@ -1208,7 +1416,8 @@ void all_string_sub(char *s,const char *pattern,const char *insert, size_t len)
  Use with caution!
 **/
 
-static smb_ucs2_t *all_string_sub_w(const smb_ucs2_t *s, const smb_ucs2_t *pattern,
+static smb_ucs2_t *all_string_sub_w(const smb_ucs2_t *s,
+                               const smb_ucs2_t *pattern,
                                const smb_ucs2_t *insert)
 {
        smb_ucs2_t *r, *rp;
@@ -1271,7 +1480,8 @@ smb_ucs2_t *all_string_sub_wa(smb_ucs2_t *s, const char *pattern,
  Splits out the front and back at a separator.
 **/
 
-static void split_at_last_component(char *path, char *front, char sep, char *back)
+static void split_at_last_component(char *path, char *front, char sep,
+               char *back)
 {
        char *p = strrchr_m(path, sep);
 
@@ -1318,16 +1528,17 @@ char *string_truncate(char *s, unsigned int length)
 }
 
 /**
- Strchr and strrchr_m are very hard to do on general multi-byte strings. 
+ Strchr and strrchr_m are very hard to do on general multi-byte strings.
  We convert via ucs2 for now.
 **/
 
 char *strchr_m(const char *src, char c)
 {
-       wpstring ws;
-       pstring s2;
+       smb_ucs2_t *ws = NULL;
+       char *s2 = NULL;
        smb_ucs2_t *p;
        const char *s;
+       char *ret;
 
        /* characters below 0x3F are guaranteed to not appear in
           non-initial position in multi-byte charsets */
@@ -1353,13 +1564,25 @@ char *strchr_m(const char *src, char c)
        s = src;
 #endif
 
-       push_ucs2(NULL, ws, s, sizeof(ws), STR_TERMINATE);
+       if (push_ucs2_allocate(&ws, s)==(size_t)-1) {
+               /* Wrong answer, but what can we do... */
+               return strchr(src, c);
+       }
        p = strchr_w(ws, UCS2_CHAR(c));
-       if (!p)
+       if (!p) {
+               SAFE_FREE(ws);
                return NULL;
+       }
        *p = 0;
-       pull_ucs2_pstring(s2, ws);
-       return (char *)(s+strlen(s2));
+       if (pull_ucs2_allocate(&s2, ws)==(size_t)-1) {
+               SAFE_FREE(ws);
+               /* Wrong answer, but what can we do... */
+               return strchr(src, c);
+       }
+       ret = (char *)(s+strlen(s2));
+       SAFE_FREE(ws);
+       SAFE_FREE(s2);
+       return ret;
 }
 
 char *strrchr_m(const char *s, char c)
@@ -1382,7 +1605,7 @@ char *strrchr_m(const char *s, char c)
        {
                size_t len = strlen(s);
                const char *cp = s;
-               BOOL got_mb = False;
+               bool got_mb = false;
 
                if (len == 0)
                        return NULL;
@@ -1390,9 +1613,10 @@ char *strrchr_m(const char *s, char c)
                do {
                        if (c == *cp) {
                                /* Could be a match. Part of a multibyte ? */
-                               if ((cp > s) && (((unsigned char)cp[-1]) & 0x80)) {
+                               if ((cp > s) &&
+                                       (((unsigned char)cp[-1]) & 0x80)) {
                                        /* Yep - go slow :-( */
-                                       got_mb = True;
+                                       got_mb = true;
                                        break;
                                }
                                /* No - we have a match ! */
@@ -1405,17 +1629,30 @@ char *strrchr_m(const char *s, char c)
 
        /* String contained a non-ascii char. Slow path. */
        {
-               wpstring ws;
-               pstring s2;
+               smb_ucs2_t *ws = NULL;
+               char *s2 = NULL;
                smb_ucs2_t *p;
+               char *ret;
 
-               push_ucs2(NULL, ws, s, sizeof(ws), STR_TERMINATE);
+               if (push_ucs2_allocate(&ws,s)==(size_t)-1) {
+                       /* Wrong answer, but what can we do. */
+                       return strrchr(s, c);
+               }
                p = strrchr_w(ws, UCS2_CHAR(c));
-               if (!p)
+               if (!p) {
+                       SAFE_FREE(ws);
                        return NULL;
+               }
                *p = 0;
-               pull_ucs2_pstring(s2, ws);
-               return (char *)(s+strlen(s2));
+               if (pull_ucs2_allocate(&s2,ws)==(size_t)-1) {
+                       SAFE_FREE(ws);
+                       /* Wrong answer, but what can we do. */
+                       return strrchr(s, c);
+               }
+               ret = (char *)(s+strlen(s2));
+               SAFE_FREE(ws);
+               SAFE_FREE(s2);
+               return ret;
        }
 }
 
@@ -1426,17 +1663,30 @@ char *strrchr_m(const char *s, char c)
 
 char *strnrchr_m(const char *s, char c, unsigned int n)
 {
-       wpstring ws;
-       pstring s2;
+       smb_ucs2_t *ws = NULL;
+       char *s2 = NULL;
        smb_ucs2_t *p;
+       char *ret;
 
-       push_ucs2(NULL, ws, s, sizeof(ws), STR_TERMINATE);
+       if (push_ucs2_allocate(&ws,s)==(size_t)-1) {
+               /* Too hard to try and get right. */
+               return NULL;
+       }
        p = strnrchr_w(ws, UCS2_CHAR(c), n);
-       if (!p)
+       if (!p) {
+               SAFE_FREE(ws);
                return NULL;
+       }
        *p = 0;
-       pull_ucs2_pstring(s2, ws);
-       return (char *)(s+strlen(s2));
+       if (pull_ucs2_allocate(&s2,ws)==(size_t)-1) {
+               SAFE_FREE(ws);
+               /* Too hard to try and get right. */
+               return NULL;
+       }
+       ret = (char *)(s+strlen(s2));
+       SAFE_FREE(ws);
+       SAFE_FREE(s2);
+       return ret;
 }
 
 /***********************************************************************
@@ -1468,7 +1718,7 @@ char *strstr_m(const char *src, const char *findstr)
 
        for (s = src; *s && !(((unsigned char)s[0]) & 0x80); s++) {
                if (*s == *findstr) {
-                       if (!findstr_len) 
+                       if (!findstr_len)
                                findstr_len = strlen(findstr);
 
                        if (strncmp(s, findstr, findstr_len) == 0) {
@@ -1491,7 +1741,7 @@ char *strstr_m(const char *src, const char *findstr)
                DEBUG(0,("strstr_m: src malloc fail\n"));
                return NULL;
        }
-       
+
        if (push_ucs2_allocate(&find_w, findstr) == (size_t)-1) {
                SAFE_FREE(src_w);
                DEBUG(0,("strstr_m: find malloc fail\n"));
@@ -1505,7 +1755,7 @@ char *strstr_m(const char *src, const char *findstr)
                SAFE_FREE(find_w);
                return NULL;
        }
-       
+
        *p = 0;
        if (pull_ucs2_allocate(&s2, src_w) == (size_t)-1) {
                SAFE_FREE(src_w);
@@ -1547,7 +1797,7 @@ void strlower_m(char *s)
        len = strlen(s) + 1;
        errno_save = errno;
        errno = 0;
-       unix_strlower(s,len,s,len);     
+       unix_strlower(s,len,s,len);
        /* Catch mb conversion errors that may not terminate. */
        if (errno)
                s[len-1] = '\0';
@@ -1581,13 +1831,83 @@ void strupper_m(char *s)
        len = strlen(s) + 1;
        errno_save = errno;
        errno = 0;
-       unix_strupper(s,len,s,len);     
+       unix_strupper(s,len,s,len);
        /* Catch mb conversion errors that may not terminate. */
        if (errno)
                s[len-1] = '\0';
        errno = errno_save;
 }
 
+/**
+ Count the number of UCS2 characters in a string. Normally this will
+ be the same as the number of bytes in a string for single byte strings,
+ but will be different for multibyte.
+**/
+
+size_t strlen_m(const char *s)
+{
+       size_t count = 0;
+
+       if (!s) {
+               return 0;
+       }
+
+       while (*s && !(((uint8_t)*s) & 0x80)) {
+               s++;
+               count++;
+       }
+
+       if (!*s) {
+               return count;
+       }
+
+       while (*s) {
+               size_t c_size;
+               codepoint_t c = next_codepoint(s, &c_size);
+               if (c < 0x10000) {
+                       /* Unicode char fits into 16 bits. */
+                       count += 1;
+               } else {
+                       /* Double-width unicode char - 32 bits. */
+                       count += 2;
+               }
+               s += c_size;
+       }
+
+       return count;
+}
+
+/**
+ Count the number of UCS2 characters in a string including the null
+ terminator.
+**/
+
+size_t strlen_m_term(const char *s)
+{
+       if (!s) {
+               return 0;
+       }
+       return strlen_m(s) + 1;
+}
+
+/*
+ * Weird helper routine for the winreg pipe: If nothing is around, return 0,
+ * if a string is there, include the terminator.
+ */
+
+size_t strlen_m_term_null(const char *s)
+{
+       size_t len;
+       if (!s) {
+               return 0;
+       }
+       len = strlen_m(s);
+       if (len == 0) {
+               return 0;
+       }
+
+       return len+1;
+}
 /**
  Return a RFC2254 binary string representation of a buffer.
  Used in LDAP filters.
@@ -1599,7 +1919,7 @@ char *binary_string_rfc2254(char *buf, int len)
        char *s;
        int i, j;
        const char *hex = "0123456789ABCDEF";
-       s = SMB_MALLOC(len * 3 + 1);
+       s = (char *)SMB_MALLOC(len * 3 + 1);
        if (!s)
                return NULL;
        for (j=i=0;i<len;i++) {
@@ -1617,7 +1937,7 @@ char *binary_string(char *buf, int len)
        char *s;
        int i, j;
        const char *hex = "0123456789ABCDEF";
-       s = SMB_MALLOC(len * 2 + 1);
+       s = (char *)SMB_MALLOC(len * 2 + 1);
        if (!s)
                return NULL;
        for (j=i=0;i<len;i++) {
@@ -1659,63 +1979,22 @@ int fstr_sprintf(fstring s, const char *fmt, ...)
        return ret;
 }
 
-
-#if !defined(HAVE_STRNDUP) || defined(BROKEN_STRNDUP)
-/**
- Some platforms don't have strndup.
-**/
-#if defined(PARANOID_MALLOC_CHECKER)
-#undef strndup
-#endif
-
- char *strndup(const char *s, size_t n)
-{
-       char *ret;
-       
-       n = strnlen(s, n);
-       ret = SMB_MALLOC(n+1);
-       if (!ret)
-               return NULL;
-       memcpy(ret, s, n);
-       ret[n] = 0;
-
-       return ret;
-}
-
-#if defined(PARANOID_MALLOC_CHECKER)
-#define strndup(s,n) __ERROR_DONT_USE_STRNDUP_DIRECTLY
-#endif
-
-#endif
-
-#if !defined(HAVE_STRNLEN) || defined(BROKEN_STRNLEN)
-/**
- Some platforms don't have strnlen
-**/
-
- size_t strnlen(const char *s, size_t n)
-{
-       size_t i;
-       for (i=0; i<n && s[i] != '\0'; i++)
-               /* noop */ ;
-       return i;
-}
-#endif
-
 /**
  List of Strings manipulation functions
 **/
 
 #define S_LIST_ABS 16 /* List Allocation Block Size */
 
-static char **str_list_make_internal(TALLOC_CTX *mem_ctx, const char *string, const char *sep)
+static char **str_list_make_internal(TALLOC_CTX *mem_ctx,
+               const char *string,
+               const char *sep)
 {
        char **list, **rlist;
        const char *str;
        char *s;
        int num, lsize;
        pstring tok;
-       
+
        if (!string || !*string)
                return NULL;
        if (mem_ctx) {
@@ -1728,23 +2007,27 @@ static char **str_list_make_internal(TALLOC_CTX *mem_ctx, const char *string, co
                return NULL;
        }
        if (!sep) sep = LIST_SEP;
-       
+
        num = lsize = 0;
        list = NULL;
-       
+
        str = s;
-       while (next_token(&str, tok, sep, sizeof(tok))) {               
+       while (next_token(&str, tok, sep, sizeof(tok))) {
                if (num == lsize) {
                        lsize += S_LIST_ABS;
                        if (mem_ctx) {
-                               rlist = TALLOC_REALLOC_ARRAY(mem_ctx, list, char *, lsize +1);
+                               rlist = TALLOC_REALLOC_ARRAY(mem_ctx, list,
+                                               char *, lsize +1);
                        } else {
-                               /* We need to keep the old list on error so we can free the elements
+                               /* We need to keep the old list on
+                                * error so we can free the elements
                                   if the realloc fails. */
-                               rlist = SMB_REALLOC_ARRAY_KEEP_OLD_ON_ERROR(list, char *, lsize +1);
+                               rlist =SMB_REALLOC_ARRAY_KEEP_OLD_ON_ERROR(list,
+                                               char *, lsize +1);
                        }
                        if (!rlist) {
-                               DEBUG(0,("str_list_make: Unable to allocate memory"));
+                               DEBUG(0,("str_list_make: "
+                                       "Unable to allocate memory"));
                                str_list_free(&list);
                                if (mem_ctx) {
                                        TALLOC_FREE(s);
@@ -1755,7 +2038,8 @@ static char **str_list_make_internal(TALLOC_CTX *mem_ctx, const char *string, co
                        } else {
                                list = rlist;
                        }
-                       memset (&list[num], 0, ((sizeof(char**)) * (S_LIST_ABS +1)));
+                       memset (&list[num], 0,
+                                       ((sizeof(char**)) * (S_LIST_ABS +1)));
                }
 
                if (mem_ctx) {
@@ -1763,7 +2047,7 @@ static char **str_list_make_internal(TALLOC_CTX *mem_ctx, const char *string, co
                } else {
                        list[num] = SMB_STRDUP(tok);
                }
-               
+
                if (!list[num]) {
                        DEBUG(0,("str_list_make: Unable to allocate memory"));
                        str_list_free(&list);
@@ -1774,8 +2058,8 @@ static char **str_list_make_internal(TALLOC_CTX *mem_ctx, const char *string, co
                        }
                        return NULL;
                }
-       
-               num++;  
+
+               num++;
        }
 
        if (mem_ctx) {
@@ -1787,7 +2071,9 @@ static char **str_list_make_internal(TALLOC_CTX *mem_ctx, const char *string, co
        return list;
 }
 
-char **str_list_make_talloc(TALLOC_CTX *mem_ctx, const char *string, const char *sep)
+char **str_list_make_talloc(TALLOC_CTX *mem_ctx,
+               const char *string,
+               const char *sep)
 {
        return str_list_make_internal(mem_ctx, string, sep);
 }
@@ -1797,72 +2083,75 @@ char **str_list_make(const char *string, const char *sep)
        return str_list_make_internal(NULL, string, sep);
 }
 
-BOOL str_list_copy(char ***dest, const char **src)
+bool str_list_copy(char ***dest, const char **src)
 {
        char **list, **rlist;
        int num, lsize;
-       
+
        *dest = NULL;
        if (!src)
-               return False;
-       
+               return false;
+
        num = lsize = 0;
        list = NULL;
-               
+
        while (src[num]) {
                if (num == lsize) {
                        lsize += S_LIST_ABS;
-                       rlist = SMB_REALLOC_ARRAY_KEEP_OLD_ON_ERROR(list, char *, lsize +1);
+                       rlist = SMB_REALLOC_ARRAY_KEEP_OLD_ON_ERROR(list,
+                                       char *, lsize +1);
                        if (!rlist) {
-                               DEBUG(0,("str_list_copy: Unable to re-allocate memory"));
+                               DEBUG(0,("str_list_copy: "
+                                       "Unable to re-allocate memory"));
                                str_list_free(&list);
-                               return False;
+                               return false;
                        } else {
                                list = rlist;
                        }
-                       memset (&list[num], 0, ((sizeof(char **)) * (S_LIST_ABS +1)));
+                       memset (&list[num], 0,
+                                       ((sizeof(char **)) * (S_LIST_ABS +1)));
                }
-               
+
                list[num] = SMB_STRDUP(src[num]);
                if (!list[num]) {
                        DEBUG(0,("str_list_copy: Unable to allocate memory"));
                        str_list_free(&list);
-                       return False;
+                       return false;
                }
 
                num++;
        }
-       
+
        *dest = list;
-       return True;    
+       return true;
 }
 
 /**
  * Return true if all the elements of the list match exactly.
  **/
-BOOL str_list_compare(char **list1, char **list2)
+bool str_list_compare(char **list1, char **list2)
 {
        int num;
-       
+
        if (!list1 || !list2)
-               return (list1 == list2); 
-       
+               return (list1 == list2);
+
        for (num = 0; list1[num]; num++) {
                if (!list2[num])
-                       return False;
+                       return false;
                if (!strcsequal(list1[num], list2[num]))
-                       return False;
+                       return false;
        }
        if (list2[num])
-               return False; /* if list2 has more elements than list1 fail */
-       
-       return True;
+               return false; /* if list2 has more elements than list1 fail */
+
+       return true;
 }
 
 static void str_list_free_internal(TALLOC_CTX *mem_ctx, char ***list)
 {
        char **tlist;
-       
+
        if (!list || !*list)
                return;
        tlist = *list;
@@ -1901,58 +2190,60 @@ int str_list_count( const char **list )
                return 0;
 
        /* count the number of list members */
-       
+
        for ( i=0; *list; i++, list++ );
-       
+
        return i;
 }
 
 /******************************************************************************
- version of standard_sub_basic() for string lists; uses alloc_sub_basic() 
+ version of standard_sub_basic() for string lists; uses alloc_sub_basic()
  for the work
  *****************************************************************************/
-BOOL str_list_sub_basic( char **list, const char *smb_name )
+
+bool str_list_sub_basic( char **list, const char *smb_name,
+                        const char *domain_name )
 {
        char *s, *tmpstr;
-       
+
        while ( *list ) {
                s = *list;
-               tmpstr = alloc_sub_basic(smb_name, s);
+               tmpstr = alloc_sub_basic(smb_name, domain_name, s);
                if ( !tmpstr ) {
-                       DEBUG(0,("str_list_sub_basic: alloc_sub_basic() return NULL!\n"));
-                       return False;
+                       DEBUG(0,("str_list_sub_basic: "
+                               "alloc_sub_basic() return NULL!\n"));
+                       return false;
                }
 
                SAFE_FREE(*list);
                *list = tmpstr;
-                       
+
                list++;
        }
 
-       return True;
+       return true;
 }
 
 /******************************************************************************
  substritute a specific pattern in a string list
  *****************************************************************************/
-BOOL str_list_substitute(char **list, const char *pattern, const char *insert)
+
+bool str_list_substitute(char **list, const char *pattern, const char *insert)
 {
        char *p, *s, *t;
        ssize_t ls, lp, li, ld, i, d;
 
        if (!list)
-               return False;
+               return false;
        if (!pattern)
-               return False;
+               return false;
        if (!insert)
-               return False;
+               return false;
 
        lp = (ssize_t)strlen(pattern);
        li = (ssize_t)strlen(insert);
        ld = li -lp;
-                       
+
        while (*list) {
                s = *list;
                ls = (ssize_t)strlen(s);
@@ -1963,8 +2254,9 @@ BOOL str_list_substitute(char **list, const char *pattern, const char *insert)
                        if (ld) {
                                t = (char *) SMB_MALLOC(ls +ld +1);
                                if (!t) {
-                                       DEBUG(0,("str_list_substitute: Unable to allocate memory"));
-                                       return False;
+                                       DEBUG(0,("str_list_substitute: "
+                                               "Unable to allocate memory"));
+                                       return false;
                                }
                                memcpy(t, *list, d);
                                memcpy(t +d +li, p +lp, ls -d -lp +1);
@@ -1973,7 +2265,7 @@ BOOL str_list_substitute(char **list, const char *pattern, const char *insert)
                                ls += ld;
                                s = t +d +li;
                        }
-                       
+
                        for (i = 0; i < li; i++) {
                                switch (insert[i]) {
                                        case '`':
@@ -1989,14 +2281,13 @@ BOOL str_list_substitute(char **list, const char *pattern, const char *insert)
                                        default:
                                                t[d +i] = insert[i];
                                }
-                       }       
+                       }
                }
-               
-               
+
                list++;
        }
-       
-       return True;
+
+       return true;
 }
 
 
@@ -2016,26 +2307,55 @@ BOOL str_list_substitute(char **list, const char *pattern, const char *insert)
  *         reallocated to new length
  **/
 
-char* ipstr_list_add(char** ipstr_list, const struct ip_service *service)
+static char *ipstr_list_add(char **ipstr_list, const struct ip_service *service)
 {
-       char* new_ipstr = NULL;
-       
+       char *new_ipstr = NULL;
+       char addr_buf[INET6_ADDRSTRLEN];
+
        /* arguments checking */
-       if (!ipstr_list || !service) return NULL;
+       if (!ipstr_list || !service) {
+               return NULL;
+       }
+
+       print_sockaddr(addr_buf,
+                       sizeof(addr_buf),
+                       &service->ss);
 
        /* attempt to convert ip to a string and append colon separator to it */
        if (*ipstr_list) {
-               asprintf(&new_ipstr, "%s%s%s:%d", *ipstr_list, IPSTR_LIST_SEP,
-                       inet_ntoa(service->ip), service->port);
+               if (service->ss.ss_family == AF_INET) {
+                       /* IPv4 */
+                       asprintf(&new_ipstr, "%s%s%s:%d",
+                                       *ipstr_list,
+                                       IPSTR_LIST_SEP,
+                                       addr_buf,
+                                       service->port);
+               } else {
+                       /* IPv6 */
+                       asprintf(&new_ipstr, "%s%s[%s]:%d",
+                                       *ipstr_list,
+                                       IPSTR_LIST_SEP,
+                                       addr_buf,
+                                       service->port);
+               }
                SAFE_FREE(*ipstr_list);
        } else {
-               asprintf(&new_ipstr, "%s:%d", inet_ntoa(service->ip), service->port);
+               if (service->ss.ss_family == AF_INET) {
+                       /* IPv4 */
+                       asprintf(&new_ipstr, "%s:%d",
+                               addr_buf,
+                               service->port);
+               } else {
+                       /* IPv6 */
+                       asprintf(&new_ipstr, "[%s]:%d",
+                               addr_buf,
+                               service->port);
+               }
        }
        *ipstr_list = new_ipstr;
        return *ipstr_list;
 }
 
-
 /**
  * Allocate and initialise an ipstr list using ip adresses
  * passed as arguments.
@@ -2045,75 +2365,86 @@ char* ipstr_list_add(char** ipstr_list, const struct ip_service *service)
  * @param ip_count number of addresses stored in ip_list
  * @return pointer to allocated ip string
  **/
-char* ipstr_list_make(char** ipstr_list, const struct ip_service* ip_list, int ip_count)
+
+char *ipstr_list_make(char **ipstr_list,
+                       const struct ip_service *ip_list,
+                       int ip_count)
 {
        int i;
-       
+
        /* arguments checking */
-       if (!ip_list && !ipstr_list) return 0;
+       if (!ip_list || !ipstr_list) {
+               return 0;
+       }
 
        *ipstr_list = NULL;
-       
+
        /* process ip addresses given as arguments */
-       for (i = 0; i < ip_count; i++)
+       for (i = 0; i < ip_count; i++) {
                *ipstr_list = ipstr_list_add(ipstr_list, &ip_list[i]);
-       
+       }
+
        return (*ipstr_list);
 }
 
 
 /**
  * Parse given ip string list into array of ip addresses
- * (as ip_service structures)  
- *    e.g. 192.168.1.100:389,192.168.1.78, ...
+ * (as ip_service structures)
+ *    e.g. [IPv6]:port,192.168.1.100:389,192.168.1.78, ...
  *
- * @param ipstr ip string list to be parsed 
+ * @param ipstr ip string list to be parsed
  * @param ip_list pointer to array of ip addresses which is
  *        allocated by this function and must be freed by caller
  * @return number of succesfully parsed addresses
  **/
-int ipstr_list_parse(const charipstr_list, struct ip_service **ip_list)
+
+int ipstr_list_parse(const char *ipstr_list, struct ip_service **ip_list)
 {
        fstring token_str;
        size_t count;
        int i;
 
-       if (!ipstr_list || !ip_list) 
+       if (!ipstr_list || !ip_list)
                return 0;
-       
+
        count = count_chars(ipstr_list, IPSTR_LIST_CHAR) + 1;
        if ( (*ip_list = SMB_MALLOC_ARRAY(struct ip_service, count)) == NULL ) {
-               DEBUG(0,("ipstr_list_parse: malloc failed for %lu entries\n", (unsigned long)count));
+               DEBUG(0,("ipstr_list_parse: malloc failed for %lu entries\n",
+                                       (unsigned long)count));
                return 0;
        }
-       
-       for ( i=0; 
-               next_token(&ipstr_list, token_str, IPSTR_LIST_SEP, FSTRING_LEN) && i<count; 
-               i++ ) 
-       {
-               struct in_addr addr;
-               unsigned port = 0;      
-               char *p = strchr(token_str, ':');
-               
+
+       for ( i=0; next_token(&ipstr_list, token_str,
+                               IPSTR_LIST_SEP, FSTRING_LEN) && i<count; i++ ) {
+               char *s = token_str;
+               char *p = strrchr(token_str, ':');
+
                if (p) {
                        *p = 0;
-                       port = atoi(p+1);
+                       (*ip_list)[i].port = atoi(p+1);
                }
 
                /* convert single token to ip address */
-               if ( (addr.s_addr = inet_addr(token_str)) == INADDR_NONE )
-                       break;
-                               
-               (*ip_list)[i].ip = addr;
-               (*ip_list)[i].port = port;
+               if (token_str[0] == '[') {
+                       /* IPv6 address. */
+                       s++;
+                       p = strchr(token_str, ']');
+                       if (!p) {
+                               continue;
+                       }
+                       *p = '\0';
+               }
+               if (!interpret_string_addr(&(*ip_list)[i].ss,
+                                       s,
+                                       AI_NUMERICHOST)) {
+                       continue;
+               }
        }
-       
+
        return count;
 }
 
-
 /**
  * Safely free ip string list
  *
@@ -2125,7 +2456,6 @@ void ipstr_list_free(char* ipstr_list)
        SAFE_FREE(ipstr_list);
 }
 
-
 /**
  Unescape a URL encoded string, in place.
 **/
@@ -2153,7 +2483,7 @@ void rfc1738_unescape(char *buf)
                else if (c2 >= 'a' && c2 <= 'f')
                        c2 = 10 + c2 - 'a';
                else {p++; continue;}
-                       
+
                *p = (c1<<4) | c2;
 
                memmove(p+1, p+3, strlen(p+3)+1);
@@ -2223,9 +2553,11 @@ void base64_decode_inplace(char *s)
 /**
  * Encode a base64 string into a malloc()ed string caller to free.
  *
- *From SQUID: adopted from http://ftp.sunet.se/pub2/gnu/vm/base64-encode.c with adjustments
+ * From SQUID: adopted from http://ftp.sunet.se/pub2/gnu/vm/base64-encode.c
+ * with adjustments
  **/
-char * base64_encode_data_blob(DATA_BLOB data)
+
+char *base64_encode_data_blob(DATA_BLOB data)
 {
        int bits = 0;
        int char_count = 0;
@@ -2238,7 +2570,8 @@ char * base64_encode_data_blob(DATA_BLOB data)
        out_cnt = 0;
        len = data.length;
        output_len = data.length * 2;
-       result = SMB_MALLOC(output_len); /* get us plenty of space */
+       result = TALLOC_ARRAY(talloc_tos(), char, output_len); /* get us plenty of space */
+       SMB_ASSERT(result != NULL);
 
        while (len-- && out_cnt < (data.length * 2) - 5) {
                int c = (unsigned char) *(data.data++);
@@ -2248,27 +2581,27 @@ char * base64_encode_data_blob(DATA_BLOB data)
                        result[out_cnt++] = b64[bits >> 18];
                        result[out_cnt++] = b64[(bits >> 12) & 0x3f];
                        result[out_cnt++] = b64[(bits >> 6) & 0x3f];
-           result[out_cnt++] = b64[bits & 0x3f];
-           bits = 0;
-           char_count = 0;
-       } else {
-           bits <<= 8;
-       }
-    }
-    if (char_count != 0) {
-       bits <<= 16 - (8 * char_count);
-       result[out_cnt++] = b64[bits >> 18];
-       result[out_cnt++] = b64[(bits >> 12) & 0x3f];
-       if (char_count == 1) {
-           result[out_cnt++] = '=';
-           result[out_cnt++] = '=';
-       } else {
-           result[out_cnt++] = b64[(bits >> 6) & 0x3f];
-           result[out_cnt++] = '=';
+                       result[out_cnt++] = b64[bits & 0x3f];
+                       bits = 0;
+                       char_count = 0;
+               } else {
+                       bits <<= 8;
+               }
+       }
+       if (char_count != 0) {
+               bits <<= 16 - (8 * char_count);
+               result[out_cnt++] = b64[bits >> 18];
+               result[out_cnt++] = b64[(bits >> 12) & 0x3f];
+               if (char_count == 1) {
+                       result[out_cnt++] = '=';
+                       result[out_cnt++] = '=';
+               } else {
+                       result[out_cnt++] = b64[(bits >> 6) & 0x3f];
+                       result[out_cnt++] = '=';
+               }
        }
-    }
-    result[out_cnt] = '\0';    /* terminate */
-    return result;
+       result[out_cnt] = '\0'; /* terminate */
+       return result;
 }
 
 /* read a SMB_BIG_UINT from a string */
@@ -2277,9 +2610,11 @@ SMB_BIG_UINT STR_TO_SMB_BIG_UINT(const char *nptr, const char **entptr)
 
        SMB_BIG_UINT val = -1;
        const char *p = nptr;
-       
+
        if (!p) {
-               *entptr = p;
+               if (entptr) {
+                       *entptr = p;
+               }
                return val;
        }
 
@@ -2287,7 +2622,7 @@ SMB_BIG_UINT STR_TO_SMB_BIG_UINT(const char *nptr, const char **entptr)
                p++;
 
 #ifdef LARGE_SMB_OFF_T
-       sscanf(p,"%llu",&val);  
+       sscanf(p,"%llu",&val);
 #else /* LARGE_SMB_OFF_T */
        sscanf(p,"%lu",&val);
 #endif /* LARGE_SMB_OFF_T */
@@ -2300,16 +2635,83 @@ SMB_BIG_UINT STR_TO_SMB_BIG_UINT(const char *nptr, const char **entptr)
        return val;
 }
 
+/* Convert a size specification to a count of bytes. We accept the following
+ * suffixes:
+ *         bytes if there is no suffix
+ *     kK  kibibytes
+ *     mM  mebibytes
+ *     gG  gibibytes
+ *     tT  tibibytes
+ *     pP  whatever the ISO name for petabytes is
+ *
+ *  Returns 0 if the string can't be converted.
+ */
+SMB_OFF_T conv_str_size(const char * str)
+{
+        SMB_OFF_T lval;
+       char * end;
+
+        if (str == NULL || *str == '\0') {
+                return 0;
+        }
+
+#ifdef HAVE_STRTOULL
+       if (sizeof(SMB_OFF_T) == 8) {
+           lval = strtoull(str, &end, 10 /* base */);
+       } else {
+           lval = strtoul(str, &end, 10 /* base */);
+       }
+#else
+       lval = strtoul(str, &end, 10 /* base */);
+#endif
+
+        if (end == NULL || end == str) {
+                return 0;
+        }
+
+        if (*end) {
+               SMB_OFF_T lval_orig = lval;
+
+                if (strwicmp(end, "K") == 0) {
+                        lval *= (SMB_OFF_T)1024;
+                } else if (strwicmp(end, "M") == 0) {
+                        lval *= ((SMB_OFF_T)1024 * (SMB_OFF_T)1024);
+                } else if (strwicmp(end, "G") == 0) {
+                        lval *= ((SMB_OFF_T)1024 * (SMB_OFF_T)1024 *
+                               (SMB_OFF_T)1024);
+                } else if (strwicmp(end, "T") == 0) {
+                        lval *= ((SMB_OFF_T)1024 * (SMB_OFF_T)1024 *
+                               (SMB_OFF_T)1024 * (SMB_OFF_T)1024);
+                } else if (strwicmp(end, "P") == 0) {
+                        lval *= ((SMB_OFF_T)1024 * (SMB_OFF_T)1024 *
+                               (SMB_OFF_T)1024 * (SMB_OFF_T)1024 *
+                               (SMB_OFF_T)1024);
+                } else {
+                        return 0;
+                }
+
+               /* Primitive attempt to detect wrapping on platforms with
+                * 4-byte SMB_OFF_T. It's better to let the caller handle
+                * a failure than some random number.
+                */
+               if (lval_orig <= lval) {
+                       return 0;
+               }
+        }
+
+       return lval;
+}
+
 void string_append(char **left, const char *right)
 {
        int new_len = strlen(right) + 1;
 
        if (*left == NULL) {
-               *left = SMB_MALLOC(new_len);
+               *left = (char *)SMB_MALLOC(new_len);
                *left[0] = '\0';
        } else {
                new_len += strlen(*left);
-               *left = SMB_REALLOC(*left, new_len);
+               *left = (char *)SMB_REALLOC(*left, new_len);
        }
 
        if (*left == NULL) {
@@ -2319,20 +2721,23 @@ void string_append(char **left, const char *right)
        safe_strcat(*left, right, new_len-1);
 }
 
-BOOL add_string_to_array(TALLOC_CTX *mem_ctx,
+bool add_string_to_array(TALLOC_CTX *mem_ctx,
                         const char *str, const char ***strings,
                         int *num)
 {
        char *dup_str = talloc_strdup(mem_ctx, str);
 
-       *strings = TALLOC_REALLOC_ARRAY(mem_ctx, *strings, const char *, (*num)+1);
+       *strings = TALLOC_REALLOC_ARRAY(mem_ctx, *strings,
+                       const char *, (*num)+1);
 
-       if ((*strings == NULL) || (dup_str == NULL))
-               return False;
+       if ((*strings == NULL) || (dup_str == NULL)) {
+               *num = 0;
+               return false;
+       }
 
        (*strings)[*num] = dup_str;
        *num += 1;
-       return True;
+       return true;
 }
 
 /* Append an sprintf'ed string. Double buffer size on demand. Usable without
@@ -2345,7 +2750,7 @@ void sprintf_append(TALLOC_CTX *mem_ctx, char **string, ssize_t *len,
        va_list ap;
        char *newstr;
        int ret;
-       BOOL increased;
+       bool increased;
 
        /* len<0 is an internal marker that something failed */
        if (*len < 0)
@@ -2355,11 +2760,7 @@ void sprintf_append(TALLOC_CTX *mem_ctx, char **string, ssize_t *len,
                if (*bufsize == 0)
                        *bufsize = 128;
 
-               if (mem_ctx != NULL)
-                       *string = TALLOC_ARRAY(mem_ctx, char, *bufsize);
-               else
-                       *string = SMB_MALLOC_ARRAY(char, *bufsize);
-
+               *string = TALLOC_ARRAY(mem_ctx, char, *bufsize);
                if (*string == NULL)
                        goto error;
        }
@@ -2371,23 +2772,18 @@ void sprintf_append(TALLOC_CTX *mem_ctx, char **string, ssize_t *len,
        if (ret < 0)
                goto error;
 
-       increased = False;
+       increased = false;
 
        while ((*len)+ret >= *bufsize) {
-               increased = True;
+               increased = true;
                *bufsize *= 2;
                if (*bufsize >= (1024*1024*256))
                        goto error;
        }
 
        if (increased) {
-               if (mem_ctx != NULL) {
-                       *string = TALLOC_REALLOC_ARRAY(mem_ctx, *string, char,
-                                                      *bufsize);
-               } else {
-                       *string = SMB_REALLOC_ARRAY(*string, char, *bufsize);
-               }
-
+               *string = TALLOC_REALLOC_ARRAY(mem_ctx, *string, char,
+                                              *bufsize);
                if (*string == NULL) {
                        goto error;
                }
@@ -2435,17 +2831,231 @@ char *sstring_sub(const char *src, char front, char back)
  characters.
 ********************************************************************/
 
-BOOL validate_net_name( const char *name, const char *invalid_chars, int max_len )
+bool validate_net_name( const char *name,
+               const char *invalid_chars,
+               int max_len)
 {
        int i;
 
        for ( i=0; i<max_len && name[i]; i++ ) {
                /* fail if strchr_m() finds one of the invalid characters */
                if ( name[i] && strchr_m( invalid_chars, name[i] ) ) {
-                       return False;
+                       return false;
                }
        }
 
-       return True;
+       return true;
+}
+
+
+/**
+return the number of bytes occupied by a buffer in ASCII format
+the result includes the null termination
+limited by 'n' bytes
+**/
+size_t ascii_len_n(const char *src, size_t n)
+{
+       size_t len;
+
+       len = strnlen(src, n);
+       if (len+1 <= n) {
+               len += 1;
+       }
+
+       return len;
 }
 
+/**
+return the number of bytes occupied by a buffer in CH_UTF16 format
+the result includes the null termination
+**/
+size_t utf16_len(const void *buf)
+{
+       size_t len;
+
+       for (len = 0; SVAL(buf,len); len += 2) ;
+
+       return len + 2;
+}
+
+/**
+return the number of bytes occupied by a buffer in CH_UTF16 format
+the result includes the null termination
+limited by 'n' bytes
+**/
+size_t utf16_len_n(const void *src, size_t n)
+{
+       size_t len;
+
+       for (len = 0; (len+2 < n) && SVAL(src, len); len += 2) ;
+
+       if (len+2 <= n) {
+               len += 2;
+       }
+
+       return len;
+}
+
+/*******************************************************************
+ Add a shell escape character '\' to any character not in a known list
+ of characters. UNIX charset format.
+*******************************************************************/
+
+#define INCLUDE_LIST "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz_/ \t.,"
+#define INSIDE_DQUOTE_LIST "$`\n\"\\"
+
+char *escape_shell_string(const char *src)
+{
+       size_t srclen = strlen(src);
+       char *ret = SMB_MALLOC_ARRAY(char, (srclen * 2) + 1);
+       char *dest = ret;
+       bool in_s_quote = false;
+       bool in_d_quote = false;
+       bool next_escaped = false;
+
+       if (!ret) {
+               return NULL;
+       }
+
+       while (*src) {
+               size_t c_size;
+               codepoint_t c = next_codepoint(src, &c_size);
+
+               if (c == INVALID_CODEPOINT) {
+                       SAFE_FREE(ret);
+                       return NULL;
+               }
+
+               if (c_size > 1) {
+                       memcpy(dest, src, c_size);
+                       src += c_size;
+                       dest += c_size;
+                       next_escaped = false;
+                       continue;
+               }
+
+               /*
+                * Deal with backslash escaped state.
+                * This only lasts for one character.
+                */
+
+               if (next_escaped) {
+                       *dest++ = *src++;
+                       next_escaped = false;
+                       continue;
+               }
+
+               /*
+                * Deal with single quote state. The
+                * only thing we care about is exiting
+                * this state.
+                */
+
+               if (in_s_quote) {
+                       if (*src == '\'') {
+                               in_s_quote = false;
+                       }
+                       *dest++ = *src++;
+                       continue;
+               }
+
+               /*
+                * Deal with double quote state. The most
+                * complex state. We must cope with \, meaning
+                * possibly escape next char (depending what it
+                * is), ", meaning exit this state, and possibly
+                * add an \ escape to any unprotected character
+                * (listed in INSIDE_DQUOTE_LIST).
+                */
+
+               if (in_d_quote) {
+                       if (*src == '\\') {
+                               /*
+                                * Next character might be escaped.
+                                * We have to peek. Inside double
+                                * quotes only INSIDE_DQUOTE_LIST
+                                * characters are escaped by a \.
+                                */
+
+                               char nextchar;
+
+                               c = next_codepoint(&src[1], &c_size);
+                               if (c == INVALID_CODEPOINT) {
+                                       SAFE_FREE(ret);
+                                       return NULL;
+                               }
+                               if (c_size > 1) {
+                                       /*
+                                        * Don't escape the next char.
+                                        * Just copy the \.
+                                        */
+                                       *dest++ = *src++;
+                                       continue;
+                               }
+
+                               nextchar = src[1];
+
+                               if (nextchar && strchr(INSIDE_DQUOTE_LIST,
+                                                       (int)nextchar)) {
+                                       next_escaped = true;
+                               }
+                               *dest++ = *src++;
+                               continue;
+                       }
+
+                       if (*src == '\"') {
+                               /* Exit double quote state. */
+                               in_d_quote = false;
+                               *dest++ = *src++;
+                               continue;
+                       }
+
+                       /*
+                        * We know the character isn't \ or ",
+                        * so escape it if it's any of the other
+                        * possible unprotected characters.
+                        */
+
+                       if (strchr(INSIDE_DQUOTE_LIST, (int)*src)) {
+                               *dest++ = '\\';
+                       }
+                       *dest++ = *src++;
+                       continue;
+               }
+
+               /*
+                * From here to the end of the loop we're
+                * not in the single or double quote state.
+                */
+
+               if (*src == '\\') {
+                       /* Next character must be escaped. */
+                       next_escaped = true;
+                       *dest++ = *src++;
+                       continue;
+               }
+
+               if (*src == '\'') {
+                       /* Go into single quote state. */
+                       in_s_quote = true;
+                       *dest++ = *src++;
+                       continue;
+               }
+
+               if (*src == '\"') {
+                       /* Go into double quote state. */
+                       in_d_quote = true;
+                       *dest++ = *src++;
+                       continue;
+               }
+
+               /* Check if we need to escape the character. */
+
+               if (!strchr(INCLUDE_LIST, (int)*src)) {
+                       *dest++ = '\\';
+               }
+               *dest++ = *src++;
+       }
+       *dest++ = '\0';
+       return ret;
+}