After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 509536 - Leak in CalUtil.ICalToCalComponentArray
Leak in CalUtil.ICalToCalComponentArray
Status: RESOLVED FIXED
Product: evolution-sharp
Classification: Deprecated
Component: bindings
0.15.x
Other All
: Urgent critical
: ---
Assigned To: Johnny Jacob
Evolution QA team
Depends on:
Blocks: 507105
 
 
Reported: 2008-01-15 02:23 UTC by Debajyoti Bera
Modified: 2008-02-26 10:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed Patch (2.38 KB, patch)
2008-02-21 13:27 UTC, Johnny Jacob
committed Details | Review

Description Debajyoti Bera 2008-01-15 02:23:27 UTC
Please describe the problem:
Using the test program TestCal.cs, there is posssibly a leak in CalUtil.ICalToCalComponentArray.

Steps to reproduce:
1. Add a webcal calender to evolution with rapid sync rate (e.g. 2/3 minutes). It will reproduce the bug quicker.
2. Compile and run TestCal. Observe that memory RSS keep increasing.

3. Comment out the body of the callback methods that use the above function and repeat the test. Memory stabilizes after some point.

In my testing (with 4 webcal source), 20 mins saw more than 5 MB increase of RSS.

Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Debajyoti Bera 2008-01-15 02:27:35 UTC
Up-ing the priority since the leak rate is pretty high and applications using evo-sharp are badly affected.
Comment 2 Debajyoti Bera 2008-01-15 16:02:21 UTC
This is similar to the problem fixed in http://svn.gnome.org/viewvc/evolution-sharp?view=revision&revision=64
A memory leak when creating an array of CalComponent objects from icalcomponent and ECalComponent objects.
Comment 3 Johnny Jacob 2008-02-21 13:27:09 UTC
Created attachment 105691 [details] [review]
Proposed Patch

Patch releases e_cal_component. Would like to get this tested ( runs good with TestCal.exe. ) and commit it in trunk asap.
Comment 4 Debajyoti Bera 2008-02-21 22:19:25 UTC
Hmm... testing this (i.e. building evo-sharp from source) is going to be difficult :-(
If you build them and share the glue .so and the .dll file, I can try to put them in the right place and see if that works. On the other hand, if testcal worked, then the problem should be fixed.
Comment 5 Debajyoti Bera 2008-02-25 17:55:20 UTC
I was able to test this patch, thanks to Johnny and it works perfectly.
Comment 6 Johnny Jacob 2008-02-26 10:43:00 UTC
Thanks a lot  dBera for the test run. Patch committed to trunk svn rev 161.

Closing bug.