[Bug 722145] tsdemux: fix memleak of PCROffsetCurrent

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Jan 14 05:20:51 PST 2014


https://bugzilla.gnome.org/show_bug.cgi?id=722145
  GStreamer | gst-plugins-bad | git

--- Comment #2 from Andrey Utkin <me at andrey-utkin.pp.ua> 2014-01-14 13:20:49 UTC ---
(In reply to comment #1)
> commit 4d957e53e4415c72440af0281b4a479101a3dc63
> Author: Andrey Utkin <andrey.krieger.utkin at gmail.com>
> Date:   Tue Jan 14 02:19:12 2014 +0200
> 
>     tsdemux: fix memleak of PCROffsetCurrent
> 
>     https://bugzilla.gnome.org/show_bug.cgi?id=722145

Thank you for applying Sebastian, but i'm afraid you added a non-NULL-checking
condition without actual need: MpegTSPCR structure is allocated in single place
in code, and it always unconditionally allocates also PCROffsetCurrent into
res->current.

-- 
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.


More information about the gstreamer-bugs mailing list