After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 446766 - return code for gst_base_rtp_payload_audio_handle_event()
return code for gst_base_rtp_payload_audio_handle_event()
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.15
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-06-12 14:52 UTC by Laurent Glayal
Modified: 2007-09-05 23:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
change return code of gst_base_rtp_payload_audio_handle_event() (716 bytes, patch)
2007-06-12 14:53 UTC, Laurent Glayal
committed Details | Review

Description Laurent Glayal 2007-06-12 14:52:51 UTC
This patch changes return code of gst_base_rtp_payload_audio_handle_event() to FALSE. This modifications forces caller of the function in base class to unref the event (not sure it's the right way to do as the function should have ownership of the event). Event unref avoids memleak for us.
Comment 1 Laurent Glayal 2007-06-12 14:53:22 UTC
Created attachment 89813 [details] [review]
change return code of gst_base_rtp_payload_audio_handle_event()
Comment 2 Wim Taymans 2007-09-05 23:01:44 UTC
        Patch by: Laurent Glayal <spglegle at yahoo dot fr>

        * gst-libs/gst/rtp/gstbasertpaudiopayload.c:
        (gst_base_rtp_payload_audio_handle_event):
        Return FALSE from the event handler to let the parent class handle the
        event. Fixes #446766.