[svn-commits] rmudgett: trunk r294127 - in /trunk: ./ channels/chan_misdn.c

SVN commits to the Digium repositories svn-commits at lists.digium.com
Mon Nov 8 11:19:08 CST 2010


Author: rmudgett
Date: Mon Nov  8 11:19:04 2010
New Revision: 294127

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

........
  r294125 | rmudgett | 2010-11-08 11:16:01 -0600 (Mon, 08 Nov 2010) | 33 lines
  
  valgrind reported references to freed memory during a mISDN hangup collision.
  
  Bad things have been happening in chan_misdn because the chan_misdn
  channel private struct chan_list is not protected from reentrancy.  Hangup
  collisions have be causing read and write accesses to freed memory.
  
  Converted chan_misdn struct chan_list to an ao2 object for its reference
  counting feature.
  
  **********
  Removed an impediment to converting chan_list to an ao2 object.
  
  The use of the other_ch member in chan_list is shaky at best.  It is set
  if the incoming and outgoing call legs are mISDN.  The use of the other_ch
  member goes against the Asterisk architecture and can even cause problems.
  
  1) It is used to disable echo cancellation.  This could be bad if the call
  is forked and the winning call leg is not mISDN or the winning call leg is
  not the last mISDN channel called by the fork.  The other_ch would become
  a dangling pointer.
  
  2) It is used when the far end is alerting to hear the far end's inband
  audio instead of Asterisk's generated ringback tone.  This is bad if the
  call is forked.  You would only hear the last forked mISDN channel and it
  may not be ringing yet.
  
  The other_ch would become a dangling pointer if the call is later
  transferred.
  **********
  
  JIRA SWP-2423
  JIRA ABE-2614
........

Modified:
    trunk/   (props changed)
    trunk/channels/chan_misdn.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: trunk/channels/chan_misdn.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_misdn.c?view=diff&rev=294127&r1=294126&r2=294127
==============================================================================
--- trunk/channels/chan_misdn.c (original)
+++ trunk/channels/chan_misdn.c Mon Nov  8 11:19:04 2010
@@ -338,6 +338,9 @@
 	int channel;
 };
 
+#define chan_list_ref(obj, debug) (ao2_t_ref((obj), +1, (debug)), (obj))
+#define chan_list_unref(obj, debug) (ao2_t_ref((obj), -1, (debug)), NULL)
+
 /*!
  * \brief Channel call record structure
  */
@@ -543,19 +546,6 @@
 	int nttimeout;
 
 	/*!
-	 * \brief Other channel call record PID
-	 * \note Value imported from Asterisk environment variable MISDN_PID
-	 */
-	int other_pid;
-
-	/*!
-	 * \brief Bridged other channel call record
-	 * \note Pointer set when other_pid imported from Asterisk environment
-	 * variable MISDN_PID by either side.
-	 */
-	struct chan_list *other_ch;
-
-	/*!
 	 * \brief Tone zone sound used for dialtone generation.
 	 * \note Used as a boolean.  Non-NULL to prod generation if enabled.
 	 */
@@ -744,6 +734,7 @@
 	return 0;
 }
 
