[Serdev] [Tracker] Updated: (SER-60) nathelper support for t38 (SDP w/image media type)

[Jan Janak (JIRA)]

     [ http://tracker.iptel.org/browse/SER-60?page=all ]

Jan Janak updated SER-60:
-------------------------

    Fix Version/s: Ipteldorf
                       (was: Wishlist)

> nathelper support for t38 (SDP w/image media type)
> --------------------------------------------------
>
>                 Key: SER-60
>                 URL: http://tracker.iptel.org/browse/SER-60
>             Project: SER
>          Issue Type: Improvement
>          Components: NAT Traversal
>    Affects Versions: 0.9.3, 0.9.4, 0.9.5, 0.9.6, Wishlist
>         Environment: The patch is usefull for the new stable ser 0.9.3 tested on netbsd/i386 with GrandStream HT286 firmware 1.0.6.7 and Cisco AS5350 IOS 12.3(10b).
>            Reporter: Jan Janak
>         Assigned To: Jan Janak
>             Fix For: Ipteldorf
>
>
> Grandstream with firmware release 1.0.6.7 support T38 FAX relay, when FAX tone is detected it trigger a re-invite to change the media stream to T38. I see that the new SDP body use m= line with 'image' as the media type and this cause nathelper/rtpproxy to fail due to the lack of media port parsing. I make this little patch to support this with nathelper/rtpproxy and like to share with us to be tested and included in CVS.
>  == Initial error detected in nathelper ==
> Jul  1 11:26:38 HPBSD01 /sbin/ser[15641]: ERROR: extract_mediaport: can't parse `m=' in SDP 
> Jul  1 11:26:38 HPBSD01 /sbin/ser[15641]: ERROR: force_rtp_proxy2: can't extract media port from the message 
>  == SDP content with image sent by HT286 ==
>  v=0
>  o=telarcenter 8000 8001 IN IP4 192.168.1.224
>  s=SIP Call
>  c=IN IP4 192.168.1.224
>  t=0 0
>  m=image 44914 udptl t38
>  a=T38FaxVersion:0
>  a=T38MaxBitRate:9600
>  a=T38FaxFillBitRemoval:0
>  a=T38FaxTranscodingMMR:0
>  a=T38FaxTranscodingJBIG:0
>  a=T38FaxRateManagement:transferredTCF
>  a=T38FaxMaxBuffer:400
>  a=T38FaxMaxDatagram:280
>  a=T38FaxUdpEC:t38UDPRedundancy
>    ======   The path   ======
> cat > nathelper.t38.patch << EOF
> 929a930,938
> >       /*
> >        * ecolombo: support added for 'image' SDP content
> >        * Grandstream generated re-invites use some like
> >        * 'm=image 44914 udptl t38' in SDP
> >        */
> >       } else if (mediaport->len > 6 && memcmp(mediaport-s, "image", 5) == 0 &&
> >           isspace((int)mediaport->s[5])) {
> >               mediaport->s += 5;
> >               mediaport->len -= 5;
> EOF
> Regards
> Ezequiel Colombo

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://tracker.iptel.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira