[asterisk-bugs] [Asterisk 0013160]: Memory leak in iax2-parser.c

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13160 
====================================================================== 
Reported By:                tavius
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13160
Category:                   Channels/chan_iax2
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.4.21.2 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-07-25 10:10 CDT
Last Modified:              2008-07-31 11:37 CDT
====================================================================== 
Summary:                    Memory leak in iax2-parser.c
Description: 
We have found compiling asterisk without LOW_MEMORY has a memory leak in
iax_frame_new. It seems that the AST_LIST_TRAVERSE_SAFE_BEGIN macro is not
traversing the linked list in good way.
We have traced adding some ast_log in the code that iax_frame_free add
buffers in the frame cache linked list and after that iax_frame_new does
not find this free buffer in the list and allocates a new buffer with
ast_calloc_cache.
With the time and with our system, that receives too many calls, asterisk
gets all the available memory in the system and produces an error.
If we compile with LOW_MEMORY, then this piece of code is not compiled and
asterisk does not have memory leaks.
====================================================================== 

---------------------------------------------------------------------- 
 (0090915) svnbot (reporter) - 2008-07-31 11:37
 http://bugs.digium.com/view.php?id=13160#c90915 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 134814

U   branches/1.4/channels/iax2-parser.c

------------------------------------------------------------------------
r134814 | russell | 2008-07-31 11:37:22 -0500 (Thu, 31 Jul 2008) | 7 lines

In case we have some processing threads that free more frames than they
allocate,
do not let the frame cache grow forever.

(closes issue http://bugs.digium.com/view.php?id=13160)
Reported by: tavius
Tested by: tavius, russell

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-07-31 11:37 svnbot         Checkin                                      
2008-07-31 11:37 svnbot         Note Added: 0090915                          
======================================================================