creating databases fails

Bug #63948 reported by Ashley Hooper
10
Affects Status Importance Assigned to Milestone
mpd (Debian)
Fix Released
Unknown
mpd (Ubuntu)
Fix Released
High
William Grant

Bug Description

Binary package hint: mpd

Since updating MPD this week to version Version: 0.12.0-0ubuntu1 in Edgy, I have been unable to create the MPD song database. When running 'sudo mpd --create-db' I get the following result:

mpd: /build/buildd/mpd-0.12.0/./src/utils.c:126: xmalloc: Assertion `__builtin_expect (!!(size), 1)' failed.

Revision history for this message
William Grant (wgrant) wrote :

In the changelog for Debian 0.12.0-2:

   * Add patch to fix xmalloc assertion with some MP3 files. This does not
     really fix #390417, as the files do not play, but mpd --create-db will no
     longer crash.

Syncing 0.12.0-3 will fix this problem.

Revision history for this message
William Grant (wgrant) wrote :

Please sync mpd (universe) 0.12.0-2 from Debian Sid (main). Overwriting Ubuntu changes is OK, as it was just a new upstream version, and the package builds fine in a clean Edgy pbuilder. Changelog entries follow:

mpd (0.12.0-3) unstable; urgency=low

  * Fix missing build-depends (Closes: #390779)

 -- Decklin Foster <email address hidden> Mon, 2 Oct 2006 21:32:27 -0400

mpd (0.12.0-2) unstable; urgency=low

  * Add patch to fix xmalloc assertion with some MP3 files. This does not
    really fix #390417, as the files do not play, but mpd --create-db will no
    longer crash.
  * When cleaning up after 0.11, check that the PID in the legacy pidfile
    is actually running before attempting to kill it.

 -- Decklin Foster <email address hidden> Mon, 2 Oct 2006 14:55:17 -0400

mpd (0.12.0-1) unstable; urgency=low

  * New upstream release (Closes: #280100)
    - Enable Ogg speex support (Closes: #341698)
    - Fixed ID3v2 support (Closes: #288101)
  * Removed debconf (Closes: #329887, #360113)
    - No more 'restart' setting, just restart in postinst (Closes: #369590)
  * Removed lintian override.

 -- Decklin Foster <email address hidden> Sat, 23 Sep 2006 14:18:36 -0400

Changed in mpd:
importance: Undecided → Low
status: Unconfirmed → Confirmed
Revision history for this message
William Grant (wgrant) wrote :

Sorry, the above should read `please sync mpd (universe) 0.12.0-3'.

Revision history for this message
Reinhard Tartler (siretart) wrote :

this seems rather critical to me, because it renders the package unusable

Changed in mpd:
importance: Low → Critical
Revision history for this message
Reinhard Tartler (siretart) wrote :

William, did you check the old ubuntu diff? You need to show that you verified that the old changes are not necessary anymore and can be dropped before requesting a sync.

If they are necessary, the need to be merged.

Revision history for this message
Reinhard Tartler (siretart) wrote :

lowering severity, because after reading the related debian bug, this seems to occur only on some mp3 files

Changed in mpd:
importance: Critical → High
Revision history for this message
William Grant (wgrant) wrote :

I uploaded the last version. It was simply the final 0.12.0 with the debian/ from one of the 0.12.0 RCs in Debian experimental. The Ubuntu delta is just the new upstream version, and is fine to override, I said that in the original sync request.

Changed in mpd:
status: Unknown → Fix Released
Revision history for this message
Reinhard Tartler (siretart) wrote :

okay, I just rechecked the diff to debian, its allright.

Archive team, please sync mpd_0.12.0-3 from debian/experimental into edgy

Revision history for this message
Colin Watson (cjwatson) wrote :

Debian unstable now has 0.12.1-1, with some important-looking fixes. Could you recheck, please?

Changed in mpd:
status: Confirmed → Needs Info
Revision history for this message
William Grant (wgrant) wrote :

