[gstreamer-bugs] [Bug 622725] New: [mpgtsparse] Doesn't remove pids from pes_pids

[GStreamer (bugzilla.gnome.org)]

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

           Summary: [mpgtsparse] Doesn't remove pids from pes_pids
    Classification: Desktop
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: sebp at k-d-w.org
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


Created an attachment (id=164618)
 View: https://bugzilla.gnome.org/attachment.cgi?id=164618
 Review: https://bugzilla.gnome.org/review?bug=622725&attachment=164618

Don't free pat structure

This is a problem if you tune to a channel which uses pid X and later tune to
another channel where X is used for another table (e.g. PMT).

The code that does that was actually already there but never used because the
pat structure was freed before. The commit that introduced those lines intended
to fix a memory leak, I checked that thoroughly, no leak is introduced.

I already talked to Zaheer about this and he agrees with my patch.

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