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 623960 - regression in ms_excel_chart_read()
regression in ms_excel_chart_read()
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export MS Excel (tm)
1.10.x
Other Linux
: Normal normal
: ---
Assigned To: Jean Bréfort
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2010-07-09 16:08 UTC by Kees Cook
Modified: 2010-07-13 15:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
XLS that causes the crash (113.00 KB, application/vnd.ms-excel)
2010-07-10 18:02 UTC, Kees Cook
Details

Description Kees Cook 2010-07-09 16:08:24 UTC
With 1.10.1 I could open an Excel document that now crashes version 1.10.7.  (Ubuntu 10.04 vs Ubuntu 10.10.)

https://launchpad.net/bugs/603319


 xl_chart_read_end (handle=<value optimized out>, s=0x7fff4af178c0, ...
 ms_excel_chart_read (q=0x1fc0d50, ...
 ms_excel_chart_read_BOF (q=0x1fc0d50, ...
 ms_read_OBJ (q=0x1fc0d50, c=0x20eac50, ...
 ms_escher_read_ClientData (state=0x7fff4af17fc0, ...


=> 0x7f37d343c623:	mov    0x78(%rax),%rdi
   0x7f37d343c627:	mov    %rax,%rbp
   0x7f37d343c62a:	callq  0x7f37d3401af8 <g_slist_length@plt>
   0x7f37d343c62f:	cmp    $0x1,%eax
   0x7f37d343c632:	jne    0x7f37d343c023

rax            0x0	0

http://launchpadlibrarian.net/51612119/Stacktrace.txt
Comment 1 Morten Welinder 2010-07-09 17:35:10 UTC
Please attach a sample spreadsheet that exhibits this problem.  If it
is confidential, please provide it by email and we'll delete after
diagnosing.
Comment 2 Kees Cook 2010-07-10 18:02:11 UTC
Created attachment 165637 [details]
XLS that causes the crash

I've stripped out the data (set everything to "1"), and it still demonstrates the crash, so I've attached it.  Thanks!
Comment 3 Jean Bréfort 2010-07-10 19:07:41 UTC
Interesting file. Seems there are charts without a plot which is a bit weird. How was it generated? Excel? Calc? or other?
Comment 4 Jean Bréfort 2010-07-10 19:26:58 UTC
Not without  plot actually, with two plots.  I have fixed locally the crash but now get another apparently unrelated one.
Comment 5 Jean Bréfort 2010-07-10 19:47:12 UTC
Fixed in git. No more crashes but the charts are empty. Not sure it is the desired behavior.
Comment 6 Kees Cook 2010-07-13 15:28:00 UTC
Uhm, yeah, no.  The graphs are busted now.  :P  I'll open a new bug.