[asterisk-dev] [asterisk-commits] dhubbard: branch 1.4 r84018 - in /branches/1.4: channels/ include/asterisk/ m...

Kevin P. Fleming kpfleming at digium.com
Thu Sep 27 18:27:03 CDT 2007


SVN commits to the Asterisk project wrote:

> @@ -1572,6 +1572,14 @@
>  		astman_send_error(s, m, buf);
>  		return 0;
>  	}
> +	if (chan->tech->get_base_channel) {
> +		base = chan->tech->get_base_channel(chan);
> +		if (base) {
> +			ast_mutex_unlock(&chan->lock);
> +			chan = base;
> +			ast_mutex_lock(&chan->lock);
> +		}
> +	}

I believe we will need an API change here; the ->get_base_channel()
callback will need to return the base channel already locked, otherwise
it could disappear after the pointer to it has been returned.

-- 
Kevin P. Fleming
Director of Software Technologies
Digium, Inc. - "The Genuine Asterisk Experience" (TM)



More information about the asterisk-dev mailing list