From 0c5d50207326d74a4805bcd898bfac887540f12b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Sebastian=20Dr=C3=B6ge?= Date: Tue, 6 Sep 2011 12:19:58 +0200 Subject: [PATCH] pad: Check for subsets, not non-empty intersections to check if caps are compatible Pads should only accept caps that are a subset of the pad caps, e.g. they should accept only caps that have a non-empty intersection and at least all fields of the pad caps. Without this a pad that wants for example "video/x-h264,stream-format=byte-stream" will be happy to accept "video/x-h264". --- gst/gstpad.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gst/gstpad.c b/gst/gstpad.c index cfb5681..368c878 100644 --- a/gst/gstpad.c +++ b/gst/gstpad.c @@ -2554,7 +2554,7 @@ gst_pad_acceptcaps_default (GstPad * pad, GstCaps * caps) GST_DEBUG_OBJECT (pad, "allowed caps %" GST_PTR_FORMAT, allowed); - result = gst_caps_can_intersect (allowed, caps); + result = gst_caps_is_subset (caps, allowed); gst_caps_unref (allowed); @@ -2760,7 +2760,7 @@ gst_pad_configure_sink (GstPad * pad, GstCaps * caps) gboolean res; /* See if pad accepts the caps */ - if (!gst_caps_can_intersect (caps, gst_pad_get_pad_template_caps (pad))) + if (!gst_caps_is_subset (caps, gst_pad_get_pad_template_caps (pad))) goto not_accepted; /* set caps on pad if call succeeds */ -- 2.7.4