pcmanx crashes with fancy texts ( on both FreeBSD 9-STABLE amd64/i386)

[GoogleCodeExporter]

What steps will reproduce the problem?
1.log into bbs.newsmth.net
2.browse some threads with fancy characters.
3.pcmanx crashes occasionally

What is the expected output? What do you see instead?
pcmanx should deal with fancy texts correctly, but failed.(maybe problems 
displaying unicode texts)

What version of the product are you using? On what operating system?
pcmanx 1.1 (1.0 works well)

Please provide any additional information below.

Original issue reported on code.google.com by jok...@gmail.com on 13 Jun 2012 at 5:31

[GoogleCodeExporter]

I tried to connect to bbs.newsmth.net (in Big5).  However, I can not reproduce.
Can you use git verison of pcmanx and grab GDB stack when it is going to crash?

Original comment by jserv.tw@gmail.com on 13 Jun 2012 at 5:40

[GoogleCodeExporter]

OK.

It's interesting that 1.0 works well here. But 1.1 does not.

Original comment by jok...@gmail.com on 13 Jun 2012 at 6:22

[GoogleCodeExporter]

gdb output when pcmanx 1.1 crashes browsing some threads.
on FreeBSD 9-STABLE i386. I'll recompile pcmanx with debugging
symbols and try again if possible.
------------------------------------------------
[New Thread 2940be00 (LWP 100465/pcmanx)]
Assertion failed: (result != NULL), function IA__FcFontSort, file fcmatch.c, 
line 850.

Program received signal SIGABRT, Aborted.
[Switching to Thread 29404300 (LWP 100281/pcmanx)]
0x28a6e3bb in thr_kill () from /lib/libc.so.7
-----------------------------------------------

Original comment by jok...@gmail.com on 13 Jun 2012 at 6:42

[GoogleCodeExporter]

Can you provide backtrace in certain thread?  Thanks!

Original comment by jserv.tw@gmail.com on 13 Jun 2012 at 6:58

[GoogleCodeExporter]

I think the following is the key for solution:
"Assertion failed: (result != NULL), function IA__FcFontSort, file fcmatch.c, 
line 850."

Can you try the GIT version?  We should already fix.

Original comment by jserv.tw@gmail.com on 23 Jun 2012 at 7:56

[GoogleCodeExporter]

Original comment by jserv.tw@gmail.com on 23 Jun 2012 at 7:56

[GoogleCodeExporter]

Original comment by fourdollars on 24 Jun 2012 at 5:39

• Changed state: Accepted

[GoogleCodeExporter]

I ran it in gdb and got the following. 

------------------------------------------------------------------------------

GNU gdb 6.1.1 [FreeBSD]
Copyright 2004 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB.  Type "show warranty" for details.
This GDB was configured as "i386-marcel-freebsd"...
(gdb) r
Starting program: 
/opt/obj/portbld/usr/ports/chinese/pcmanx/work/pcmanx-gtk2-1.1/src/pcmanx 
[New LWP 100214]
[New Thread 29404300 (LWP 100214/pcmanx)]
[New Thread 29408200 (LWP 100560/pcmanx)]
[New Thread 29408e00 (LWP 100561/pcmanx)]
[New Thread 2940be00 (LWP 100562/pcmanx)]
[Thread 2940be00 (LWP 100562/pcmanx) exited]
[New Thread 2940be00 (LWP 100562/pcmanx)]
Assertion failed: (result != NULL), function IA__FcFontSort, file fcmatch.c, 
line 850.

Program received signal SIGABRT, Aborted.
[Switching to Thread 29404300 (LWP 100214/pcmanx)]
0x28a6e3bb in thr_kill () from /lib/libc.so.7
(gdb) where
#0  0x28a6e3bb in thr_kill () from /lib/libc.so.7
#1  0x289fef96 in pthread_sigmask () from /lib/libthr.so.3
#2  0x289ff66b in raise () from /lib/libthr.so.3
#3  0x28b16f6a in abort () from /lib/libc.so.7
#4  0x28af9496 in __assert () from /lib/libc.so.7
#5  0x287a969e in IA__FcFontSort () from /usr/local/lib/libfontconfig.so.1
#6  0x0808de9e in CFontConfig::CFontConfig ()
#7  0x0808e072 in CFontConfig::Instance ()
#8  0x08089a55 in CTermView::DrawChar ()
#9  0x080833e1 in CTermData::DoUpdateDisplay ()
#10 0x08084ff1 in CTermData::UpdateDisplay ()
#11 0x08060935 in CTelnetCon::OnRecv ()
#12 0x080609ac in CTelnetCon::OnSocket ()
#13 0x28734abd in g_io_channel_unix_get_fd () from 
/usr/local/lib/libglib-2.0.so.0
#14 0x286ee247 in g_main_context_dispatch () from 
/usr/local/lib/libglib-2.0.so.0
#15 0x286f22ce in g_main_context_prepare () from /usr/local/lib/libglib-2.0.so.0
#16 0x286f26d7 in g_main_loop_run () from /usr/local/lib/libglib-2.0.so.0
#17 0x282e65a4 in gtk_main () from /usr/local/lib/libgtk-x11-2.0.so.0
#18 0x0806d648 in main ()
--------------------------------------------------------------------------------
-

Original comment by jok...@gmail.com on 29 Jun 2012 at 4:43

[GoogleCodeExporter]

Was the above backtrace produced by GIT version?

commit log:
commit 6a14892408364090985bf89fd4efa81f13da6bc2
Author: Jim Huang <jserv@0xlab.org>
Date:   Tue May 1 18:50:45 2012 +0800

    FcFontSort() should no longer use `NULL' as the value of `result'

    After a recent commit[1] in the fontconfig code, FcFontSort() no longer
    accepts NULL as the `result' argument.
    Therefore, the code in src/core/cfontconfig.cpp should be updated
    accordingly.
    Attached is a patch to fix this issue.

    [1] http://cgit.freedesktop.org/fontconfig/commit/?id=a18ca17b6211f62fbd1d893811b94b8c83db4cc0

    Patched by: CasperVector@gmail.com

Original comment by jserv.tw@gmail.com on 29 Jun 2012 at 3:13

[GoogleCodeExporter]

Can't reproduce here.  Marked as FIXED.

Original comment by jserv.tw@gmail.com on 3 Jul 2012 at 6:01

• Changed state: Fixed