[gstreamer-bugs] [Bug 639056] TDT tables are not parsed correctly

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Fri Jan 14 01:45:19 PST 2011


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

--- Comment #17 from Andoni Morales <ylatuya at gmail.com> 2011-01-14 09:45:17 UTC ---
I would revert 2611b12970d2c2254ecc3f0a6f02a09ffa82a16a and include the
following the following hunk in the Karol's patch:

index a86ef0e..b61dca4 100644
--- a/gst/mpegdemux/mpegtspacketizer.c
+++ b/gst/mpegdemux/mpegtspacketizer.c
@@ -2192,22 +2192,6 @@ mpegts_packetizer_push_section (MpegTSPacketizer *
packetizer,

     data += pointer;
   }
-  /* TDT and TOT sections (see ETSI EN 300 468 5.2.5)
-   *  these sections do not extend to several packets so we don't need to use
the
-   *  sections filter. */
-  if (packet->pid == 0x14) {
-    table_id = data[0];
-    section->section_length = GST_READ_UINT24_BE (data) & 0x000FFF;
-    section->buffer = gst_buffer_create_sub (packet->buffer,
-        data - GST_BUFFER_DATA (packet->buffer), section->section_length + 3);
-    section->table_id = table_id;
-    section->complete = TRUE;
-    res = TRUE;
-    GST_DEBUG ("TDT section pid:%d table_id:%d section_length: %d\n",
-        packet->pid, table_id, section->section_length);
-    goto out;
-  }
-
   /* create a sub buffer from the start of the section (table_id and
    * section_length included) to the end */
   sub_buf = gst_buffer_create_sub (packet->buffer,

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