r23792: convert Samba4 to GPLv3
[samba.git] / source4 / librpc / ndr / ndr_string.c
index e9d6425f54fd14b135440513942fcb3718c24366..fca864a5ebf1eb56e76aca750a1feffee6668e01 100644 (file)
@@ -7,7 +7,7 @@
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
-   the Free Software Foundation; either version 2 of the License, or
+   the Free Software Foundation; either version 3 of the License, or
    (at your option) any later version.
    
    This program is distributed in the hope that it will be useful,
    GNU General Public License for more details.
    
    You should have received a copy of the GNU General Public License
-   along with this program; if not, write to the Free Software
-   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "includes.h"
-#include "system/network.h"
+#include "librpc/ndr/libndr.h"
 
-/*
+/**
   pull a general string from the wire
 */
-NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
+_PUBLIC_ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
 {
        char *as=NULL;
        uint32_t len1, ofs, len2;
@@ -66,7 +65,6 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
        switch (flags & LIBNDR_STRING_FLAGS) {
        case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_SIZE4:
        case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_SIZE4|LIBNDR_FLAG_STR_NOTERM:
-       case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_SIZE4|LIBNDR_FLAG_STR_NOTERM|LIBNDR_FLAG_STR_LARGE_SIZE:
                NDR_CHECK(ndr_pull_uint32(ndr, NDR_SCALARS, &len1));
                NDR_CHECK(ndr_pull_uint32(ndr, NDR_SCALARS, &ofs));
                if (ofs != 0) {
@@ -79,31 +77,24 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                                              "Bad string lengths len1=%u ofs=%u len2=%u\n", 
                                              len1, ofs, len2);
                }
-               if (len2 == 0) {
-                       *s = talloc_strdup(ndr, "");
-                       break;
-               }
                NDR_PULL_NEED_BYTES(ndr, (len2 + c_len_term)*byte_mul);
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
-                                           ndr->data+ndr->offset, 
-                                           (len2 + c_len_term)*byte_mul,
-                                           (void **)&as);
-               if (ret == -1) {
-                       return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
+               if (len2 == 0) {
+                       as = talloc_strdup(ndr->current_mem_ctx, "");
+               } else {
+                       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                                   chset, CH_UNIX, 
+                                                   ndr->data+ndr->offset, 
+                                                   (len2 + c_len_term)*byte_mul,
+                                                   (void **)&as);
+                       if (ret == -1) {
+                               return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
+                                                     "Bad character conversion");
+                       }
                }
                NDR_CHECK(ndr_pull_advance(ndr, (len2 + c_len_term)*byte_mul));
 