+/*! Returns a reference to the found chan_list. */
 static struct chan_list *get_chan_by_ast(struct ast_channel *ast)
 {
 	struct chan_list *tmp;
@@ -751,6 +742,7 @@
 	ast_mutex_lock(&cl_te_lock);
 	for (tmp = cl_te; tmp; tmp = tmp->next) {
 		if (tmp->ast == ast) {
+			chan_list_ref(tmp, "Found chan_list by ast");
 			ast_mutex_unlock(&cl_te_lock);
 			return tmp;
 		}
@@ -760,6 +752,7 @@
 	return NULL;
 }
 
+/*! Returns a reference to the found chan_list. */
 static struct chan_list *get_chan_by_ast_name(const char *name)
 {
 	struct chan_list *tmp;
@@ -767,6 +760,7 @@
 	ast_mutex_lock(&cl_te_lock);
 	for (tmp = cl_te; tmp; tmp = tmp->next) {
 		if (tmp->ast && strcmp(tmp->ast->name, name) == 0) {
+			chan_list_ref(tmp, "Found chan_list by ast name");
 			ast_mutex_unlock(&cl_te_lock);
 			return tmp;
 		}
@@ -5199,12 +5193,15 @@
 			ast_verbose("Sending CD with nr %s to %s failed: Channel does not exist.\n", nr, channame);
 			return 0;
 		}
+		ao2_lock(tmp);
 
 #if defined(AST_MISDN_ENHANCEMENTS)
 		max_len = sizeof(tmp->bc->fac_out.u.CallDeflection.Component.Invoke.Deflection.Party.Number) - 1;
 		if (max_len < strlen(nr)) {
 			ast_verbose("Sending CD with nr %s to %s failed: Number too long (up to %u digits are allowed).\n",
 				nr, channame, max_len);
+			ao2_unlock(tmp);
+			chan_list_unref(tmp, "Number too long");
 			return 0;
 		}
 		tmp->bc->fac_out.Function = Fac_CallDeflection;
@@ -5223,6 +5220,8 @@
 		if (max_len < strlen(nr)) {
 			ast_verbose("Sending CD with nr %s to %s failed: Number too long (up to %u digits are allowed).\n",
 				nr, channame, max_len);
+			ao2_unlock(tmp);
+			chan_list_unref(tmp, "Number too long");
 			return 0;
 		}
 		tmp->bc->fac_out.Function = Fac_CD;
@@ -5233,7 +5232,9 @@
 
 		/* Send message */
 		print_facility(&tmp->bc->fac_out, tmp->bc);
+		ao2_unlock(tmp);
 		misdn_lib_send_event(tmp->bc, EVENT_FACILITY);
+		chan_list_unref(tmp, "Send facility complete");
 #if defined(AST_MISDN_ENHANCEMENTS)
 	} else if (strstr(a->argv[3], "callrerouteing") || strstr(a->argv[3], "callrerouting")) {
 		if (a->argc < 6) {
@@ -5249,11 +5250,14 @@
 			ast_verbose("Sending Call Rerouting with nr %s to %s failed: Channel does not exist.\n", nr, channame);
 			return 0;
 		}
+		ao2_lock(tmp);
 
 		max_len = sizeof(tmp->bc->fac_out.u.CallRerouteing.Component.Invoke.CalledAddress.Party.Number) - 1;
 		if (max_len < strlen(nr)) {
 			ast_verbose("Sending Call Rerouting with nr %s to %s failed: Number too long (up to %u digits are allowed).\n",
 				nr, channame, max_len);
+			ao2_unlock(tmp);
+			chan_list_unref(tmp, "Number too long");
 			return 0;
 		}
 		tmp->bc->fac_out.Function = Fac_CallRerouteing;
@@ -5284,11 +5288,13 @@
 
 		/* Send message */
 		print_facility(&tmp->bc->fac_out, tmp->bc);
+		ao2_unlock(tmp);
 		misdn_lib_send_event(tmp->bc, EVENT_FACILITY);
+		chan_list_unref(tmp, "Send facility complete");
 #endif	/* defined(AST_MISDN_ENHANCEMENTS) */
-		} else if (strstr(a->argv[3], "CFActivate")) {
-			if (a->argc < 7) {
-				ast_verbose("CFActivate requires 2 args: 1.FromNumber, 2.ToNumber\n\n");
+	} else if (strstr(a->argv[3], "CFActivate")) {
+		if (a->argc < 7) {
+			ast_verbose("CFActivate requires 2 args: 1.FromNumber, 2.ToNumber\n\n");
 			return 0;
 		}
 		port = atoi(a->argv[4]);
@@ -5373,20 +5379,23 @@
 		}
 		port = atoi(a->argv[4]);
 
-		channame = argv[4];
+		channame = a->argv[4];
 		tmp = get_chan_by_ast_name(channame);
 		if (tmp) {
 			/* We are going to send this FACILITY message out on an existing connection */
-			msg_number = atoi(argv[5]);
+			msg_number = atoi(a->argv[5]);
 			if (msg_number < ARRAY_LEN(Fac_Msgs)) {
+				ao2_lock(tmp);
 				tmp->bc->fac_out = Fac_Msgs[msg_number];
 
 				/* Send message */
 				print_facility(&tmp->bc->fac_out, tmp->bc);
+				ao2_unlock(tmp);
 				misdn_lib_send_event(tmp->bc, EVENT_FACILITY);
 			} else {
 				ast_verbose("test <channel-name> <msg#>\n\n");
 			}
+			chan_list_unref(tmp, "Facility test done");
 		} else if (a->argc < 6) {
 			for (msg_number = 0; msg_number < ARRAY_LEN(Fac_Msgs); ++msg_number) {
 				misdn_make_dummy(bc, port, 0, misdn_lib_port_is_nt(port), 0);
@@ -5410,16 +5419,16 @@
 				ast_verbose("test <port> [<msg#>]\n\n");
 			}
 		}
-	} else if (strstr(argv[3], "register")) {
-		if (argc < 5) {
-			ast_cli(fd, "register <port>\n\n");
+	} else if (strstr(a->argv[3], "register")) {
+		if (a->argc < 5) {
+			ast_verbose("register <port>\n\n");
 			return 0;
 		}
-		port = atoi(argv[4]);
+		port = atoi(a->argv[4]);
 
 		bc = misdn_lib_get_register_bc(port);
 		if (!bc) {
-			ast_cli(fd, "Could not allocate REGISTER bc struct\n\n");
+			ast_verbose("Could not allocate REGISTER bc struct\n\n");
 			return 0;
 		}
 		bc->fac_out = Fac_Msgs[45];
@@ -5501,6 +5510,9 @@
 	}
 #if 1
 	for (i = 0; i < msglen; i++) {
+		if (!tmp->ast) {
+			break;
+		}
 		ast_cli(a->fd, "Sending: %c\n", msg[i]);
 		send_digit_to_chan(tmp, msg[i]);
 		/* res = ast_safe_sleep(tmp->ast, 250); */
@@ -5508,8 +5520,11 @@
 		/* res = ast_waitfor(tmp->ast,100); */
 	}
 #else
-	ast_dtmf_stream(tmp->ast, NULL, msg, 250);
+	if (tmp->ast) {
+		ast_dtmf_stream(tmp->ast, NULL, msg, 250);
+	}
 #endif
+	chan_list_unref(tmp, "Digit(s) sent");
 
 	return CLI_SUCCESS;
 }
@@ -5556,6 +5571,7 @@
 	} else {
 		manager_ec_disable(tmp->bc);
 	}
