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 499722 - Crash in scg_make_cell_visible()
Crash in scg_make_cell_visible()
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: GUI Expression Entry Widget
git master
Other All
: High blocker
: ---
Assigned To: Morten Welinder
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2007-11-26 12:28 UTC by sum1
Modified: 2007-11-27 00:22 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch (1.43 KB, patch)
2007-11-26 15:10 UTC, Morten Welinder 		accepted-commit_now Details | Review

Description sum1 2007-11-26 12:28:34 UTC 
Version: r16203
OS: Ubuntu Gutsy

Steps to reproduce:
- Run Gnumeric
- Where it says A1, type A100 and press enter
- Where it now says A100, type A1 and press enter to crash Gnumeric


Backtrace:
Program received signal SIGSEGV, Segmentation fault.

+ Trace 179683

Thread NaN (LWP 5573)

  • #0 scg_make_cell_visible
    at sheet-control-gui.c line 1028
  • #1 scg_make_cell_visible_virt
    at sheet-control-gui.c line 1056
  • #2 sc_make_cell_visible
    at sheet-control.c line 110
  • #3 sv_make_cell_visible
    at sheet-view.c line 354
  • #4 wb_control_jump
    at workbook-control.c line 250
  • #5 wb_control_parse_and_jump
    at workbook-control.c line 297
  • #6 cb_statusbox_activate
    at wbc-gtk.c line 1454
  • #7 g_cclosure_marshal_VOID__VOID
    from /usr/lib/libgobject-2.0.so.0
  • #8 g_closure_invoke
    from /usr/lib/libgobject-2.0.so.0
  • #9 ??
    from /usr/lib/libgobject-2.0.so.0 






Comment 1


Morten Welinder





          2007-11-26 15:10:42 UTC
        



Created attachment 99664 [details] [review]
Patch






Comment 2


Morten Welinder





          2007-11-26 15:11:18 UTC
        



Needs review.

(I am already tired of this piece of code.)







Comment 3


Andreas J. Guelzow





          2007-11-26 21:42:41 UTC
        



Looks good to me. (But it makes me think that the position handling in general seems (unnecessarily?) convoluted.)






Comment 4


Morten Welinder





          2007-11-27 00:22:35 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.


The sheer number of bugs in this little unimportant corner of Gnumeric
strongly suggests you are right re. convolution.