-               if (ndr->flags & LIBNDR_FLAG_STR_LARGE_SIZE) {
-                       if (len1 != 0 && len2 == 0) {
-                               DEBUG(6,("len1[%u] != (len2[%u]) '%s'\n", len1, len2, as));
-                       } else if (len1 != (len2 + 1)) {
-                               DEBUG(6,("len1[%u] != (len2[%u]+1) '%s'\n", len1, len2, as));
-                       }
-               } else {
-                       if (len1 != len2) {
-                               DEBUG(6,("len1[%u] != len2[%u] '%s'\n", len1, len2, as));
-                       }
+               if (len1 != len2) {
+                       DEBUG(6,("len1[%u] != len2[%u] '%s'\n", len1, len2, as));
                }
 
                /* this is a way of detecting if a string is sent with the wrong
@@ -125,16 +116,17 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                NDR_CHECK(ndr_pull_uint32(ndr, NDR_SCALARS, &len1));
                NDR_PULL_NEED_BYTES(ndr, (len1 + c_len_term)*byte_mul);
                if (len1 == 0) {
-                       *s = talloc_strdup(ndr, "");
-                       break;
-               }
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
-                                           ndr->data+ndr->offset, 
-                                           (len1 + c_len_term)*byte_mul,
-                                           (void **)&as);
-               if (ret == -1) {
-                       return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
+                       as = talloc_strdup(ndr->current_mem_ctx, "");
+               } else {
+                       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                                   chset, CH_UNIX, 
+                                                   ndr->data+ndr->offset, 
+                                                   (len1 + c_len_term)*byte_mul,
+                                                   (void **)&as);
+                       if (ret == -1) {
+                               return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
+                                                     "Bad character conversion");
+                       }
                }
                NDR_CHECK(ndr_pull_advance(ndr, (len1 + c_len_term)*byte_mul));
 
@@ -162,16 +154,17 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                NDR_CHECK(ndr_pull_uint32(ndr, NDR_SCALARS, &len1));
                NDR_PULL_NEED_BYTES(ndr, (len1 + c_len_term)*byte_mul);
                if (len1 == 0) {
-                       *s = talloc_strdup(ndr, "");
-                       break;
-               }
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
-                                           ndr->data+ndr->offset, 
-                                           (len1 + c_len_term)*byte_mul,
-                                           (void **)&as);
-               if (ret == -1) {
-                       return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
+                       as = talloc_strdup(ndr->current_mem_ctx, "");
+               } else {
+                       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                                   chset, CH_UNIX, 
+                                                   ndr->data+ndr->offset, 
+                                                   (len1 + c_len_term)*byte_mul,
+                                                   (void **)&as);
+                       if (ret == -1) {
+                               return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
+                                                     "Bad character conversion");
+                       }
                }
                NDR_CHECK(ndr_pull_advance(ndr, (len1 + c_len_term)*byte_mul));
 
@@ -195,16 +188,17 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                NDR_CHECK(ndr_pull_uint16(ndr, NDR_SCALARS, &len3));
                NDR_PULL_NEED_BYTES(ndr, (len3 + c_len_term)*byte_mul);
                if (len3 == 0) {
-                       *s = talloc_strdup(ndr, "");
-                       break;
-               }
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
-                                           ndr->data+ndr->offset, 
-                                           (len3 + c_len_term)*byte_mul,
-                                           (void **)&as);
-               if (ret == -1) {
-                       return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
+                       as = talloc_strdup(ndr->current_mem_ctx, "");
+               } else {
+                       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                                   chset, CH_UNIX, 
+                                                   ndr->data+ndr->offset, 
+                                                   (len3 + c_len_term)*byte_mul,
+                                                   (void **)&as);
+                       if (ret == -1) {
+                               return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
+                                                     "Bad character conversion");
+                       }
                }
                NDR_CHECK(ndr_pull_advance(ndr, (len3 + c_len_term)*byte_mul));
 
@@ -226,16 +220,17 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                NDR_CHECK(ndr_pull_uint16(ndr, NDR_SCALARS, &len3));
                NDR_PULL_NEED_BYTES(ndr, len3);
                if (len3 == 0) {
-                       *s = talloc_strdup(ndr, "");
-                       break;
-               }
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
-                                           ndr->data+ndr->offset, 
-                                           len3,
-                                           (void **)&as);
-               if (ret == -1) {
-                       return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
+                       as = talloc_strdup(ndr->current_mem_ctx, "");
+               } else {
+                       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                                   chset, CH_UNIX, 
+                                                   ndr->data+ndr->offset, 
+                                                   len3,
+                                                   (void **)&as);
+                       if (ret == -1) {
+                               return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
+                                                     "Bad character conversion");
+                       }
                }
                NDR_CHECK(ndr_pull_advance(ndr, len3));
                *s = as;
@@ -247,7 +242,8 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                } else {
                        len1 = utf16_len_n(ndr->data+ndr->offset, ndr->data_size - ndr->offset);
                }
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
+               ret = convert_string_talloc(ndr->current_mem_ctx,
+                                           chset, CH_UNIX, 
                                            ndr->data+ndr->offset, 
                                            len1,
                                            (void **)&as);
@@ -263,7 +259,8 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
        case LIBNDR_FLAG_STR_FIXLEN32:
                len1 = (flags & LIBNDR_FLAG_STR_FIXLEN32)?32:15;
                NDR_PULL_NEED_BYTES(ndr, len1*byte_mul);
-               ret = convert_string_talloc(ndr, chset, CH_UNIX, 
+               ret = convert_string_talloc(ndr->current_mem_ctx,
+                                           chset, CH_UNIX, 
                                            ndr->data+ndr->offset, 
                                            len1*byte_mul,
                                            (void **)&as);
@@ -275,6 +272,33 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
                *s = as;
                break;
 
+       case LIBNDR_FLAG_STR_NOTERM:
+               if (!(ndr->flags & LIBNDR_FLAG_REMAINING)) {
+                       return ndr_pull_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x (missing NDR_REMAINING)\n",
+                                             ndr->flags & LIBNDR_STRING_FLAGS);
+               }
+
+               len1 = ndr->data_size - ndr->offset;
+
+               NDR_PULL_NEED_BYTES(ndr, len1);
+               if (len1 == 0) {
+                       as = talloc_strdup(ndr->current_mem_ctx, "");
+               } else {
+                       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                                   chset, CH_UNIX, 
+                                                   ndr->data+ndr->offset, 
+                                                   len1,
+                                                   (void **)&as);
+                       if (ret == -1) {
+                               return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
+                                                     "Bad character conversion");
+                       }
+               }
+               NDR_CHECK(ndr_pull_advance(ndr, len1));
+
+               *s = as;
+               break;
+
        default:
                return ndr_pull_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x\n",
                                      ndr->flags & LIBNDR_STRING_FLAGS);
@@ -284,17 +308,16 @@ NTSTATUS ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
 }
 
 
-/*
+/**
   push a general string onto the wire
 */
-NTSTATUS ndr_push_string(struct ndr_push *ndr, int ndr_flags, const char *s)
+_PUBLIC_ NTSTATUS ndr_push_string(struct ndr_push *ndr, int ndr_flags, const char *s)
 {
        ssize_t s_len, c_len, d_len;
-       int ret;
        int chset = CH_UTF16;
        unsigned flags = ndr->flags;
        unsigned byte_mul = 2;
-       unsigned c_len_term = 1;
+       uint8_t *dest = NULL;
 
        if (!(ndr_flags & NDR_SCALARS)) {
                return NT_STATUS_OK;
@@ -305,7 +328,6 @@ NTSTATUS ndr_push_string(struct ndr_push *ndr, int ndr_flags, const char *s)
        }
        
        s_len = s?strlen(s):0;
-       c_len = s?strlen_m(s):0;
 
        if (flags & LIBNDR_FLAG_STR_ASCII) {
                chset = CH_DOS;
@@ -321,154 +343,90 @@ NTSTATUS ndr_push_string(struct ndr_push *ndr, int ndr_flags, const char *s)
 
        flags &= ~LIBNDR_FLAG_STR_CONFORMANT;
 
-       if (flags & LIBNDR_FLAG_STR_CHARLEN) {
-               c_len_term = 0;
+       if (!(flags & 
+             (LIBNDR_FLAG_STR_NOTERM |
+              LIBNDR_FLAG_STR_FIXLEN15 |
+              LIBNDR_FLAG_STR_FIXLEN32))) {
+               s_len++;
+       }
+       d_len = convert_string_talloc(ndr, CH_UNIX, chset, s, s_len, (void **)&dest);
+       if (d_len == -1) {
+               return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
+                                     "Bad character conversion");
+       }
+
+       if (flags & LIBNDR_FLAG_STR_BYTESIZE) {
+               c_len = d_len;
+               flags &= ~LIBNDR_FLAG_STR_BYTESIZE;
+       } else if (flags & LIBNDR_FLAG_STR_CHARLEN) {
+               c_len = (d_len / byte_mul)-1;
                flags &= ~LIBNDR_FLAG_STR_CHARLEN;
+       } else {
+               c_len = d_len / byte_mul;
        }
 
-       switch (flags & LIBNDR_STRING_FLAGS) {
+       switch ((flags & LIBNDR_STRING_FLAGS) & ~LIBNDR_FLAG_STR_NOTERM) {
        case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_SIZE4:
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len+c_len_term));
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len+c_len_term));
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*(c_len+1));
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len+1,
-                                    ndr->data+ndr->offset, 
-                                    byte_mul*(c_len+1));
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += byte_mul*(c_len+1);
-               break;
-
-       case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_SIZE4|LIBNDR_FLAG_STR_NOTERM:
-               c_len_term = 0;
-       case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_SIZE4|LIBNDR_FLAG_STR_NOTERM|LIBNDR_FLAG_STR_LARGE_SIZE:
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len+c_len_term));
+               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len));
                NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
                NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len));
