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 322745 - [mpegdemux] segfaults
[mpegdemux] segfaults
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-ugly
0.9.x
Other Linux
: Normal normal
: 0.9.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-11-29 11:04 UTC by Michal Benes
Modified: 2005-11-29 11:29 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch fixing send_event function (1.39 KB, patch)
2005-11-29 11:06 UTC, Michal Benes 		committed Details | Review

Description Michal Benes 2005-11-29 11:04:15 UTC 
GstMpegParse seems to be pushing one event to many pads without referencing this
event.
Comment 1 Michal Benes 2005-11-29 11:06:47 UTC 
Created attachment 55368 [details] [review]
Patch fixing send_event function

This patch adds event refence before pushing the event to multiple pads. Also,
two event related memory leaks are fixed.
Comment 2 Tim-Philipp Müller 2005-11-29 11:28:16 UTC 
Applied, thanks!

2005-11-29  Michal Benes  <michal dot benes at xeris dot cz>

        * gst/mpegstream/gstmpegdemux.c: (gst_mpeg_demux_send_event):
        * gst/mpegstream/gstmpegparse.c: (gst_mpeg_parse_send_newsegment),
        (gst_mpeg_parse_send_event):
          Ref events before sending them to multiple pads, after all
          gst_pad_send_event() takes ownership of events. Don't leak
          events that have not been handled (fixes #322745).