Bug 680572 - [abrt] gnumeric-1:1.10.13-1.fc13: Process /usr/bin/gnumeric-1.10.13 was killed by signal 11 (SIGSEGV)
Summary: [abrt] gnumeric-1:1.10.13-1.fc13: Process /usr/bin/gnumeric-1.10.13 was kille...
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: gnumeric
Version: 13
Hardware: x86_64
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Huzaifa S. Sidhpurwala
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard: abrt_hash:c6a5152550b8826b056e871023f...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-02-25 22:58 UTC by Karl
Modified: 2011-02-28 07:00 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-02-28 07:00:51 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
File: backtrace (30.50 KB, text/plain)
2011-02-25 22:59 UTC, Karl
no flags Details

Description Karl 2011-02-25 22:58:59 UTC
abrt version: 1.1.14
architecture: x86_64
Attached file: backtrace
cmdline: gnumeric /home/karlh/Desktop/gnumeric-bugtest.gnumeric
comment: The crash occurs because the constraint isn't preserved on the save and reload; instead of $A$1 >= 0 it says $A$1 >= #REF#.  The solver doesn't check for the improper constraint on file open either.
component: gnumeric
crash_function: gnm_solver_constraint_get_part
executable: /usr/bin/gnumeric-1.10.13
kernel: 2.6.34.7-66.fc13.x86_64
package: gnumeric-1:1.10.13-1.fc13
rating: 4
reason: Process /usr/bin/gnumeric-1.10.13 was killed by signal 11 (SIGSEGV)
release: Fedora release 13 (Goddard)
time: 1298674152
uid: 500

How to reproduce
-----
Easily reproduced:
1. Open new file and enter random numbers in A1:A5
2. Set cell B1 to "=(A1-5)^2" and fill down to B5
3. Set cell B7 to "=SUM(B1:B5)"
4. Open Solver; tell it to set $B$7 -> Min by changing $A$1.
5. Add a constraint:  $A$1 >= 0; click "Add"
6. Save the file and close it.
7. Open the file you just saved.
8. Open Solver and click "Solve".

Comment 1 Karl 2011-02-25 22:59:01 UTC
Created attachment 481101 [details]
File: backtrace

Comment 2 Julian Sikorski 2011-02-26 05:04:03 UTC
It happens here. CCing upstream.

Comment 3 M Welinder 2011-02-26 13:28:07 UTC
Crash fixed here:

http://git.gnome.org/browse/gnumeric/commit/?id=022c38d8aaa8cc093554f1e04c5c74766aa876f1

Saving is fine, but loading mangles the right-hand side.  I'll have a look
at that too.

Comment 4 M Welinder 2011-02-26 13:58:29 UTC
Reading and a further problem fixed here.

http://git.gnome.org/browse/gnumeric/commit/?id=7c2cdb4a0ef43eeabc1618df1d351395fbcd3970

However, results are still bogus, but that's your own fault: the model
is not linear.  Use the non-linear solver instead.

Comment 5 Julian Sikorski 2011-02-26 18:49:53 UTC
Should I patch the Fedora package with these, or would it be enough if the fix shipped with the next gnumeric version?

Comment 6 M Welinder 2011-02-26 22:48:38 UTC
They'll ship with the next version in, I guess, a few weeks.
That ought to be good enough.  The reporter can build with patches if he
really needs it before then.

Comment 7 Karl 2011-02-28 01:11:04 UTC
Thank you for your prompt fixes---I look forward to the next version.


Note You need to log in before you can comment on or make changes to this bug.