FreeSWITCH

Freeswitch is removing last digit of RTP port when handling incoming SIP REFER

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: mod_sofia
  • Labels:
    None
  • Environment:
    Ubuntu 12.04 LTS updated
    compiled from latest GIT version
  • Platform:
    Linux x86_64/gcc
  • CPU Info:
    Hide
    Not available...
    Amazon EC2 server
    Show
    Not available... Amazon EC2 server
  • FreeSWITCH GIT Revision:
    c92a55d3f1b2948a90f39afd312430bf918d513c
  • Reproduced with GIT HEAD?:
    yes

Description

Please, look at the first INVITE, the RTP port is: 61712
The scenario is: A -> FS1 -> FS2
When A press some digit, FS2 send SIP REFER to FS1, and FS1 calls again to FS2 (The idea is to call any server of the cluster, this is just for debugging)
When testing everyone on same LAN, everything works ok
On this scenario, both FS are behind NAT and have a public external IP Address that points to the internal one... I don't know if this is the problem...
Tried "make current" but same problem happens
And the problem, you can see when FS1 send a RE-INVITE to A and FS2, to set the correct path of the media. Both INVITEs have one missing digit and obviously, no audio to both sides.

** Using media_bypass

So the stripped port is: 6171

http://pastebin.freeswitch.org/19382
  1. File
    refer_problem_2.pcap
    27/Jun/12 1:39 PM
    32 kB
    Daniel Zanutti Gomes
  2. File
    refer_problem.pcap
    26/Jun/12 8:05 PM
    36 kB
    Daniel Zanutti Gomes

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Peter Olsson added a comment -
Please give your git revision hash. FS-4349 is not a git revision.

Also attach full debug logs and FS sofia logging as well.

/Peter
Show
Peter Olsson added a comment - Please give your git revision hash. FS-4349 is not a git revision. Also attach full debug logs and FS sofia logging as well. /Peter
Hide
Permalink
Travis Cross added a comment -
His git revision is visible in the signaling he posted; he's running a recent build. I updated the bug with his git revision.
Show
Travis Cross added a comment - His git revision is visible in the signaling he posted; he's running a recent build. I updated the bug with his git revision.
Hide
Permalink
Travis Cross added a comment -
I've reviewed his pcap trace and I concur that our current behavior here is a bit inexplicable.
Show
Travis Cross added a comment - I've reviewed his pcap trace and I concur that our current behavior here is a bit inexplicable.
Hide
Permalink
Travis Cross added a comment -
Hi Daniel,

I've prepared a debug commit that'll help us catch what is happening if we're overflowing a buffer in mod_sofia. Please test as follows.

Pull this repository:

  https://github.com/traviscross/freeswitch.git

Rebuild, and retest with:

  /log debug
  console loglevel debug
  sofia loglevel all 9
  sofia profile <profile> siptrace on

...along with a corresponding tcpdump capture.

Attach the results to this bug.

Thanks for your report.
Show
Travis Cross added a comment - Hi Daniel, I've prepared a debug commit that'll help us catch what is happening if we're overflowing a buffer in mod_sofia. Please test as follows. Pull this repository:    https://github.com/traviscross/freeswitch.git Rebuild, and retest with:   /log debug   console loglevel debug   sofia loglevel all 9   sofia profile <profile> siptrace on ...along with a corresponding tcpdump capture. Attach the results to this bug. Thanks for your report.
Hide
Permalink
Daniel Zanutti Gomes added a comment -
Hi Travis

Thanks for your effort.

Here info you requested:
http://pastebin.freeswitch.org/19385

tcpdump is attached as: refer_problem_2.pcap

PS: Took 5 hours to compile on Amazon micro machine. :(
Show
Daniel Zanutti Gomes added a comment - Hi Travis Thanks for your effort. Here info you requested: http://pastebin.freeswitch.org/19385 tcpdump is attached as: refer_problem_2.pcap PS: Took 5 hours to compile on Amazon micro machine. :(
Hide
Permalink
Daniel Zanutti Gomes added a comment -
PCAP dump of Travis build.
Show
Daniel Zanutti Gomes added a comment - PCAP dump of Travis build.
Hide
Permalink
Travis Cross added a comment -
Thanks Daniel. That output was very helpful. Please test at or after this commit from:

  git://git.freeswitch.org/freeswitch

commit 7978bd892111e4235f93211919bd713c78d87c80
Author: Travis Cross
Date: Wed Jun 27 20:16:46 2012 +0000

    Don't truncate 5-digit port numbers
    
    The adj_port buffer was only 5 bytes wide, so we were truncating any
    port numbers >= 10000 that were written to this variable as snprintf
    ensures the last byte is null.
    
    FS-4354 --resolve
Show
Travis Cross added a comment - Thanks Daniel. That output was very helpful. Please test at or after this commit from:    git://git.freeswitch.org/freeswitch commit 7978bd892111e4235f93211919bd713c78d87c80 Author: Travis Cross Date: Wed Jun 27 20:16:46 2012 +0000     Don't truncate 5-digit port numbers          The adj_port buffer was only 5 bytes wide, so we were truncating any     port numbers >= 10000 that were written to this variable as snprintf     ensures the last byte is null.           FS-4354 --resolve
Hide
Permalink
Travis Cross added a comment -
You don't need to do a clean build, by the way. Doing something like this from your last build will speed things up for you:

  git remote add fsorg git://git.freeswitch.org/freeswitch
  git fetch fsorg && git reset --hard fsorg/master
  make && make install

Show
Travis Cross added a comment - You don't need to do a clean build, by the way. Doing something like this from your last build will speed things up for you:   git remote add fsorg git://git.freeswitch.org/freeswitch   git fetch fsorg && git reset --hard fsorg/master   make && make install
Hide
Permalink
Daniel Zanutti Gomes added a comment -
Hi travis

Your solution is working very fine! The RTP ports are OK now...

Thank you very much.
Daniel
Show
Daniel Zanutti Gomes added a comment - Hi travis Your solution is working very fine! The RTP ports are OK now... Thank you very much. Daniel
Hide
Permalink
Daniel Zanutti Gomes added a comment -
Solved with last commit
Show
Daniel Zanutti Gomes added a comment - Solved with last commit

People

  • Assignee:
    Travis Cross
    Reporter:
    Daniel Zanutti Gomes
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: