[asterisk-commits] russell: branch 1.6.0 r175296 - in /branches/1.6.0: ./ channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: russell
Date: Thu Feb 12 14:46:11 2009
New Revision: 175296

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=175296
Log:
Merged revisions 175295 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
r175295 | russell | 2009-02-12 14:45:47 -0600 (Thu, 12 Feb 2009) | 2 lines

Avoid using ast_strdupa() in a loop.

........

Modified:
    branches/1.6.0/   (props changed)
    branches/1.6.0/channels/chan_sip.c

Propchange: branches/1.6.0/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.0/channels/chan_sip.c
URL: http://svn.digium.com/svn-view/asterisk/branches/1.6.0/channels/chan_sip.c?view=diff&rev=175296&r1=175295&r2=175296
==============================================================================
--- branches/1.6.0/channels/chan_sip.c (original)
+++ branches/1.6.0/channels/chan_sip.c Thu Feb 12 14:46:11 2009
@@ -6834,7 +6834,8 @@
 	/* XXX This needs to be done per media stream, since it's media stream specific */
 	iterator = req->sdp_start;
 	while ((a = get_sdp_iterate(&iterator, req, "a"))[0] != '\0') {
-		char* mimeSubtype = ast_strdupa(a); /* ensures we have enough space */
+		char mimeSubtype[128];
+		ast_copy_string(mimeSubtype, a, sizeof(mimeSubtype));
 		if (option_debug > 1) {
 			int breakout = FALSE;