Summary: | Crash when auto-scrolling headers in compose window | ||
---|---|---|---|
Product: | Claws Mail (GTK 2) | Reporter: | Michael Schwendt <mschwendt> |
Component: | UI/Compose Window | Assignee: | users |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | andreas.bierfert |
Priority: | P3 | ||
Version: | 3.7.8 | ||
Hardware: | PC | ||
OS: | Linux |
Description
Michael Schwendt
2011-02-18 22:40:17 UTC
I can reproduce it with version 3.7.8, but not with newest cvs version. Maybe this fixed the issue: 2010-12-20 [colin] 3.7.8cvs14 * src/compose.c Don't flush events when adding an header entry. This can mess up the keypresses in the previous header. Instead, make the scroll happen later (doing it on the spot without flushing events doesn't work as the new requisitions aren't set yet). Any chances you could check CVS version? Indeed Pawel, 3.7.8cvs14 should fix it... Looks promising. I've tested the related change to compose.c, https://bugzilla.redhat.com/attachment.cgi?id=480198 and even with wild mouse-wheel usage I could not reproduce the issue. I guess our users will report back when they would encounter it again. There's a new bug report related to this with Claws Mail 3.7.8 patched with the scroll_postpone fix. Any ideas on what the user may have done? Full backtrace: https://bugzilla.redhat.com/attachment.cgi?id=482630 Bug: https://bugzilla.redhat.com/682679 Thread 1 (Thread 0x7f343b92e980 (LWP 19702)): #0 0x000000000047c3ce in compose_show_first_last_header (compose=<value optimized out>, show_first=0) at compose.c:10791 __inst = 0x-a0a0b000a0a0c __t = 35756320 __r = <value optimized out> vadj = <value optimized out> #1 0x000000000047c648 in scroll_postpone (data=0x2ab6750) at compose.c:10769 compose = 0x2ab6750 #2 0x00000036fd4435eb in g_timeout_dispatch (source=<value optimized out>, callback=<value optimized out>, user_data=<value optimized out>) at gmain.c:3585 timeout_source = 0x3f40be0 #3 0x00000036fd441e33 in g_main_dispatch (context=0x220d3c0) at gmain.c:2149 [...] In https://bugzilla.redhat.com/682679#c6 the reporter has described his typical Claws Mail usage (using the "Spam" button) when this crash in compose_show_first_last_header happens. He seems to be able to reproduce it often. In https://bugzilla.redhat.com/682679#c10 the reporter has downgraded to Claws Mail 3.7.8 (without the patch for this bug) and cannot reproduce the crash. Working on that, thanks for reporting again :) Changes related to this bug have been committed. Please check latest CVS and update the bug accordingly. You can also get the patch from: http://www.colino.net/claws-mail/ 2011-04-03 [colin] 3.7.8cvs70 * src/compose.c Fix bug #2365 again, "Crash when auto-scrolling headers in compose window". Batch-mode compose windows (from filtering actions) shouldn't try to access stuff after flushing events... Quoting reporter, | Work perfect. | Thank you. who referred to a test-build of 3.7.8cvs68 plus Colin's patch. |