+	chan_list_unref(tmp, "Done toggling echo cancel");
 
 	return CLI_SUCCESS;
 }
@@ -5586,12 +5602,16 @@
 	msg = a->argv[4];
 
 	ast_cli(a->fd, "Sending %s to %s\n", msg, channame);
+
 	tmp = get_chan_by_ast_name(channame);
-
 	if (tmp && tmp->bc) {
 		ast_copy_string(tmp->bc->display, msg, sizeof(tmp->bc->display));
 		misdn_lib_send_event(tmp->bc, EVENT_INFORMATION);
+		chan_list_unref(tmp, "Done sending display");
 	} else {
+		if (tmp) {
+			chan_list_unref(tmp, "Display failed");
+		}
 		ast_cli(a->fd, "No such channel %s\n", channame);
 		return CLI_SUCCESS;
 	}
@@ -6457,7 +6477,6 @@
 	int port = 0;
 	int r;
 	int exceed;
-	int bridging;
 	int number_type;
 	struct chan_list *ch;
 	struct misdn_bchannel *newbc;
@@ -6675,20 +6694,6 @@
 			newbc->div_leg_3_rx_wanted = 1;
 		}
 #endif	/* defined(AST_MISDN_ENHANCEMENTS) */
-
-		/*check for bridging*/
-		misdn_cfg_get(0, MISDN_GEN_BRIDGING, &bridging, sizeof(bridging));
-		if (bridging && ch->other_ch) {
-#ifdef MISDN_1_2
-			chan_misdn_log(1, port, "Disabling EC (aka Pipeline) on both Sides\n");
-			*ch->bc->pipeline = 0;
-			*ch->other_ch->bc->pipeline = 0;
-#else
-			chan_misdn_log(1, port, "Disabling EC on both Sides\n");
-			ch->bc->ec_enable = 0;
-			ch->other_ch->bc->ec_enable = 0;
-#endif
-		}
 	}
 
 	exceed = add_out_calls(port);
@@ -6937,18 +6942,6 @@
 			chan_misdn_log(2, p->bc->port, " --> * IND :\tringing pid:%d\n", p->bc->pid);
 			misdn_lib_send_event(p->bc, EVENT_ALERTING);
 
-			if (p->other_ch && p->other_ch->bc) {
-				if (misdn_inband_avail(p->other_ch->bc)) {
-					chan_misdn_log(2, p->bc->port, " --> other End is mISDN and has inband info available\n");
-					break;
-				}
-
-				if (!p->other_ch->bc->nt) {
-					chan_misdn_log(2, p->bc->port, " --> other End is mISDN TE so it has inband info for sure (?)\n");
-					break;
-				}
-			}
-
 			chan_misdn_log(3, p->bc->port, " --> * SEND: State Ring pid:%d\n", p->bc->pid);
 			ast_setstate(ast, AST_STATE_RING);
 
@@ -7034,15 +7027,22 @@
 		return -1;
 	}
 
