[asterisk-commits] jpeeler: branch 1.4 r212498 - /branches/1.4/channels/misdn_config.c

[SVN commits to the Asterisk project]

Author: jpeeler
Date: Mon Aug 17 11:34:56 2009
New Revision: 212498

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=212498
Log:
Fix segfault when reloading chan_misdn.

If more ports were specified than configured in misdn.conf a reload would crash
asterisk. The problem was the unconfigured port was using data from the
previously configured port. When the data for an unconfigured port was freed a
crash would result from the double free.

(closes issue #12113)
Reported by: agupta
Patches:
      bug12113.patch uploaded by jpeeler (license 325)

Modified:
    branches/1.4/channels/misdn_config.c

Modified: branches/1.4/channels/misdn_config.c
URL: http://svn.asterisk.org/svn-view/asterisk/branches/1.4/channels/misdn_config.c?view=diff&rev=212498&r1=212497&r2=212498
==============================================================================
--- branches/1.4/channels/misdn_config.c (original)
+++ branches/1.4/channels/misdn_config.c Mon Aug 17 11:34:56 2009
@@ -1001,6 +1001,11 @@
 	}
 
 	for (i = 0; i < (max_ports + 1); ++i) {
+		if (i > 0 && cfg_for_ports[0]) {
+			/* default category, will populate the port_cfg with additional port
+			categories in subsequent calls to this function */
+			memset(cfg_tmp, 0, sizeof(cfg_tmp));
+		}
 		if (cfg_for_ports[i]) {
 			memcpy(port_cfg[i], cfg_tmp, sizeof(cfg_tmp));
 		}