[asterisk-commits] russell: trunk r86238 - in /trunk: ./ main/utils.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Oct 17 23:41:34 CDT 2007


Author: russell
Date: Wed Oct 17 23:41:33 2007
New Revision: 86238

URL: http://svn.digium.com/view/asterisk?view=rev&rev=86238
Log:
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)

........

Modified:
    trunk/   (props changed)
    trunk/main/utils.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/main/utils.c
URL: http://svn.digium.com/view/asterisk/trunk/main/utils.c?view=diff&rev=86238&r1=86237&r2=86238
==============================================================================
--- trunk/main/utils.c (original)
+++ trunk/main/utils.c Wed Oct 17 23:41:33 2007
@@ -352,9 +352,6 @@
 			cnt++;
 		}
 	}
-	if (cnt == max)
-		dst--;
-	*dst = '\0';
 	/* Dont worry about left over bits, they're extra anyway */
 	return cnt;
 }




More information about the asterisk-commits mailing list