Ubuntu
audacity package

audacity crashed with SIGSEGV in DevicePrefs::GetDefaultPlayDevice()

Bug #436990 reported by Simon Déziel
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
audacity (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: audacity

I just changed from ALSA to OSS. This option is in Edit->Preferences->Devices.

ProblemType: Crash
Architecture: amd64
Date: Fri Sep 25 21:22:07 2009
DistroRelease: Ubuntu 9.10
ExecutablePath: /usr/bin/audacity
Package: audacity 1.3.9-3
ProcCmdline: audacity
ProcEnviron:
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-10.35-generic
SegvAnalysis:
 Segfault happened at: 0x6c17fa <_ZN11DevicePrefs20GetDefaultPlayDeviceEi+26>: mov 0x8(%rax),%rsi
 PC (0x006c17fa) ok
 source "0x8(%rax)" (0x00000008) not located in a known VMA region (needed readable region)!
 destination "%rsi" ok
SegvReason: reading NULL VMA
Signal: 11
SourcePackage: audacity
StacktraceTop:
 DevicePrefs::GetDefaultPlayDevice(int) ()
 DevicePrefs::OnHost(wxCommandEvent&) ()
 wxEvtHandler::ProcessEventIfMatches(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&) () from /usr/lib/libwx_baseu-2.8.so.0
 wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*) () from /usr/lib/libwx_baseu-2.8.so.0
 wxEvtHandler::ProcessEvent(wxEvent&) ()
Title: audacity crashed with SIGSEGV in DevicePrefs::GetDefaultPlayDevice()
Uname: Linux 2.6.31-10-generic x86_64
UserGroups:

Revision history for this message
Simon Déziel (sdeziel) wrote :
visibility: private → public
Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt (retraced)

StacktraceTop:DevicePrefs::GetDefaultPlayDevice (

Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt (retraced)
Changed in audacity (Ubuntu):
importance: Undecided → Medium
tags: removed: need-amd64-retrace
Revision history for this message
David Henningsson (diwic) wrote :

In DevicePrefs::GetDefaultPlayDevice:

   const PaDeviceInfo* devinfo = Pa_GetDeviceInfo(apiinfo->defaultOutputDevice);
   wxString name(devinfo->name, wxConvLocal);

Pa_GetDeviceInfo returns null, so devinfo->name crashes. Devinfo can be null if the apiinfo does not have a default output device. So a simple fix to avoid a crash could be to return an empty wxString in this case.

The background cause for this error could be that DevicePrefs::GetNamesAndLabels() only adds names and labels if there are devices. Could this lead to a mismatch between mHost->GetCurrentSelection() and portaudio hostapi indices?

Changed in audacity (Ubuntu):
assignee: nobody → David Henningsson (diwic)
status: New → In Progress
Revision history for this message
David Henningsson (diwic) wrote :
Revision history for this message
David Henningsson (diwic) wrote :

Hi Benjamin, since you've done some work with audacity lately I thought you might be interested in sponsoring this upload and make sure it makes its way upstream. If not, just unassign yourself. Thanks! :-)

Changed in audacity (Ubuntu):
assignee: David Henningsson (diwic) → Benjamin Drung (bdrung)
status: In Progress → Confirmed
Revision history for this message
Benjamin Drung (bdrung) wrote :

Thanks for your contribution. Can you please add some meta-data to the patch according to DEP-3 [1] and then reassing me. I will sponsor it and send the patch upstream.

http://dep.debian.net/deps/dep3/

Changed in audacity (Ubuntu):
assignee: Benjamin Drung (bdrung) → David Henningsson (diwic)
Revision history for this message
David Henningsson (diwic) wrote :
Changed in audacity (Ubuntu):
assignee: David Henningsson (diwic) → Benjamin Drung (bdrung)
Revision history for this message
Benjamin Drung (bdrung) wrote :

Thanks. Uploaded audacity 1.3.9-4 to Debian unstable.

Changed in audacity (Ubuntu):
assignee: Benjamin Drung (bdrung) → nobody
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package audacity - 1.3.9-4

---------------
audacity (1.3.9-4) unstable; urgency=low

  * Add debian/patches/switch-hostapi-crash.patch to prevent crash when
    selecting sound card driver; thanks to David Henningsson for the patch
    (LP: #436990).

 -- Benjamin Drung <email address hidden> Mon, 28 Sep 2009 12:50:16 +0100

Changed in audacity (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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