-               NDR_PUSH_NEED_BYTES(ndr, c_len*byte_mul);
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len,
-                                    ndr->data+ndr->offset, c_len*byte_mul);
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += c_len*byte_mul;
+               NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
                break;
 
        case LIBNDR_FLAG_STR_LEN4:
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len + c_len_term));
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*(c_len+1));
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len + 1,
-                                    ndr->data+ndr->offset, byte_mul*(c_len+1));
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += byte_mul*(c_len+1);
-               break;
-
-       case LIBNDR_FLAG_STR_LEN4|LIBNDR_FLAG_STR_NOTERM:
                NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
                NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len));
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*c_len);
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len,
-                                    ndr->data+ndr->offset, byte_mul*c_len);
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += byte_mul*c_len;
+               NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
                break;
 
        case LIBNDR_FLAG_STR_SIZE4:
-               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len + c_len_term));
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*(c_len+1));
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len + 1,
-                                    ndr->data+ndr->offset, byte_mul*(c_len+1));
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += byte_mul*(c_len+1);
+               NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, c_len));
+               NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
                break;
 
        case LIBNDR_FLAG_STR_SIZE2:
-               NDR_CHECK(ndr_push_uint16(ndr, NDR_SCALARS, c_len + c_len_term));
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*(c_len+1));
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len + 1,
-                                    ndr->data+ndr->offset, byte_mul*(c_len+1));
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += byte_mul*(c_len+1);
+               NDR_CHECK(ndr_push_uint16(ndr, NDR_SCALARS, c_len));
+               NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
                break;
 
        case LIBNDR_FLAG_STR_NULLTERM:
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*(c_len+1));
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len+1,
-                                    ndr->data+ndr->offset, byte_mul*(c_len+1));
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += byte_mul*(c_len+1);
-               break;
-
-       case LIBNDR_FLAG_STR_SIZE2|LIBNDR_FLAG_STR_NOTERM|LIBNDR_FLAG_STR_BYTESIZE:
-               NDR_CHECK(ndr_push_uint16(ndr, NDR_SCALARS, c_len*byte_mul));
-               NDR_PUSH_NEED_BYTES(ndr, c_len*byte_mul);
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len,
-                                    ndr->data+ndr->offset, c_len*byte_mul);
-               if (ret == -1) {
-                       return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
-                                             "Bad character conversion");
-               }
-               ndr->offset += c_len*byte_mul;
+               NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
                break;
 
        case LIBNDR_FLAG_STR_FIXLEN15:
-       case LIBNDR_FLAG_STR_FIXLEN32:
-               d_len = (flags & LIBNDR_FLAG_STR_FIXLEN32)?32:15;
-               NDR_PUSH_NEED_BYTES(ndr, byte_mul*d_len);
-               ret = convert_string(CH_UNIX, chset, 
-                                    s, s_len,
-                                    ndr->data+ndr->offset, byte_mul*d_len);
-               if (ret == -1) {
+       case LIBNDR_FLAG_STR_FIXLEN32: {
+               ssize_t fix_len = (flags & LIBNDR_FLAG_STR_FIXLEN32)?32:15;
+               uint32_t pad_len = fix_len - d_len;
+               if (d_len > fix_len) {
                        return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
                                              "Bad character conversion");
                }
-               ndr->offset += ret;
-               if ((byte_mul*d_len) > ret) {
-                       uint32_t _padding_len = (byte_mul*d_len) - ret;
-                       NDR_CHECK(ndr_push_zero(ndr, _padding_len));
+               NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
+               if (pad_len != 0) {
+                       NDR_CHECK(ndr_push_zero(ndr, pad_len));
                }
                break;
+       }
 
        default:
+               if (ndr->flags & LIBNDR_FLAG_REMAINING) {
+                       NDR_CHECK(ndr_push_bytes(ndr, dest, d_len));
+                       break;          
+               }
+
                return ndr_push_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x\n",
                                      ndr->flags & LIBNDR_STRING_FLAGS);
        }
 
