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 627611 - Memory leak in efh_format_header()
Memory leak in efh_format_header()
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.32.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
Depends on:
Blocks: 627707
 
 
Reported: 2010-08-21 23:31 UTC by David Woodhouse
Modified: 2013-09-13 01:04 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description David Woodhouse 2010-08-21 23:31:10 UTC 
==30229== 7,535 (200 direct, 7,335 indirect) bytes in 5 blocks are definitely lost in loss record 25,325 of 25,778
==30229==    at 0x4A04481: calloc (vg_replace_malloc.c:418)
==30229==    by 0x3A4DC456A7: g_malloc0 (in /lib64/libglib-2.0.so.0.2512.0)
==30229==    by 0x79D292D: camel_header_address_new (camel-mime-utils.c:4501)
==30229==    by 0x79D29AD: camel_header_address_new_name (camel-mime-utils.c:4511)
==30229==    by 0x79D90C1: header_decode_mailbox (camel-mime-utils.c:2571)
==30229==    by 0x79D9611: camel_header_address_decode (camel-mime-utils.c:2628)
==30229==    by 0xF15C711: efh_format_header (em-format-html.c:2442)
==30229==    by 0xF15D384: efh_format_message (em-format-html.c:2727)
==30229==    by 0xF15B278: efh_format_exec (em-format-html.c:204)
==30229==    by 0xF16F027: mail_msg_proxy (mail-mt.c:469)
==30229==    by 0x3A4DC67D2A: ??? (in /lib64/libglib-2.0.so.0.2512.0)
==30229==    by 0x3A4DC65DE3: ??? (in /lib64/libglib-2.0.so.0.2512.0)
Comment 1 David Woodhouse 2010-08-21 23:54:02 UTC 
camel_header_address_decode() returns a list. But efh_format_header() and two other places in em-format-html.c seem to be calling camel_header_address_unref() only on the *first* item in that list.
Comment 2 David Woodhouse 2010-08-22 00:16:58 UTC 
s/camel_header_address_unref(addrs)/camel_header_address_list_clear(&addrs) seems to fix it.
Comment 3 David Woodhouse 2010-08-24 00:15:41 UTC 
To ssh://dwmw2@git.gnome.org/git/evolution
   7a2e06e..480b9e1  master -> master