| 2008-01-16 03:05:50 |
desrt |
bug |
|
|
added bug |
| 2008-01-16 03:12:54 |
desrt |
bug |
|
|
added subscriber Martin Pitt |
| 2008-01-16 03:13:25 |
desrt |
bug |
|
|
added attachment 'esd.debdiff' (fix the bug!!!!!!) |
| 2008-01-16 06:05:08 |
Launchpad Janitor |
esound: status |
New |
Fix Released |
|
| 2008-01-18 08:31:55 |
Martin Pitt |
bug |
|
|
added subscriber Ubuntu Stable Release Updates Team |
| 2008-01-18 08:32:02 |
Martin Pitt |
bug |
|
|
added subscriber SRU Verification |
| 2008-01-18 08:32:36 |
Martin Pitt |
esound: status |
New |
Fix Committed |
|
| 2008-01-18 08:32:36 |
Martin Pitt |
esound: assignee |
|
desrt |
|
| 2008-01-18 08:44:37 |
desrt |
description |
my sister has been using her shiny new ubuntu system
i'm not sure what she does with it exactly, but after a while she gets "too many open files!" messages from nautilus
i tracked the problem down. pitti made an ubuntu vendor patch to esound to have it check for non-existent /usr/bin/esd and bypass a bunch of spawning code if it's not there.
unfortunately he also had it bypass the code to close the pipe file descriptors that had already been opened by this point.....
it's a pretty simple fix. will attach. |
my sister has been using her shiny new ubuntu system
i'm not sure what she does with it exactly, but after a while she gets "too many open files!" messages from nautilus
i tracked the problem down. pitti made an ubuntu vendor patch to esound to have it check for non-existent /usr/bin/esd and bypass a bunch of spawning code if it's not there.
unfortunately he also had it bypass the code to close the pipe file descriptors that had already been opened by this point.....
it's a pretty simple fix. will attach.
TEST CASE:
1) ensure that "Enable software sound mixing (ESD)" checkbox is enabled in "Sound Preferences" (this is the default)
2) ensure that the 'esound' package is not installed (this is also the default)
3) ensure "preview sound files" is enabled in nautilus preferences (also the default)
3) open a terminal
4) $ cd /proc/`pidof nautilus`/fd
5) 'ls' and notice a small number of 'pipe' entries
6) open a folder with a bunch of ogg vorbis files in it and move the mouse around a lot, moving over top of various sound files (as if you would when you are trying to preview them).
7) 'ls' again and notice that the number of 'pipe' entries has increased
8) notice that once nautilus has 1024 open file descriptors further attempts to open folders (or anything else) in nautilus will fail. |
|
| 2008-08-06 07:01:20 |
Martin Pitt |
esound: status |
Fix Committed |
Fix Released |
|
| 2008-08-06 07:01:20 |
Martin Pitt |
esound: statusexplanation |
Sponsored and accepted into gutsy-proposed:
esound (0.2.38-0ubuntu4.1) gutsy-proposed; urgency=low
.
* esdlib.c: The previous patch leaks pipe file descriptors in the case
that /usr/bin/esd doesn't exist. Fix that bug by only opening the
pipes after we know that the esd executable exists (LP: #183411).
Ryan, please add a TEST CASE: to the description, as per SRU policy. Thanks!
|
Copied to gutsy-updates. |
|
| 2009-12-05 19:39:12 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/esound |
|
| 2009-12-05 19:47:40 |
Launchpad Janitor |
branch linked |
|
lp:ubuntu/gutsy-proposed/esound |
|
