[asterisk-commits] russell: branch 1.4 r314822 - /branches/1.4/res/res_agi.c

[SVN commits to the Asterisk project]

Author: russell
Date: Fri Apr 22 09:34:23 2011
New Revision: 314822

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=314822
Log:
Initialize buffers in getvar and getvarfull.

Initialize the buffers used to hold the result from GET VARIABLE or
GET VARIABLE FULL.  The bug report shows func_read returning garbage in
the result.  It assumed that the buffer passed in was initialized, like many
other functions do.  In the more common code path (through the dialplan), it
is initialized, so just initialize it here too.

(closes issue #19050)
Reported by: johnz

Modified:
    branches/1.4/res/res_agi.c

Modified: branches/1.4/res/res_agi.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.4/res/res_agi.c?view=diff&rev=314822&r1=314821&r2=314822
==============================================================================
--- branches/1.4/res/res_agi.c (original)
+++ branches/1.4/res/res_agi.c Fri Apr 22 09:34:23 2011
@@ -1215,7 +1215,7 @@
 static int handle_getvariable(struct ast_channel *chan, AGI *agi, int argc, char **argv)
 {
 	char *ret;
-	char tempstr[1024];
+	char tempstr[1024] = "";
 
 	if (argc != 3)
 		return RESULT_SHOWUSAGE;