[asterisk-commits] russell: branch 1.4 r86237 - /branches/1.4/main/utils.c

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


Author: russell
Date: Wed Oct 17 23:40:52 2007
New Revision: 86237

URL: http://svn.digium.com/view/asterisk?view=rev&rev=86237
Log:
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:
    branches/1.4/main/utils.c

Modified: branches/1.4/main/utils.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/main/utils.c?view=diff&rev=86237&r1=86236&r2=86237
==============================================================================
--- branches/1.4/main/utils.c (original)
+++ branches/1.4/main/utils.c Wed Oct 17 23:40:52 2007
@@ -347,9 +347,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