Bug 132060 – Crash deleting plotline with labels, when there's another plotline

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 132060 - Crash deleting plotline with labels, when there's another plotline


Summary:	Crash deleting plotline with labels, when there's another plotline


Status:	RESOLVED FIXED

Product:	Gnumeric
Classification:	Applications
Component:	Charting
Version:	1.2.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	Jody Goldberg
QA Contact:	Jody Goldberg

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2004-01-21 04:31 UTC by Luke Hutchison
Modified:	2004-12-22 21:47 UTC

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

Description Luke Hutchison 2004-01-21 04:31:34 UTC

Create two plotlines in a graph.  Attach a label series to the first, but
not to the second.  Delete the first plotline (the one with the label
series).  Crash.  Presumably the second plotline is using the first
plotline's label series.

I've also had this spin and take 100% CPU rather than crash.

--

Backtrace was generated from '/home/lukeh/GARNOME/bin/gnumeric'

[New Thread 1092943808 (LWP 7963)]
0x4001544e in _dl_sysinfo_int80 () from /lib/ld-linux.so.2

+ Trace 43391

Thread 1 (Thread 1092943808 (LWP 7963))

#0 _dl_sysinfo_int80
from /lib/ld-linux.so.2
#1 waitpid
from /lib/tls/libpthread.so.0
#2 libgnomeui_segv_handle
at gnome-ui-init.c line 738
#3 <signal handler called>
#4 ??
#5 gog_axis_num_markers
at gog-axis.c line 906
#6 gog_axis_view_size_request
at gog-axis.c line 983
#7 gog_view_size_request
at gog-view.c line 419
#8 gog_chart_view_size_allocate
at gog-chart.c line 519
#9 gog_view_size_allocate
at gog-view.c line 446
#10 gog_graph_view_size_allocate
at gog-graph.c line 557
#11 gog_view_size_allocate
at gog-view.c line 446
#12 gog_view_update_sizes
at gog-view.c line 461
#13 gog_renderer_pixbuf_update
at gog-renderer-pixbuf.c line 681
#14 gog_control_foocanvas_update
at gog-control-foocanvas.c line 197
#15 foo_canvas_item_invoke_update
at foo-canvas.c line 433
#16 foo_canvas_group_update
at foo-canvas.c line 1383
#17 foo_canvas_item_invoke_update
at foo-canvas.c line 433
#18 do_update
at foo-canvas.c line 2916
#19 idle_handler
at foo-canvas.c line 2951
#20 g_idle_dispatch
at gmain.c line 3273
#21 g_main_dispatch
at gmain.c line 1752
#22 g_main_context_dispatch
at gmain.c line 2300
#23 g_main_context_iterate
at gmain.c line 2381
#24 g_main_loop_run
at gmain.c line 2601
#25 gtk_main
at gtkmain.c line 1124
#26 main
at main-application.c line 237
#27 __libc_start_main
from /lib/tls/libc.so.6
#0 _dl_sysinfo_int80
from /lib/ld-linux.so.2

Comment 1 Jody Goldberg 2004-01-26 15:52:38 UTC

What version is this for ?
It seems similar to 132059

Comment 2 Luke Hutchison 2004-01-26 18:10:08 UTC

v1.2.2.

Comment 3 Jody Goldberg 2004-01-27 15:20:57 UTC

Looks very similar.  Please give this a shot with 1.2.5 too

Comment 4 Luke Hutchison 2004-01-28 00:01:08 UTC

Yes, this seems related to bug 132059, since the change of behaviour
upon upgrade has been very similar.

Now I can't get the crash, but it might be due to the same
non-updating behaviour observed in 132059: Deleting the second
plotline, in this example, leaves the first plotline's axis labels on
the axis, but changing the contents of the Labels series does not
update the axis (i.e. the axis labels are not linked to the data
series).  Because it is not updating the graph, it may not be
accessing the memory that was nonexistent in the original segfault
problem.

Feel free to mark this as a dupe of 132059 if you think it is.

Also, in general, it would make more sense, IMHO, to have the Labels
series as an attribute of the X axis, not for each plotline, since
only one is displayed even if there is more than one series.

Comment 5 Jody Goldberg 2004-02-02 05:24:24 UTC

Should also be fixed in both trees