[Home]

Summary:ASTERISK-13407: [patch] tcptls.c doesn't set correct remote_address
Reporter:James Golovich (jamesgolovich)Labels:
Date Opened:2009-01-20 13:28:27.000-0600Date Closed:2009-01-21 04:29:08.000-0600
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) bug-14287.diff.txt
Description:With the recent changes to the tcptls.c code the local info was being stored in the tcptls_session struct instead of the remote info for client side work.  This fixes that so the real info instead of 0.0.0.0/0 is shown

This also makes sip_tcp_locate find the correct session for clients.

I was digging through this looking for what was leaking fd's since I was eventually running out of them since the client code was opening a new session every time because sip_tcp_locate was never finding the same session
Comments:By: Digium Subversion (svnbot) 2009-01-21 04:25:52.000-0600

Repository: asterisk
Revision: 169620

U   trunk/main/tcptls.c

------------------------------------------------------------------------
r169620 | russell | 2009-01-21 04:25:52 -0600 (Wed, 21 Jan 2009) | 10 lines

Fix a regression in TCP support.

This patch fixes a problem that caused chan_sip to think that every open TCP session
was to a remote address of 0.0.0.0:0.

(closes issue ASTERISK-13407)
Reported by: jamesgolovich
Patches:
     bug-14287.diff.txt uploaded by jamesgolovich (license 176)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=169620

By: Digium Subversion (svnbot) 2009-01-21 04:26:40.000-0600

Repository: asterisk
Revision: 169621

_U  branches/1.6.0/

------------------------------------------------------------------------
r169621 | russell | 2009-01-21 04:26:40 -0600 (Wed, 21 Jan 2009) | 17 lines

Blocked revisions 169620 via svnmerge

........
r169620 | russell | 2009-01-21 04:26:07 -0600 (Wed, 21 Jan 2009) | 10 lines

Fix a regression in TCP support.

This patch fixes a problem that caused chan_sip to think that every open TCP session
was to a remote address of 0.0.0.0:0.

(closes issue ASTERISK-13407)
Reported by: jamesgolovich
Patches:
     bug-14287.diff.txt uploaded by jamesgolovich (license 176)

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=169621

By: Digium Subversion (svnbot) 2009-01-21 04:29:07.000-0600

Repository: asterisk
Revision: 169622

_U  branches/1.6.1/
U   branches/1.6.1/main/tcptls.c

------------------------------------------------------------------------
r169622 | russell | 2009-01-21 04:29:07 -0600 (Wed, 21 Jan 2009) | 18 lines

Merged revisions 169620 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r169620 | russell | 2009-01-21 04:26:07 -0600 (Wed, 21 Jan 2009) | 10 lines

Fix a regression in TCP support.

This patch fixes a problem that caused chan_sip to think that every open TCP session
was to a remote address of 0.0.0.0:0.

(closes issue ASTERISK-13407)
Reported by: jamesgolovich
Patches:
     bug-14287.diff.txt uploaded by jamesgolovich (license 176)

........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=169622