+       talloc_free(dest);
+
        return NT_STATUS_OK;
 }
 
-/*
+/**
   push a general string onto the wire
 */
-size_t ndr_string_array_size(struct ndr_push *ndr, const char *s)
+_PUBLIC_ size_t ndr_string_array_size(struct ndr_push *ndr, const char *s)
 {
        size_t c_len;
        unsigned flags = ndr->flags;
@@ -501,7 +459,7 @@ size_t ndr_string_array_size(struct ndr_push *ndr, const char *s)
        return c_len;
 }
 
-void ndr_print_string(struct ndr_print *ndr, const char *name, const char *s)
+_PUBLIC_ void ndr_print_string(struct ndr_print *ndr, const char *name, const char *s)
 {
        if (s) {
                ndr->print(ndr, "%-25s: '%s'", name, s);
@@ -510,66 +468,152 @@ void ndr_print_string(struct ndr_print *ndr, const char *name, const char *s)
        }
 }
 
-uint32_t ndr_size_string(int ret, const char * const* string, int flags) 
+_PUBLIC_ uint32_t ndr_size_string(int ret, const char * const* string, int flags) 
 {
        /* FIXME: Is this correct for all strings ? */
        if(!(*string)) return ret;
        return ret+strlen(*string)+1;
 }
 
-/*
+/**
   pull a general string array from the wire
 */
-NTSTATUS ndr_pull_string_array(struct ndr_pull *ndr, int ndr_flags, const char ***_a)
+_PUBLIC_ NTSTATUS ndr_pull_string_array(struct ndr_pull *ndr, int ndr_flags, const char ***_a)
 {
        const char **a = *_a;
        uint32_t count;
+       unsigned flags = ndr->flags;
+       unsigned saved_flags = ndr->flags;
 
        if (!(ndr_flags & NDR_SCALARS)) {
                return NT_STATUS_OK;
        }
 
-       for (count = 0;; count++) {
-               const char *s = NULL;
-               a = talloc_realloc(ndr, a, const char *, count + 2);
-               NT_STATUS_HAVE_NO_MEMORY(a);
-               a[count]   = NULL;
-               a[count+1]   = NULL;
+       switch (flags & LIBNDR_STRING_FLAGS) {
+       case LIBNDR_FLAG_STR_NULLTERM:
+               /* 
+                * here the strings are null terminated
+                * but also the array is null terminated
+                */
+               for (count = 0;; count++) {
+                       TALLOC_CTX *tmp_ctx;
+                       const char *s = NULL;
+                       a = talloc_realloc(ndr->current_mem_ctx, a, const char *, count + 2);
+                       NT_STATUS_HAVE_NO_MEMORY(a);
+                       a[count]   = NULL;
+                       a[count+1]   = NULL;
+
+                       tmp_ctx = ndr->current_mem_ctx;
+                       ndr->current_mem_ctx = a;
+                       NDR_CHECK(ndr_pull_string(ndr, ndr_flags, &s));
+                       ndr->current_mem_ctx = tmp_ctx;
+                       if (strcmp("", s)==0) {
+                               a[count] = NULL;
+                               break;
+                       } else {
+                               a[count] = s;
+                       }
+               }
 
-               NDR_CHECK(ndr_pull_string(ndr, ndr_flags, &s));
-               if (strcmp("", s)==0) {
-                       a[count] = NULL;
-                       break;
-               } else {
+               *_a =a;
+               break;
+
+       case LIBNDR_FLAG_STR_NOTERM:
+               if (!(ndr->flags & LIBNDR_FLAG_REMAINING)) {
+                       return ndr_pull_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x (missing NDR_REMAINING)\n",
+                                             ndr->flags & LIBNDR_STRING_FLAGS);
+               }
+               /*
+                * here the strings are not null terminated
+                * but serarated by a null terminator
+                *
+                * which means the same as:
+                * very string is null terminated exept the last
+                * string is terminated by the end of the buffer
+                *
+                * as LIBNDR_FLAG_STR_NULLTERM also end at the end
+                * of the buffer, we can pull each string with this flag
+                */
+               ndr->flags &= ~(LIBNDR_FLAG_STR_NOTERM|LIBNDR_FLAG_REMAINING);
+               ndr->flags |= LIBNDR_FLAG_STR_NULLTERM;
+
+               for (count = 0; ((ndr->data_size - ndr->offset) > 0); count++) {
+                       TALLOC_CTX *tmp_ctx;
+                       const char *s = NULL;
+                       a = talloc_realloc(ndr->current_mem_ctx, a, const char *, count + 2);
+                       NT_STATUS_HAVE_NO_MEMORY(a);
+                       a[count]   = NULL;
+                       a[count+1]   = NULL;
+
+                       tmp_ctx = ndr->current_mem_ctx;
+                       ndr->current_mem_ctx = a;
+                       NDR_CHECK(ndr_pull_string(ndr, ndr_flags, &s));
+                       ndr->current_mem_ctx = tmp_ctx;
                        a[count] = s;
                }
+
+               *_a =a;
+               break;
+
+       default:
+               return ndr_pull_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x\n",
+                                     ndr->flags & LIBNDR_STRING_FLAGS);
        }
 
-       *_a =a;
+       ndr->flags = saved_flags;
        return NT_STATUS_OK;
 }
 
