[Shr-Devel] [Debian] Segfault in phoneui-contacts

[Sebastian Reichel]

On Mon, Jan 18, 2010 at 09:48:17PM +0200, Tom Hacohen wrote:
> On Mon, Jan 18, 2010 at 2:37 PM, Sebastian Reichel <elektranox at gmail.com>
> wrote:
> 
>     On Mon, Jan 11, 2010 at 05:51:03PM +0200, Tom Hacohen wrote:
>     > On Mon, Jan 11, 2010 at 4:57 PM, Sebastian Reichel <elektranox at gmail.com>
>     > wrote:
>     >
>     >     Hi,
>     >
>     >     I got phoneui-contacts segfaulting by doing:
>     >      * Select Contact
>     >      * Options -> Show
>     >      * Add Field
>     >      * Back
>     >
>     >     Backtrace:
>     >     (gdb) bt
>     >     #0  0x4027d588 in strcmp () from /lib/libc.so.6
>     >     #1  0x40509808 in frame_edit_data_changed (data=0x184ba8) at view/
>     >     contact-show-view.c:534
>     >     #2  0x40509ae8 in frame_edit_close_clicked (_data=0x184ba8, obj=<value
>     >     optimized out>, event_info=<value optimized out>) at view/
>     >     contact-show-view.c:641
>     >     #3  0x405506ec in evas_object_smart_callback_call () from /usr/lib/
>     >     libevas-ver-svn-05.so.0
>     >     #4  0x406d0924 in ?? () from /usr/lib/libelementary-ver-svn-05.so.0
>     >     #5  0x406d0924 in ?? () from /usr/lib/libelementary-ver-svn-05.so.0
>     >
>     >     I can prepare a patch if you need one, but I first I want to fix the
>     >     Debian SHR packages, so you are probably faster.
>     >
>     >     Apart from the library has not been linked against ecore-x for me.
>     >     You should add ecore-x to libphone-ui-shr's configure.ac.
>     >
>     >     -- Sebastian
>     >
>     > I remember something similar, I think we started taking a look at it, but
>     > stopped as we figured we are going to overhaul all of the messaging app
>     anyway.
>     > As for ecore-x, the only thing that actually uses it is obsolete, but
>     yeah, we
>     > should probably add it (and eventually remove it), I'm starring this
>     message,
>     > will do.
> 
>     I just found another segfault in phoneui-contacts:
>      * Select Contact
>      * Options -> Show
>      * Edit Field
>      * Delete all text from one of the fields
>      * Save
> 
>     Here is the backtrace:
> 
>     (gdb) bt
>     #0  0x4027e588 in strcmp () from /lib/libc.so.6
>     #1  0x40508830 in frame_edit_data_changed (data=<value optimized out>) at
>     view/contact-show-view.c:536
>     #2  0x40508878 in frame_edit_save_clicked (_data=<value optimized out>, obj
>     =<value optimized out>, event_info=<value optimized out>) at view/
>     contact-show-view.c:569
>     #3  0x405496ec in evas_object_smart_callback_call () from /usr/lib/
>     libevas-ver-svn-05.so.0
> 
>     -- Sebastian
> 
>     -----BEGIN PGP SIGNATURE-----
>     Version: GnuPG v1.4.10 (GNU/Linux)
> 
>     iEYEARECAAYFAktUVfgACgkQH0JwilpTmKhregCfeEU6yrKIlWb65DDePECnFGWU
>     MKYAoIB8u/IaRCSS5pZzCWKYGsWcpfY7
>     =e57V
>     -----END PGP SIGNATURE-----
> 
>     _______________________________________________
> 
> Told you we need a rewrite :P
> Anyhow, I wrote this down as well, thanks.

Hi,

I found some time today to fix it. You can find the patch attached
to the bug report I just created:

http://shr-project.org/trac/ticket/881

This patch will fix both problems :)

-- Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.shr-project.org/pipermail/shr-devel/attachments/20100120/7f7a8184/attachment-0001.pgp>