[Home]

Summary:ASTERISK-12586: [patch] Memory leak in chan_gtalk while trying to unload the module.
Reporter:Eliel Sardanons (eliel)Labels:
Date Opened:2008-08-15 10:32:40Date Closed:2009-09-15 16:54:43
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_gtalk
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) chan_gtalk.c.patch
Description:While trying to unload the chan_gtalk module, and we call ASTOBJ_CONTAINER_DESTROYALL() the member_detroy function is never called cause every member in gtalk_list has refcount > 1.
Also when calling find_gtalk the refcount of the found member is incremented and never decremented when not used anymore.
Comments:By: Digium Subversion (svnbot) 2008-08-21 04:46:40

Repository: asterisk
Revision: 139281

U   trunk/channels/chan_gtalk.c

------------------------------------------------------------------------
r139281 | phsultan | 2008-08-21 04:46:38 -0500 (Thu, 21 Aug 2008) | 5 lines

Fix two memory leaks in chan_gtalk, thanks Eliel!
(closes issue ASTERISK-12586)
Reported by: eliel
Patches:
     chan_gtalk.c.patch uploaded by eliel (license 64)
------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=139281
By: Digium Subversion (svnbot) 2008-08-21 05:02:12

Repository: asterisk
Revision: 139283

U   branches/1.4/channels/chan_gtalk.c

------------------------------------------------------------------------
r139283 | phsultan | 2008-08-21 05:02:11 -0500 (Thu, 21 Aug 2008) | 1 line

Apply fix for issue ASTERISK-12586 to branch 1.4, too.
------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=139283
By: Digium Subversion (svnbot) 2009-09-15 16:54:42

Repository: asterisk
Revision: 218727

_U  branches/1.6.1/
U   branches/1.6.1/channels/chan_gtalk.c

------------------------------------------------------------------------
r218727 | tilghman | 2009-09-15 16:54:41 -0500 (Tue, 15 Sep 2009) | 44 lines

Merged revisions 139281,175058,175089 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
(closes issue ASTERISK-13134)

................
 r139281 | phsultan | 2008-08-21 04:55:31 -0500 (Thu, 21 Aug 2008) | 5 lines
 
 Fix two memory leaks in chan_gtalk, thanks Eliel!
 (closes issue ASTERISK-12586)
 Reported by: eliel
 Patches:
       chan_gtalk.c.patch uploaded by eliel (license 64)
................
 r175058 | phsultan | 2009-02-12 04:31:36 -0600 (Thu, 12 Feb 2009) | 20 lines
 
 Merged revisions 175029 via svnmerge from
 https://origsvn.digium.com/svn/asterisk/branches/1.4
 
 ........
 r175029 | phsultan | 2009-02-12 11:16:21 +0100 (Thu, 12 Feb 2009) | 12 lines
 
 Set the initiator attribute to lowercase in our replies when receiving calls.
 
 This attribute contains a JID that identifies the initiator of the GoogleTalk
 voice session. The GoogleTalk client discards Asterisk's replies if the
 initiator attribute contains uppercase characters.
 
 (closes issue ASTERISK-13133)
 Reported by: jcovert
 Patches:
       chan_gtalk.2.patch uploaded by jcovert (license 551)
 Tested by: jcovert
 
 ........
................
 r175089 | phsultan | 2009-02-12 08:25:03 -0600 (Thu, 12 Feb 2009) | 6 lines
 
 Issue a warning message if our candidate's IP is the loopback address.
 
 (closes issue ASTERISK-13134)
 Reported by: jcovert
 Tested by: phsultan
................

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

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