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 348993 - cd_cache_new_hal_ctx needs to call libhal_ctx_init
cd_cache_new_hal_ctx needs to call libhal_ctx_init
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: general
1.5.x
Other All
: High critical
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
: 349123 350118 351177 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-07-27 21:25 UTC by Tomasz Sterna
Modified: 2006-08-15 01:03 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
totem patch (764 bytes, patch)
2006-08-13 22:25 UTC, Jonathan Matthew
none Details | Review

Description Tomasz Sterna 2006-07-27 21:25:44 UTC
What were you doing when the application crashed?
I launched it.


Distribution: Gentoo Base System version 1.12.1
Gnome Release: 2.15.90 2006-07-26 (Gentoo)
BugBuddy Version: 2.15.90

Memory status: size: 172609536 vsize: 0 resident: 172609536 share: 0 rss: 22323200 rss_rlim: 0
CPU usage: start_time: 1154035491 rtime: 0 utime: 122 stime: 0 cutime:116 cstime: 0 timeout: 6 it_real_value: 0 frequency: 8

Backtrace was generated from '/usr/bin/rhythmbox'

(no debugging symbols found)
Using host libthread_db library "/lib/libthread_db.so.1".
(no debugging symbols found)
[Thread debugging using libthread_db enabled]
[New Thread -1210538320 (LWP 16070)]
0xffffe410 in __kernel_vsyscall ()

Thread 1 (Thread -1210538320 (LWP 16070))

  • #0 __kernel_vsyscall
  • #1 __waitpid_nocancel
    from /lib/libpthread.so.0
  • #2 gnome_gtk_module_info_get
    from /usr/lib/libgnomeui-2.so.0
  • #3 <signal handler called>
  • #4 dbus_connection_remove_filter
    from /usr/lib/libdbus-1.so.2
  • #5 libhal_ctx_shutdown
    from /usr/lib/libhal.so.1
  • #6 totem_cd_get_human_readable_name
    from /usr/lib/libtotem-plparser.so.1
  • #7 totem_cd_detect_type_with_url
    from /usr/lib/libtotem-plparser.so.1
  • #8 totem_cd_detect_type
    from /usr/lib/libtotem-plparser.so.1
  • #9 rb_audiocd_is_volume_audiocd
  • #10 rb_removable_media_manager_queue_transfer
  • #11 rb_removable_media_manager_scan
  • #12 rb_shell_new
  • #13 g_object_newv
    from /usr/lib/libgobject-2.0.so.0
  • #14 g_object_new_valist
    from /usr/lib/libgobject-2.0.so.0
  • #15 g_object_new
    from /usr/lib/libgobject-2.0.so.0
  • #16 rb_shell_new
  • #17 main
  • #0 __kernel_vsyscall

Comment 1 Karsten Bräckelmann 2006-07-28 20:38:14 UTC
*** Bug 349123 has been marked as a duplicate of this bug. ***
Comment 2 Jonathan Matthew 2006-07-30 08:53:36 UTC
What versions of totem, dbus, and hal are you using?
Comment 3 Tomasz Sterna 2006-07-30 09:03:47 UTC
*  media-video/totem
      Latest version available: 1.5.90-r2
      Latest version installed: 1.5.90-r2

*  sys-apps/dbus
      Latest version available: 0.62
      Latest version installed: 0.62

*  sys-apps/hal
      Latest version available: 0.5.7-r3
      Latest version installed: 0.5.7-r3
Comment 4 Tomasz Sterna 2006-07-30 09:07:19 UTC
I've noticed the problem occurs when there is a CD in a drive.
Rhythmbox crashes when I launch it with CD mounted. And with ejected CD, Rhythmbox crashes at the moment when I insert a CD.
Comment 5 Jonathan Matthew 2006-07-30 09:17:28 UTC
any messages printed to standard out before the crash?
Comment 6 Tomasz Sterna 2006-07-30 09:31:15 UTC
smoku@wing ~ $ LANG=POSIX rhythmbox

** (bug-buddy:9781): WARNING **: Couldn't load icon for Dziennik systemowy

** (bug-buddy:9781): WARNING **: Couldn't load icon for Open Folder

** (bug-buddy:9781): WARNING **: Couldn't load icon for Przeglądarka komponentów Bonobo
glibtop: This machine has 1 CPUs, 1 are being monitored.
smoku@wing ~ $ 
Comment 7 Tomasz Sterna 2006-07-30 09:52:49 UTC
smoku@wing ~ $ LANG=POSIX rhythmbox
NameOwnerChanged org.gnome.Rhythmbox ->:1.68
NameOwnerChanged :1.69 ->:1.69
NameOwnerChanged org.freedesktop.Notifications ->:1.69
NameOwnerChanged org.freedesktop.Notifications :1.69->
NameOwnerChanged :1.69 :1.69->


>>> NOW I INSERTED A CD... IT WAS MOUNTED AND: <<<


** (bug-buddy:10606): WARNING **: Couldn't load icon for Dziennik systemowy

** (bug-buddy:10606): WARNING **: Couldn't load icon for Open Folder

** (bug-buddy:10606): WARNING **: Couldn't load icon for Przeglądarka komponentów Bonobo
glibtop: This machine has 1 CPUs, 1 are being monitored.
smoku@wing ~ $ 
Comment 8 Jonathan Matthew 2006-08-13 21:35:41 UTC
*** Bug 351177 has been marked as a duplicate of this bug. ***
Comment 9 Jonathan Matthew 2006-08-13 21:42:17 UTC
Does gentoo build dbus with DBUS_DISABLE_CHECKS by default?  I just get a message saying 'Attempt to remove filter function 0xb7b9c8c0 user data 0x95bac30, but no such filter has been added' when I insert a CD.
Comment 10 Jonathan Matthew 2006-08-13 22:25:55 UTC
Created attachment 70835 [details] [review]
totem patch

totem-disc needs to call libhal_ctx_init(), and probably shouldn't call libhal_ctx_shutdown() when hal context initialization fails, since it only undoes work done by libhal_ctx_init().  This fixes the 'attempt to remove filter..' messages for me, and it should fix the crash.  Please test.
Comment 11 Tomasz Sterna 2006-08-13 22:43:02 UTC
Yes. This patch fixed the crash.
Many thanks. :-)
Comment 12 Jonathan Matthew 2006-08-13 22:50:53 UTC
OK, reassigning to totem.  Thanks for testing it.
Comment 13 Bastien Nocera 2006-08-14 06:47:15 UTC
bug 350118 is the same one.
Comment 14 Bastien Nocera 2006-08-14 06:57:40 UTC
Thanks for the patch Jonathan.

2006-08-14  Bastien Nocera  <hadess@hadess.net>

        * src/plparse/totem-disc.c: (cd_cache_new_hal_ctx):
        Patch from Jonathan Matthew <jonathan@kaolin.wh9.net> to avoid
        crashing when using HAL to detect a disc type (Closes: #348993)
Comment 15 Bastien Nocera 2006-08-15 01:03:47 UTC
*** Bug 350118 has been marked as a duplicate of this bug. ***