Bug 14875 - crash with oss backend if /dev/dsp can't be opened
Summary: crash with oss backend if /dev/dsp can't be opened
Status: RESOLVED WONTFIX
Alias: None
Product: swfdec
Classification: Unclassified
Component: library (show other bugs)
Version: git
Hardware: x86 (IA32) Linux (All)
: medium critical
Assignee: swfdec ml
QA Contact: swfdec ml
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-07 02:26 UTC by Riccardo Magliocchetti
Modified: 2008-09-12 12:02 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
remove assert from oss playback (367 bytes, patch)
2008-08-15 02:30 UTC, Riccardo Magliocchetti
Details | Splinter Review

Description Riccardo Magliocchetti 2008-03-07 02:26:25 UTC
Latest git, i'm using alsa oss emulation.

Failed to open /dev/dsp
**
** Swfdec-Gtk:ERROR:(swfdec_playback.c:225):audio_removed: code should not be reached

Program received signal SIGABRT, Aborted.
---Type <return> to continue, or q <return> to quit---
[Switching to Thread 0xb7078a40 (LWP 12545)]
0xb721e546 in raise () from /lib/libc.so.6
(gdb) bt full
#0  0xb721e546 in raise () from /lib/libc.so.6
No symbol table info available.
#1  0xb721fd58 in abort () from /lib/libc.so.6
No symbol table info available.
#2  0xb74c934a in g_assertion_message () from /usr/lib/libglib-2.0.so.0
No symbol table info available.
#3  0xb03f9110 in audio_removed (player=0xb6803c8, audio=0x3101, 
    sound=0xb292f98) at swfdec_playback.c:225
	walk = (GList *) 0x6
	__PRETTY_FUNCTION__ = "audio_removed"
#4  0xb75343a5 in g_cclosure_marshal_VOID__OBJECT ()
   from /usr/lib/libgobject-2.0.so.0
No symbol table info available.
#5  0xb7526729 in g_closure_invoke () from /usr/lib/libgobject-2.0.so.0
No symbol table info available.
#6  0xb753c445 in ?? () from /usr/lib/libgobject-2.0.so.0
No symbol table info available.
#7  0x0adc6bd0 in ?? ()
No symbol table info available.
#8  0x00000000 in ?? ()
No symbol table info available.
Comment 1 Riccardo Magliocchetti 2008-05-28 01:47:39 UTC
Crashers are severity critical. I must add that this will probably closed when 0.8.0 will be out since oss backend is gone.
Comment 2 Riccardo Magliocchetti 2008-08-15 02:30:09 UTC
Created attachment 18295 [details] [review]
remove assert from oss playback

I had this change locally, it workarounds it removing the assert. I haven't noticed anything wrong with this.
Comment 3 Riccardo Magliocchetti 2008-09-08 04:34:02 UTC
0.8 is out, oss is gone.
Comment 4 Rafael Diniz 2008-09-12 07:10:47 UTC
People,
This is a mistake in my opinion.
IMHO, OSS 4 is out and is really much better than ALSA (kernel mode mixing, independent volume control for applications, ...).
OSS 4 is widely used as sound system in Solaris, FreeBSD, Unixware and Linux.

If you plan to make swfdec portable, keep OSS supported is a nice thing :)
Comment 5 Riccardo Magliocchetti 2008-09-12 07:27:03 UTC
The problem (In reply to comment #4)
> People,
> This is a mistake in my opinion.
> IMHO, OSS 4 is out and is really much better than ALSA (kernel mode mixing,
> independent volume control for applications, ...).
> OSS 4 is widely used as sound system in Solaris, FreeBSD, Unixware and Linux.

I think if someone step up to ressurrect, improve and maintain oss audio backend it can be put back. I have tried to do this myself with the patches posted here and at bug 13772. And failed :(
 
> If you plan to make swfdec portable, keep OSS supported is a nice thing :)

There is a pulseaudio backend for that and other purposes. But i have to agree that pulseaudio could be a bit painful at the moment. But as you can see from bug 
13772#c12 is the swfdec future.
Comment 6 Rafael Diniz 2008-09-12 12:02:28 UTC
It's true that I can run pulseaudio on top of OSS.
BTW, if I have time in the next week, I'll try to write a good OSS backend using the OSS 4 ioctl()'s (and, may be, just don't support the old dead OSS 3 API).


Thank you all for the great swfdec :)


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.