[Bug 622588] thread problem in rtspsrc

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=622588
  GStreamer | gst-plugins-good | 0.10.23

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEEDINFO

--- Comment #9 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2012-01-07 20:24:34 UTC ---
> sometimes it crashed at plugin loading:
> 
> 0:00:28.024064013  [331m10992 [00m  0x8e37d20  [37mLOG     [00m
>  [00;01;36m  GST_PLUGIN_LOADING
> gstplugin.c:782:gst_plugin_load_file: [00m Plugin 0x92640800 for file
> "/usr/lib/gstreamer-0.10/libgstrtsp.so" prepared, registering...
> 0:00:28.024091155  [331m10992 [00m  0x8e37d20  [37mLOG     [00m
>  [00;01;36m  GST_PLUGIN_LOADING
> gstplugin.c:538:gst_plugin_register_func: [00m plugin
> "/usr/lib/gstreamer-0.10/libgstrtsp.so" looks good
> 
> ERROR: Caught a segmentation fault while loading plugin file:
> /usr/lib/gstreamer-0.10/libgstrtsp.so
> 
> I have a similar crashlog with libgstffmpeg.so, too. After that I had the idea
> to load the elements I'm using at the start of my app. I did this with creating
> every elements and throwing them away. The goal was to preload the .so files.
> With this workaround the crashes are gone and all the rtsp streams are showed
> up. 
> 
> So I think maybe the loading of the plugins is not thread safe: while a thread
> initializing a plugin, an another thread can use this partially loaded plugin,
> and the result is crash or some weird behavior. Can this be correct?

That seems unlikely. Plugin loading should be serialised in GStreamer core.

Please provide a stack trace for the crash you get during plugin loading.

I've just pushed the following fix which may or may not be related:

 commit 9f042ae224b3d84643bf4ad977080987fe80d36f
 Author: Tim-Philipp Müller <tim.muller at collabora.co.uk>
 Date:   Sat Jan 7 20:12:17 2012 +0000

    rtspconnection: make hostname lookup more thread-safe

    Don't write IP number string to return into a static
    array which is shared amongst all threads (note: of
    course a copy is returned).

    https://bugzilla.gnome.org/show_bug.cgi?id=666711


Setting NEEDINFO for:

 (a) please re-test with current git and confirm the problem still exists
 (b) please provide a stack trace of the crash during rtsp plugin loading

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.