summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2003-04-02 00:03:52 +0000
committerAndrew Bartlett <abartlet@samba.org>2003-04-02 00:03:52 +0000
commita4fe384f1d3ba07c4b91c7c5530e862b41355555 (patch)
treec015ae6d72bb411479dd1f62b7d7b5f9b0d9e94f
parent63c87884d494b22b6b7f055ed75dc259c2997e79 (diff)
downloadsamba-a4fe384f1d3ba07c4b91c7c5530e862b41355555.tar.gz
samba-a4fe384f1d3ba07c4b91c7c5530e862b41355555.tar.bz2
samba-a4fe384f1d3ba07c4b91c7c5530e862b41355555.zip
OHKAWA Yuichi (kuri@makino.ecei.tohoku.ac.jp) points out that using
decoded.length after it's been zero'ed out by data_blob_free() doesn't give you the original length... Andrew Bartlett (This used to be commit 647a720cfb06bdf0dfc8fcdcab50aadcceb0494d)
-rw-r--r--source3/lib/util_str.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/lib/util_str.c b/source3/lib/util_str.c
index 4d955c59a7..e2f9f19f58 100644
--- a/source3/lib/util_str.c
+++ b/source3/lib/util_str.c
@@ -1662,10 +1662,10 @@ void base64_decode_inplace(char *s)
{
DATA_BLOB decoded = base64_decode_data_blob(s);
memcpy(s, decoded.data, decoded.length);
- data_blob_free(&decoded);
-
/* null terminate */
s[decoded.length] = '\0';
+
+ data_blob_free(&decoded);
}
/**