El dom, 07-06-2009 a las 00:47 +0900, Kouhei Sutou escribió:
> Hi,
> 
> In <1244299332.4158.3.ca...@charmaleon>
>   "Re: [poppler] unit test for GLib bindings" on Sat, 06 Jun 2009 16:42:12 
> +0200,
>   Carlos Garcia Campos <[email protected]> wrote:
> 
> > I've just pulled again from you repo and it still crashes with
> > attachments:
> > 
> > $ CUTTER_DEBUG=yes glib/test/run-test.sh
> > GNU gdb 6.8-debian
> > Copyright (C) 2008 Free Software Foundation, Inc.
> > License GPLv3+: GNU GPL version 3 or later 
> > <http://gnu.org/licenses/gpl.html>
> > This is free software: you are free to change and redistribute it.
> > There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
> > and "show warranty" for details.
> > This GDB was configured as "i486-linux-gnu"...
> > (gdb) r
> > Starting program: /home/carlos/gnome-cvs/bin/cutter --keep-opening-modules 
> > --verbose=verbose -s glib/test --exclude-directory lib --exclude-directory 
> > fixtures glib/test
> > [Thread debugging using libthread_db enabled]
> > [New Thread 0xb7c8d6c0 (LWP 10641)]
> > test-layer:
> >   test_nested:                                              .: (0.016063)
> >   test_title:                                               .: (0.018539)
> >   test_visible:                                             .: (0.001037)
> > test-form-field:
> >   test_multi_line_text:                                     .: (0.003734)
> >   test_combo_choice:                                        .: (0.003480)
> >   test_list_choice:                                 .: (0.005432)
> >   test_check_button:                                        .: (0.008952)
> >   test_radio_button:                                        .: (0.002963)
> >   test_general:                                             .: (0.002682)
> >   test_normal_text:                                 .: (0.002978)
> >   test_file_select_text:                            .: (0.003110)
> >   test_push_button:                                 .: (0.002925)
> >   test_normal_password_text:                                .: (0.003063)
> > test-page:
> >   test_index_multi_pages:                           .: (0.000797)
> >   test_get_size:                                    .: (0.000638)
> >   test_render_pixbuf:                                       .: (0.029454)
> >   test_transition:                                  .: (0.000983)
> >   test_index_single_page:                           .: (0.000672)
> >   test_get_form_field_mapping:                              .: (0.002568)
> >   test_get_text:                                    .: (0.010651)
> > test-document:
> >   test_attachment:                                  
> > Program received signal SIGSEGV, Segmentation fault.
> > [Switching to Thread 0xb7c8d6c0 (LWP 10641)]
> > attachment_equal (data1=0x0, data2=0x8c62e00) at test-document.c:494
> > 494   if ((attachment1->checksum != attachment2->checksum) &&
> > (gdb) bt
> > #0  attachment_equal (data1=0x0, data2=0x8c62e00) at test-document.c:494
> 
> OK. I've added NULL check.
>   
> http://github.com/kou/poppler-test/commit/01c2b6f6fd71718dbd0258abbbc19e7d4c0f69ac
> 
> Does the change fix the problem? (I couldn't reproduce it on
> my environment.)

yes, but still crashes:

attachment_inspect (string=0x84137c0, data=0x0, user_data=0x0) at 
test-document.c:520
520       g_string_append_printf (string, "name=<%s>, ", attachment->name);
(gdb) bt
#0  attachment_inspect (string=0x84137c0, data=0x0, user_data=0x0) at 
test-document.c:520
#1  0xb7fc88ab in gcut_list_inspect (list=0x8410330, inspect_func=0xb7a2b950 
<attachment_inspect>, user_data=0x0) at gcut-list.c:57
#2  0xb7fca9e7 in gcut_assert_equal_list_helper (expected=0x8410330, 
actual=0x83eda80, equal_function=0xb7a2ba50 <attachment_equal>, 
    inspect_function=0xb7a2b950 <attachment_inspect>, inspect_user_data=0x0, 
expression_expected=0xb7a2d2c7 "expected", 
    expression_actual=0xb7a2d2c0 "actual", expression_equal_function=0xb7a2d2af 
"attachment_equal") at gcut-assertions-helper.c:98
#3  0xb7a2b93a in test_attachment () at test-document.c:558
#4  0xb7faa039 in invoke (test=0x8405550, test_context=0x8409b40, 
run_context=0x83f7800) at cut-test.c:403
#5  0xb7faa72b in run (test=0x8405550, test_context=0x8409b40, 
run_context=0x83f7800) at cut-test.c:429
#6  0xb7fa96af in cut_test_run (test=0x8405550, test_context=0x8409b40, 
run_context=0x83f7800) at cut-test.c:461
#7  0xb7fad3bb in cut_test_case_run_with_filter (test_case=0x8404140, 
run_context=0x83f7800, test_names=0x0) at cut-test-case.c:338
#8  0xb7fadc5a in run (data=0x8406448, user_data=0xbf8f1e28) at 
cut-test-suite.c:244
#9  0xb7fae4af in cut_test_suite_run_test_cases (test_suite=0x83fa018, 
run_context=0x83f7800, test_cases=0x840a2b0, test_names=0x0) at 
cut-test-suite.c:290
#10 0xb7fae759 in cut_test_suite_run_with_filter (test_suite=0x83fa018, 
run_context=0x83f7800, test_case_names=0x0, test_names=0x0) at 
cut-test-suite.c:582
#11 0xb7fae7e6 in cut_test_suite_run (suite=0x83fa018, run_context=0x83f7800) 
at cut-test-suite.c:505
#12 0xb7fa6769 in cut_test_runner_run_test_suite (runner=0x83f7800, 
test_suite=0x83fa018) at cut-test-runner.c:767
#13 0xb7fa895c in runner_run (runner=0x83f7800) at cut-test-runner.c:722
#14 0xb7fa653e in cut_runner_run (runner=0x83f7800) at cut-runner.c:68
#15 0xb7fa30a4 in cut_run_context_start (context=0x83f7800) at 
cut-run-context.c:1758
#16 0xb7f79f1d in run (ui=0x83eea00, run_context=0x83f7800) at 
cut-console-ui.c:912
#17 0xb7fb6546 in cut_ui_run (ui=0x83eea00, run_context=0x83f7800) at 
cut-ui.c:103
#18 0xb7fa8d8a in cut_start_run_context (run_context=0x83f7800) at 
cut-main.c:411
#19 0xb7fa8f72 in cut_run () at cut-main.c:439
#20 0x080485aa in main (argc=138552184, argv=0x15) at main.c:35


> 
> BTW, I've released Cutter 1.0.7. It includes a change
> "including test name on crash" requested by you. :-)

great :-)

> If you are working on Fedora on x86_64, you can use RPM to
> install Cutter:
> 
>   % wget http://downloads.sourceforge.net/cutter/cutter-1.0.7-0.x86_64.rpm
>   % sudo rpm -Uvh cutter-1.0.7-0.x86_64.rpm
> 
> ref. http://cutter.sourceforge.net/reference/install-to-fedora.html

Ubuntu here, I'll install it from sources. 

> 
> Thanks,
> --
> kou


-- 
Carlos Garcia Campos
   [email protected]
   [email protected]
   http://carlosgc.linups.org
PGP key: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x523E6462

Attachment: signature.asc
Description: Esta parte del mensaje está firmada digitalmente

_______________________________________________
poppler mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/poppler

Reply via email to