Bug 306536 – warning when activating the rss extension

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 306536 - warning when activating the rss extension


Summary:	warning when activating the rss extension


Status:	RESOLVED FIXED

Product:	epiphany-extensions
Classification:	Deprecated
Component:	general
Version:	1.6.x
Hardware:	Other Linux

Importance:	Normal critical
Target Milestone:	---
Assigned To:	epiphany-extensions-maint
QA Contact:	Christian Persch

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2005-06-05 09:22 UTC by Christian Persch
Modified:	2005-06-05 13:11 UTC

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

Attachments
rss-fix.patch (1.22 KB, patch) 2005-06-05 12:51 UTC, Raphael Slinckx	none	Details \| Review

Description Christian Persch 2005-06-05 09:22:22 UTC

Steps to reproduce:
0) Tools-Extensions
1) Check "News Feed Subscription"

Result:
** (epiphany:30687): CRITICAL **: ephy_rss_update_action: assertion `data !=
NULL' failed

Program received signal SIGTRAP, Trace/breakpoint trap.

+ Trace 60562

Thread NaN (LWP 30687)

#0 trap_handler
at ../../lib/ephy-debug.c line 109
#1 g_logv
from /usr/lib/libglib-2.0.so.0
#2 g_log
from /usr/lib/libglib-2.0.so.0
#3 g_return_if_fail_warning
from /usr/lib/libglib-2.0.so.0
#4 ephy_rss_update_action
at ../../../extensions/rss/ephy-rss-extension.c line 272
#5 impl_attach_window
at ../../../extensions/rss/ephy-rss-extension.c line 448
#6 ephy_extension_attach_window
at ../../src/ephy-extension.c line 55
#7 attach_window
at ../../src/ephy-extensions-manager.c line 815
#8 g_list_foreach
from /usr/lib/libglib-2.0.so.0
#9 load_extension
at ../../src/ephy-extensions-manager.c line 860
#10 sync_loaded_extensions
at ../../src/ephy-extensions-manager.c line 987
#11 gconf_client_change_set_from_current
from /usr/lib/libgconf-2.so.4
#12 gconf_listeners_remove_if
from /usr/lib/libgconf-2.so.4
#13 gconf_listeners_remove_if
from /usr/lib/libgconf-2.so.4
#14 gconf_client_change_set_from_current
from /usr/lib/libgconf-2.so.4
#15 gconf_client_change_set_from_current
from /usr/lib/libgconf-2.so.4
#16 gconf_client_change_set_from_current
from /usr/lib/libgconf-2.so.4
#17 g_child_watch_add
from /usr/lib/libglib-2.0.so.0
#18 g_main_depth
from /usr/lib/libglib-2.0.so.0
#19 g_main_context_dispatch
from /usr/lib/libglib-2.0.so.0
#20 g_main_context_dispatch
from /usr/lib/libglib-2.0.so.0
#21 g_main_loop_run
from /usr/lib/libglib-2.0.so.0
#22 IA__gtk_main
at gtkmain.c line 963
#23 main
at ../../src/ephy-main.c line 313

Comment 1 Raphael Slinckx 2005-06-05 12:11:57 UTC

Ok, i don't understand why the line numbers don't match, but basically the
rss_sync_tab is called before we set the data on the window, which isn't a
problem except when you load the extension for the first time cause the window
is already realized ..

Testing and attaching the patch

Comment 2 Raphael Slinckx 2005-06-05 12:51:03 UTC

Created attachment 47269 [details] [review]
rss-fix.patch

This should fix the problem

Comment 3 Christian Persch 2005-06-05 13:11:41 UTC

Checked in with a further tweak, thanks for the patch!