[asterisk-commits] russell: branch 1.6.2 r289425 - in /branches/1.6.2: ./ apps/app_sms.c

[SVN commits to the Asterisk project]

Author: russell
Date: Thu Sep 30 10:37:29 2010
New Revision: 289425

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=289425
Log:
Merged revisions 289424 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r289424 | russell | 2010-09-30 10:34:29 -0500 (Thu, 30 Sep 2010) | 8 lines
  
  Fix a crash in app_sms.
  
  Since the data being passed to the generator callback is on the stack of the
  SMS() application, we must ensure that the generator is stopped before the
  application exits.
  
  ABE-2543
........

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/apps/app_sms.c

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

Modified: branches/1.6.2/apps/app_sms.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/apps/app_sms.c?view=diff&rev=289425&r1=289424&r2=289425
==============================================================================
--- branches/1.6.2/apps/app_sms.c (original)
+++ branches/1.6.2/apps/app_sms.c Thu Sep 30 10:37:29 2010
@@ -2039,6 +2039,12 @@
 	}
 	res = h.err;                            /* XXX */
 
+	/* 
+	 * The SMS generator data is on the stack.  We _MUST_ make sure the generator
+	 * is stopped before returning from this function.
+	 */
+	ast_deactivate_generator(chan);
+
 	sms_log(&h, '?');                       /* log incomplete message */
 done:
 	return (res);