SOFTWARE VERSION: tumbler 1:0.1.13-1+rq498670+0m6 /tumbler/tumblerd/tumbler-manager.c line 1455 -- *list = g_list_delete_link (*list, lp); line 1490 -- *list = g_list_delete_link (*list, lp); lp is deleted in these lines but still using in line 1442 for (lp = *list; lp != NULL; lp = lp->next) valgrind generate Invalid read errors for similar places For people with access to this bugzilla: https://projects.maemo.org/bugzilla/show_bug.cgi?id=235928
Created attachment 3567 Proposed fix for upstream
Note that downstream this bug is marked as invalid, due to the patches that add #ifdef MONITOR_CONFIG isn't this code being used / never ran on the Harmattan platform. The valgrind point the reporter makes is vague. I think he was seeing valgrind reporting about different lines. Perhaps a reopen will happen with the right lines, in which case I'll either fix those invalid reads (if they are relevant for Harmattan) and propose a patch. Or I'll again forward this info here or in a new bug. Cheers
I've pushed a similar but slightly different fix to master: commit 3704cf3502bc8cafb4e7cd7956c0ae983a415b3e Author: Jannis Pohlmann <jannis@xfce.org> Date: Sat Feb 25 15:09:30 2012 +0000 Avoid accessing list elements after they have been freed (bug #7418).