Merged revisions 86237 via svnmerge from
authorRussell Bryant <russell@russellbryant.com>
Thu, 18 Oct 2007 04:41:33 +0000 (04:41 +0000)
committerRussell Bryant <russell@russellbryant.com>
Thu, 18 Oct 2007 04:41:33 +0000 (04:41 +0000)
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

index 25de47e..653c313 100644 (file)
@@ -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;
 }