jack open file descriptor leak in alsa audioadapter

Bug #1014839 reported by ruff
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
jackit
Fix Released
Unknown
jackd2 (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

See http://trac.jackaudio.org/ticket/286

In linux/alsa/JackAlsaAdapter.h file in *info methods of AudioInterface? class control device is opened twice on each load but never closed.

Result is that with each unload/load number of opened FDs grows until full jackd/jackdbus restart.

Attached patch fixes that.

Revision history for this message
ruff (rufferson) wrote :

the patch applies as to svn so to upstream src deb

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Changed in jackit:
status: Unknown → New
Gary M (garym)
tags: added: patch-forwarded-upstream
ruff (rufferson)
description: updated
Changed in jackit:
status: New → Fix Released
Gary M (garym)
affects: jack-audio-connection-kit (Ubuntu) → jackd2 (Ubuntu)
summary: - jack-audio-connectio-kit open file descriptor leak in alsa audioadapter
+ jack open file descriptor leak in alsa audioadapter
Gary M (garym)
Changed in jackd2 (Ubuntu):
status: New → Confirmed
Gary M (garym)
tags: added: patch-accepted-upstream
removed: patch-forwarded-upstream
Revision history for this message
David Henningsson (diwic) wrote :

I downloaded the current jackd2 source package (for 13.04) and verified that your patch was included there. Thanks for your contribution!

Changed in jackd2 (Ubuntu):
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

Patches

Remote bug watches

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