+	ast_debug(1, "misdn_hangup(%s)\n", ast->name);
+
+	/* Take the ast_channel's tech_pvt reference. */
 	ast_mutex_lock(&release_lock);
 	p = MISDN_ASTERISK_TECH_PVT(ast);
-	if (!p || !misdn_chan_is_valid(p)) {
+	if (!p) {
 		ast_mutex_unlock(&release_lock);
 		return -1;
 	}
 	MISDN_ASTERISK_TECH_PVT(ast) = NULL;
 
-	ast_debug(1, "misdn_hangup(%s)\n", ast->name);
+	if (!misdn_chan_is_valid(p)) {
+		ast_mutex_unlock(&release_lock);
+		chan_list_unref(p, "Release ast_channel reference.  Was not active?");
+		return 0;
+	}
 
 	if (p->hold.state == MISDN_HOLD_IDLE) {
 		bc = p->bc;
@@ -7054,6 +7054,7 @@
 				"misdn_hangup: Could not find held bc for (%s)\n", ast->name);
 			release_chan_early(p);
 			ast_mutex_unlock(&release_lock);
+			chan_list_unref(p, "Release ast_channel reference");
 			return 0;
 		}
 	}
@@ -7066,6 +7067,7 @@
 			misdn_lib_release(bc);
 		}
 		ast_mutex_unlock(&release_lock);
+		chan_list_unref(p, "Release ast_channel reference");
 		return 0;
 	}
 	if (!bc) {
@@ -7073,6 +7075,7 @@
 			misdn_get_ch_state(p), p->l3id);
 		release_chan_early(p);
 		ast_mutex_unlock(&release_lock);
+		chan_list_unref(p, "Release ast_channel reference");
 		return 0;
 	}
 
@@ -7132,6 +7135,7 @@
 		release_chan(p, bc);
 		misdn_lib_send_event(bc, EVENT_RELEASE_COMPLETE);
 		ast_mutex_unlock(&release_lock);
+		chan_list_unref(p, "Release ast_channel reference");
 		return 0;
 	case MISDN_DIALING:
 		if (p->hold.state == MISDN_HOLD_IDLE) {
@@ -7185,6 +7189,7 @@
 
 	case MISDN_CLEANING:
 		ast_mutex_unlock(&release_lock);
+		chan_list_unref(p, "Release ast_channel reference");
 		return 0;
 
 	case MISDN_BUSY:
@@ -7208,6 +7213,7 @@
 		misdn_get_ch_state(p));
 
 	ast_mutex_unlock(&release_lock);
+	chan_list_unref(p, "Release ast_channel reference");
 	return 0;
 }
 
@@ -7494,20 +7500,25 @@
 	int bridging;
 
 	ch1 = get_chan_by_ast(c0);
+	if (!ch1) {
+		return -1;
+	}
 	ch2 = get_chan_by_ast(c1);
+	if (!ch2) {
+		chan_list_unref(ch1, "Failed to find ch2");
+		return -1;
+	}
 
 	carr[0] = c0;
 	carr[1] = c1;
 
-	if (!(ch1 && ch2)) {
-		return -1;
-	}
-
 	misdn_cfg_get(ch1->bc->port, MISDN_CFG_BRIDGING, &p1_b, sizeof(p1_b));
 	misdn_cfg_get(ch2->bc->port, MISDN_CFG_BRIDGING, &p2_b, sizeof(p2_b));
 
 	if (! p1_b || ! p2_b) {
 		ast_log(LOG_NOTICE, "Falling back to Asterisk bridging\n");
+		chan_list_unref(ch1, "Bridge fallback ch1");
+		chan_list_unref(ch2, "Bridge fallback ch2");
 		return AST_BRIDGE_FAILED;
 	}
 
@@ -7581,6 +7592,8 @@
 
 	misdn_lib_split_bridge(ch1->bc, ch2->bc);
 
+	chan_list_unref(ch1, "Bridge complete ch1");
+	chan_list_unref(ch2, "Bridge complete ch2");
 	return AST_BRIDGE_COMPLETE;
 }
 
@@ -7663,12 +7676,58 @@
 	return 0;
 }
 
