r26261: Use enum type.
[jelmer/samba4-debian.git] / source / librpc / ndr / ndr_string.c
index 95e9df39f06aa649777ea43b1cb9c0908fe512e1..a15855493c142a557f98aaffca782587ea3e185b 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_ enum ndr_err_code ndr_pull_string(struct ndr_pull *ndr, int ndr_flags, const char **s)
 {
        char *as=NULL;
        uint32_t len1, ofs, len2;
        uint16_t len3;
        int ret;
-       int chset = CH_UTF16;
+       charset_t chset = CH_UTF16;
        unsigned byte_mul = 2;
        unsigned flags = ndr->flags;
        unsigned c_len_term = 0;
 
        if (!(ndr_flags & NDR_SCALARS)) {
-               return NT_STATUS_OK;
+               return NDR_ERR_SUCCESS;
        }
 
        if (NDR_BE(ndr)) {
@@ -78,21 +77,26 @@ 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 (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
                   termination */
                if (ndr->flags & LIBNDR_FLAG_STR_NOTERM) {
@@ -112,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));
 
@@ -149,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));
 
@@ -182,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));
 
@@ -213,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;
@@ -234,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);
@@ -250,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);
@@ -262,29 +272,55 @@ 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);
        }
 
-       return NT_STATUS_OK;
+       return NDR_ERR_SUCCESS;
 }
 
 
-/*
+/**
   push a general string onto the wire
 */
-NTSTATUS ndr_push_string(struct ndr_push *ndr, int ndr_flags, const char *s)
+_PUBLIC_ enum ndr_err_code 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;
+               return NDR_ERR_SUCCESS;
        }
 
        if (NDR_BE(ndr)) {
@@ -292,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;
@@ -308,148 +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:
                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 += byte_mul*d_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);
        }
 
-       return NT_STATUS_OK;
+       talloc_free(dest);
+
+       return NDR_ERR_SUCCESS;
 }
 
-/*
+/**
   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;
@@ -482,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);
@@ -491,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_ enum ndr_err_code 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;
+               return NDR_ERR_SUCCESS;
        }
 
-       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);
+                       NDR_ERR_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);
+                       NDR_ERR_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;
-       return NT_STATUS_OK;
+       ndr->flags = saved_flags;
+       return NDR_ERR_SUCCESS;
 }
 
-/*
+/**
   push a general string array onto the wire
 */
-NTSTATUS ndr_push_string_array(struct ndr_push *ndr, int ndr_flags, const char **a)
+_PUBLIC_ enum ndr_err_code 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;
+               return NDR_ERR_SUCCESS;
        }
 
-       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);
+               }
 
-       return NT_STATUS_OK;
+               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 NDR_ERR_SUCCESS;
 }
 
-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;
@@ -570,65 +633,103 @@ 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(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};
-       char *var = _var;
+       const char *var = (const char *)_var;
 
        for (i = 0; memcmp(var+i*element_size,zero,element_size) != 0; i++);
 
        return i+1;
 }
 
-NTSTATUS ndr_check_string_terminator(struct ndr_pull *ndr, const void *_var, uint32_t count, uint32_t element_size)
+_PUBLIC_ enum ndr_err_code ndr_check_string_terminator(struct ndr_pull *ndr, uint32_t count, uint32_t element_size)
 {
-       const char *var = _var;
        uint32_t i;
+       uint32_t save_offset;
 
-       var += element_size*(count-1);
+       save_offset = ndr->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->offset = save_offset;
+
+                       return ndr_pull_error(ndr, NDR_ERR_ARRAY_SIZE, "String terminator not present or outside string boundaries");
                 }
        }
 
-       return NT_STATUS_OK;
+       ndr->offset = save_offset;
 
+       return NDR_ERR_SUCCESS;
 }
 
-NTSTATUS ndr_pull_charset(struct ndr_pull *ndr, int ndr_flags, char **var, uint32_t length, uint8_t byte_mul, int chset)
+_PUBLIC_ enum ndr_err_code 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->current_mem_ctx, "");
+               return NDR_ERR_SUCCESS;
+       }
+
+       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,
-                                   (void **)var);
+                                   discard_const_p(void *, var));
        if (ret == -1) {
                return ndr_pull_error(ndr, NDR_ERR_CHARCNV, 
                                      "Bad character conversion");
        }
        NDR_CHECK(ndr_pull_advance(ndr, length*byte_mul));
 
-       return NT_STATUS_OK;
+       return NDR_ERR_SUCCESS;
 }
 
-NTSTATUS ndr_push_charset(struct ndr_push *ndr, int ndr_flags, const char *var, uint32_t length, uint8_t byte_mul, int chset)
+_PUBLIC_ enum ndr_err_code ndr_push_charset(struct ndr_push *ndr, int ndr_flags, const char *var, uint32_t length, uint8_t byte_mul, charset_t chset)
 {
-       int ret;
-       NDR_PUSH_NEED_BYTES(ndr, byte_mul*length);
+       ssize_t ret, required;
+
+       if (NDR_BE(ndr) && chset == CH_UTF16) {
+               chset = CH_UTF16BE;
+       }
+
+       required = byte_mul * length;
+       
+       NDR_PUSH_NEED_BYTES(ndr, required);
        ret = convert_string(CH_UNIX, chset, 
-                            var, length,
-                            ndr->data+ndr->offset, 
-                            byte_mul*length);
+                            var, strlen(var),
+                            ndr->data+ndr->offset, required);
        if (ret == -1) {
                return ndr_push_error(ndr, NDR_ERR_CHARCNV, 
                                      "Bad character conversion");
        }
-       ndr->offset += byte_mul*length;
 
-       return NT_STATUS_OK;
+       /* Make sure the remaining part of the string is filled with zeroes */
+       if (ret < required) {
+               memset(ndr->data+ndr->offset+ret, 0, required-ret);
+       }
+
+       ndr->offset += required;
+
+       return NDR_ERR_SUCCESS;
+}
+
+/* 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((const char *)var)+1;
 }