-/*
+/**
   push a general string array onto the wire
 */
-NTSTATUS ndr_push_string_array(struct ndr_push *ndr, int ndr_flags, const char **a)
+_PUBLIC_ NTSTATUS ndr_push_string_array(struct ndr_push *ndr, int ndr_flags, const char **a)
 {
        uint32_t count;
+       unsigned flags = ndr->flags;
+       unsigned saved_flags = ndr->flags;
 
        if (!(ndr_flags & NDR_SCALARS)) {
                return NT_STATUS_OK;
        }
 
-       for (count = 0; a && a[count]; count++) {
-               NDR_CHECK(ndr_push_string(ndr, ndr_flags, a[count]));
-       }
+       switch (flags & LIBNDR_STRING_FLAGS) {
+       case LIBNDR_FLAG_STR_NULLTERM:
+               for (count = 0; a && a[count]; count++) {
+                       NDR_CHECK(ndr_push_string(ndr, ndr_flags, a[count]));
+               }
 
-       NDR_CHECK(ndr_push_string(ndr, ndr_flags, ""));
+               NDR_CHECK(ndr_push_string(ndr, ndr_flags, ""));
+               break;
 
+       case LIBNDR_FLAG_STR_NOTERM:
+               if (!(ndr->flags & LIBNDR_FLAG_REMAINING)) {
+                       return ndr_push_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x (missing NDR_REMAINING)\n",
+                                             ndr->flags & LIBNDR_STRING_FLAGS);
+               }
+
+               for (count = 0; a && a[count]; count++) {
+                       if (count > 0) {
+                               ndr->flags &= ~(LIBNDR_FLAG_STR_NOTERM|LIBNDR_FLAG_REMAINING);
+                               ndr->flags |= LIBNDR_FLAG_STR_NULLTERM;
+                               NDR_CHECK(ndr_push_string(ndr, ndr_flags, ""));
+                               ndr->flags = saved_flags;
+                       }
+                       NDR_CHECK(ndr_push_string(ndr, ndr_flags, a[count]));
+               }
+
+               break;
+
+       default:
+               return ndr_push_error(ndr, NDR_ERR_STRING, "Bad string flags 0x%x\n",
+                                     ndr->flags & LIBNDR_STRING_FLAGS);
+       }
+       
+       ndr->flags = saved_flags;
        return NT_STATUS_OK;
 }
 
