[Swfdec] [Bug 13604] New: Assertion failure when seeking on youtube with pulseaudio backend.

[Riccardo Magliocchetti]

Hi eric, all,

On Tue, 11 Dec 2007 11:03:05 -0800 (PST)
bugzilla-daemon at freedesktop.org wrote:

> http://bugs.freedesktop.org/show_bug.cgi?id=13604
> 
> firefox-bin: pulse/operation.c:68: pa_operation_unref: Assertion `o'
> failed.
> 
> Program received signal SIGABRT, Aborted.
> [Switching to Thread 0x2b641677d9c0 (LWP 11689)]
> 0x00002b6413542fd5 in raise () from /lib/libc.so.6
> (gdb) bt
> #0  0x00002b6413542fd5 in raise () from /lib/libc.so.6
> #1  0x00002b6413544a30 in abort () from /lib/libc.so.6
> #2  0x00002b641353c3df in __assert_fail () from /lib/libc.so.6
> #3  0x00002aaaafaa7be4 in pa_operation_unref ()
> #from /usr/lib/libpulse.so.0 4  0x00002aaaaf0ddaa6 in
> #swfdec_stream_close ()
>    from /opt/swfdec/lib/libswfdec-gtk-0.5.so.4
> #5  0x00002aaaaf0ddd78 in audio_removed ()
>    from /opt/swfdec/lib/libswfdec-gtk-0.5.so.4

I've taken a look at the code and at pulseaudio example pacat.c,
it looks that we have to check if pa_stream_drain returns null
because if so pa_operation_unref asserts. I'm unsure about this:

g_printerr("PA stream drain failed: %s\n",
                 pa_strerror(pa_context_errno(stream->sound->pa)));

Is the right thing to print?

Attached a patch that does this, the code compiles but i'm not able to
test it.

thanks,
Riccardo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: try-to-fix-pulseaudio-assertion-failure.diff
Type: text/x-diff
Size: 924 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/swfdec/attachments/20080202/4a3a9114/attachment.diff