Re: [evolution-patches] Another valgrind-warnings fix, this one is in CamelFolderSearch

[Philip Van Hoof <spam pvanhoof be>]

On Wed, 2007-06-27 at 21:12 +0200, Philip Van Hoof wrote:
> What about this one?
> 
> http://tinymail.org/trac/tinymail/changeset/2279?format=diff
> 
> Fixes these valgrind warnings:

The code that was causing these (I'm not sure about the line numbers,
since things have changed since the tester invoked these warnings):

~230  func = *((void **)(((char *)klass)+builtins[i].offset));
~234  if (func == NULL && builtins[i].flags&1) {


> ==8434== Conditional jump or move depends on uninitialised value(s)
> ==8434==    at 0x412B39F: camel_folder_search_construct (camel-folder-search.c:230)
> ==8434==    by 0x5B2292B: camel_imap_search_new (camel-imap-search.c:165)
> ==8434==    by 0x5B16B2A: camel_imap_folder_new (camel-imap-folder.c:336)
> ==8434==    by 0x5B290F5: get_folder_offline (camel-imap-store.c:2630)
> ==8434==    by 0x5B28813: get_folder_online (camel-imap-store.c:2410)
> ==8434==    by 0x41263C9: disco_get_folder (camel-disco-store.c:235)
> ==8434==    by 0x414D6D6: camel_store_get_folder (camel-store.c:297)
> 
> ==8434== Conditional jump or move depends on uninitialised value(s)
> ==8434==    at 0x412B42D: camel_folder_search_construct (camel-folder-search.c:234)
> ==8434==    by 0x5B2292B: camel_imap_search_new (camel-imap-search.c:165)
> ==8434==    by 0x5B16B2A: camel_imap_folder_new (camel-imap-folder.c:336)
> ==8434==    by 0x5B290F5: get_folder_offline (camel-imap-store.c:2630)
> ==8434==    by 0x5B28813: get_folder_online (camel-imap-store.c:2410)
> ==8434==    by 0x41263C9: disco_get_folder (camel-disco-store.c:235)
> ==8434==    by 0x414D6D6: camel_store_get_folder (camel-store.c:297)
> 
> 
-- 
Philip Van Hoof, software developer
home: me at pvanhoof dot be 
gnome: pvanhoof at gnome dot org 
http://www.pvanhoof.be/blog