diff options
| author | Russell Bryant <russell@russellbryant.com> | 2007-10-18 04:41:33 +0000 |
|---|---|---|
| committer | Russell Bryant <russell@russellbryant.com> | 2007-10-18 04:41:33 +0000 |
| commit | ac4d66cc738ce45c90926066bd7f3a21f8ea3a60 (patch) | |
| tree | 8a3f5dc71811469988773976bf5e0c5c1cb87001 /main/utils.c | |
| parent | d1a0d959a71e3e17d01ba84d0079063f533e8475 (diff) | |
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
Diffstat (limited to 'main/utils.c')
| -rw-r--r-- | main/utils.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/main/utils.c b/main/utils.c index 25de47ef3..653c31312 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; } |