Calling set_state(NULL) from glib bus error callback

[Tim-Philipp Müller]

On Wed, 2011-12-07 at 13:05 -0800, Max Paklin wrote:

Hi,

> The typical reaction to getting an error posted to glib bus is to quick the
> loop and hence the app.

GStreamer bus you meant, presumably.

> I my case I have multiple pipelines running, so I can't just exit the app.
> What I want to do is to stop the failing pipeline and later delete it.
> Is it safe to call gst_element_set_state(NULL) from the bus error callback?

Yes, assuming the main loop / main context iteration is happening in a
thread other than a thread created by GStreamer. Usually this is the
case, because usually you run gtk_main() or g_main_loop_run() from your
application's main thread.

> From what I can read in the source it seems like it should OK, but I'd love
> for people in the know to confirm that.
> I wouldn't want to cause an occasional deadlock.

Should be fine. _set_state() is only problematic when done from the
context of a GStreamer streaming thread (e.g. a signal callback emitted
by an element, a *sync* bus watch or a pad probe callback or so).

Cheers
 -Tim