paman crashed with SIGFPE in pa_cvolume_avg()

Bug #352959 reported by Yann Simon
20
This bug affects 1 person
Affects Status Importance Assigned to Milestone
paman (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: paman

Steps to reproduce the crash:

1. Open "Pulseaudio Manager"
2. switch to tab "Sample Cache"
3. Select the first ".ogg" in the list
4. Click "play"
5. Click "Properties"

ProblemType: Crash
Architecture: i386
CrashCounter: 1
DistroRelease: Ubuntu 9.04
ExecutablePath: /usr/bin/paman
Package: paman 0.9.4-1ubuntu1
ProcCmdline: paman
ProcEnviron:
 SHELL=/bin/bash
 LANG=en_US.UTF-8
Signal: 8
SourcePackage: paman
StacktraceTop:
 ?? () from /usr/lib/libpulse.so.0
 pa_cvolume_avg () from /usr/lib/libpulse.so.0
 SampleWindow::updateInfo (this=0x8de7ad8, i=@0x8514070)
 SampleInfo::showWindow (this=0x8514070)
 ServerInfoManager::showSampleWindow (this=0x82b39c0,
Title: paman crashed with SIGFPE in pa_cvolume_avg()
Uname: Linux 2.6.29-020629-generic i686
UserGroups: adm admin cdrom dialout lpadmin plugdev sambashare

Revision history for this message
Yann Simon (yann-simon-fr) wrote :
Revision history for this message
Apport retracing service (apport) wrote : Symbolic stack trace

StacktraceTop:?? () from /usr/lib/libpulse.so.0
pa_cvolume_avg () from /usr/lib/libpulse.so.0
SampleWindow::updateInfo (this=0x8de7ad8, i=@0x8514070)
SampleInfo::showWindow (this=0x8514070)
ServerInfoManager::showSampleWindow (this=0x82b39c0,

Revision history for this message
Apport retracing service (apport) wrote : Symbolic threaded stack trace
Revision history for this message
Apport retracing service (apport) wrote : Stack trace with source code
visibility: private → public
Revision history for this message
Ryan (ubuntu-draziw) wrote :

Had the same issue
$ paman
** Message: Starting in record mode.
** Message: Using device 'alsa_output.pci_8086_27d8_sound_card_0_alsa_playback_0.monitor'
** Message: Using sample format: float32le 2ch 44100Hz
** Message: Using channel map: front-left,front-right
Floating point exception (core dumped)
ryan@lt:~$
(pavumeter:32483): Gtk-CRITICAL **: gtk_main_quit: assertion `main_loops != NULL' failed

Revision history for this message
Yann Simon (yann-simon-fr) wrote :

The error comes probably from the following line:

    snprintf(t, sizeof(t), "%0.0f%%", (double) pa_sw_volume_to_linear(pa_cvolume_avg(&i.volume)) * 100);

In SampleWindow.cc
void SampleWindow::updateInfo(const SampleInfo &i) {

Revision history for this message
Yann Simon (yann-simon-fr) wrote :

The problem comes from lib-pulse0
pa_cvolume_avg()

affects: paman (Ubuntu) → pulseaudio (Ubuntu)
Revision history for this message
Yann Simon (yann-simon-fr) wrote :
Revision history for this message
Yann Simon (yann-simon-fr) wrote :

paman did not check for numbers of channel

affects: pulseaudio (Ubuntu) → paman (Ubuntu)
Revision history for this message
C de-Avillez (hggdh2) wrote :

Thank you for both the bug and the patch. I have, nevertheless, a few questions:

(1) did you test the patch? If so, did it work?
(2) Have you checked upstream if this has already been fixed on the current head (although it looks that upstream has been inactive for a while...)?
(3) have you reported it upstream? If so, can you link the report here?

Thank you.

Changed in paman (Ubuntu):
importance: Undecided → Medium
status: New → Incomplete
Revision history for this message
C de-Avillez (hggdh2) wrote :

well, I went ahead and tested it. I can confirm the fix.

Debdiff attached, probably good also for Intrepid.

Changed in paman (Ubuntu):
status: Incomplete → Confirmed
Revision history for this message
C de-Avillez (hggdh2) wrote :
Revision history for this message
Yann Simon (yann-simon-fr) wrote :

1. I tested the patch, but I could only test the case when the number of channels = 0. I think that there is antoher problem with an used library because we cannot get any information from the sound files.

2. I compared the ubuntu's version and the upstream one, and I did not find any differences.

3. I did not find any tracking tool. I send an email to the author of the project.

Thank you for the debdiff

Revision history for this message
C de-Avillez (hggdh2) wrote :

I also tried to find a BTS for paman, but the freshmeat site http://freshmeat.net/projects/paman does not provide a BTS (the "report a bug" link there is to report issues with freshmeat, *not* the project :-(

And, you are welcome. Thank YOU for your work here.

Revision history for this message
Daniel T Chen (crimsun) wrote :

Lennart advises us to use pa_cvolume_valid() instead, so I've made that change.

Changed in paman (Ubuntu):
assignee: nobody → crimsun
status: Confirmed → In Progress
Revision history for this message
Daniel T Chen (crimsun) wrote :
Changed in paman (Ubuntu):
assignee: crimsun → nhandler
Revision history for this message
Nathan Handler (nhandler) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Fri, 03 Apr 2009 17:58:11 -0400
Source: paman
Binary: paman
Architecture: source
Version: 0.9.4-1ubuntu2
Distribution: jaunty
Urgency: low
Maintainer: Ubuntu MOTU Developers <email address hidden>
Changed-By: Daniel T Chen <email address hidden>
Description:
 paman - PulseAudio Manager
Launchpad-Bugs-Fixed: 352959
Changes:
 paman (0.9.4-1ubuntu2) jaunty; urgency=low
 .
   * Add 0002-Fix-sigfpe-with-invalid-number-of-channels.patch based on a
     patch by Yann Simon (LP: #352959)
Checksums-Sha1:
 775e22b4838c17a75f78543ed444ba574331e566 1352 paman_0.9.4-1ubuntu2.dsc
 87982dc874a19f5bdfd8588c041d232c34a043cf 3969 paman_0.9.4-1ubuntu2.diff.gz
Checksums-Sha256:
 f80ffe271215988e598d5a362cd4e91ebba2fa37e481025cc678d83ebaff3508 1352 paman_0.9.4-1ubuntu2.dsc
 bc20964b14310118a572be5c6f8d149842799567e1df01872809d132e1e02585 3969 paman_0.9.4-1ubuntu2.diff.gz
Files:
 3dc67e5bd887ab051631988a0400dd33 1352 sound optional paman_0.9.4-1ubuntu2.dsc
 559e79dba65c86b536189a2d8b38d718 3969 sound optional paman_0.9.4-1ubuntu2.diff.gz
Original-Maintainer: CJ van den Berg <email address hidden>

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAknWjCQACgkQS7NiVFf3leiVbwCfdgtpDQ8DnLdjFTiFJgA6MjGY
o4QAnjm+aUYQrCQkr9eU4dODpO/M/FJm
=r9w5
-----END PGP SIGNATURE-----

Changed in paman (Ubuntu):
assignee: nhandler → nobody
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package paman - 0.9.4-1ubuntu2

---------------
paman (0.9.4-1ubuntu2) jaunty; urgency=low

  * Add 0002-Fix-sigfpe-with-invalid-number-of-channels.patch based on a
    patch by Yann Simon (LP: #352959)

 -- Daniel T Chen <email address hidden> Fri, 03 Apr 2009 17:58:11 -0400

Changed in paman (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Daniel T Chen (crimsun) wrote :

Merged upstream in c12cf2bfa6497e742cc52f2d71cb8e2f2f52050d (http://git.0pointer.de/?p=paman.git;a=commitdiff;h=c12cf2bfa6497e742cc52f2d71cb8e2f2f52050d)

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.