util/charset/convert: when retrying, retry from the start

iconv() advances the inbuf pointer; if we decide to realloc and re-iconv,
we need to reset inbuf to the source string

Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/lib/util/charset/convert_string.c b/lib/util/charset/convert_string.c
index 459121426788d3d2a8deb1731d4a8945b3c52668..baff3d9e30882b2b184d1beab7014b549108bf7e 100644 (file)
--- a/lib/util/charset/convert_string.c
+++ b/lib/util/charset/convert_string.c
@@ -351,7 +351,7 @@ bool convert_string_talloc_handle(TALLOC_CTX *ctx, struct smb_iconv_handle *ic,
 {
        size_t i_len, o_len, destlen;
        size_t retval;
-       const char *inbuf = (const char *)src;
+       const char *inbuf = NULL;
        char *outbuf = NULL, *ob = NULL;
        smb_iconv_t descriptor;
        void **dest = (void **)dst;
@@ -430,6 +430,7 @@ bool convert_string_talloc_handle(TALLOC_CTX *ctx, struct smb_iconv_handle *ic,
        outbuf = ob;
        i_len = srclen;
        o_len = destlen;
+       inbuf = (const char *)src;
 
        retval = smb_iconv(descriptor,
                           &inbuf, &i_len,