From 61a885613316ce7657c36a6cd215b43f9dc67b79 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20P=C3=B6lsterl?= Date: Wed, 30 Jun 2010 18:20:13 +0100 Subject: [PATCH] mpegtsparse: don't free PAT structure which may still be needed later 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, but we clean things up elsewhere. Fixes #622725. --- gst/mpegdemux/mpegtsparse.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/gst/mpegdemux/mpegtsparse.c b/gst/mpegdemux/mpegtsparse.c index e430ec36cb..7dc68c5938 100644 --- a/gst/mpegdemux/mpegtsparse.c +++ b/gst/mpegdemux/mpegtsparse.c @@ -267,9 +267,6 @@ mpegts_parse_reset (MpegTSParse * parse) /* PAT */ g_hash_table_insert (parse->psi_pids, GINT_TO_POINTER (0), GINT_TO_POINTER (1)); - if (parse->pat != NULL) - gst_structure_free (parse->pat); - parse->pat = NULL; /* pmt pids will be added and removed dynamically */ }