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 345784 - Crashes on sorting threads by subject
Crashes on sorting threads by subject
Status: RESOLVED FIXED
Product: Pan
Classification: Other
Component: general
pre-1.0 betas
Other Windows
: Normal critical
: 1.0
Assigned To: Charles Kerr
Pan QA Team
Depends on:
Blocks:
 
 
Reported: 2006-06-24 00:02 UTC by Artur Jachacy
Modified: 2006-06-27 19:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix utf8 corruption in the header pane by subject/author sorting. (598 bytes, patch)
2006-06-27 19:43 UTC, Charles Kerr
none Details | Review

Description Artur Jachacy 2006-06-24 00:02:04 UTC
When I try to sort headers by subject in Pan .100, it crashes every time. (WinXP)

Program received signal SIGSEGV, Segmentation fault.
0x10037a2c in ?? ()

(gdb) bt
  • #0 ??
  • #1 ??

Comment 1 Charles Kerr 2006-06-25 03:35:12 UTC
I won't be able to test this on Windows until Monday.
Can't reproduce on Linux.

That's an interersting backtrace -- I wonder, is Pan
scrambling the stack so badly that a backtrace is
impossible?  Did you compile with -ggdb3 ?

Anyway, I'll give this a spin on XP on Monday w/0.101.
Comment 2 Artur Jachacy 2006-06-25 14:06:37 UTC
Yes, I compiled with -ggdb3, as in readme.mingw. Actually, most of the backtraces I get in Insight look like that.
The bug must have something to do with contents of those subjects, because I can sort some newsgroups just fine. Perhaps it's a throwback to the bug fixed for 0.97?
Comment 3 Charles Kerr 2006-06-25 17:45:46 UTC
Well now that's helpful, why didn't you mention that before? :)

Which groups does it crash on?
Comment 4 Artur Jachacy 2006-06-25 18:54:19 UTC
That's because I didn't know, honestly. :-) The first three newsgroups I tried crashed, so I didn't check any further.

For crashing newsgroups, you can try gmane.science.linguistics.wikipedia.arabic, gmane.science.linguistics.wikipedia.chinese or gmane.science.linguistics.wikipedia.polish on news.gmane.org. Some other groups: pl.hum.polszczyzna, pl.pregierz, alt.chinese.text.big5.
Comment 5 Charles Kerr 2006-06-26 04:00:05 UTC
Not crashing for me in Linux, but Pan *is* handling the
groups strangely.  It appears to save the headers right,
but won't reload them.

Regarding this crash, I'll have to retry with valgrind
to see if there are any memory errors that Linux is treating
as non-fatal so that Pan soldiers on.

Regarding the missing headers, this sounds suspiciously
like #345787 (threading falls apart) which you reported.
By any chance was that behavior in the same groups,
he asked hopefully? :)
Comment 6 Artur Jachacy 2006-06-26 16:38:09 UTC
Unfortunately, #345787 happens in all groups.
Comment 7 Charles Kerr 2006-06-27 18:36:16 UTC
confirmed for gmane.science.linguistics.wikipedia.chinese on XP.
works fine on RHEL4.
Comment 8 Charles Kerr 2006-06-27 19:27:57 UTC
d'oh, we're corrupting utf8 strings by stripping out leading whitespace
with isalpha() and *pch++ in header_pane.cc::do_collate().
Comment 9 Charles Kerr 2006-06-27 19:43:42 UTC
Created attachment 68082 [details] [review]
Fix utf8 corruption in the header pane by subject/author sorting.
Comment 10 Charles Kerr 2006-06-27 19:47:12 UTC
The above patch seems to fix it.

I can consistently crash 0.101 in XP by going to gmane.s.l.wikipedia.chinese,
downloading all, and sorting by subject.  For some reason it works under RHEL4
though we must surely be corrupting the utf8 string there too.

0.101 + the above patch passes the same test under both Windows and RHEL4.

I'll include the patch in 0.102.