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 657185 - Rhythmbox crashes when attempting to extract a CD
Rhythmbox crashes when attempting to extract a CD
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: general
HEAD
Other Linux
: Normal major
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 657628 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-08-23 18:03 UTC by Adam Williamson
Modified: 2011-08-29 21:59 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
traceback of the crash (16.36 KB, text/plain)
2011-08-23 18:03 UTC, Adam Williamson 		  Details
try this (1.85 KB, patch)
2011-08-23 23:03 UTC, Jonathan Matthew 		none Details | Review

Description Adam Williamson 2011-08-23 18:03:32 UTC 
Created attachment 194521 [details]
traceback of the crash

With current git master on Fedora 16, RB reliably crashes immediately upon hitting 'Extract' for an audio CD. Tried with both the Vorbis and FLAC profiles. Note that the FLAC entry in the profile list appears blank - you can only tell it's the FLAC entry because when you click it, the example output filename extension changes to .flac.

Traceback attached.
Comment 1 Jonathan Matthew 2011-08-23 22:30:46 UTC 
Can you try adding some debug output to rb_gst_encoding_profile_get_media_type to see where it's failing?  It appears to be returning NULL when it clearly shouldn't be.

> Note that the FLAC entry in the profile list appears blank - you can
> only tell it's the FLAC entry because when you click it, the example output
> filename extension changes to .flac.

This was bug 652342.
Comment 2 Adam Williamson 2011-08-23 22:38:45 UTC 
"Can you try adding some debug output to rb_gst_encoding_profile_get_media_type
to see where it's failing?"

I don't code, so I would need a bit more hand-holding than that =) If you could send me a patch which adds whatever debug code you want added there, I can rebuild RB with the patch applied and run it.
Comment 3 Adam Williamson 2011-08-23 22:39:15 UTC 
actually, hold your horses, I have a suspicion...
Comment 4 Adam Williamson 2011-08-23 22:41:27 UTC 
nope, my suspicion didn't pan out, so go ahead with the debug code.
Comment 5 Jonathan Matthew 2011-08-23 23:03:26 UTC 
Created attachment 194537 [details] [review]
try this

run 'rhythmbox -D media-type'
Comment 6 Adam Williamson 2011-08-24 23:44:01 UTC 
(16:43:39) [0x210e3a0] [rb_gst_encoding_profile_get_media_type] rb-gst-media-types.c:238: profile: oggvorbis
(16:43:39) [0x210e3a0] [rb_gst_encoding_profile_get_media_type] rb-gst-media-types.c:240: is container
(16:43:39) [0x210e3a0] [rb_gst_encoding_profile_get_media_type] rb-gst-media-types.c:244: stream: (null)
(16:43:39) [0x210e3a0] [rb_gst_encoding_profile_get_media_type] rb-gst-media-types.c:246: is audio profile
(16:43:39) [0x210e3a0] [rb_gst_encoding_profile_get_media_type] rb-gst-media-types.c:238: profile: copy
(16:43:39) [0x210e3a0] [rb_gst_encoding_profile_get_media_type] rb-gst-media-types.c:255: not container
(16:43:39) [0x210e3a0] [rb_gst_caps_to_media_type] rb-gst-media-types.c:106: zero size caps?
Segmentation fault (core dumped)
Comment 7 Jonathan Matthew 2011-08-25 22:59:28 UTC 
fixed in commit 8ff760e
Comment 8 Jonathan Matthew 2011-08-29 21:59:17 UTC 
*** Bug 657628 has been marked as a duplicate of this bug. ***