* BUG 446
[samba.git] / source3 / lib / charcnv.c
index d42dc994b0032b90e90c06dedac65648d3c7530a..5f9392b62019a45560c27e1cd7c2130544d9bbc8 100644 (file)
@@ -4,6 +4,7 @@
    Copyright (C) Igor Vergeichik <iverg@mail.ru> 2001
    Copyright (C) Andrew Tridgell 2001
    Copyright (C) Simo Sorce 2001
+   Copyright (C) Martin Pool 2003
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 */
 #include "includes.h"
 
-static pstring cvtbuf;
+/**
+ * @file
+ *
+ * @brief Character-set conversion routines built on our iconv.
+ * 
+ * @note Samba's internal character set (at least in the 3.0 series)
+ * is always the same as the one for the Unix filesystem.  It is
+ * <b>not</b> necessarily UTF-8 and may be different on machines that
+ * need i18n filenames to be compatible with Unix software.  It does
+ * have to be a superset of ASCII.  All multibyte sequences must start
+ * with a byte with the high bit set.
+ *
+ * @sa lib/iconv.c
+ */
+
 
 static smb_iconv_t conv_handles[NUM_CHARSETS][NUM_CHARSETS];
+static BOOL conv_silent; /* Should we do a debug if the conversion fails ? */
 
-/****************************************************************************
-return the name of a charset to give to iconv()
-****************************************************************************/
-static char *charset_name(charset_t ch)
+/**
+ * Return the name of a charset to give to iconv().
+ **/
+static const char *charset_name(charset_t ch)
 {
-       char *ret = NULL;
+       const char *ret = NULL;
 
        if (ch == CH_UCS2) ret = "UCS-2LE";
        else if (ch == CH_UNIX) ret = lp_unix_charset();
@@ -39,12 +55,35 @@ static char *charset_name(charset_t ch)
        else if (ch == CH_DISPLAY) ret = lp_display_charset();
        else if (ch == CH_UTF8) ret = "UTF8";
 
+#if defined(HAVE_NL_LANGINFO) && defined(CODESET)
+       if (ret && !strcmp(ret, "LOCALE")) {
+               const char *ln = NULL;
+
+#ifdef HAVE_SETLOCALE
+               setlocale(LC_ALL, "");
+#endif
+               ln = nl_langinfo(CODESET);
+               if (ln) {
+                       /* Check whether the charset name is supported
+                          by iconv */
+                       smb_iconv_t handle = smb_iconv_open(ln,"UCS-2LE");
+                       if (handle == (smb_iconv_t) -1) {
+                               DEBUG(5,("Locale charset '%s' unsupported, using ASCII instead\n", ln));
+                               ln = NULL;
+                       } else {
+                               DEBUG(5,("Substituting charset '%s' for LOCALE\n", ln));
+                               smb_iconv_close(handle);
+                       }
+               }
+               ret = ln;
+       }
+#endif
+
        if (!ret || !*ret) ret = "ASCII";
        return ret;
 }
 
-
-static void lazy_initialize_conv(void)
+void lazy_initialize_conv(void)
 {
        static int initialized = False;
 
@@ -52,13 +91,16 @@ static void lazy_initialize_conv(void)
                initialized = True;
                load_case_tables();
                init_iconv();
-               init_valid_table();
        }
 }
 
