kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-12 15:45:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

I thought I noticed a memory leak earlier when I saw that the contents of this

Browse Source 
list were not destroyed when the module is unloaded.  However, after reading
the code related to the use of this list a lot today, I realized that it isn't
necessary.  So, I have added a comment to explain why it isn't necessary.


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@75806 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Russell Bryant
2007-07-18 22:52:54 +00:00
parent 9c99cba373
commit 5272289355
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
channels/chan_iax2.c
View File

@@ -646,6 +646,13 @@ struct chan_iax2_pvt {
int frames_received;
};
/*!
* \brief a list of frames that may need to be retransmitted
*
* \note The contents of this list do not need to be explicitly destroyed
* on module unload. This is because all active calls are destroyed, and
* all frames in this queue will get destroyed as a part of that process.
*/
static AST_LIST_HEAD_STATIC(frame_queue, iax_frame);
static AST_LIST_HEAD_STATIC(users, iax2_user);
@@ -10807,9 +10814,10 @@ static int __unload_module(void)
ast_netsock_release(netsock);
ast_netsock_release(outsock);
for (x=0;x<IAX_MAX_CALLS;x++)
for (x = 0; x < IAX_MAX_CALLS; x++) {
if (iaxs[x])
iax2_destroy(x);
}
ast_manager_unregister( "IAXpeers" );
ast_manager_unregister( "IAXnetstats" );
ast_unregister_application(papp);