-
-static struct chan_list *init_chan_list(int orig)
+/*!
+ * \internal
+ * \brief Destroy the chan_list object.
+ *
+ * \param obj chan_list object to destroy.
+ *
+ * \return Nothing
+ */
+static void chan_list_destructor(void *obj)
+{
+	struct chan_list *ch = obj;
+
+#if defined(AST_MISDN_ENHANCEMENTS)
+	if (ch->peer) {
+		ao2_ref(ch->peer, -1);
+		ch->peer = NULL;
+	}
+#endif /* AST_MISDN_ENHANCEMENTS */
+
+	if (ch->dsp) {
+		ast_dsp_free(ch->dsp);
+		ch->dsp = NULL;
+	}
+
+	/* releasing jitterbuffer */
+	if (ch->jb) {
+		misdn_jb_destroy(ch->jb);
+		ch->jb = NULL;
+	}
+
+	if (ch->overlap_dial) {
+		if (ch->overlap_dial_task != -1) {
+			misdn_tasks_remove(ch->overlap_dial_task);
+			ch->overlap_dial_task = -1;
+		}
+		ast_mutex_destroy(&ch->overlap_tv_lock);
+	}
+
+	if (-1 < ch->pipe[0]) {
+		close(ch->pipe[0]);
+	}
+	if (-1 < ch->pipe[1]) {
+		close(ch->pipe[1]);
+	}
+}
+
+/*! Returns a reference to the new chan_list. */
+static struct chan_list *chan_list_init(int orig)
 {
 	struct chan_list *cl;
 
-	cl = ast_calloc(1, sizeof(*cl));
+	cl = ao2_alloc(sizeof(*cl), chan_list_destructor);
 	if (!cl) {
 		chan_misdn_log(-1, 0, "misdn_request: malloc failed!");
 		return NULL;
@@ -7682,6 +7741,8 @@
 #if defined(AST_MISDN_ENHANCEMENTS)
 	cl->record_id = -1;
 #endif	/* defined(AST_MISDN_ENHANCEMENTS) */
+	cl->pipe[0] = -1;
+	cl->pipe[1] = -1;
 
 	return cl;
 }
@@ -7905,7 +7966,7 @@
 	}
 
 	/* create ast_channel and link all the objects together */
