rhythmbox crashed with SIGSEGV in save_playlist()

Bug #567399 reported by Peter Schaefer
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rhythmbox
Fix Released
Undecided
Unassigned
rhythmbox (Ubuntu)
Fix Released
Medium
Sebastien Bacher
Lucid
Fix Released
Medium
Sebastien Bacher

Bug Description

Binary package hint: rhythmbox

Steps to reproduce:

1) Insert a SD-memory card containing the magic file ".is_audio_player" into the pc's card reader.

2) Wait till the removable device comes up in rhythmbox.

3) Create a playlist on the device.

4) Drag-and-drop an entry from a playlist residing already on the computer onto the newly created playlist.

5) Crash

ProblemType: Crash
DistroRelease: Ubuntu 10.04
Package: rhythmbox 0.12.8-0ubuntu3
ProcVersionSignature: Ubuntu 2.6.32-21.32-generic 2.6.32.11+drm33.2
Uname: Linux 2.6.32-21-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Tue Apr 20 20:08:12 2010
ExecutablePath: /usr/bin/rhythmbox
ProcCmdline: rhythmbox
ProcEnviron:
 PATH=(custom, user)
 LANG=de_DE.UTF-8
 SHELL=/bin/bash
SegvAnalysis:
 Segfault happened at: 0x7fc963f9dd68 <save_playlist+504>: mov 0x8(%rax),%rcx
 PC (0x7fc963f9dd68) ok
 source "0x8(%rax)" (0x00000008) not located in a known VMA region (needed readable region)!
 destination "%rcx" ok
SegvReason: reading NULL VMA
Signal: 11
SourcePackage: rhythmbox
StacktraceTop:
 save_playlist (source=<value optimized out>) at rb-generic-player-playlist-source.c:276
 ?? () from /lib/libglib-2.0.so.0
 g_main_context_dispatch () from /lib/libglib-2.0.so.0
 ?? () from /lib/libglib-2.0.so.0
 g_main_loop_run () from /lib/libglib-2.0.so.0
Title: rhythmbox crashed with SIGSEGV in save_playlist()
UserGroups: adm admin audio cdrom dialout dip fax floppy fuse libvirtd lpadmin plugdev sambashare tape vboxusers video

Revision history for this message
Peter Schaefer (derschaefer) wrote :
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for your bug report. The issue is an upstream one and it would be nice if somebody having it could send the bug the to the people writting the software (https://wiki.ubuntu.com/Bugs/Upstream/GNOME)

Changed in rhythmbox (Ubuntu):
importance: Undecided → Medium
Angel Abad (angelabad)
Changed in rhythmbox:
importance: Undecided → Unknown
status: New → Unknown
Changed in rhythmbox (Ubuntu):
status: New → Triaged
Revision history for this message
Sebastien Bacher (seb128) wrote :

the crash has been fixed upstream now

Changed in rhythmbox (Ubuntu):
status: Triaged → Fix Committed
Changed in rhythmbox:
status: Unknown → New
Changed in rhythmbox (Ubuntu Lucid):
assignee: nobody → Sebastien Bacher (seb128)
Revision history for this message
Sebastien Bacher (seb128) wrote :

SRU team, the debdiff is on bug #568887

Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted rhythmbox into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Revision history for this message
Peter Schaefer (derschaefer) wrote :

Just tested rhythmbox from lucid-proposed. Well, rhythmbox still doesn't honour my "playlist_format=audio/x-mpegurl" line in the .is_audio_player file and keeps creating *.pls instead *.m3u playlists, but:

The crasher is gone. Verified fixed.

However, dropping files directly from playlists or the music library into a playlist on removable media does nothing. One must first drop the file onto the removable media and from there on the playlist on the removable media. Seems that the file must reside on the same media as the playlist.

Alas, thanks guys.

Martin Pitt (pitti)
tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package rhythmbox - 0.12.8-0ubuntu4

---------------
rhythmbox (0.12.8-0ubuntu4) lucid-proposed; urgency=low

  * debian/patches/92_git_mc5_status.patch:
    - upstream change to correctly update the im status (lp: #548851)
  * debian/patches/93_git_no_empty_playlist_crasher.patch:
    - upstream change to not crash on empty playlist saving (lp: #567399)
  * debian/patches/94_git_escaping_crasher.patch:
    - upstream change to fix a crash on copies (lp: #565517)
  * debian/patches/95_git_correct_dbversion.patch:
    - upstream change to db version detection issues in some locates leading
      to trying to convert datas on every run (lp: #568887)
  * debian/patches/96_git_magnatune_genre.patch:
    - upstream change to set correct magnatune genre names (lp: #537958)
 -- Sebastien Bacher <email address hidden> Wed, 28 Apr 2010 16:27:51 +0200

Changed in rhythmbox (Ubuntu Lucid):
status: Fix Committed → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Copied lucid-proposed to maverick.

Changed in rhythmbox (Ubuntu):
status: Fix Committed → Fix Released
Changed in rhythmbox (Ubuntu Lucid):
status: Fix Released → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package rhythmbox - 0.12.8-0ubuntu4

---------------
rhythmbox (0.12.8-0ubuntu4) lucid-proposed; urgency=low

  * debian/patches/92_git_mc5_status.patch:
    - upstream change to correctly update the im status (lp: #548851)
  * debian/patches/93_git_no_empty_playlist_crasher.patch:
    - upstream change to not crash on empty playlist saving (lp: #567399)
  * debian/patches/94_git_escaping_crasher.patch:
    - upstream change to fix a crash on copies (lp: #565517)
  * debian/patches/95_git_correct_dbversion.patch:
    - upstream change to db version detection issues in some locates leading
      to trying to convert datas on every run (lp: #568887)
  * debian/patches/96_git_magnatune_genre.patch:
    - upstream change to set correct magnatune genre names (lp: #537958)
 -- Sebastien Bacher <email address hidden> Wed, 28 Apr 2010 16:27:51 +0200

Changed in rhythmbox (Ubuntu Lucid):
status: Fix Committed → Fix Released
Revision history for this message
Angel Abad (angelabad) wrote :

From Gnome Bugzilla:

"The only time this can happen is if the playlist is empty. Or if there's
codepaths in GIO that don't return errors on failure, which is probably
unlikely.

Fixed in 09b2bae."

Changed in rhythmbox:
importance: Unknown → Undecided
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.