[asterisk-bugs] [Asterisk 0018521]: Memory leaking in calendars

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18521 
====================================================================== 
Reported By:                pitel
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18521
Category:                   Resources/res_calendar
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.8.1.1 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-12-23 09:15 CST
Last Modified:              2010-12-31 04:49 CST
====================================================================== 
Summary:                    Memory leaking in calendars
Description: 
Memory is leaking in calendaring module.

Noticed after backporting calendaring to Asterisk 1.6.1.20, but it also
appear in 1.8.1.1!
====================================================================== 

---------------------------------------------------------------------- 
 (0130075) cervajs (reporter) - 2010-12-31 04:49
 https://issues.asterisk.org/view.php?id=18521#c130075 
---------------------------------------------------------------------- 
i found this commit in libical

------------------------------------------------------------------------
r1037 | awinterz | 2010-10-07 00:33:54 +0200 (Thu, 07 Oct 2010) | 18
lines

fix some memory leaks in libical
patch from qkmiao -- Thanks!
ID: 3046011

Details:
 1. In icalparameter_set_* series function, param->string is overridden
with new string. Then the old string gets lost. icalparameter_set_*
series
functions are in icalderivedparameter.c. This c file is generated by
libical/scripts/mkderivedparameters.pl. Just add code to free non-empty
param->string before it is overridden. 

 2. Function "icalcomponent_remove_property" will remove property
instance
from properties list but don't free the space it holds. So, when call
this
function, the property instance remove from properties list should be
free
explicitly. Similarly for "pvl_remove (prop->parameters, p);" it also
should
be explicitly free memory after remove parameter instance form parameter
list.
---end---

i will try latest libical 0.46 with pitel (we are using 0.43)

happy new year! 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-12-31 04:49 cervajs        Note Added: 0130075                          
======================================================================