-	cl = init_chan_list(ORG_AST);
+	cl = chan_list_init(ORG_AST);
 	if (!cl) {
 		ast_log(LOG_ERROR, "Could not create call record for Dial(%s)\n", dial_str);
 		return NULL;
@@ -7914,11 +7975,10 @@
 
 	ast = misdn_new(cl, AST_STATE_RESERVED, args.ext, NULL, format, requestor ? requestor->linkedid : NULL, port, channel);
 	if (!ast) {
-		ast_free(cl);
+		chan_list_unref(cl, "Failed to create a new channel");
 		ast_log(LOG_ERROR, "Could not create Asterisk channel for Dial(%s)\n", dial_str);
 		return NULL;
 	}
-	cl->ast = ast;
 
 #if defined(AST_MISDN_ENHANCEMENTS)
 	cl->record_id = record_id;
@@ -7933,13 +7993,14 @@
 	/* important */
 	cl->need_hangup = 0;
 
+	chan_list_unref(cl, "Successful misdn_request()");
 	return ast;
 }
 
 
 static int misdn_send_text(struct ast_channel *chan, const char *text)
 {
-	struct chan_list *tmp = chan->tech_pvt;
+	struct chan_list *tmp = MISDN_ASTERISK_TECH_PVT(chan);
 
 	if (tmp && tmp->bc) {
 		ast_copy_string(tmp->bc->display, text, sizeof(tmp->bc->display));
@@ -8050,7 +8111,10 @@
 		tmp->writeformat = format;
 		tmp->rawwriteformat = format;
 
-		tmp->tech_pvt = chlist;
+		/* Link the channel and private together */
+		chan_list_ref(chlist, "Give a reference to ast_channel");
+		MISDN_ASTERISK_TECH_PVT(tmp) = chlist;
+		chlist->ast = tmp;
 
 		misdn_cfg_get(0, MISDN_GEN_BRIDGING, &bridging, sizeof(bridging));
 		tmp->tech = bridging ? &misdn_tech : &misdn_tech_wo_bridge;
@@ -8087,6 +8151,7 @@
 	return tmp;
 }
 
+/*! Returns a reference to the found chan_list. */
 static struct chan_list *find_chan_by_bc(struct misdn_bchannel *bc)
 {
 	struct chan_list *help;
@@ -8094,6 +8159,7 @@
 	ast_mutex_lock(&cl_te_lock);
 	for (help = cl_te; help; help = help->next) {
 		if (help->bc == bc) {
+			chan_list_ref(help, "Found chan_list by bc");
 			ast_mutex_unlock(&cl_te_lock);
 			return help;
 		}
@@ -8109,24 +8175,7 @@
 	return NULL;
 }
 
-static struct chan_list *find_chan_by_pid(int pid)
-{
-	struct chan_list *help;
-
-	ast_mutex_lock(&cl_te_lock);
-	for (help = cl_te; help; help = help->next) {
-		if (help->bc && (help->bc->pid == pid)) {
-			ast_mutex_unlock(&cl_te_lock);
-			return help;
-		}
-	}
-	ast_mutex_unlock(&cl_te_lock);
-
-	chan_misdn_log(6, 0, "$$$ find_chan_by_pid: No channel found for pid:%d\n", pid);
-
-	return NULL;
-}
-
+/*! Returns a reference to the found chan_list. */
 static struct chan_list *find_hold_call(struct misdn_bchannel *bc)
 {
 	struct chan_list *help;
@@ -8144,6 +8193,7 @@
 	for (help = cl_te; help; help = help->next) {
 		chan_misdn_log(4, bc->port, "$$$ find_hold_call: --> hold:%d channel:%d\n", help->hold.state, help->hold.channel);
 		if (help->hold.state == MISDN_HOLD_ACTIVE && help->hold.port == bc->port) {
+			chan_list_ref(help, "Found chan_list hold call");
 			ast_mutex_unlock(&cl_te_lock);
 			return help;
 		}
@@ -8159,6 +8209,7 @@
 }
 
 
+/*! Returns a reference to the found chan_list. */
 static struct chan_list *find_hold_call_l3(unsigned long l3_id)
 {
 	struct chan_list *help;
@@ -8166,6 +8217,7 @@
 	ast_mutex_lock(&cl_te_lock);
 	for (help = cl_te; help; help = help->next) {
 		if (help->hold.state != MISDN_HOLD_IDLE && help->l3id == l3_id) {
+			chan_list_ref(help, "Found chan_list hold call l3");
 			ast_mutex_unlock(&cl_te_lock);
 			return help;
 		}
@@ -8184,6 +8236,8 @@
  * \param bc B channel record.
  *
  * \return Found call record or NULL.
+ *
+ * \note Returns a reference to the found chan_list.
  *
  * \note There could be a possibility where we find the wrong active call to transfer.
  * This concern is mitigated by the fact that there could be at most one other call
@@ -8203,6 +8257,7 @@
 			case MISDN_PROGRESS:
 			case MISDN_ALERTING:
 			case MISDN_CONNECTED:
+				chan_list_ref(list, "Found chan_list hold active call");
 				ast_mutex_unlock(&cl_te_lock);
 				return list;
 			default:
@@ -8219,6 +8274,7 @@
 {
 	chan_misdn_log(4, chan->bc ? chan->bc->port : 0, "* Queuing chan %p\n", chan);
 
+	chan_list_ref(chan, "Adding chan_list to list");
 	ast_mutex_lock(&cl_te_lock);
 	chan->next = NULL;
 	if (!cl_te) {
@@ -8251,6 +8307,7 @@
 		/* What we want is the head of the list. */
 		cl_te = cl_te->next;
 		ast_mutex_unlock(&cl_te_lock);
+		chan_list_unref(chan, "Removed chan_list from list head");
 		return 1;
 	}
 
@@ -8265,6 +8322,9 @@
 	}
 
 	ast_mutex_unlock(&cl_te_lock);
+	if (found_it) {
+		chan_list_unref(chan, "Removed chan_list from list");
+	}
 	return found_it;
 }
 
@@ -8326,7 +8386,7 @@
  *
  * \return Nothing
  *
- * \note ch must not be referenced after calling.
+ * \note The only valid thing to do with ch after calling is to chan_list_unref(ch, "").
  */
 static void release_chan(struct chan_list *ch, struct misdn_bchannel *bc)
 {
@@ -8353,6 +8413,9 @@
 	ch->state = MISDN_CLEANING;
 	ch->ast = NULL;
 	if (ast) {
+		struct chan_list *ast_ch;
+
+		ast_ch = MISDN_ASTERISK_TECH_PVT(ast);
 		MISDN_ASTERISK_TECH_PVT(ast) = NULL;
 		chan_misdn_log(1, bc->port,
 			"* RELEASING CHANNEL pid:%d context:%s dialed:%s caller:\"%s\" <%s>\n",
@@ -8367,36 +8430,9 @@
 			ast_setstate(ast, AST_STATE_DOWN);
 		}
 		ast_channel_unlock(ast);
-	}
-
-#if defined(AST_MISDN_ENHANCEMENTS)
-	if (ch->peer) {
-		ao2_ref(ch->peer, -1);
-		ch->peer = NULL;
-	}
-#endif /* AST_MISDN_ENHANCEMENTS */
-
-	if (ch->dsp) {
-		ast_dsp_free(ch->dsp);
-		ch->dsp = NULL;
-	}
-
-	/* releasing jitterbuffer */
-	if (ch->jb) {
-		misdn_jb_destroy(ch->jb);
-		ch->jb = NULL;
-	} else {
-		if (!bc->nojitter) {
-			chan_misdn_log(5, bc->port, "Jitterbuffer already destroyed.\n");
-		}
-	}
-
-	if (ch->overlap_dial) {
-		if (ch->overlap_dial_task != -1) {
-			misdn_tasks_remove(ch->overlap_dial_task);
-			ch->overlap_dial_task = -1;
-		}
-		ast_mutex_destroy(&ch->overlap_tv_lock);
+		if (ast_ch) {
+			chan_list_unref(ast_ch, "Release ast_channel reference.");
+		}
 	}
 
 	if (ch->originator == ORG_AST) {
@@ -8405,11 +8441,6 @@
 		--misdn_in_calls[bc->port];
 	}
 
-	close(ch->pipe[0]);
-	close(ch->pipe[1]);
-
-	ast_free(ch);
-
 	ast_mutex_unlock(&release_lock);
 }
 
@@ -8421,7 +8452,7 @@
  *
  * \return Nothing
  *
- * \note ch must not be referenced after calling.
+ * \note The only valid thing to do with ch after calling is to chan_list_unref(ch, "").
  */
 static void release_chan_early(struct chan_list *ch)
 {
@@ -8446,37 +8477,18 @@
 	ch->state = MISDN_CLEANING;
 	ch->ast = NULL;
 	if (ast) {
+		struct chan_list *ast_ch;
+
+		ast_ch = MISDN_ASTERISK_TECH_PVT(ast);
 		MISDN_ASTERISK_TECH_PVT(ast) = NULL;
+
 		if (ast->_state != AST_STATE_RESERVED) {
 			ast_setstate(ast, AST_STATE_DOWN);
 		}
 		ast_channel_unlock(ast);
-	}
-
-#if defined(AST_MISDN_ENHANCEMENTS)
-	if (ch->peer) {
-		ao2_ref(ch->peer, -1);
-		ch->peer = NULL;
-	}
-#endif /* AST_MISDN_ENHANCEMENTS */
-
-	if (ch->dsp) {
-		ast_dsp_free(ch->dsp);
-		ch->dsp = NULL;
-	}
-
-	/* releasing jitterbuffer */
-	if (ch->jb) {
-		misdn_jb_destroy(ch->jb);
-		ch->jb = NULL;
-	}
-
-	if (ch->overlap_dial) {
-		if (ch->overlap_dial_task != -1) {
-			misdn_tasks_remove(ch->overlap_dial_task);
-			ch->overlap_dial_task = -1;
-		}
-		ast_mutex_destroy(&ch->overlap_tv_lock);
+		if (ast_ch) {
+			chan_list_unref(ast_ch, "Release ast_channel reference.");
+		}
 	}
 
 	if (ch->hold.state != MISDN_HOLD_IDLE) {
@@ -8486,11 +8498,6 @@
 			--misdn_in_calls[ch->hold.port];
 		}
 	}
-
-	close(ch->pipe[0]);
-	close(ch->pipe[1]);
-
-	ast_free(ch);
 
 	ast_mutex_unlock(&release_lock);
 }
@@ -8796,18 +8803,6 @@
 	const char *tmp;
 
 	ast_channel_lock(chan);
-	tmp = pbx_builtin_getvar_helper(chan, "MISDN_PID");
-	if (tmp) {
-		ch->other_pid = atoi(tmp);
-		chan_misdn_log(3, bc->port, " --> IMPORT_PID: importing pid:%s\n", tmp);
-		if (ch->other_pid > 0) {
-			ch->other_ch = find_chan_by_pid(ch->other_pid);
-			if (ch->other_ch) {
-				ch->other_ch->other_ch = ch;
-			}
-		}
-	}
-
 	tmp = pbx_builtin_getvar_helper(chan, "MISDN_ADDRESS_COMPLETE");
 	if (tmp && (atoi(tmp) == 1)) {
 		bc->sending_complete = 1;
@@ -8832,6 +8827,12 @@
 {
 	char tmp[32];
 
+	/*
+	 * The only use for MISDN_PID is if there is a problem and you
+	 * have to use the "misdn restart pid" CLI command.  Otherwise,
+	 * the pid is not used by anyone.  The internal use of MISDN_PID
+	 * has been deleted.
+	 */
 	chan_misdn_log(3, bc->port, " --> EXPORT_PID: pid:%d\n", bc->pid);
 	snprintf(tmp, sizeof(tmp), "%d", bc->pid);
 	pbx_builtin_setvar_helper(chan, "_MISDN_PID", tmp);
@@ -9962,9 +9963,7 @@
 			chan_misdn_log(1, bc->port, "Chan not existing at the moment bc->l3id:%x bc:%p event:%s port:%d channel:%d\n", bc->l3_id, bc, manager_isdn_get_info(event), bc->port, bc->channel);
 			return -1;
 		}
-	}
-
-	if (ch) {
+	} else {
 		switch (event) {
 		case EVENT_TONE_GENERATE:
 			break;
@@ -9982,6 +9981,7 @@
 				if (event != EVENT_BCHAN_DATA) {
 					ast_log(LOG_NOTICE, "No Ast or No private Pointer in Event (%d:%s)\n", event, manager_isdn_get_info(event));
 				}
+				chan_list_unref(ch, "No Ast or Ast private pointer");
 				return -1;
 			}
 			break;
@@ -10157,7 +10157,6 @@
 		break;
 	case EVENT_SETUP:
 	{
-		struct chan_list *ch = find_chan_by_bc(bc);
 		struct ast_channel *chan;
 		int exceed;
 		int ai;
@@ -10167,9 +10166,11 @@
 		if (ch) {
 			switch (ch->state) {
 			case MISDN_NOTHING:
+				chan_list_unref(ch, "Ignore found ch.  Is it for an outgoing call?");
 				ch = NULL;
 				break;
 			default:
+				chan_list_unref(ch, "Already have a call.");
 				chan_misdn_log(1, bc->port, " --> Ignoring Call we have already one\n");
 				return RESPONSE_IGNORE_SETUP_WITHOUT_CLOSE; /*  Ignore MSNs which are not in our List */
 			}
@@ -10190,7 +10191,7 @@
 
 		print_bearer(bc);
 
-		ch = init_chan_list(ORG_MISDN);
+		ch = chan_list_init(ORG_MISDN);
 		if (!ch) {
 			chan_misdn_log(-1, bc->port, "cb_events: malloc for chan_list failed!\n");
 			return 0;
@@ -10202,13 +10203,11 @@
 
 		chan = misdn_new(ch, AST_STATE_RESERVED, bc->dialed.number, bc->caller.number, AST_FORMAT_ALAW, NULL, bc->port, bc->channel);
 		if (!chan) {
-			ast_free(ch);
+			chan_list_unref(ch, "Failed to create a new channel");
 			misdn_lib_send_event(bc,EVENT_RELEASE_COMPLETE);
 			ast_log(LOG_ERROR, "cb_events: misdn_new failed !\n");
 			return 0;
 		}
-
-		ch->ast = chan;
 
 		if ((exceed = add_in_calls(bc->port))) {
 			char tmp[16];
@@ -10292,6 +10291,7 @@
 
 				ch->state = MISDN_EXTCANTMATCH;
 				misdn_lib_send_event(bc, EVENT_RELEASE_COMPLETE);
+				chan_list_unref(ch, "BC not allowed, releasing call");
 				return RESPONSE_OK;
 			}
 		}
@@ -10679,6 +10679,9 @@
 				}
 			}
 		}
+		if (held_ch) {
+			chan_list_unref(held_ch, "Done with held call");
+		}
 		bc->out_cause = -1;
 		if (bc->need_release) {
 			misdn_lib_send_event(bc, EVENT_RELEASE);
@@ -11045,6 +11048,9 @@
 		break;
 	}
 
+	if (ch) {
+		chan_list_unref(ch, "cb_event complete OK");
+	}
 	return RESPONSE_OK;
 }
 
@@ -12483,13 +12489,22 @@
 
 int chan_misdn_jb_empty(struct misdn_bchannel *bc, char *buf, int len)
 {
-	struct chan_list *ch = find_chan_by_bc(bc);
-
-	if (ch && ch->jb) {
-		return misdn_jb_empty(ch->jb, buf, len);
-	}
-
-	return -1;
+	struct chan_list *ch;
+	int res;
+
+	ch = find_chan_by_bc(bc);
+	if (!ch) {
+		return 0;
+	}
+
+	if (ch->jb) {
+		res = misdn_jb_empty(ch->jb, buf, len);
+	} else {
+		res = 0;
+	}
+	chan_list_unref(ch, "Done emptying jb");
+
+	return res;
 }
 
 




More information about the svn-commits mailing list