[Home]

Summary:ASTERISK-14867: memory leak, tcptls_session never destroyed in chan_sip for client connections
Reporter:David Vossel (dvossel)Labels:
Date Opened:2009-09-22 17:52:10Date Closed:2009-09-24 15:44:55
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_sip/TCP-TLS
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:A sip_socket's tcptls_session is dereferenced properly, but due to an extra ref count given in tcptls.c's ast_tcptls_client_start() function, the tcptls_session is never destroyed.  This causes a memory leak for every tcp/tls client connection made.
Comments:By: Elazar Broad (ebroad) 2009-09-24 14:24:58

A patch for this issue can be found at https://reviewboard.asterisk.org/r/375/
By: Digium Subversion (svnbot) 2009-09-24 15:39:40

Repository: asterisk
Revision: 220365

U   trunk/main/tcptls.c

------------------------------------------------------------------------
r220365 | dvossel | 2009-09-24 15:39:39 -0500 (Thu, 24 Sep 2009) | 8 lines

fixes tcptls_session memory leak caused by ref count error

(closes issue ASTERISK-14867)
Reported by: dvossel

Review: https://reviewboard.asterisk.org/r/375/


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

http://svn.digium.com/view/asterisk?view=rev&revision=220365
By: Digium Subversion (svnbot) 2009-09-24 15:40:21

Repository: asterisk
Revision: 220369

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

------------------------------------------------------------------------
r220369 | dvossel | 2009-09-24 15:40:21 -0500 (Thu, 24 Sep 2009) | 14 lines

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

........
 r220365 | dvossel | 2009-09-24 15:37:20 -0500 (Thu, 24 Sep 2009) | 8 lines
 
 fixes tcptls_session memory leak caused by ref count error
 
 (closes issue ASTERISK-14867)
 Reported by: dvossel
 
 Review: https://reviewboard.asterisk.org/r/375/
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=220369
By: Digium Subversion (svnbot) 2009-09-24 15:40:51

Repository: asterisk
Revision: 220371

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

------------------------------------------------------------------------
r220371 | dvossel | 2009-09-24 15:40:50 -0500 (Thu, 24 Sep 2009) | 14 lines

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

........
 r220365 | dvossel | 2009-09-24 15:37:20 -0500 (Thu, 24 Sep 2009) | 8 lines
 
 fixes tcptls_session memory leak caused by ref count error
 
 (closes issue ASTERISK-14867)
 Reported by: dvossel
 
 Review: https://reviewboard.asterisk.org/r/375/
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=220371
By: Digium Subversion (svnbot) 2009-09-24 15:44:55

Repository: asterisk
Revision: 220372

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

------------------------------------------------------------------------
r220372 | dvossel | 2009-09-24 15:44:54 -0500 (Thu, 24 Sep 2009) | 14 lines

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

........
 r220365 | dvossel | 2009-09-24 15:37:20 -0500 (Thu, 24 Sep 2009) | 8 lines
 
 fixes tcptls_session memory leak caused by ref count error
 
 (closes issue ASTERISK-14867)
 Reported by: dvossel
 
 Review: https://reviewboard.asterisk.org/r/375/
........

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

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