From c7103ce4b8c1da7dcfbcf2ec83a42a376fb896e1 Mon Sep 17 00:00:00 2001 From: Vincent Penquerc'h Date: Mon, 3 Nov 2014 17:46:57 +0000 Subject: [PATCH] pad: fail dropped queries Previously, dropping a query from a pad probe would deem the query succeeded, and the caller might then assume the query's results are valid, and thus dereference an invalid object such as a GstCaps. We now assume dropped queries did not succeed. Dropped events and buffers are still deemed a success. --- gst/gstpad.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/gst/gstpad.c b/gst/gstpad.c index e835e29..4c063fe 100644 --- a/gst/gstpad.c +++ b/gst/gstpad.c @@ -3643,11 +3643,8 @@ probe_stopped: GST_PAD_STREAM_UNLOCK (pad); /* if a probe dropped, we don't sent it further but assume that the probe - * answered the query and return TRUE */ - if (ret == GST_FLOW_CUSTOM_SUCCESS) - res = TRUE; - else - res = FALSE; + * did not answer the query and return FALSE */ + res = FALSE; return res; } @@ -3760,11 +3757,8 @@ probe_stopped: GST_OBJECT_UNLOCK (pad); /* if a probe dropped, we don't sent it further but assume that the probe - * answered the query and return TRUE */ - if (ret == GST_FLOW_CUSTOM_SUCCESS) - res = TRUE; - else - res = FALSE; + * did not answer the query and return FALSE */ + res = FALSE; return res; } -- 2.7.4