Closed
Bug 1337261
Opened 9 years ago
Closed 8 years ago
Misconfigured ALSA (or setup with no audio hardware) can cause crashes in cubeb when trying to play audio
Categories
(Core :: Audio/Video: cubeb, defect, P3)
Tracking
()
RESOLVED
DUPLICATE
of bug 1337805
People
(Reporter: martin, Assigned: kinetik)
References
()
Details
Attachments
(1 file)
2.08 KB,
patch
|
Details | Diff | Splinter Review |
There was a bogus freeing of an uninitialized cond var in an error path if cubeb failed to initialize alsa. This caused firefox to dump core for me when web sites tried to play audio.
I reported the issue upstream and it was quickly fixed:
https://github.com/kinetiknz/cubeb/commit/0c2b114e5b9436d17680afe5bc45a10eb43163
90
Depending on pthread immplementations this could have caused less visible corruption in other systems, but the "no working audio" setup probably is quite exceptional when using a browser.
Reporter | ||
Updated•9 years ago
|
Comment 1•8 years ago
|
||
P3 because IIRC we don't officially support alsa anymore. Should be an easy fix however.
Rank: 35
Priority: -- → P3
Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8836810 -
Flags: review?(padenot)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Assignee | ||
Updated•8 years ago
|
Summary: Misconfiugred alsa (or setup with no audio hardware) can cause crashes in cubeb when trying to play audio → Misconfigured ALSA (or setup with no audio hardware) can cause crashes in cubeb when trying to play audio
Updated•8 years ago
|
Attachment #8836810 -
Flags: review?(padenot) → review+
Comment 3•8 years ago
|
||
Comment on attachment 8836810 [details] [diff] [review]
p1 - Update libcubeb to afad5aae.
Matthew, Gecko's copy is already at 21e96a, handled in bug 1337805. We did this bump so we could have :chunmin OSX multichannel code in, and this includes the alsa fix.
Attachment #8836810 -
Flags: review+
Updated•8 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•