From a49918b152aa7b0e998e0f159d4b0128c297c347 Mon Sep 17 00:00:00 2001 From: Alessandro Decina Date: Tue, 18 Nov 2008 16:52:11 +0000 Subject: ext/metadata/gstbasemetadata.c: Avoid deadlock during pad activation. Original commit message from CVS: * ext/metadata/gstbasemetadata.c: Avoid deadlock during pad activation. Fixes #561241. --- ext/metadata/gstbasemetadata.c | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) (limited to 'ext/metadata') diff --git a/ext/metadata/gstbasemetadata.c b/ext/metadata/gstbasemetadata.c index e86c9ce1..0e4eb512 100644 --- a/ext/metadata/gstbasemetadata.c +++ b/ext/metadata/gstbasemetadata.c @@ -1807,16 +1807,24 @@ gst_base_metadata_sink_activate (GstPad * pad) } if (ret) { - gst_pad_activate_pull (pad, FALSE); - gst_pad_activate_push (filter->srcpad, FALSE); - if (!gst_pad_is_active (pad)) { - ret = gst_pad_activate_push (filter->srcpad, TRUE); - ret = ret && gst_pad_activate_push (pad, TRUE); + GstActivateMode mode; + + /* in gst_base_metadata_pull_range_parse() we could have triggered + * negotiation and plugged new downstream elements. + * If GST_PAD_ACTIVATE_MODE (filter->srcpad) is GST_ACTIVATE_PULL it means + * that downstream is active in pull mode so we don't deactivate pull mode. + */ + GST_OBJECT_LOCK (filter->srcpad); + mode = GST_PAD_ACTIVATE_MODE (filter->srcpad); + GST_OBJECT_UNLOCK (filter->srcpad); + + if (mode != GST_ACTIVATE_PULL) { + /* change from PULL to PUSH */ + gst_pad_activate_push (pad, TRUE); } } return ret; - } static gboolean -- cgit v1.2.1