pad: Check data NULL-ness when probes are stopped

We were correctly handling this for buffers, but not events and queries.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/7493>
This commit is contained in:
Arun Raghavan 2024-09-10 16:03:05 -04:00 committed by GStreamer Marge Bot
parent e4220fa9a8
commit f30321538b

View file

@ -4578,7 +4578,7 @@ probe_handled:
probe_stopped:
{
/* We unref the buffer, except if the probe handled it (CUSTOM_SUCCESS_1) */
if (!handled)
if (data && !handled)
gst_mini_object_unref (GST_MINI_OBJECT_CAST (data));
switch (ret) {
@ -5647,7 +5647,7 @@ inactive:
probe_stopped:
{
GST_OBJECT_FLAG_SET (pad, GST_PAD_FLAG_PENDING_EVENTS);
if (ret != GST_FLOW_CUSTOM_SUCCESS_1)
if (event && ret != GST_FLOW_CUSTOM_SUCCESS_1)
gst_event_unref (event);
switch (ret) {
@ -6056,7 +6056,7 @@ probe_stopped:
if (need_unlock)
GST_PAD_STREAM_UNLOCK (pad);
/* Only unref if unhandled */
if (ret != GST_FLOW_CUSTOM_SUCCESS_1)
if (event && ret != GST_FLOW_CUSTOM_SUCCESS_1)
gst_event_unref (event);
switch (ret) {