0.12.1 fixes this issue, as well as a couple of others.

Changed in mpd:
assignee: nobody → motu-uvf
Revision history for this message
William Grant (wgrant) wrote :
Revision history for this message
William Grant (wgrant) wrote :
Revision history for this message
William Grant (wgrant) wrote :
Revision history for this message
Lionel Porcheron (lionel.porcheron) wrote :

It builds well but it does not run well: see #64293 (0.12.0 gets rid of Ubuntu changes).

The patch I submited for this bug also applies on the new Debian release and it runs well. This is not a sync, this should be a merge.

Revision history for this message
Lionel Porcheron (lionel.porcheron) wrote :

Here's a debdiff between the current Debian and new Ubuntu version.

Revision history for this message
William Grant (wgrant) wrote :

Implementing that patch is a good idea, however your statement that 0.12.0 got rid of Ubuntu changes is false. There have been no Ubuntu changes since Breezy, so it was just a plain sync.

Revision history for this message
Lionel Porcheron (lionel.porcheron) wrote :

Sorry William, you are right: in Dapper version, the PID file was in /var/run (not a subdirectory).

However, the patch is still needed.

Revision history for this message
Daniel Holbach (dholbach) wrote :

Looks like a very good idea.

Reinhard, Sebastian, Andrew?

Changed in mpd:
status: Needs Info → Unconfirmed
Revision history for this message
Qball Cow (qball-qballcow) wrote :

Please update to 0.12.1, it fixes f.e. this problem.

Revision history for this message
Reinhard Tartler (siretart) wrote :

Yes, I agree. Please proceed with uploading

Changed in mpd:
assignee: motu-uvf → fujitsu
status: Unconfirmed → Confirmed
Revision history for this message
William Grant (wgrant) wrote :

Changelog since current Ubuntu version 0.12.0-0ubuntu1:

mpd (0.12.1-1ubuntu1) edgy; urgency=low

  * Sync from Debian Sid
  * debian/mpd.init.d:
    - create /var/run/ directory if necessary and set proper permissions
      (Closes Ubuntu 64293)

 -- Lionel Porcheron <email address hidden> Sat, 14 Oct 2006 13:02:49 +0200

mpd (0.12.1-1) unstable; urgency=low

  * New upstream release
    - Fixes scanning of MP3s with a 0-frame Xing tag (Closes: #390417)

 -- Decklin Foster <email address hidden> Wed, 11 Oct 2006 15:22:46 -0400

mpd (0.12.0-3) unstable; urgency=low

  * Fix missing build-depends (Closes: #390779)

 -- Decklin Foster <email address hidden> Mon, 2 Oct 2006 21:32:27 -0400

mpd (0.12.0-2) unstable; urgency=low

  * Add patch to fix xmalloc assertion with some MP3 files. This does not
    really fix #390417, as the files do not play, but mpd --create-db will no
    longer crash.
  * When cleaning up after 0.11, check that the PID in the legacy pidfile
    is actually running before attempting to kill it.

 -- Decklin Foster <email address hidden> Mon, 2 Oct 2006 14:55:17 -0400

mpd (0.12.0-1) unstable; urgency=low

  * New upstream release (Closes: #280100)
    - Enable Ogg speex support (Closes: #341698)
    - Fixed ID3v2 support (Closes: #288101)
  * Removed debconf (Closes: #329887, #360113)
    - No more 'restart' setting, just restart in postinst (Closes: #369590)
  * Removed lintian override.

 -- Decklin Foster <email address hidden> Sat, 23 Sep 2006 14:18:36 -0400

Revision history for this message
Colin Watson (cjwatson) wrote :

This appears to have been uploaded.

       mpd | 0.12.1-1ubuntu1 | edgy/universe | source, amd64, i386, ia64, powerpc, sparc

Changed in mpd:
status: Confirmed → 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.