Re: [evolution-patches] Conditional jump or move depends on uninitialised value(s)

[Philip Van Hoof <spam pvanhoof be>]

Here are two other such valgrind warnings:

==22389== Conditional jump or move depends on uninitialised value(s)
==22389==    at 0x4F65FEE: g_ascii_strcasecmp (gstrfuncs.c:1840)
==22389==    by 0x4184A1E: camel_header_raw_find (camel-mime-utils.c:3868)
==22389==    by 0x417D146: folder_scan_step (camel-mime-parser.c:1609)
==22389==    by 0x417DA13: camel_mime_parser_step (camel-mime-parser.c:623)
==22389==    by 0x417F24C: construct_from_parser (camel-mime-part.c:936)
==22389==    by 0x417ADB2: construct_from_parser (camel-mime-message.c:600)
==22389==    by 0x417F435: camel_mime_part_construct_from_parser (camel-mime-part.c:989)
==22389==    by 0x417F491: construct_from_stream (camel-mime-part.c:1005)
==22389==    by 0x416DC93: camel_data_wrapper_construct_from_stream (camel-data-wrapper.c:270)
==22389==    by 0x53DBDBE: imap_update_summary (camel-imap-folder.c:2833)
==22389==    by 0x53DCC5D: camel_imap_folder_changed (camel-imap-folder.c:3818)
==22389==    by 0x53DCE1E: imap_rescan (camel-imap-folder.c:1086)
==22389==    by 0x53DDB2E: camel_imap_folder_selected (camel-imap-folder.c:652)
==22389==    by 0x53DE113: imap_refresh_info (camel-imap-folder.c:799)
==22389==    by 0x4118228: disco_refresh_info (camel-disco-folder.c:268)
==22389==    by 0x4127A8E: camel_folder_refresh_info (camel-folder.c:320)
==22389==    by 0x40F3F96: tny_camel_folder_refresh_async_thread (tny-camel-folder.c:861)
==22389==    by 0x4F694F8: g_thread_create_proxy (gthread.c:591)
==22389==    by 0x462AF9A: pthread_start_thread (in /targets/i386-csgcc34/lib/libpthread-0.10.so)
==22389==    by 0x45890B9: clone (in /targets/i386-csgcc34/lib/libc-2.3.6.so)

==22389== Use of uninitialised value of size 4
==22389==    at 0x4180543: header_decode_lwsp (camel-mime-utils.c:994)
==22389==    by 0x4181E13: decode_token (camel-mime-utils.c:1810)
==22389==    by 0x4183F16: camel_content_type_decode (camel-mime-utils.c:3375)
==22389==    by 0x417D156: folder_scan_step (camel-mime-parser.c:1609)
==22389==    by 0x417DA13: camel_mime_parser_step (camel-mime-parser.c:623)
==22389==    by 0x417F24C: construct_from_parser (camel-mime-part.c:936)
==22389==    by 0x417ADB2: construct_from_parser (camel-mime-message.c:600)
==22389==    by 0x417F435: camel_mime_part_construct_from_parser (camel-mime-part.c:989)
==22389==    by 0x417F491: construct_from_stream (camel-mime-part.c:1005)
==22389==    by 0x416DC93: camel_data_wrapper_construct_from_stream (camel-data-wrapper.c:270)
==22389==    by 0x53DBDBE: imap_update_summary (camel-imap-folder.c:2833)
==22389==    by 0x53DCC5D: camel_imap_folder_changed (camel-imap-folder.c:3818)
==22389==    by 0x53DCE1E: imap_rescan (camel-imap-folder.c:1086)
==22389==    by 0x53DDB2E: camel_imap_folder_selected (camel-imap-folder.c:652)
==22389==    by 0x53DE113: imap_refresh_info (camel-imap-folder.c:799)
==22389==    by 0x4118228: disco_refresh_info (camel-disco-folder.c:268)
==22389==    by 0x4127A8E: camel_folder_refresh_info (camel-folder.c:320)
==22389==    by 0x40F3F96: tny_camel_folder_refresh_async_thread (tny-camel-folder.c:861)
==22389==    by 0x4F694F8: g_thread_create_proxy (gthread.c:591)
==22389==    by 0x462AF9A: pthread_start_thread (in /targets/i386-csgcc34/lib/libpthread-0.10.so)


On Wed, 2007-06-27 at 10:57 +0200, Philip Van Hoof wrote:
> Based on what valgrind is giving me, I'm proposing this patch (attached). 
> 
> ==25759== Conditional jump or move depends on uninitialised value(s)
> ==25759==    at 0x417BB66: folder_scan_content (camel-mime-parser.c:1353)
> ==25759==    by 0x417C032: folder_scan_step (camel-mime-parser.c:1662)
> 
> _______________________________________________
> Evolution-patches mailing list
> Evolution-patches gnome org
> http://mail.gnome.org/mailman/listinfo/evolution-patches
-- 
Philip Van Hoof, software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://www.pvanhoof.be/blog