[asterisk-commits] mmichelson: branch group/appdocsxml r143131 - /team/group/appdocsxml/main/pbx.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Mon Sep 15 12:49:44 2008
New Revision: 143131

URL: http://svn.digium.com/view/asterisk?view=rev&rev=143131
Log:
We should only attempt to free these variables if XML_DOCUMENTATION
is defined. Otherwise, we're freeing memory that was allocated
using alloca(). Freeing stacked memory will cause a crash.


Modified:
    team/group/appdocsxml/main/pbx.c

Modified: team/group/appdocsxml/main/pbx.c
URL: http://svn.digium.com/view/asterisk/team/group/appdocsxml/main/pbx.c?view=diff&rev=143131&r1=143130&r2=143131
==============================================================================
--- team/group/appdocsxml/main/pbx.c (original)
+++ team/group/appdocsxml/main/pbx.c Mon Sep 15 12:49:44 2008
@@ -5938,9 +5938,11 @@
 
 					ast_cli(a->fd,"%s%s%s\n\n%s%s\n\n%s%s\n\n%s%s\n", infotitle, stxtitle, syntax, argtitle, 
 						arguments, syntitle, synopsis, destitle, description);
+#ifdef XML_DOCUMENTATION
 					ast_free(description);
 					ast_free(arguments);
 					ast_free(synopsis);
+#endif
 				} else {
 					/* ... one of our applications, show info ...*/
 					ast_cli(a->fd,"\n  -= Info about application '%s' =- \n\n"