-/****************************************************************************
- Initialize iconv conversion descriptors 
-****************************************************************************/
+/**
+ * Initialize iconv conversion descriptors.
+ *
+ * This is called the first time it is needed, and also called again
+ * every time the configuration is reloaded, because the charset or
+ * codepage might have changed.
+ **/
 void init_iconv(void)
 {
        int c1, c2;
@@ -66,27 +108,26 @@ void init_iconv(void)
 
        /* so that charset_name() works we need to get the UNIX<->UCS2 going
           first */
-       if (!conv_handles[CH_UNIX][CH_UCS2]) {
+       if (!conv_handles[CH_UNIX][CH_UCS2])
                conv_handles[CH_UNIX][CH_UCS2] = smb_iconv_open("UCS-2LE", "ASCII");
-       }
-       if (!conv_handles[CH_UCS2][CH_UNIX]) {
+
+       if (!conv_handles[CH_UCS2][CH_UNIX])
                conv_handles[CH_UCS2][CH_UNIX] = smb_iconv_open("ASCII", "UCS-2LE");
-       }
-       
 
        for (c1=0;c1<NUM_CHARSETS;c1++) {
                for (c2=0;c2<NUM_CHARSETS;c2++) {
-                       char *n1 = charset_name((charset_t)c1);
-                       char *n2 = charset_name((charset_t)c2);
+                       const char *n1 = charset_name((charset_t)c1);
+                       const char *n2 = charset_name((charset_t)c2);
                        if (conv_handles[c1][c2] &&
                            strcmp(n1, conv_handles[c1][c2]->from_name) == 0 &&
-                           strcmp(n2, conv_handles[c1][c2]->to_name) == 0) continue;
+                           strcmp(n2, conv_handles[c1][c2]->to_name) == 0)
+                               continue;
 
                        did_reload = True;
 
-                       if (conv_handles[c1][c2]) {
+                       if (conv_handles[c1][c2])
                                smb_iconv_close(conv_handles[c1][c2]);
-                       }
+
                        conv_handles[c1][c2] = smb_iconv_open(n2,n1);
                        if (conv_handles[c1][c2] == (smb_iconv_t)-1) {
                                DEBUG(0,("Conversion from %s to %s not supported\n",
@@ -97,21 +138,31 @@ void init_iconv(void)
        }
 
        if (did_reload) {
+               /* XXX: Does this really get called every time the dos
+                * codepage changes? */
+               /* XXX: Is the did_reload test too strict? */
+               conv_silent = True;
+               init_doschar_table();
                init_valid_table();
+               conv_silent = False;
        }
 }
 
 /**
  * Convert string from one encoding to another, making error checking etc
+ * Slow path version - uses (slow) iconv.
  *
- * @param descriptor conversion descriptor, created in init_iconv()
  * @param src pointer to source string (multibyte or singlebyte)
  * @param srclen length of the source string in bytes
  * @param dest pointer to destination string (multibyte or singlebyte)
  * @param destlen maximal length allowed for string
- * @retval the number of bytes occupied in the destination
+ * @returns the number of bytes occupied in the destination
+ *
+ * Ensure the srclen contains the terminating zero.
+ *
  **/
-size_t convert_string(charset_t from, charset_t to,
+
+static size_t convert_string_internal(charset_t from, charset_t to,
                      void const *src, size_t srclen, 
                      void *dest, size_t destlen)
 {
@@ -121,120 +172,320 @@ size_t convert_string(charset_t from, charset_t to,
        char* outbuf = (char*)dest;
        smb_iconv_t descriptor;
 
-       if (srclen == -1) srclen = strlen(src)+1;
-
        lazy_initialize_conv();
 
        descriptor = conv_handles[from][to];
 
+       if (srclen == (size_t)-1) {
+               if (from == CH_UCS2) {
+                       srclen = (strlen_w((const smb_ucs2_t *)src)+1) * 2;
+               } else {
+                       srclen = strlen((const char *)src)+1;
+               }
+       }
+
        if (descriptor == (smb_iconv_t)-1 || descriptor == (smb_iconv_t)0) {
-               /* conversion not supported, use as is */
-               int len = MIN(srclen,destlen);
-               memcpy(dest,src,len);
-               return len;
+               if (!conv_silent)
+                       DEBUG(0,("convert_string_internal: Conversion not supported.\n"));
+               goto use_as_is;
        }
 
        i_len=srclen;
        o_len=destlen;
-       retval = smb_iconv(descriptor,  &inbuf, &i_len, &outbuf, &o_len);
-       if(retval==-1)          
-       {
-               char *reason="unknown error";
-               switch(errno)
-               { case EINVAL: reason="Incomplete multibyte sequence"; break;
-                 case E2BIG:  reason="No more room"; 
-                              DEBUG(0, ("convert_string: Required %d, available %d\n",
-                              srclen, destlen));
-                              /* we are not sure we need srclen bytes,
+       retval = smb_iconv(descriptor, (char **)&inbuf, &i_len, &outbuf, &o_len);
+       if(retval==(size_t)-1) {
+               const char *reason="unknown error";
+               switch(errno) {
+                       case EINVAL:
+                               reason="Incomplete multibyte sequence";
+                               if (!conv_silent)
+                                       DEBUG(3,("convert_string_internal: Conversion error: %s(%s)\n",reason,inbuf));
+                               goto use_as_is;
+                       case E2BIG:
+                               reason="No more room"; 
+                               if (!conv_silent)
+                                       DEBUG(3, ("convert_string_internal: Required %lu, available %lu\n",
+                                               (unsigned long)srclen, (unsigned long)destlen));
+                               /* we are not sure we need srclen bytes,
                                  may be more, may be less.
                                  We only know we need more than destlen
                                  bytes ---simo */
-                               
-                                               
                               break;
-                 case EILSEQ: reason="Illegal multibyte sequence"; break;
+                       case EILSEQ:
+                               reason="Illegal multibyte sequence";
+                               if (!conv_silent)
+                                       DEBUG(3,("convert_string_internal: Conversion error: %s(%s)\n",reason,inbuf));
+                               goto use_as_is;
+                       default:
+                               if (!conv_silent)
+                                       DEBUG(0,("convert_string_internal: Conversion error: %s(%s)\n",reason,inbuf));
+                               break;
                }
                /* smb_panic(reason); */
        }
        return destlen-o_len;
+
+ use_as_is:
+
+       /* conversion not supported, use as is */
+       {
+               size_t len = MIN(srclen,destlen);
+               if (len)
+                       memcpy(dest,src,len);
+               return len;
+       }
+}
+
+/**
+ * Convert string from one encoding to another, making error checking etc
+ * Fast path version - handles ASCII first.
+ *
+ * @param src pointer to source string (multibyte or singlebyte)
+ * @param srclen length of the source string in bytes
+ * @param dest pointer to destination string (multibyte or singlebyte)
+ * @param destlen maximal length allowed for string
+ * @returns the number of bytes occupied in the destination
+ *
+ * Ensure the srclen contains the terminating zero.
+ *
+ * This function has been hand-tuned to provide a fast path.
+ * Don't change unless you really know what you are doing. JRA.
+ **/
+
+size_t convert_string(charset_t from, charset_t to,
+                     void const *src, size_t srclen, 
+                     void *dest, size_t destlen)
+{
+       /*
+        * NB. We deliberately don't do a strlen here is srclen == -1.
+        * This is very expensive over millions of calls and is taken
+        * care of in the slow path in convert_string_internal. JRA.
+        */
+
+       if (srclen == 0)
+               return 0;
+
+       if (from != CH_UCS2 && to != CH_UCS2) {
+               const unsigned char *p = (const unsigned char *)src;
+               unsigned char *q = (unsigned char *)dest;
+               size_t slen = srclen;
+               size_t dlen = destlen;
+               unsigned char lastp;
+               size_t retval = 0;
+
+               /* If all characters are ascii, fast path here. */
+               while (slen && dlen) {
+                       if ((lastp = *p) <= 0x7f) {
+                               *q++ = *p++;
+                               if (slen != (size_t)-1) {
+                                       slen--;
+                               }
+                               dlen--;
+                               retval++;
+                               if (!lastp)
+                                       break;
+                       } else {
+#ifdef BROKEN_UNICODE_COMPOSE_CHARACTERS
+                               goto general_case;
+#else
+                               return retval + convert_string_internal(from, to, p, slen, q, dlen);
+#endif
+                       }
+               }
+               return retval;
+       } else if (from == CH_UCS2 && to != CH_UCS2) {
+               const unsigned char *p = (const unsigned char *)src;
+               unsigned char *q = (unsigned char *)dest;
+               size_t retval = 0;
+               size_t slen = srclen;
+               size_t dlen = destlen;
+               unsigned char lastp;
+
+               /* If all characters are ascii, fast path here. */
+               while ((slen >= 2) && dlen) {
+                       if (((lastp = *p) <= 0x7f) && (p[1] == 0)) {
+                               *q++ = *p;
+                               if (slen != (size_t)-1) {
+                                       slen -= 2;
+                               }
+                               p += 2;
+                               dlen--;
+                               retval++;
+                               if (!lastp)
+                                       break;
+                       } else {
+#ifdef BROKEN_UNICODE_COMPOSE_CHARACTERS
+                               goto general_case;
+#else
+                               return retval + convert_string_internal(from, to, p, slen, q, dlen);
+#endif
+                       }
+               }
+               return retval;
+       } else if (from != CH_UCS2 && to == CH_UCS2) {
+               const unsigned char *p = (const unsigned char *)src;
+               unsigned char *q = (unsigned char *)dest;
+               size_t retval = 0;
+               size_t slen = srclen;
+               size_t dlen = destlen;
+               unsigned char lastp;
+
+               /* If all characters are ascii, fast path here. */
+               while (slen && (dlen >= 2)) {
+                       if ((lastp = *p) <= 0x7F) {
+                               *q++ = *p++;
+                               *q++ = '\0';
+                               if (slen != (size_t)-1) {
+                                       slen--;
+                               }
+                               dlen -= 2;
+                               retval += 2;
+                               if (!lastp)
+                                       break;
+                       } else {
+#ifdef BROKEN_UNICODE_COMPOSE_CHARACTERS
+                               goto general_case;
+#else
+                               return retval + convert_string_internal(from, to, p, slen, q, dlen);
+#endif
+                       }
+               }
+               return retval;
+       }
+
+#ifdef BROKEN_UNICODE_COMPOSE_CHARACTERS
+  general_case:
+#endif
+       return convert_string_internal(from, to, src, srclen, dest, destlen);
 }
 
 /**
  * Convert between character sets, allocating a new buffer for the result.
  *
+ * @param ctx TALLOC_CTX to use to allocate with. If NULL use malloc.
  * @param srclen length of source buffer.
  * @param dest always set at least to NULL
  * @note -1 is not accepted for srclen.
  *
- * @retval Size in bytes of the converted string; or -1 in case of error.
+ * @returns Size in bytes of the converted string; or -1 in case of error.
+ *
+ * Ensure the srclen contains the terminating zero.
  **/
-size_t convert_string_allocate(charset_t from, charset_t to,
-                               void const *src, size_t srclen, void **dest)
+
+size_t convert_string_allocate(TALLOC_CTX *ctx, charset_t from, charset_t to,
+                              void const *src, size_t srclen, void **dest)
 {
-       size_t i_len, o_len, destlen;
+       size_t i_len, o_len, destlen = MAX(srclen, 512);
        size_t retval;
        const char *inbuf = (const char *)src;
-       char *outbuf, *ob;
+       char *outbuf = NULL, *ob = NULL;
        smb_iconv_t descriptor;
 
        *dest = NULL;
 
-       if (src == NULL || srclen == -1) return -1;
+       if (src == NULL || srclen == (size_t)-1)
+               return (size_t)-1;
+       if (srclen == 0)
+               return 0;
 
        lazy_initialize_conv();
 
        descriptor = conv_handles[from][to];
 
        if (descriptor == (smb_iconv_t)-1 || descriptor == (smb_iconv_t)0) {
-               /* conversion not supported, return -1*/
-               DEBUG(3, ("convert_string_allocate: conversion not supported!\n"));
-               return -1;
+               if (!conv_silent)
+                       DEBUG(0,("convert_string_allocate: Conversion not supported.\n"));
+               goto use_as_is;
        }
 
-       destlen = MAX(srclen, 512);
-       outbuf = NULL;
 convert:
-       destlen = destlen * 2;
-       ob = (char *)realloc(outbuf, destlen);
+       if ((destlen*2) < destlen) {
+               /* wrapped ! abort. */
+               if (!conv_silent)
+                       DEBUG(0, ("convert_string_allocate: destlen wrapped !\n"));
+               if (!ctx)
+                       SAFE_FREE(outbuf);
+               return (size_t)-1;
+       } else {
+               destlen = destlen * 2;
+       }
+
+       if (ctx)
+               ob = (char *)talloc_realloc(ctx, ob, destlen);
+       else
+               ob = (char *)Realloc(ob, destlen);
+
        if (!ob) {
                DEBUG(0, ("convert_string_allocate: realloc failed!\n"));
-               SAFE_FREE(outbuf);
-               return -1;
+               if (!ctx)
+                       SAFE_FREE(outbuf);
+               return (size_t)-1;
+       } else {
+               outbuf = ob;
        }
-       else outbuf = ob;
        i_len = srclen;
        o_len = destlen;
        retval = smb_iconv(descriptor,
-                          &inbuf, &i_len,
+                          (char **)&inbuf, &i_len,
                           &outbuf, &o_len);
-       if(retval == -1)                
-       {
-               char *reason="unknown error";
-               switch(errno)
-               {
-               case EINVAL:
-                       reason="Incomplete multibyte sequence";
-                       break;
-               case E2BIG:
-                       goto convert;           
-               case EILSEQ:
-                       reason="Illegal multibyte sequence";
-                       break;
+       if(retval == (size_t)-1)                {
+               const char *reason="unknown error";
+               switch(errno) {
+                       case EINVAL:
+                               reason="Incomplete multibyte sequence";
+                               if (!conv_silent)
+                                       DEBUG(3,("convert_string_allocate: Conversion error: %s(%s)\n",reason,inbuf));
+                               goto use_as_is;
+                       case E2BIG:
+                               goto convert;           
+                       case EILSEQ:
+                               reason="Illegal multibyte sequence";
+                               if (!conv_silent)
+                                       DEBUG(3,("convert_string_allocate: Conversion error: %s(%s)\n",reason,inbuf));
+                               goto use_as_is;
                }
-               DEBUG(0,("Conversion error: %s(%s)\n",reason,inbuf));
+               if (!conv_silent)
+                       DEBUG(0,("Conversion error: %s(%s)\n",reason,inbuf));
                /* smb_panic(reason); */
-               return -1;
+               return (size_t)-1;
        }
        
        destlen = destlen - o_len;
-       *dest = (char *)Realloc(ob,destlen);
-       if (!*dest) {
+       if (ctx)
+               *dest = (char *)talloc_realloc(ctx,ob,destlen);
+       else
+               *dest = (char *)Realloc(ob,destlen);
+       if (destlen && !*dest) {
                DEBUG(0, ("convert_string_allocate: out of memory!\n"));
-               SAFE_FREE(ob);
-               return -1;
+               if (!ctx)
+                       SAFE_FREE(ob);
+               return (size_t)-1;
        }
 
        return destlen;
+
+  use_as_is:
+
+       /* conversion not supported, use as is */
+       {
+               if (srclen && (destlen != srclen)) {
+                       destlen = srclen;
+                       if (ctx)
+                               ob = (char *)talloc_realloc(ctx, ob, destlen);
+                       else
+                               ob = (char *)Realloc(ob, destlen);
+                       if (!ob) {
+                               DEBUG(0, ("convert_string_allocate: realloc failed!\n"));
+                               if (!ctx)
+                                       SAFE_FREE(outbuf);
+                               return (size_t)-1;
+                       }
+               }
+               if (srclen && ob)
+                       memcpy(ob,(const char *)src,srclen);
+               *dest = (char *)ob;
+               return srclen;
+       }
 }
 
 /**
@@ -244,178 +495,286 @@ convert:
  * @param dest always set at least to NULL 
  * @note -1 is not accepted for srclen.
  *
- * @retval Size in bytes of the converted string; or -1 in case of error.
+ * @returns Size in bytes of the converted string; or -1 in case of error.
  **/
-size_t convert_string_talloc(TALLOC_CTX *ctx, charset_t from, charset_t to,
+static size_t convert_string_talloc(TALLOC_CTX *ctx, charset_t from, charset_t to,
                                void const *src, size_t srclen, void **dest)
 {
-       void *ob;
        size_t dest_len;
 
        *dest = NULL;
-       dest_len=convert_string_allocate(from, to, src, srclen, (void **)&ob);
-       if (dest_len == -1)
-               return -1;
-       *dest = talloc_strdup(ctx, (char *)ob);
-       SAFE_FREE(ob);
+       dest_len=convert_string_allocate(ctx, from, to, src, srclen, dest);
+       if (dest_len == (size_t)-1)
+               return (size_t)-1;
        if (*dest == NULL)
-               return -1;
+               return (size_t)-1;
        return dest_len;
 }
 
-int unix_strupper(const char *src, size_t srclen, char *dest, size_t destlen)
+size_t unix_strupper(const char *src, size_t srclen, char *dest, size_t destlen)
+{
+       size_t size;
+       smb_ucs2_t *buffer;
+       
+       size = push_ucs2_allocate(&buffer, src);
+       if (size == -1) {
+               smb_panic("failed to create UCS2 buffer");
+       }
+       if (!strupper_w(buffer) && (dest == src)) {
+               free(buffer);
+               return srclen;
+       }
+       
+       size = convert_string(CH_UCS2, CH_UNIX, buffer, size, dest, destlen);
+       free(buffer);
+       return size;
+}
+
+/**
+ strdup() a unix string to upper case.
+ Max size is pstring.
+**/
+
+char *strdup_upper(const char *s)
 {
-       int size;
-       smb_ucs2_t *buffer=(smb_ucs2_t*)cvtbuf;
-       size=convert_string(CH_UNIX, CH_UCS2, src, srclen, buffer, sizeof(cvtbuf));
-       if (!strupper_w(buffer) && (dest == src)) return srclen;
-       return convert_string(CH_UCS2, CH_UNIX, buffer, size, dest, destlen);
+       pstring out_buffer;
+       const unsigned char *p = (const unsigned char *)s;
+       unsigned char *q = (unsigned char *)out_buffer;
+
+       /* this is quite a common operation, so we want it to be
+          fast. We optimise for the ascii case, knowing that all our
+          supported multi-byte character sets are ascii-compatible
+          (ie. they match for the first 128 chars) */
+
+       while (1) {
+               if (*p & 0x80)
+                       break;
+               *q++ = toupper(*p);
+               if (!*p)
+                       break;
+               p++;
+               if (p - ( const unsigned char *)s >= sizeof(pstring))
+                       break;
+       }
+
+       if (*p) {
+               /* MB case. */
+               size_t size;
+               wpstring buffer;
+               size = convert_string(CH_UNIX, CH_UCS2, s, -1, buffer, sizeof(buffer));
+               if (size == -1) {
+                       return NULL;
+               }
+
+               strupper_w(buffer);
+       
+               size = convert_string(CH_UCS2, CH_UNIX, buffer, -1, out_buffer, sizeof(out_buffer));
+               if (size == -1) {
+                       return NULL;
+               }
+       }
+
+       return strdup(out_buffer);
 }
 
-int unix_strlower(const char *src, size_t srclen, char *dest, size_t destlen)
+size_t unix_strlower(const char *src, size_t srclen, char *dest, size_t destlen)
 {
-       int size;
-       smb_ucs2_t *buffer=(smb_ucs2_t*)cvtbuf;
-       size=convert_string(CH_UNIX, CH_UCS2, src, srclen, buffer, sizeof(cvtbuf));
-       if (!strlower_w(buffer) && (dest == src)) return srclen;
-       return convert_string(CH_UCS2, CH_UNIX, buffer, size, dest, destlen);
+       size_t size;
+       smb_ucs2_t *buffer = NULL;
+       
+       size = convert_string_allocate(NULL, CH_UNIX, CH_UCS2, src, srclen,
+                                      (void **) &buffer);
+       if (size == -1 || !buffer) {
+               smb_panic("failed to create UCS2 buffer");
+       }
+       if (!strlower_w(buffer) && (dest == src)) {
+               SAFE_FREE(buffer);
+               return srclen;
+       }
+       size = convert_string(CH_UCS2, CH_UNIX, buffer, size, dest, destlen);
+       SAFE_FREE(buffer);
+       return size;
 }
 
+/**
+ strdup() a unix string to lower case.
+**/
+
+char *strdup_lower(const char *s)
+{
+       size_t size;
+       smb_ucs2_t *buffer = NULL;
+       char *out_buffer;
+       
+       size = push_ucs2_allocate(&buffer, s);
+       if (size == -1 || !buffer) {
+               return NULL;
+       }
+
+       strlower_w(buffer);
+       
+       size = pull_ucs2_allocate(&out_buffer, buffer);
+       SAFE_FREE(buffer);
+
+       if (size == -1) {
+               return NULL;
+       }
+       
+       return out_buffer;
+}
 
-int ucs2_align(const void *base_ptr, const void *p, int flags)
+static size_t ucs2_align(const void *base_ptr, const void *p, int flags)
 {
-       if (flags & (STR_NOALIGN|STR_ASCII)) return 0;
+       if (flags & (STR_NOALIGN|STR_ASCII))
+               return 0;
        return PTR_DIFF(p, base_ptr) & 1;
 }
 
 
-/****************************************************************************
-copy a string from a char* unix src to a dos codepage string destination
-return the number of bytes occupied by the string in the destination
-flags can have:
-  STR_TERMINATE means include the null termination
-  STR_UPPER     means uppercase in the destination
-dest_len is the maximum length allowed in the destination. If dest_len
-is -1 then no maxiumum is used
-****************************************************************************/
-int push_ascii(void *dest, const char *src, int dest_len, int flags)
+/**
+ * Copy a string from a char* unix src to a dos codepage string destination.
+ *
+ * @return the number of bytes occupied by the string in the destination.
+ *
+ * @param flags can include
+ * <dl>
+ * <dt>STR_TERMINATE</dt> <dd>means include the null termination</dd>
+ * <dt>STR_UPPER</dt> <dd>means uppercase in the destination</dd>
+ * </dl>
+ *
+ * @param dest_len the maximum length in bytes allowed in the
+ * destination.  If @p dest_len is -1 then no maximum is used.
+ **/
+size_t push_ascii(void *dest, const char *src, size_t dest_len, int flags)
 {
-       int src_len = strlen(src);
+       size_t src_len = strlen(src);
        pstring tmpbuf;
 
        /* treat a pstring as "unlimited" length */
-       if (dest_len == -1) {
+       if (dest_len == (size_t)-1)
                dest_len = sizeof(pstring);
-       }
 
        if (flags & STR_UPPER) {
                pstrcpy(tmpbuf, src);
-               strupper(tmpbuf);
+               strupper_m(tmpbuf);
                src = tmpbuf;
        }
 
-       if (flags & STR_TERMINATE) {
+       if (flags & (STR_TERMINATE | STR_TERMINATE_ASCII))
                src_len++;
-       }
 
        return convert_string(CH_UNIX, CH_DOS, src, src_len, dest, dest_len);
 }
 
-int push_ascii_fstring(void *dest, const char *src)
+size_t push_ascii_fstring(void *dest, const char *src)
 {
        return push_ascii(dest, src, sizeof(fstring), STR_TERMINATE);
 }
 
-int push_ascii_pstring(void *dest, const char *src)
+size_t push_ascii_pstring(void *dest, const char *src)
 {
        return push_ascii(dest, src, sizeof(pstring), STR_TERMINATE);
 }
 
-int push_pstring(void *dest, const char *src)
+size_t push_ascii_nstring(void *dest, const char *src)
 {
-       return push_ascii(dest, src, sizeof(pstring), STR_TERMINATE);
+       return push_ascii(dest, src, sizeof(nstring), STR_TERMINATE);
 }
 
-
-/****************************************************************************
-copy a string from a dos codepage source to a unix char* destination
-flags can have:
-  STR_TERMINATE means the string in src is null terminated
-if STR_TERMINATE is set then src_len is ignored
-src_len is the length of the source area in bytes
-return the number of bytes occupied by the string in src
-the resulting string in "dest" is always null terminated
-****************************************************************************/
-int pull_ascii(char *dest, const void *src, int dest_len, int src_len, int flags)
+/**
+ * Copy a string from a dos codepage source to a unix char* destination.
+ *
+ * The resulting string in "dest" is always null terminated.
+ *
+ * @param flags can have:
+ * <dl>
+ * <dt>STR_TERMINATE</dt>
+ * <dd>STR_TERMINATE means the string in @p src
+ * is null terminated, and src_len is ignored.</dd>
+ * </dl>
+ *
+ * @param src_len is the length of the source area in bytes.
+ * @returns the number of bytes occupied by the string in @p src.
+ **/
+size_t pull_ascii(char *dest, const void *src, size_t dest_len, size_t src_len, int flags)
 {
-       int ret;
+       size_t ret;
 
-       if (dest_len == -1) {
+       if (dest_len == (size_t)-1)
                dest_len = sizeof(pstring);
-       }
 
        if (flags & STR_TERMINATE) {
-               if (src_len == -1) {
+               if (src_len == (size_t)-1) {
                        src_len = strlen(src) + 1;
                } else {
-                       int len = strnlen(src, src_len);
-                       if (len < src_len) len++;
+                       size_t len = strnlen(src, src_len);
+                       if (len < src_len)
+                               len++;
                        src_len = len;
                }
        }
 
        ret = convert_string(CH_DOS, CH_UNIX, src, src_len, dest, dest_len);
 
-       if (dest_len) dest[MIN(ret, dest_len-1)] = 0;
+       if (dest_len)
+               dest[MIN(ret, dest_len-1)] = 0;
+       else 
+               dest[0] = 0;
 
        return src_len;
 }
 
-int pull_ascii_pstring(char *dest, const void *src)
+size_t pull_ascii_pstring(char *dest, const void *src)
 {
        return pull_ascii(dest, src, sizeof(pstring), -1, STR_TERMINATE);
 }
 
-int pull_ascii_fstring(char *dest, const void *src)
+size_t pull_ascii_fstring(char *dest, const void *src)
 {
        return pull_ascii(dest, src, sizeof(fstring), -1, STR_TERMINATE);
 }
 
-/****************************************************************************
-copy a string from a char* src to a unicode destination
-return the number of bytes occupied by the string in the destination
-flags can have:
-  STR_TERMINATE means include the null termination
-  STR_UPPER     means uppercase in the destination
-  STR_NOALIGN   means don't do alignment
-dest_len is the maximum length allowed in the destination. If dest_len
-is -1 then no maxiumum is used
-****************************************************************************/
-int push_ucs2(const void *base_ptr, void *dest, const char *src, int dest_len, int flags)
-{
-       int len=0;
-       int src_len = strlen(src);
-       pstring tmpbuf;
+size_t pull_ascii_nstring(char *dest, const void *src)
+{
+       return pull_ascii(dest, src, sizeof(nstring), sizeof(nstring), STR_TERMINATE);
+}
+
+/**
+ * Copy a string from a char* src to a unicode destination.
+ *
+ * @returns the number of bytes occupied by the string in the destination.
+ *
+ * @param flags can have:
+ *
+ * <dl>
+ * <dt>STR_TERMINATE <dd>means include the null termination.
+ * <dt>STR_UPPER     <dd>means uppercase in the destination.
+ * <dt>STR_NOALIGN   <dd>means don't do alignment.
+ * </dl>
+ *
+ * @param dest_len is the maximum length allowed in the
+ * destination. If dest_len is -1 then no maxiumum is used.
+ **/
+
+size_t push_ucs2(const void *base_ptr, void *dest, const char *src, size_t dest_len, int flags)
+{
+       size_t len=0;
+       size_t src_len;
 
        /* treat a pstring as "unlimited" length */
-       if (dest_len == -1) {
+       if (dest_len == (size_t)-1)
                dest_len = sizeof(pstring);
-       }
 
-       if (flags & STR_UPPER) {
-               pstrcpy(tmpbuf, src);
-               strupper(tmpbuf);
-               src = tmpbuf;
-       }
-
-       if (flags & STR_TERMINATE) {
-               src_len++;
-       }
+       if (flags & STR_TERMINATE)
+               src_len = (size_t)-1;
+       else
+               src_len = strlen(src);
 
        if (ucs2_align(base_ptr, dest, flags)) {
                *(char *)dest = 0;
                dest = (void *)((char *)dest + 1);
-               if (dest_len) dest_len--;
+               if (dest_len)
+                       dest_len--;
                len++;
        }
 
@@ -423,94 +782,107 @@ int push_ucs2(const void *base_ptr, void *dest, const char *src, int dest_len, i
        dest_len &= ~1;
 
        len += convert_string(CH_UNIX, CH_UCS2, src, src_len, dest, dest_len);
+
+       if (flags & STR_UPPER) {
+               smb_ucs2_t *dest_ucs2 = dest;
+               size_t i;
+               for (i = 0; i < (dest_len / 2) && dest_ucs2[i]; i++) {
+                       smb_ucs2_t v = toupper_w(dest_ucs2[i]);
+                       if (v != dest_ucs2[i]) {
+                               dest_ucs2[i] = v;
+                       }
+               }
+       }
+
        return len;
 }
 
+
 /**
- * Copy a string from a unix char* src to a UCS2 destination, allocating a buffer using talloc
+ * Copy a string from a unix char* src to a UCS2 destination,
+ * allocating a buffer using talloc().
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
+ *         or -1 in case of error.
  **/
-int push_ucs2_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
+size_t push_ucs2_talloc(TALLOC_CTX *ctx, smb_ucs2_t **dest, const char *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = strlen(src)+1;
 
        *dest = NULL;
-       return convert_string_talloc(ctx, CH_UNIX, CH_UCS2, src, src_len, dest);
+       return convert_string_talloc(ctx, CH_UNIX, CH_UCS2, src, src_len, (void **)dest);
 }
 
+
 /**
  * Copy a string from a unix char* src to a UCS2 destination, allocating a buffer
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
+ *         or -1 in case of error.
  **/
-int push_ucs2_allocate(void **dest, const char *src)
+
+size_t push_ucs2_allocate(smb_ucs2_t **dest, const char *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = strlen(src)+1;
 
        *dest = NULL;
-       return convert_string_allocate(CH_UNIX, CH_UCS2, src, src_len, dest);   
+       return convert_string_allocate(NULL, CH_UNIX, CH_UCS2, src, src_len, (void **)dest);    
 }
 
-/****************************************************************************
-copy a string from a char* src to a UTF-8 destination
-return the number of bytes occupied by the string in the destination
-flags can have:
+/**
+ Copy a string from a char* src to a UTF-8 destination.
+ Return the number of bytes occupied by the string in the destination
+ Flags can have:
   STR_TERMINATE means include the null termination
   STR_UPPER     means uppercase in the destination
-dest_len is the maximum length allowed in the destination. If dest_len
-is -1 then no maxiumum is used
-****************************************************************************/
-int push_utf8(void *dest, const char *src, int dest_len, int flags)
+ dest_len is the maximum length allowed in the destination. If dest_len
+ is -1 then no maxiumum is used.
+**/
+
+static size_t push_utf8(void *dest, const char *src, size_t dest_len, int flags)
 {
-       int src_len = strlen(src);
+       size_t src_len = strlen(src);
        pstring tmpbuf;
 
        /* treat a pstring as "unlimited" length */
-       if (dest_len == -1) {
+       if (dest_len == (size_t)-1)
                dest_len = sizeof(pstring);
-       }
 
        if (flags & STR_UPPER) {
                pstrcpy(tmpbuf, src);
-               strupper(tmpbuf);
+               strupper_m(tmpbuf);
                src = tmpbuf;
        }
 
-       if (flags & STR_TERMINATE) {
+       if (flags & STR_TERMINATE)
                src_len++;
-       }
 
        return convert_string(CH_UNIX, CH_UTF8, src, src_len, dest, dest_len);
 }
 
-int push_utf8_fstring(void *dest, const char *src)
+size_t push_utf8_fstring(void *dest, const char *src)
 {
        return push_utf8(dest, src, sizeof(fstring), STR_TERMINATE);
 }
 
-int push_utf8_pstring(void *dest, const char *src)
-{
-       return push_utf8(dest, src, sizeof(pstring), STR_TERMINATE);
-}
-
 /**
  * Copy a string from a unix char* src to a UTF-8 destination, allocating a buffer using talloc
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
  **/
-int push_utf8_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
+
+size_t push_utf8_talloc(TALLOC_CTX *ctx, char **dest, const char *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = strlen(src)+1;
 
        *dest = NULL;
-       return convert_string_talloc(ctx, CH_UNIX, CH_UTF8, src, src_len, dest);
+       return convert_string_talloc(ctx, CH_UNIX, CH_UTF8, src, src_len, (void**)dest);
 }
 
 /**
@@ -518,64 +890,74 @@ int push_utf8_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
  **/
-int push_utf8_allocate(void **dest, const char *src)
+
+size_t push_utf8_allocate(char **dest, const char *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = strlen(src)+1;
 
        *dest = NULL;
-       return convert_string_allocate(CH_UNIX, CH_UTF8, src, src_len, dest);   
+       return convert_string_allocate(NULL, CH_UNIX, CH_UTF8, src, src_len, (void **)dest);    
 }
 
-/****************************************************************************
-copy a string from a ucs2 source to a unix char* destination
-flags can have:
-  STR_TERMINATE means the string in src is null terminated
-  STR_NOALIGN   means don't try to align
-if STR_TERMINATE is set then src_len is ignored if it is -1
-src_len is the length of the source area in bytes
-return the number of bytes occupied by the string in src
-the resulting string in "dest" is always null terminated
-****************************************************************************/
-int pull_ucs2(const void *base_ptr, char *dest, const void *src, int dest_len, int src_len, int flags)
+/**
+ Copy a string from a ucs2 source to a unix char* destination.
+ Flags can have:
+  STR_TERMINATE means the string in src is null terminated.
+  STR_NOALIGN   means don't try to align.
+ if STR_TERMINATE is set then src_len is ignored if it is -1.
+ src_len is the length of the source area in bytes
+ Return the number of bytes occupied by the string in src.
+ The resulting string in "dest" is always null terminated.
+**/
+
+size_t pull_ucs2(const void *base_ptr, char *dest, const void *src, size_t dest_len, size_t src_len, int flags)
 {
-       int ret;
+       size_t ret;
 
-       if (dest_len == -1) {
+       if (dest_len == (size_t)-1)
                dest_len = sizeof(pstring);
-       }
 
        if (ucs2_align(base_ptr, src, flags)) {
                src = (const void *)((const char *)src + 1);
-               if (src_len > 0) src_len--;
+               if (src_len > 0)
+                       src_len--;
        }
 
        if (flags & STR_TERMINATE) {
-               if (src_len == -1) {
-                       src_len = strlen_w(src)*2 + 2;
-               } else {
-                       int len = strnlen_w(src, src_len/2);
-                       if (len < src_len/2) len++;
+               /* src_len -1 is the default for null terminated strings. */
+               if (src_len != (size_t)-1) {
+                       size_t len = strnlen_w(src, src_len/2);
+                       if (len < src_len/2)
+                               len++;
                        src_len = len*2;
                }
        }
 
        /* ucs2 is always a multiple of 2 bytes */
-       src_len &= ~1;
+       if (src_len != (size_t)-1)
+               src_len &= ~1;
        
        ret = convert_string(CH_UCS2, CH_UNIX, src, src_len, dest, dest_len);
-       if (dest_len) dest[MIN(ret, dest_len-1)] = 0;
+       
+       if (src_len == (size_t)-1)
+               src_len = ret*2;
+               
+       if (dest_len)
+               dest[MIN(ret, dest_len-1)] = 0;
+       else 
+               dest[0] = 0;
 
        return src_len;
 }
 
-int pull_ucs2_pstring(char *dest, const void *src)
+size_t pull_ucs2_pstring(char *dest, const void *src)
 {
        return pull_ucs2(NULL, dest, src, sizeof(pstring), -1, STR_TERMINATE);
 }
 
-int pull_ucs2_fstring(char *dest, const void *src)
+size_t pull_ucs2_fstring(char *dest, const void *src)
 {
        return pull_ucs2(NULL, dest, src, sizeof(fstring), -1, STR_TERMINATE);
 }
@@ -585,13 +967,14 @@ int pull_ucs2_fstring(char *dest, const void *src)
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
  **/
-int pull_ucs2_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
+
+size_t pull_ucs2_talloc(TALLOC_CTX *ctx, char **dest, const smb_ucs2_t *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = (strlen_w(src)+1) * sizeof(smb_ucs2_t);
        *dest = NULL;
-       return convert_string_talloc(ctx, CH_UCS2, CH_UNIX, src, src_len, dest);        
+       return convert_string_talloc(ctx, CH_UCS2, CH_UNIX, src, src_len, (void **)dest);
 }
 
 /**
@@ -599,56 +982,14 @@ int pull_ucs2_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
  **/
-int pull_ucs2_allocate(void **dest, const char *src)
-{
-       int src_len = strlen(src)+1;
-       *dest = NULL;
-       return convert_string_allocate(CH_UCS2, CH_UNIX, src, src_len, dest);   
-}
-
-/****************************************************************************
-copy a string from a utf-8 source to a unix char* destination
-flags can have:
-  STR_TERMINATE means the string in src is null terminated
-if STR_TERMINATE is set then src_len is ignored
-src_len is the length of the source area in bytes
-return the number of bytes occupied by the string in src
-the resulting string in "dest" is always null terminated
-****************************************************************************/
-int pull_utf8(char *dest, const void *src, int dest_len, int src_len, int flags)
-{
-       int ret;
-
-       if (dest_len == -1) {
-               dest_len = sizeof(pstring);
-       }
-
-       if (flags & STR_TERMINATE) {
-               if (src_len == -1) {
-                       src_len = strlen(src) + 1;
-               } else {
-                       int len = strnlen(src, src_len);
-                       if (len < src_len) len++;
-                       src_len = len;
-               }
-       }
-
-       ret = convert_string(CH_UTF8, CH_UNIX, src, src_len, dest, dest_len);
-       if (dest_len) dest[MIN(ret, dest_len-1)] = 0;
-
-       return src_len;
-}
-
-int pull_utf8_pstring(char *dest, const void *src)
-{
-       return pull_utf8(dest, src, sizeof(pstring), -1, STR_TERMINATE);
-}
 
-int pull_utf8_fstring(char *dest, const void *src)
+size_t pull_ucs2_allocate(char **dest, const smb_ucs2_t *src)
 {
-       return pull_utf8(dest, src, sizeof(fstring), -1, STR_TERMINATE);
+       size_t src_len = (strlen_w(src)+1) * sizeof(smb_ucs2_t);
+       *dest = NULL;
+       return convert_string_allocate(NULL, CH_UCS2, CH_UNIX, src, src_len, (void **)dest);    
 }
 
 /**
@@ -656,13 +997,14 @@ int pull_utf8_fstring(char *dest, const void *src)
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
  **/
-int pull_utf8_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
+
+size_t pull_utf8_talloc(TALLOC_CTX *ctx, char **dest, const char *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = strlen(src)+1;
        *dest = NULL;
-       return convert_string_talloc(ctx, CH_UTF8, CH_UNIX, src, src_len, dest);        
+       return convert_string_talloc(ctx, CH_UTF8, CH_UNIX, src, src_len, (void **)dest);       
 }
 
 /**
@@ -670,30 +1012,48 @@ int pull_utf8_talloc(TALLOC_CTX *ctx, void **dest, const char *src)
  *
  * @param dest always set at least to NULL 
  *
- * @retval The number of bytes occupied by the string in the destination
+ * @returns The number of bytes occupied by the string in the destination
  **/
-int pull_utf8_allocate(void **dest, const char *src)
+
+size_t pull_utf8_allocate(char **dest, const char *src)
 {
-       int src_len = strlen(src)+1;
+       size_t src_len = strlen(src)+1;
        *dest = NULL;
-       return convert_string_allocate(CH_UTF8, CH_UNIX, src, src_len, dest);   
+       return convert_string_allocate(NULL, CH_UTF8, CH_UNIX, src, src_len, (void **)dest);    
 }
  
-/****************************************************************************
-copy a string from a char* src to a unicode or ascii
-dos codepage destination choosing unicode or ascii based on the 
-flags in the SMB buffer starting at base_ptr
-return the number of bytes occupied by the string in the destination
-flags can have:
-  STR_TERMINATE means include the null termination
-  STR_UPPER     means uppercase in the destination
-  STR_ASCII     use ascii even with unicode packet
-  STR_NOALIGN   means don't do alignment
-dest_len is the maximum length allowed in the destination. If dest_len
-is -1 then no maxiumum is used
-****************************************************************************/
-int push_string(const void *base_ptr, void *dest, const char *src, int dest_len, int flags)
+/**
+ Copy a string from a char* src to a unicode or ascii
+ dos codepage destination choosing unicode or ascii based on the 
+ flags in the SMB buffer starting at base_ptr.
+ Return the number of bytes occupied by the string in the destination.
+ flags can have:
+  STR_TERMINATE means include the null termination.
+  STR_UPPER     means uppercase in the destination.
+  STR_ASCII     use ascii even with unicode packet.
+  STR_NOALIGN   means don't do alignment.
+ dest_len is the maximum length allowed in the destination. If dest_len
+ is -1 then no maxiumum is used.
+**/
+
+size_t push_string_fn(const char *function, unsigned int line, const void *base_ptr, void *dest, const char *src, size_t dest_len, int flags)
 {
+#ifdef DEVELOPER
+       /* We really need to zero fill here, not clobber
+        * region, as we want to ensure that valgrind thinks
+        * all of the outgoing buffer has been written to
+        * so a send() or write() won't trap an error.
+        * JRA.
+        */
+#if 0
+       if (dest_len != (size_t)-1)
+               clobber_region(function, line, dest, dest_len);
+#else
+       if (dest_len != (size_t)-1)
+               memset(dest, '\0', dest_len);
+#endif
+#endif
+
        if (!(flags & STR_ASCII) && \
            ((flags & STR_UNICODE || \
              (SVAL(base_ptr, smb_flg2) & FLAGS2_UNICODE_STRINGS)))) {
@@ -703,22 +1063,27 @@ int push_string(const void *base_ptr, void *dest, const char *src, int dest_len,
 }
 
 
-/****************************************************************************
-copy a string from a unicode or ascii source (depending on
-the packet flags) to a char* destination
-flags can have:
-  STR_TERMINATE means the string in src is null terminated
-  STR_UNICODE   means to force as unicode
-  STR_ASCII     use ascii even with unicode packet
-  STR_NOALIGN   means don't do alignment
-if STR_TERMINATE is set then src_len is ignored is it is -1
-src_len is the length of the source area in bytes
-return the number of bytes occupied by the string in src
-the resulting string in "dest" is always null terminated
-****************************************************************************/
-int pull_string(const void *base_ptr, char *dest, const void *src, int dest_len, int src_len, 
-               int flags)
+/**
+ Copy a string from a unicode or ascii source (depending on
+ the packet flags) to a char* destination.
+ Flags can have:
+  STR_TERMINATE means the string in src is null terminated.
+  STR_UNICODE   means to force as unicode.
+  STR_ASCII     use ascii even with unicode packet.
+  STR_NOALIGN   means don't do alignment.
+ if STR_TERMINATE is set then src_len is ignored is it is -1
+ src_len is the length of the source area in bytes.
+ Return the number of bytes occupied by the string in src.
+ The resulting string in "dest" is always null terminated.
+**/
+
+size_t pull_string_fn(const char *function, unsigned int line, const void *base_ptr, char *dest, const void *src, size_t dest_len, size_t src_len, int flags)
 {
+#ifdef DEVELOPER
+       if (dest_len != (size_t)-1)
+               clobber_region(function, line, dest, dest_len);
+#endif
+
        if (!(flags & STR_ASCII) && \
            ((flags & STR_UNICODE || \
              (SVAL(base_ptr, smb_flg2) & FLAGS2_UNICODE_STRINGS)))) {
@@ -727,7 +1092,7 @@ int pull_string(const void *base_ptr, char *dest, const void *src, int dest_len,
        return pull_ascii(dest, src, dest_len, src_len, flags);
 }
 
-int align_string(const void *base_ptr, const char *p, int flags)
+size_t align_string(const void *base_ptr, const char *p, int flags)
 {
        if (!(flags & STR_ASCII) && \
            ((flags & STR_UNICODE || \
@@ -736,77 +1101,3 @@ int align_string(const void *base_ptr, const char *p, int flags)
        }
        return 0;
 }
-
-
-
-/****************************************************************************
-convert from ucs2 to unix charset and return the
-allocated and converted string or NULL if an error occurred.
-you must provide a zero terminated string.
-the returning string will be zero terminated.
-****************************************************************************/
-char *acnv_u2ux(const smb_ucs2_t *src)
-{
-       size_t slen;
-       size_t dlen;
-       void *dest;
-       
-       slen = (strlen_w(src) + 1) * sizeof(smb_ucs2_t);
-       dlen = convert_string_allocate(CH_UCS2, CH_UNIX, src, slen, &dest);
-       if (dlen == -1) return NULL;
-       else return dest;
-}
-
-/****************************************************************************
-convert from unix to ucs2 charset and return the
-allocated and converted string or NULL if an error occurred.
-you must provide a zero terminated string.
-the returning string will be zero terminated.
-****************************************************************************/
-smb_ucs2_t *acnv_uxu2(const char *src)
-{
-       size_t slen;
-       size_t dlen;
-       void *dest;
-       
-       slen = strlen(src) + 1;
-       dlen = convert_string_allocate(CH_UNIX, CH_UCS2, src, slen, &dest);
-       if (dlen == -1) return NULL;
-       else return dest;
-}
-
-/****************************************************************************
-convert from ucs2 to dos charset and return the
-allocated and converted string or NULL if an error occurred.
-you must provide a zero terminated string.
-the returning string will be zero terminated.
-****************************************************************************/
-char *acnv_u2dos(const smb_ucs2_t *src)
-{
-       size_t slen;
-       size_t dlen;
-       void *dest;
-       
-       slen = (strlen_w(src) + 1) * sizeof(smb_ucs2_t);
-       dlen = convert_string_allocate(CH_UCS2, CH_DOS, src, slen, &dest);
-       if (dlen == -1) return NULL;
-       else return dest;
-}
-
-/****************************************************************************
-convert from dos to ucs2 charset and return the
-allocated and converted string or NULL if an error occurred.
-you must provide a zero terminated string.
-the returning string will be zero terminated.
-****************************************************************************/
-smb_ucs2_t *acnv_dosu2(const char *src)
-{
-       size_t slen;
-       size_t dlen;
-       void *dest;
-       
-       slen = strlen(src) + 1;
-       dlen = convert_string_allocate(CH_DOS, CH_UCS2, src, slen, &dest);
-       if (dlen == -1) return NULL;
-       else return dest;
-}