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 628651 - memory leak in imapx_append_message()
memory leak in imapx_append_message()
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: Mailer
2.32.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[imapx]
Depends on:
Blocks: 627707
 
 
Reported: 2010-09-03 00:31 UTC by David Woodhouse
Modified: 2013-09-14 16:54 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
eds patch (2.01 KB, patch)
2010-09-06 19:22 UTC, Milan Crha 		committed Details | Review

Description David Woodhouse 2010-09-03 00:31:42 UTC 
==21464== 1,840 (864 direct, 976 indirect) bytes in 6 blocks are definitely lost in loss record 31,
798 of 33,527
==21464==    at 0x4A05481: calloc (vg_replace_malloc.c:418)
==21464==    by 0x3D3D645CE7: g_malloc0 (gmem.c:157)
==21464==    by 0x1A263D5F: camel_imapx_server_append_message (camel-imapx-server.c:5090)
==21464==    by 0x1A2615D3: imapx_append_message (camel-imapx-folder.c:362)
==21464==    by 0x68EFCE3: camel_folder_append_message (camel-folder.c:1415)
==21464==    by 0xF3A3138: send_queue_exec (mail-ops.c:620)
==21464==    by 0xF39E407: mail_msg_proxy (mail-mt.c:469)
==21464==    by 0x3D3D668EC3: g_thread_pool_thread_proxy (gthreadpool.c:314)
==21464==    by 0x3D3D666745: g_thread_create_proxy (gthread.c:1897)
==21464==    by 0x359B007760: start_thread (pthread_create.c:301)
==21464==    by 0x359A8E14EC: clone (clone.S:115)
Comment 1 Milan Crha 2010-09-06 19:22:40 UTC 
Created attachment 169608 [details] [review]
eds patch

for evolution-data-server;

This fixes it, together with an 'uid' leak in the same function and an incorrect assignment to CamelMessageInfo::uid, as it should always come from camel_pstring_..., because it's freed with camel_pstring_free function at the end.
Comment 2 Milan Crha 2010-09-06 19:23:46 UTC 
Created commit ba23727 in eds master (2.31.92+)