bugGnash - The GNU Flash player - Bugs: bug #32069, yt: crash on exit

 
 

bug #32069: yt: crash on exit

Submitter:  Gabriele Giacone <gg0>
Submitted:  Thu 06 Jan 2011 06:10:08 PM UTC
   
 
Category:  sound-sdl Severity:  4 - Important
Release:  None Status:  Fixed
Privacy:  Public Assigned to:  strk
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 09 Feb 2011 02:10:24 PM UTC, comment #9: 

It's fixed, not invalid :)

Sandro Santilli <strk>
Group Member
Wed 09 Feb 2011 01:56:10 PM UTC, comment #8: 

Good, fixed. It doesn't crash anymore.

Gabriele Giacone <gg0>
Group Member
Wed 09 Feb 2011 01:06:32 PM UTC, comment #7: 

Please try master 571f23b

Sandro Santilli <strk>
Group Member
Wed 09 Feb 2011 12:48:23 PM UTC, comment #6: 

I've found NetStream destructor is making no attempt at unplugging the aux streamer. A patch is in testing.

Sandro Santilli <strk>
Group Member
Wed 09 Feb 2011 12:18:36 PM UTC, comment #5: 

Oh, ok. I see it is 100% reproducible with standalone.
Not a plugin bug at all.

gui/gtk-gnash -v http://www.youtube.com/v/OyvZq3Yn9_4j

Now I think #31766 is different.

This one should really have to do with the fact that the SDL
thread is not killed at the appropriate time.

Sandro Santilli <strk>
Group Member
Wed 09 Feb 2011 12:10:21 PM UTC, comment #4: 

I don't have the backtrace anymore, wasn't easy to obtain as the problem doesn't occur by only using the launcher.

Can you reproduce this one with the launcher ?

Sandro Santilli <strk>
Group Member
Wed 09 Feb 2011 11:40:28 AM UTC, comment #3: 

It seems it is I guess by comparing backtraces.
What else after #8 in your backtrace?

Anyway in this case it's easily reproducable and it crashes just on exit, when you close the window started by CLI.
It doesn't compromise normal behavior.

Gabriele Giacone <gg0>
Group Member
Wed 09 Feb 2011 10:43:52 AM UTC, comment #2: 

Is this the same as bug #31766 ?

Sandro Santilli <strk>
Group Member
Thu 06 Jan 2011 06:31:20 PM UTC, comment #1: 

It looks like it's related to thread cleanup in the sound handler, which has an unhealthy interaction with GC-collected as_objects.

Possibly the SDL callback is being executed here after the GC has cleaned everything. It's pretty much a guess; I think strk could be more helpful.

Benjamin Wolsey <bwy>
Group Member
Thu 06 Jan 2011 06:10:08 PM UTC, original submission:  


Playing yt videos, I've noticed both plugin and cli crashes on exit.

To reproduce, convert yt address
from
  http://www.youtube.com/watch?v=XXXXXXXXXXX
to
  http://www.youtube.com/v/XXXXXXXXXXX

and run
  $ gtk-gnash -vvv http://www.youtube.com/v/XXXXXXXXXXX

Then click on the movie so it starts, then close window.

You'll get:
gtk-gnash: /usr/include/boost/thread/pthread/mutex.hpp:50: void boost::mutex::lock(): Assertion `!pthread_mutex_lock(&m)' failed.

Backtrace:

#0  0x00007f0716182165 in *__GI_raise (sig=<value optimized out>) at ../nptl/sysdeps/unix/sysv/linux/raise.c:64
#1  0x00007f0716184f70 in *__GI_abort () at abort.c:92
#2  0x00007f071617b2b1 in *__GI___assert_fail (assertion=0x7f07191f28a8 "!pthread_mutex_lock(&m)", file=<value optimized out>, line=50, function=0x7f071932cfd0 "void boost::mutex::lock()") at assert.c:81
#3  0x00007f071903f8de in boost::mutex::lock (this=0x1423518, samples=0x163e000, nSamples=<value optimized out>, eof=<value optimized out>) at /usr/include/boost/thread/pthread/mutex.hpp:50
#4  boost::unique_lock<boost::mutex>::lock (this=0x1423518, samples=0x163e000, nSamples=<value optimized out>, eof=<value optimized out>) at /usr/include/boost/thread/locks.hpp:349
#5  unique_lock (this=0x1423518, samples=0x163e000, nSamples=<value optimized out>, eof=<value optimized out>) at /usr/include/boost/thread/locks.hpp:227
#6  gnash::BufferedAudioStreamer::fetch (this=0x1423518, samples=0x163e000, nSamples=<value optimized out>, eof=<value optimized out>) at ../../libcore/asobj/NetStream_as.cpp:1500
#7  0x00007f071893269f in gnash::sound::AuxStream::fetchSamples (this=0x1e94300, to=0x2994, nSamples=6) at ../../libsound/AuxStream.h:47
#8  0x00007f07189316ef in gnash::sound::sound_handler::fetchSamples (this=0x14a6040, to=0x1c57000, nSamples=4096) at ../../libsound/sound_handler.cpp:583
#9  0x00007f07189411be in gnash::sound::SDL_sound_handler::fetchSamples (this=0x14a6040, to=0x1c57000, nSamples=4096) at ../../libsound/sdl/sound_handler_sdl.cpp:231
#10 0x00007f0718941b38 in gnash::sound::SDL_sound_handler::sdl_audio_callback (udata=0x2979, buf=0x2994 <Address 0x2994 out of bounds>, bufLenIn=8192) at ../../libsound/sdl/sound_handler_sdl.cpp:270
#11 0x00007f071339bd18 in ?? () from /usr/lib/libSDL-1.2.so.0
#12 0x00007f07133a3b85 in ?? () from /usr/lib/libSDL-1.2.so.0
#13 0x00007f07133ef499 in ?? () from /usr/lib/libSDL-1.2.so.0
#14 0x00007f07196d38ba in start_thread (arg=<value optimized out>) at pthread_create.c:300
#15 0x00007f071621f02d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:112
#16 0x0000000000000000 in ?? ()



Gabriele Giacone <gg0>
Group Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by strk (Posted a comment)
  • -email is unavailable- added by bwy (Posted a comment)
  • -email is unavailable- added by gg0 (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2011-02-09 strk StatusInvalid Fixed
        Open/ClosedOpen Closed
    2011-02-09 gg0 StatusReady For Test Invalid
    2011-02-09 strk StatusIn Progress Ready For Test
    2011-02-09 strk StatusNone In Progress
        Assigned toNone strk
    2011-02-09 strk CategoryNone sound-sdl
        Severity3 - Normal 4 - Important

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code