-void ndr_print_string_array(struct ndr_print *ndr, const char *name, const char **a)
+_PUBLIC_ void ndr_print_string_array(struct ndr_print *ndr, const char *name, const char **a)
 {
        uint32_t count;
        uint32_t i;
@@ -589,8 +633,10 @@ void ndr_print_string_array(struct ndr_print *ndr, const char *name, const char
        ndr->depth--;
 }
 
-/* Return number of elements in a string including the last (zeroed) element */
-uint32_t ndr_string_length(const void *_var, uint32_t element_size)
+/**
+ * Return number of elements in a string including the last (zeroed) element 
+ */
+_PUBLIC_ uint32_t ndr_string_length(const void *_var, uint32_t element_size)
 {
        uint32_t i;
        uint8_t zero[4] = {0,0,0,0};
@@ -601,32 +647,44 @@ uint32_t ndr_string_length(const void *_var, uint32_t element_size)
        return i+1;
 }
 
-NTSTATUS ndr_check_string_terminator(struct ndr_pull *ndr, const void *_var, uint32_t count, uint32_t element_size)
+_PUBLIC_ NTSTATUS ndr_check_string_terminator(struct ndr_pull *ndr, uint32_t count, uint32_t element_size)
 {
-       const char *var = _var;
        uint32_t i;
+       struct ndr_pull_save save_offset;
 
-       var += element_size*(count-1);
+       ndr_pull_save(ndr, &save_offset);
+       ndr_pull_advance(ndr, (count - 1) * element_size);
+       NDR_PULL_NEED_BYTES(ndr, element_size);
 
        for (i = 0; i < element_size; i++) {
-                if (var[i] != 0) {
-                       return NT_STATUS_UNSUCCESSFUL;
+                if (ndr->data[ndr->offset+i] != 0) {
+                       ndr_pull_restore(ndr, &save_offset);
+
+                       return ndr_pull_error(ndr, NDR_ERR_ARRAY_SIZE, "String terminator not present or outside string boundaries");
                 }
        }
 
-       return NT_STATUS_OK;
+       ndr_pull_restore(ndr, &save_offset);
 
+       return NT_STATUS_OK;
 }
 
-NTSTATUS ndr_pull_charset(struct ndr_pull *ndr, int ndr_flags, const char **var, uint32_t length, uint8_t byte_mul, int chset)
+_PUBLIC_ NTSTATUS ndr_pull_charset(struct ndr_pull *ndr, int ndr_flags, const char **var, uint32_t length, uint8_t byte_mul, charset_t chset)
 {
        int ret;
        if (length == 0) {
-               *var = talloc_strdup(ndr, "");
+               *var = talloc_strdup(ndr->current_mem_ctx, "");
                return NT_STATUS_OK;
        }
+
+       if (NDR_BE(ndr) && chset == CH_UTF16) {
+               chset = CH_UTF16BE;
+       }
+
        NDR_PULL_NEED_BYTES(ndr, length*byte_mul);
-       ret = convert_string_talloc(ndr, chset, CH_UNIX, 
+
+       ret = convert_string_talloc(ndr->current_mem_ctx,
+                                   chset, CH_UNIX, 
                                    ndr->data+ndr->offset, 
                                    length*byte_mul,
                                    discard_const_p(void *, var));
@@ -639,10 +697,14 @@ NTSTATUS ndr_pull_charset(struct ndr_pull *ndr, int ndr_flags, const char **var,
        return NT_STATUS_OK;
 }
 
-NTSTATUS ndr_push_charset(struct ndr_push *ndr, int ndr_flags, const char *var, uint32_t length, uint8_t byte_mul, int chset)
+_PUBLIC_ NTSTATUS ndr_push_charset(struct ndr_push *ndr, int ndr_flags, const char *var, uint32_t length, uint8_t byte_mul, charset_t chset)
 {
        ssize_t ret, required;
 
+       if (NDR_BE(ndr) && chset == CH_UTF16) {
+               chset = CH_UTF16BE;
+       }
+
        required = byte_mul * length;
        
        NDR_PUSH_NEED_BYTES(ndr, required);
@@ -663,3 +725,11 @@ NTSTATUS ndr_push_charset(struct ndr_push *ndr, int ndr_flags, const char *var,
 
        return NT_STATUS_OK;
 }
+
+/* Return number of elements in a string in the specified charset */
+_PUBLIC_ uint32_t ndr_charset_length(const void *var, charset_t chset)
+{
+       /* FIXME: Treat special chars special here, taking chset into account */
+       /* Also include 0 byte */
+       return strlen(var)+1;
+}