Bug 564078 – map_pattern_to_xl critical

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 564078 - map_pattern_to_xl critical


Summary:	map_pattern_to_xl critical


Status:	RESOLVED FIXED

Product:	Gnumeric
Classification:	Applications
Component:	import/export MS Excel (tm)
Version:	git master
Hardware:	Other All

Importance:	Normal normal
Target Milestone:	---
Assigned To:	Morten Welinder
QA Contact:	Jody Goldberg

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2008-12-11 07:54 UTC by sum1
Modified:	2008-12-12 21:28 UTC

See Also:
GNOME target:	---
GNOME version:	---

Description sum1 2008-12-11 07:54:15 UTC

Version: r16998
OS: Ubuntu Intrepid

Steps to reproduce:
- ssconvert gnumeric/samples/applix/sample.as /tmp/foo.xls


Partial console output:

** (/home/s/local/bin/ssconvert:19396): WARNING **: Unknown color (ffff00), converting it to black

** (/home/s/local/bin/ssconvert:19396): CRITICAL **: map_pattern_to_xl: assertion `i >= 0 && i < (int)G_N_ELEMENTS (map_to_excel)' failed

** (/home/s/local/bin/ssconvert:19396): WARNING **: Unknown color (ff0000), converting it to black

** (/home/s/local/bin/ssconvert:19396): CRITICAL **: map_pattern_to_xl: assertion `i >= 0 && i < (int)G_N_ELEMENTS (map_to_excel)' failed

** (/home/s/local/bin/ssconvert:19396): CRITICAL **: map_pattern_to_xl: assertion `i >= 0 && i < (int)G_N_ELEMENTS (map_to_excel)' failed


Backtrace:

Breakpoint 1, 0xb6f14926 in g_log () from /usr/lib/libglib-2.0.so.0
(gdb) f 2

+ Trace 210687

#2 map_pattern_to_xl
at ms-excel-write.c line 179
#0 g_log
from /usr/lib/libglib-2.0.so.0
#1 g_return_if_fail_warning
from /usr/lib/libglib-2.0.so.0
#2 map_pattern_to_xl
at ms-excel-write.c line 179
#3 build_xf_data
at ms-excel-write.c line 2768
#4 excel_write_XFs
at ms-excel-write.c line 3053
#5 excel_write_workbook
at ms-excel-write.c line 5573
#6 excel_write_v7
at ms-excel-write.c line 5705
#7 excel_save
at boot.c line 271
#8 excel_dsf_file_save
at boot.c line 308
#9 go_plugin_loader_module_func_file_save
at go-plugin-loader-module.c line 323
#10 go_plugin_file_saver_save
at go-plugin-service.c line 749
#11 go_file_saver_save
at file.c line 705
#12 wbv_save_to_output
at workbook-view.c line 850
#13 wbv_save_to_uri
at workbook-view.c line 887
#14 wb_view_save_as
at workbook-view.c line 923
#15 convert
at ssconvert.c line 336
#16 main
at ssconvert.c line 401

Comment 1 Morten Welinder 2008-12-11 20:33:49 UTC

The table contains 19 patterns, but pattern.c defines 26.

Comment 2 Morten Welinder 2008-12-11 20:48:51 UTC

Make that 25 but a placeholder for 0.  (Vs 18 + #0)

Anyways, 19-23 are applix-specific so we need to map.  And 24-25 probably
are too.

Comment 3 Morten Welinder 2008-12-11 21:38:59 UTC

Criticals fixed, but something is still screwed up in two_way_table_move
causing the colour trouble.

Comment 4 Morten Welinder 2008-12-12 21:28:08 UTC

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.