[Bug 721096] New: Pad BLOCKING probe callback issues

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=721096
  GStreamer | gstreamer (core) | git

           Summary: Pad BLOCKING probe callback issues
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: me at andrey-utkin.pp.ua
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


I see there a race condition and plain defect:
1) with GST_PAD_PROBE_TYPE_BLOCKING, inside of which i do
gst_pad_remove_probe (pad, GST_PAD_PROBE_INFO_ID (info)), is still
entered more than once;
2) in gstpad.c:1270 the return value of callback is ignored.

This is in fresh git master checkout, HEAD
b03361ac38f020bac6f06c6092adc691b3419bdf.

It is reasonable to assume that application maker should guard his
callback procedure with his own mutex, but it's pity that a feature of
return value GST_PAD_PROBE_REMOVE does not guarantee that callback
will be never more entered (i.e. there is a indeterminate delay).

You can check these issues with my sketch app
https://github.com/krieger-od/gstreamer_test

To build, run ./build.sh
To launch test app infinitely, run ./loop_forever.sh (problems show up
not every time, so this is necessary).

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.