[mythtv-commits] mythtv commit: r10097 - in trunk/mythtv/libs/libmythtv by danielk

[mythtv at cvs.mythtv.org]

      Author: danielk
        Date: 2006-06-01 20:39:59 +0000 (Thu, 01 Jun 2006)
New Revision: 10097
   Changeset: http://cvs.mythtv.org/trac/changeset/10097

Modified:

   trunk/mythtv/libs/libmythtv/dtvsignalmonitor.cpp
   trunk/mythtv/libs/libmythtv/dvbrecorder.cpp
   trunk/mythtv/libs/libmythtv/mpeg/mpegstreamdata.cpp
   trunk/mythtv/libs/libmythtv/mpeg/mpegstreamdata.h
   trunk/mythtv/libs/libmythtv/mpeg/scanstreamdata.cpp
   trunk/mythtv/libs/libmythtv/mpeg/scanstreamdata.h
   trunk/mythtv/libs/libmythtv/siscan.cpp

Log:

Refs #1887. Fixes the underlaying cause of the invalid PAT in the DVB recordings which causes the reported crash.

Multiple PAT tables do not seem to be precluded in the MPEG-2 spec, but seem to have been banned in DVB-S, DVB-T, DVB-C and ATSC. But they are now allowed in DVB-S2, and have been seen in the wild.

Multiple PAT sections have always been allowed by the spec, but we haven't seen them in the wild.

This extends mpegstreamdata to allow both multiple sections per PAT table and multiple PAT tables. This changes the interface a bit, hence the changes in the other files.

I've left the ticket open because still haven't looked at having ffmpeg not crash on bad tables.