Index: channels/chan_skinny.c =================================================================== --- channels/chan_skinny.c (revision 66882) +++ channels/chan_skinny.c (working copy) @@ -783,6 +783,7 @@ static char mailbox[AST_MAX_EXTENSION]; static int amaflags = 0; static int callnums = 1; +static int canreinvite = 0; #define SKINNY_DEVICE_UNKNOWN -1 #define SKINNY_DEVICE_NONE 0 @@ -919,6 +920,7 @@ /* int lastout; */ /* Unused */ int cxmode; int nat; + int canreinvite; int outgoing; int alreadygone; @@ -967,6 +969,7 @@ int immediate; int hookstate; int nat; + int canreinvite; struct ast_codec_pref prefs; struct skinny_subchannel *sub; @@ -1064,7 +1067,7 @@ .fixup = skinny_fixup, .send_digit_begin = skinny_senddigit_begin, .send_digit_end = skinny_senddigit_end, -/* .bridge = ast_rtp_bridge, */ + .bridge = ast_rtp_bridge, }; static int skinny_extensionstate_cb(char *context, char* exten, int state, void *data); @@ -1836,24 +1839,98 @@ static enum ast_rtp_get_result skinny_get_rtp_peer(struct ast_channel *c, struct ast_rtp **rtp) { struct skinny_subchannel *sub = NULL; + struct skinny_line *l; + enum ast_rtp_get_result res = AST_RTP_TRY_NATIVE; - if (!(sub = c->tech_pvt) || !(sub->rtp)) + if (skinnydebug) + ast_verbose("skinny_get_rtp_peer() Channel = %s\n", c->name); + + + if (!(sub = c->tech_pvt)) return AST_RTP_GET_FAILED; + ast_mutex_lock(&sub->lock); + + if (!(sub->rtp)){ + ast_mutex_unlock(&sub->lock); + return AST_RTP_GET_FAILED; + } + *rtp = sub->rtp; - return AST_RTP_TRY_NATIVE; + l = sub->parent; + + if (!l->canreinvite || l->nat){ + res = AST_RTP_TRY_PARTIAL; + if (skinnydebug) + ast_verbose("skinny_get_rtp_peer() Using AST_RTP_TRY_PARTIAL = %s\n"); + } + + ast_mutex_unlock(&sub->lock); + + return res; + } static int skinny_set_rtp_peer(struct ast_channel *c, struct ast_rtp *rtp, struct ast_rtp *vrtp, struct ast_rtp *trtp, int codecs, int nat_active) { struct skinny_subchannel *sub; + struct skinny_line *l; + struct skinny_device *d; + struct skinnysession *s; + struct ast_format_list fmt; + struct sockaddr_in us; + struct sockaddr_in them; + struct skinny_req *req; + sub = c->tech_pvt; - if (sub) { - /* transmit_modify_with_sdp(sub, rtp); @@FIXME@@ if needed */ + + if (c->_state != AST_STATE_UP) return 0; + + if (!sub) { + return -1; } - return -1; + + l = sub->parent; + d = l->parent; + s = d->session; + + if (rtp){ + ast_rtp_get_peer(rtp, &them); + + if (skinnydebug) + ast_verbose("Peerip = %s:%d\n", ast_inet_ntoa(them.sin_addr), ntohs(them.sin_port)); + + if (!(req = req_alloc(sizeof(struct start_media_transmission_message), START_MEDIA_TRANSMISSION_MESSAGE))) + return -1; + + fmt = ast_codec_pref_getsize(&l->prefs, ast_best_codec(l->capability)); + + if (skinnydebug) + ast_verbose("Setting payloadType to '%d' (%d ms)\n", fmt.bits, fmt.cur_ms); + + req->data.startmedia.conferenceId = htolel(0); + req->data.startmedia.passThruPartyId = htolel(sub->callid); + if (!(l->canreinvite) || (l->nat)){ + req->data.startmedia.remoteIp = htolel(d->ourip.s_addr); + req->data.startmedia.remotePort = htolel(ntohs(us.sin_port)); + } else { + req->data.startmedia.remoteIp = htolel(them.sin_addr.s_addr); + req->data.startmedia.remotePort = htolel(ntohs(them.sin_port)); + } + req->data.startmedia.packetSize = htolel(fmt.cur_ms); + req->data.startmedia.payloadType = htolel(codec_ast2skinny(fmt.bits)); + req->data.startmedia.qualifier.precedence = htolel(127); + req->data.startmedia.qualifier.vad = htolel(0); + req->data.startmedia.qualifier.packets = htolel(0); + req->data.startmedia.qualifier.bitRate = htolel(0); + transmit_response(s, req); + + return 0; + } + /* Need a return here to break the bridge */ + return 0; } static struct ast_rtp_protocol skinny_rtp = { @@ -2154,6 +2231,8 @@ ast_parse_allow_disallow(&d->prefs, &d->capability, v->value, 0); } else if (!strcasecmp(v->name, "version")) { ast_copy_string(d->version_id, v->value, sizeof(d->version_id)); + } else if (!strcasecmp(v->name, "canreinvite")) { + canreinvite = ast_true(v->value); } else if (!strcasecmp(v->name, "nat")) { nat = ast_true(v->value); } else if (!strcasecmp(v->name, "callerid")) { @@ -2285,6 +2364,7 @@ /* ASSUME we're onhook at this point */ l->hookstate = SKINNY_ONHOOK; l->nat = nat; + l->canreinvite = canreinvite; l->next = d->lines; d->lines = l; @@ -2861,6 +2941,7 @@ d->lastcallreference = sub->callid; sub->cxmode = SKINNY_CX_INACTIVE; sub->nat = l->nat; + sub->canreinvite = l->canreinvite; sub->parent = l; sub->onhold = 0; @@ -3838,7 +3919,7 @@ struct skinny_device *d = s->device; struct skinny_line *l; struct skinny_subchannel *sub; - struct ast_format_list fmt; + struct ast_format_list fmt; struct sockaddr_in sin; struct sockaddr_in us; uint32_t addr; @@ -3874,11 +3955,12 @@ return 0; } - if (skinnydebug) { + if (skinnydebug) ast_verbose("ipaddr = %s:%d\n", ast_inet_ntoa(sin.sin_addr), ntohs(sin.sin_port)); - ast_verbose("ourip = %s:%d\n", ast_inet_ntoa(d->ourip), ntohs(us.sin_port)); - } + + + if (!(req = req_alloc(sizeof(struct start_media_transmission_message), START_MEDIA_TRANSMISSION_MESSAGE))) return -1; @@ -3898,7 +3980,7 @@ req->data.startmedia.qualifier.packets = htolel(0); req->data.startmedia.qualifier.bitRate = htolel(0); transmit_response(s, req); - + return 1; }