From ac4d66cc738ce45c90926066bd7f3a21f8ea3a60 Mon Sep 17 00:00:00 2001 From: Russell Bryant Date: Thu, 18 Oct 2007 04:41:33 +0000 Subject: [PATCH] Merged revisions 86237 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r86237 | russell | 2007-10-17 23:40:52 -0500 (Wed, 17 Oct 2007) | 9 lines Revert a change that I made for issue #10979 which, as has been pointed out to me in issue #11018, doesn't really make sense. There is no reason to have the base64 decode function force a '\0' terminated buffer, when the result is almost always binary, anyway. In fact, this caused some breakage, as some code in res_crypto passed in a buffer exactly the right size to get its binary result, which got stomped on by this patch. (closes issue #11018, reported by dimas) ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@86238 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- main/utils.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/main/utils.c b/main/utils.c index 25de47ef3d..653c31312b 100644 --- a/main/utils.c +++ b/main/utils.c @@ -352,9 +352,6 @@ int ast_base64decode(unsigned char *dst, const char *src, int max) cnt++; } } - if (cnt == max) - dst--; - *dst = '\0'; /* Dont worry about left over bits, they're extra anyway */ return cnt; }