Bug 547766 – Crash in gpm_cell_array_update gpm-cell-array.c:273

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 547766 - Crash in gpm_cell_array_update gpm-cell-array.c:273


Summary:	Crash in gpm_cell_array_update gpm-cell-array.c:273


Status:	RESOLVED FIXED

Product:	gnome-power-manager
Classification:	Deprecated
Component:	general
Version:	2.23.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	GNOME Power Manager Maintainer(s)
QA Contact:	GNOME Power Manager Maintainer(s)

URL:
Whiteboard:

Duplicates:	549772 (view as bug list)
Depends on:
Blocks:

Reported:	2008-08-14 13:19 UTC by Priit Laes (IRC: plaes)
Modified:	2008-09-02 04:21 UTC

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

Description Priit Laes (IRC: plaes) 2008-08-14 13:19:16 UTC

I'm getting following crash in 2.23.6

Program received signal SIGSEGV, Segmentation fault.

+ Trace 205147

Thread 140288982226768 (LWP 32139)

#0 gpm_cell_array_update
at gpm-cell-array.c line 273
#1 gpm_cell_percent_changed_cb
at gpm-cell-array.c line 597
#2 IA__g_closure_invoke
at gclosure.c line 767
#3 signal_emit_unlocked_R
at gsignal.c line 3244
#4 IA__g_signal_emit_valist
at gsignal.c line 2977
#5 IA__g_signal_emit
at gsignal.c line 3034
#6 hal_device_property_modified_cb
at gpm-cell.c line 326
#7 IA__g_closure_invoke
at gclosure.c line 767
#8 signal_emit_unlocked_R
at gsignal.c line 3244
#9 IA__g_signal_emit_valist
at gsignal.c line 2977
#10 IA__g_signal_emit
at gsignal.c line 3034
#11 watch_device_properties_modified_cb
at libhal-gdevice.c line 311
#12 marshal_dbus_message_to_g_marshaller
at dbus-gproxy.c line 1638
#13 IA__g_closure_invoke
at gclosure.c line 767
#14 signal_emit_unlocked_R
at gsignal.c line 3244
#15 IA__g_signal_emit_valist
at gsignal.c line 2977
#16 IA__g_signal_emit
at gsignal.c line 3034
#17 dbus_g_proxy_manager_filter
at dbus-gproxy.c line 1691
#18 dbus_connection_dispatch
at dbus-connection.c line 4406
#19 message_queue_dispatch
at dbus-gmain.c line 101
#20 IA__g_main_context_dispatch
at gmain.c line 2072
#21 g_main_context_iterate
at gmain.c line 2705
#22 IA__g_main_loop_run
at gmain.c line 2928
#23 main
at gpm-main.c line 254

Comment 1 David Zeuthen (not reading bugmail) 2008-08-14 15:35:42 UTC

I'm also seeing this on Rawhide

Comment 2 Richard Hughes 2008-08-14 19:32:28 UTC

2008-08-14  Richard Hughes  <richard@hughsie.com>

	* src/gpm-cell-array.c: (gpm_cell_percent_changed_cb):
	The callback passes a gfloat, not a guint. Fixes #547766

Building for rawhide now.

Comment 3 David Zeuthen (not reading bugmail) 2008-08-14 20:38:21 UTC

(In reply to comment #2)
> Building for rawhide now.

Thanks for fixing this; however JFYI the rawhide build fails

 1 out of 1 hunk FAILED -- saving rejects to file src/gpm-cell-array.c.rej

Comment 4 David Zeuthen (not reading bugmail) 2008-08-14 21:28:20 UTC

I've fixed the rawhide build for you (and, wooh, also verified that the patch works!)

Comment 5 Christopher Halse Rogers 2008-09-02 04:21:54 UTC

*** Bug 549772 has been marked as a duplicate of this bug. ***