304629 – media-libs/alsa-libs leaks FDs

Bug 304629 - media-libs/alsa-libs leaks FDs

Summary: media-libs/alsa-libs leaks FDs

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Gentoo ALSA team

URL:	https://bugs.launchpad.net/ubuntu/+so...
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2010-02-11 20:59 UTC by James Brown
Modified:	2010-02-12 16:10 UTC (History)
CC List:	0 users

See Also:
Package list:
Runtime testing required:	---

Attachments
Patch from LaunchPad (dont_leak_timer_fd_on_pcm_slave_close.patch,1.60 KB, patch) 2010-02-11 21:00 UTC, James Brown	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description James Brown 2010-02-11 20:59:43 UTC

alsa-lib has a bug (fixed in the latest HEAD, apparently) which causes it to leak FDs sometimes. This leads to PulseAudio (and possibly other apps?) hanging. The Ubuntu folks at the referenced URL debugged it and backported the patch from HEAD. Their patch applies cleanly against our 1.0.22 ebuild and seems to fix the problem. Any chance we could push their patch into portage?

Reproducible: Always

Comment 1 James Brown 2010-02-11 21:00:29 UTC

Created attachment 219279 [details, diff]
Patch from LaunchPad

Attaching patch from Ubuntu folks

Comment 2 Tony Vroon (RETIRED) gentoo-dev

2010-02-12 16:10:04 UTC

+*alsa-lib-1.0.22-r1 (12 Feb 2010)
+
+  12 Feb 2010; <chainsaw@gentoo.org> +alsa-lib-1.0.22-r1.ebuild,
+  +files/alsa-lib-1.0.22-fd-leak.patch:
+  Apply patch from Daniel T Chen (Ubuntu) to avoid a file-descriptor leak.
+  It can be triggered by PulseAudio. With thanks to James Brown for
+  identifying the required patch. Closes bug #304629.