OSDN Git Service

Sanity check in encfaacClose or we'll abort on an illegal channel spec or sample...
[handbrake-jp/handbrake-jp-git.git] / libhb / stream.c
index fe2c962..dd5c709 100755 (executable)
@@ -1,21 +1,91 @@
 /* $Id$
 
    This file is part of the HandBrake source code.
-   Homepage: <http://handbrake.m0k.org/>.
+   Homepage: <http://handbrake.fr/>.
    It may be used under the terms of the GNU General Public License. */
 
+#include <string.h>
+#include <ctype.h>
+#include <errno.h>
+
 #include "hb.h"
 #include "lang.h"
 #include "a52dec/a52.h"
-
-#include <string.h>
+#include "libavcodec/avcodec.h"
+#include "libavformat/avformat.h"
 
 #define min(a, b) a < b ? a : b
 
+/*
+ * This table defines how ISO MPEG stream type codes map to HandBrake
+ * codecs. It is indexed by the 8 bit stream type and contains the codec
+ * worker object id and a parameter for that worker proc (ignored except
+ * for the ffmpeg-based codecs in which case it is the ffmpeg codec id).
+ *
+ * Entries with a worker proc id of 0 or a kind of 'U' indicate that HB
+ * doesn't handle the stream type.
+ */
+typedef struct {
+    enum { U, A, V } kind; /* unknown / audio / video */
+    int codec;          /* HB worker object id of codec */
+    int codec_param;    /* param for codec (usually ffmpeg codec id) */
+    const char* name;   /* description of type */
+} stream2codec_t;
+
+#define st(id, kind, codec, codec_param, name) \
+ [id] = { kind, codec, codec_param, name }
+
+static const stream2codec_t st2codec[256] = {
+    st(0x01, V, WORK_DECMPEG2,     0,              "MPEG1"),
+    st(0x02, V, WORK_DECMPEG2,     0,              "MPEG2"),
+    st(0x03, A, HB_ACODEC_MPGA,    CODEC_ID_MP2,   "MPEG1"),
+    st(0x04, A, HB_ACODEC_MPGA,    CODEC_ID_MP2,   "MPEG2"),
+    st(0x05, U, 0,                 0,              "ISO 13818-1 private section"),
+    st(0x06, U, 0,                 0,              "ISO 13818-1 PES private data"),
+    st(0x07, U, 0,                 0,              "ISO 13522 MHEG"),
+    st(0x08, U, 0,                 0,              "ISO 13818-1 DSM-CC"),
+    st(0x09, U, 0,                 0,              "ISO 13818-1 auxiliary"),
+    st(0x0a, U, 0,                 0,              "ISO 13818-6 encap"),
+    st(0x0b, U, 0,                 0,              "ISO 13818-6 DSM-CC U-N msgs"),
+    st(0x0c, U, 0,                 0,              "ISO 13818-6 Stream descriptors"),
+    st(0x0d, U, 0,                 0,              "ISO 13818-6 Sections"),
+    st(0x0e, U, 0,                 0,              "ISO 13818-1 auxiliary"),
+    st(0x0f, A, HB_ACODEC_MPGA,    CODEC_ID_AAC,   "ISO 13818-7 AAC Audio"),
+    st(0x10, V, WORK_DECAVCODECV,  CODEC_ID_MPEG4, "MPEG4"),
+    st(0x11, A, HB_ACODEC_MPGA,    CODEC_ID_AAC_LATM, "MPEG4 LATM AAC"),
+    st(0x12, U, 0,                 0,              "MPEG4 generic"),
+
+    st(0x14, U, 0,                 0,              "ISO 13818-6 DSM-CC download"),
+
+    st(0x1b, V, WORK_DECAVCODECV,  CODEC_ID_H264,  "H.264"),
+
+    st(0x80, U, 0,                 0,              "DigiCipher II Video"),
+    st(0x81, A, HB_ACODEC_AC3,     0,              "AC-3"),
+    st(0x82, A, HB_ACODEC_MPGA,    CODEC_ID_DTS,   "HDMV DTS"),
+    st(0x83, A, HB_ACODEC_LPCM,    0,              "LPCM"),
+    st(0x84, A, 0,                 0,              "SDDS"),
+    st(0x85, U, 0,                 0,              "ATSC Program ID"),
+    st(0x86, U, 0,                 0,              "SCTE 35 splice info"),
+    st(0x87, A, 0,                 0,              "E-AC-3"),
+
+    st(0x8a, A, HB_ACODEC_DCA,     0,              "DTS"),
+
+    st(0x91, A, HB_ACODEC_AC3,     0,              "AC-3"),
+    st(0x92, U, 0,                 0,              "Subtitle"),
+
+    st(0x94, A, 0,                 0,              "SDDS"),
+    st(0xa0, V, 0,                 0,              "MSCODEC"),
+
+    st(0xea, V, WORK_DECAVCODECV,  CODEC_ID_VC1,   "VC1"),
+};
+#undef st
+
 typedef enum {
     hb_stream_type_unknown = 0,
-    hb_stream_type_transport,
-    hb_stream_type_program
+    transport,
+    program,
+    dvd_program,
+    ffmpeg
 } hb_stream_type_t;
 
 #define kMaxNumberVideoPIDS 1
@@ -30,10 +100,12 @@ struct hb_stream_s
     int     errors;             /* total errors so far */
     int     last_error_frame;   /* frame # at last error message */
     int     last_error_count;   /* # errors at last error message */
+    int     packetsize;         /* Transport Stream packet size */
 
     int64_t ts_lastpcr;         /* the last pcr we found in the TS stream */
     int64_t ts_nextpcr;         /* the next pcr to put in a PS packet */
 
+    uint8_t *ts_packet;         /* buffer for one TS packet */
     uint8_t *ts_buf[kMaxNumberDecodeStreams];
     int     ts_pos[kMaxNumberDecodeStreams];
     int8_t  ts_foundfirst[kMaxNumberDecodeStreams];
@@ -50,19 +122,23 @@ struct hb_stream_s
      * we learn during the initial scan but cache so it can be
      * reused during the conversion read.
      */
-    int16_t ts_video_pids[kMaxNumberVideoPIDS];
-    int16_t ts_audio_pids[kMaxNumberAudioPIDS];
-
     uint8_t ts_number_video_pids;
     uint8_t ts_number_audio_pids;
 
+    int16_t ts_video_pids[kMaxNumberVideoPIDS];
+    int16_t ts_audio_pids[kMaxNumberAudioPIDS];
+
     uint8_t ts_streamid[kMaxNumberDecodeStreams];
-    uint8_t ts_audio_stream_type[kMaxNumberDecodeStreams];
+    uint8_t ts_stream_type[kMaxNumberDecodeStreams];
 
     char    *path;
     FILE    *file_handle;
-    hb_stream_type_t stream_type;
-    int     opentype;
+    hb_stream_type_t hb_stream_type;
+    hb_title_t *title;
+
+    AVFormatContext *ffmpeg_ic;
+    AVPacket *ffmpeg_pkt;
+    double ffmpeg_tsconv[MAX_STREAMS];
 
     struct {
         int lang_code;
@@ -111,7 +187,13 @@ static void hb_ts_stream_reset(hb_stream_t *stream);
 static hb_audio_t *hb_ts_stream_set_audio_id_and_codec(hb_stream_t *stream,
                                                        int aud_pid_index);
 static void hb_ps_stream_find_audio_ids(hb_stream_t *stream, hb_title_t *title);
-static off_t align_to_next_packet(FILE* f);
+static off_t align_to_next_packet(hb_stream_t *stream);
+
+static int ffmpeg_open( hb_stream_t *stream, hb_title_t *title );
+static void ffmpeg_close( hb_stream_t *d );
+static hb_title_t *ffmpeg_title_scan( hb_stream_t *stream );
+static int ffmpeg_read( hb_stream_t *stream, hb_buffer_t *buf );
+static int ffmpeg_seek( hb_stream_t *stream, float frac );
 
 /*
  * streams have a bunch of state that's learned during the scan. We don't
@@ -146,7 +228,62 @@ static void hb_stream_state_delete( hb_stream_t *ss )
     free( ss );
 }
 
-static inline int check_ps_sync(const uint8_t *buf)
+/*
+ * logging routines.
+ * these frontend hb_log because transport streams can have a lot of errors
+ * so we want to rate limit messages. this routine limits the number of
+ * messages to at most one per minute of video. other errors that occur
+ * during the minute are counted & the count is output with the next
+ * error msg we print.
+ */
+static void ts_warn_helper( hb_stream_t *stream, char *log, va_list args )
+{
+    // limit error printing to at most one per minute of video (at 30fps)
+    ++stream->errors;
+    if ( stream->frames - stream->last_error_frame >= 30*60 )
+    {
+        char msg[256];
+
+        vsnprintf( msg, sizeof(msg), log, args );
+
+        if ( stream->errors - stream->last_error_count < 10 )
+        {
+            hb_log( "stream: error near frame %d: %s", stream->frames, msg );
+        }
+        else
+        {
+            int Edelta = stream->errors - stream->last_error_count;
+            double Epcnt = (double)Edelta * 100. /
+                            (stream->frames - stream->last_error_frame);
+            hb_log( "stream: %d new errors (%.0f%%) up to frame %d: %s",
+                    Edelta, Epcnt, stream->frames, msg );
+        }
+        stream->last_error_frame = stream->frames;
+        stream->last_error_count = stream->errors;
+    }
+}
+
+static void ts_warn( hb_stream_t *stream, char *log, ... )
+{
+    va_list     args;
+    va_start( args, log );
+    ts_warn_helper( stream, log, args );
+    va_end( args );
+}
+
+static void ts_err( hb_stream_t *stream, int curstream, char *log, ... )
+{
+    va_list     args;
+    va_start( args, log );
+    ts_warn_helper( stream, log, args );
+    va_end( args );
+
+    stream->ts_skipbad[curstream] = 1;
+    stream->ts_pos[curstream] = 0;
+    stream->ts_streamcont[curstream] = -1;
+}
+
+static int check_ps_sync(const uint8_t *buf)
 {
     // a legal MPEG program stream must start with a Pack header in the
     // first four bytes.
@@ -154,39 +291,61 @@ static inline int check_ps_sync(const uint8_t *buf)
            (buf[2] == 0x01) && (buf[3] == 0xba);
 }
 
-static inline int check_ts_sync(const uint8_t *buf)
+static int check_ps_sys(const uint8_t *buf)
+{
+    // a legal MPEG program stream must start with a Pack followed by a
+    // SYS. If we've already verified the pack, this skips over it and checks
+    // for the sys header.
+    int pos = 14 + ( buf[13] & 0x7 );   // skip over the PACK
+    return (buf[pos+0] == 0x00) && (buf[pos+1] == 0x00) &&
+           (buf[pos+2] == 0x01) && (buf[pos+3] == 0xbb);
+}
+
+static int check_ts_sync(const uint8_t *buf)
 {
     // must have initial sync byte, no scrambling & a legal adaptation ctrl
     return (buf[0] == 0x47) && ((buf[3] >> 6) == 0) && ((buf[3] >> 4) > 0);
 }
 
-static inline int have_ts_sync(const uint8_t *buf)
+static int have_ts_sync(const uint8_t *buf, int psize)
 {
-    return check_ts_sync(&buf[0*188]) && check_ts_sync(&buf[1*188]) &&
-           check_ts_sync(&buf[2*188]) && check_ts_sync(&buf[3*188]) &&
-           check_ts_sync(&buf[4*188]) && check_ts_sync(&buf[5*188]) &&
-           check_ts_sync(&buf[6*188]) && check_ts_sync(&buf[7*188]);
+    return check_ts_sync(&buf[0*psize]) && check_ts_sync(&buf[1*psize]) &&
+           check_ts_sync(&buf[2*psize]) && check_ts_sync(&buf[3*psize]) &&
+           check_ts_sync(&buf[4*psize]) && check_ts_sync(&buf[5*psize]) &&
+           check_ts_sync(&buf[6*psize]) && check_ts_sync(&buf[7*psize]);
 }
 
 static int hb_stream_check_for_ts(const uint8_t *buf)
 {
     // transport streams should have a sync byte every 188 bytes.
-    // search the first KB of buf looking for at least 8 consecutive
+    // search the first 8KB of buf looking for at least 8 consecutive
     // correctly located sync patterns.
     int offset = 0;
 
-    for ( offset = 0; offset < 1024; ++offset )
+    for ( offset = 0; offset < 8*1024-8*188; ++offset )
     {
-        if ( have_ts_sync( &buf[offset]) )
-            return 1;
+        if ( have_ts_sync( &buf[offset], 188) )
+            return 188 | (offset << 8);
+        if ( have_ts_sync( &buf[offset], 192) )
+            return 192 | (offset << 8);
+        if ( have_ts_sync( &buf[offset], 204) )
+            return 204 | (offset << 8);
+        if ( have_ts_sync( &buf[offset], 208) )
+            return 208 | (offset << 8);
     }
     return 0;
 }
 
 static int hb_stream_check_for_ps(const uint8_t *buf)
 {
-    // program streams should have a Pack header every 2048 bytes.
-    // check that we have 4 of these.
+    // program streams should start with a PACK then a SYS header.
+    return check_ps_sync(buf) && check_ps_sys(buf);
+}
+
+static int hb_stream_check_for_dvd_ps(const uint8_t *buf)
+{
+    // DVD program streams should have a Pack header every 2048 bytes.
+    // check that we have 4 of these in a row.
     return check_ps_sync(&buf[0*2048]) && check_ps_sync(&buf[1*2048]) &&
            check_ps_sync(&buf[2*2048]) && check_ps_sync(&buf[3*2048]);
 }
@@ -197,17 +356,28 @@ static int hb_stream_get_type(hb_stream_t *stream)
 
     if ( fread(buf, 1, sizeof(buf), stream->file_handle) == sizeof(buf) )
     {
-        if ( hb_stream_check_for_ts(buf) != 0 )
+        int psize;
+        if ( ( psize = hb_stream_check_for_ts(buf) ) != 0 )
         {
-            hb_log("file is MPEG Transport Stream");
-            stream->stream_type = hb_stream_type_transport;
+            int offset = psize >> 8;
+            psize &= 0xff;
+            hb_log("file is MPEG Transport Stream with %d byte packets"
+                   " offset %d bytes", psize, offset);
+            stream->packetsize = psize;
+            stream->hb_stream_type = transport;
             hb_ts_stream_init(stream);
             return 1;
         }
+        if ( hb_stream_check_for_dvd_ps(buf) != 0 )
+        {
+            hb_log("file is MPEG DVD Program Stream");
+            stream->hb_stream_type = dvd_program;
+            return 1;
+        }
         if ( hb_stream_check_for_ps(buf) != 0 )
         {
             hb_log("file is MPEG Program Stream");
-            stream->stream_type = hb_stream_type_program;
+            stream->hb_stream_type = program;
             return 1;
         }
     }
@@ -224,6 +394,11 @@ static void hb_stream_delete_dynamic( hb_stream_t *d )
 
        int i=0;
 
+    if ( d->ts_packet )
+    {
+        free( d->ts_packet );
+        d->ts_packet = NULL;
+    }
        for (i = 0; i < kMaxNumberDecodeStreams; i++)
        {
                if (d->ts_buf[i])
@@ -246,9 +421,8 @@ static void hb_stream_delete( hb_stream_t *d )
  ***********************************************************************
  *
  **********************************************************************/
-hb_stream_t * hb_stream_open( char *path, int opentype )
+hb_stream_t * hb_stream_open( char *path, hb_title_t *title )
 {
-
     FILE *f = fopen( path, "r" );
     if ( f == NULL )
     {
@@ -271,33 +445,34 @@ hb_stream_t * hb_stream_open( char *path, int opentype )
      * (even if we have saved state, the stream may have changed).
      */
     hb_stream_t *ss = hb_stream_lookup( path );
-    if ( opentype == 1 )
+    if ( title && ss && ss->hb_stream_type != ffmpeg )
     {
-        /* opening to read - we must have saved state */
-        if ( ss == NULL )
-        {
-            hb_log( "hb_stream_open: error: re-opening %s but no scan state", path );
-            fclose( f );
-            free( d );
-            return NULL;
-        }
         /*
          * copy the saved state since we might be encoding the same stream
          * multiple times.
          */
         memcpy( d, ss, sizeof(*d) );
         d->file_handle = f;
-        d->opentype = opentype;
+        d->title = title;
         d->path = strdup( path );
 
-        if ( d->stream_type == hb_stream_type_transport )
+        if ( d->hb_stream_type == transport )
         {
+            d->ts_packet = malloc( d->packetsize );
+
             int i = 0;
             for ( ; i < d->ts_number_video_pids + d->ts_number_audio_pids; i++)
             {
                 d->ts_buf[i] = malloc( HB_DVD_READ_BUFFER_SIZE );
             }
             hb_stream_seek( d, 0. );
+
+            if ( d->packetsize == 188 )
+            {
+                // Assume that an over-the-air transport stream can lose PCR
+                // packets and try to filter out the timing inconsistencies.
+                title->flaky_clock = 1;
+            }
         }
         return d;
     }
@@ -307,19 +482,30 @@ hb_stream_t * hb_stream_open( char *path, int opentype )
      * If it's something we can deal with (MPEG2 PS or TS) return a stream
      * reference structure & null otherwise.
      */
-
     if ( ss != NULL )
     {
         hb_stream_state_delete( ss );
     }
     d->file_handle = f;
-    d->opentype = opentype;
+    d->title = title;
     d->path = strdup( path );
-    if (d->path != NULL &&  hb_stream_get_type( d ) != 0 )
+    if (d->path != NULL )
     {
-        return d;
+        if ( hb_stream_get_type( d ) != 0 )
+        {
+            return d;
+        }
+        fclose( d->file_handle );
+               d->file_handle = NULL;
+        if ( ffmpeg_open( d, title ) )
+        {
+            return d;
+        }
+    }
+    if ( d->file_handle )
+    {
+        fclose( d->file_handle );
     }
-    fclose( d->file_handle );
     if (d->path)
     {
         free( d->path );
@@ -337,17 +523,27 @@ hb_stream_t * hb_stream_open( char *path, int opentype )
 void hb_stream_close( hb_stream_t ** _d )
 {
     hb_stream_t *stream = * _d;
+
+    if ( stream->hb_stream_type == ffmpeg )
+    {
+        ffmpeg_close( stream );
+        hb_stream_delete( stream );
+        *_d = NULL;
+        return;
+    }
+
     if ( stream->frames )
     {
         hb_log( "stream: %d good frames, %d errors (%.0f%%)", stream->frames,
                 stream->errors, (double)stream->errors * 100. /
                 (double)stream->frames );
     }
+
     /*
      * if the stream was opened for a scan, cache the result, otherwise delete
      * the state.
      */
-    if ( stream->opentype == 0 )
+    if ( stream->title == NULL )
     {
         hb_stream_delete_dynamic( stream );
         if ( stream_state_list == NULL )
@@ -376,7 +572,7 @@ static void hb_stream_delete_audio_entry(hb_stream_t *stream, int indx)
     for (i = indx+1; i < stream->ts_number_audio_pids; ++i)
     {
         stream->ts_audio_pids[indx] = stream->ts_audio_pids[i];
-        stream->ts_audio_stream_type[1 + indx] = stream->ts_audio_stream_type[1+i];
+        stream->ts_stream_type[1 + indx] = stream->ts_stream_type[1+i];
         stream->ts_streamid[1 + indx] = stream->ts_streamid[1 + i];
         ++indx;
     }
@@ -404,6 +600,9 @@ static int index_of_pid(int pid, hb_stream_t *stream)
  **********************************************************************/
 hb_title_t * hb_stream_title_scan(hb_stream_t *stream)
 {
+       if ( stream->hb_stream_type == ffmpeg )
+        return ffmpeg_title_scan( stream );
+
     // 'Barebones Title'
     hb_title_t *aTitle = hb_title_init( stream->path, 0 );
     aTitle->index = 1;
@@ -437,7 +636,7 @@ hb_title_t * hb_stream_title_scan(hb_stream_t *stream)
     //   the elementary stream is an audio type.
     // - For program streams read the first 4MB and take every unique
     //   audio stream we find.
-       if (stream->stream_type == hb_stream_type_transport)
+       if (stream->hb_stream_type == transport)
        {
         int i;
 
@@ -460,6 +659,10 @@ hb_title_t * hb_stream_title_scan(hb_stream_t *stream)
             stream->ts_audio_pids[stream->ts_number_audio_pids++] =
                 stream->pmt_info.PCR_PID;
         }
+
+        // set up the video codec to use for this title
+        aTitle->video_codec = st2codec[stream->ts_stream_type[0]].codec;
+        aTitle->video_codec_param = st2codec[stream->ts_stream_type[0]].codec_param;
        }
     else
     {
@@ -470,27 +673,80 @@ hb_title_t * hb_stream_title_scan(hb_stream_t *stream)
 }
 
 /*
+ * read the next transport stream packet from 'stream'. Return NULL if
+ * we hit eof & a pointer to the sync byte otherwise.
+ */
+static const uint8_t *next_packet( hb_stream_t *stream )
+{
+    uint8_t *buf = stream->ts_packet + stream->packetsize - 188;
+
+    while ( 1 )
+    {
+        if ( fread(stream->ts_packet, 1, stream->packetsize, stream->file_handle) !=
+             stream->packetsize )
+        {
+            return NULL;
+        }
+        if (buf[0] == 0x47)
+        {
+            return buf;
+        }
+        // lost sync - back up to where we started then try to re-establish.
+        off_t pos = ftello(stream->file_handle) - stream->packetsize;
+        off_t pos2 = align_to_next_packet(stream);
+        if ( pos2 == 0 )
+        {
+            hb_log( "next_packet: eof while re-establishing sync @ %lld", pos );
+            return NULL;
+        }
+        ts_warn( stream, "next_packet: sync lost @ %lld, regained after %lld bytes",
+                 pos, pos2 );
+    }
+}
+
+/*
+ * skip to the start of the next PACK header in program stream src_stream.
+ */
+static void skip_to_next_pack( hb_stream_t *src_stream )
+{
+    // scan forward until we find the start of the next pack
+    uint32_t strt_code = -1;
+    int c;
+
+    flockfile( src_stream->file_handle );
+    while ( ( c = getc_unlocked( src_stream->file_handle ) ) != EOF )
+    {
+        strt_code = ( strt_code << 8 ) | c;
+        if ( strt_code == 0x000001ba )
+            // we found the start of the next pack
+            break;
+    }
+    funlockfile( src_stream->file_handle );
+
+    // if we didn't terminate on an eof back up so the next read
+    // starts on the pack boundary.
+    if ( c != EOF )
+    {
+        fseeko( src_stream->file_handle, -4, SEEK_CUR );
+    }
+}
+
+/*
  * scan the next MB of 'stream' to find the next start packet for
  * the Packetized Elementary Stream associated with TS PID 'pid'.
  */
 static const uint8_t *hb_ts_stream_getPEStype(hb_stream_t *stream, uint32_t pid)
 {
-    static uint8_t buf[188];
-    int npack = 100000; // max packets to read
+    int npack = 300000; // max packets to read
 
     while (--npack >= 0)
     {
-        if (fread(buf, 1, 188, stream->file_handle) != 188)
+        const uint8_t *buf = next_packet( stream );
+        if ( buf == NULL )
         {
             hb_log("hb_ts_stream_getPEStype: EOF while searching for PID 0x%x", pid);
             return 0;
         }
-        if (buf[0] != 0x47)
-        {
-            hb_log("hb_ts_stream_getPEStype: lost sync while searching for PID 0x%x", pid);
-            align_to_next_packet(stream->file_handle);
-            continue;
-        }
 
         /*
          * The PES header is only in TS packets with 'start' set so we check
@@ -520,7 +776,11 @@ static const uint8_t *hb_ts_stream_getPEStype(hb_stream_t *stream, uint32_t pid)
                 udata += buf[4] + 1;
                 break;
         }
-        return &buf[udata];
+        /* PES hdr has to begin with an mpeg start code */
+        if (buf[udata+0] == 0x00 && buf[udata+1] == 0x00 && buf[udata+2] == 0x01)
+        {
+            return &buf[udata];
+        }
     }
 
     /* didn't find it */
@@ -540,7 +800,7 @@ static uint64_t hb_ps_stream_getVideoPTS(hb_stream_t *stream)
         hb_buffer_t *es;
 
         // 'buf' contains an MPEG2 PACK - get a list of all it's elementary streams
-        hb_demux_ps(buf, list);
+        hb_demux_ps( buf, list, 0 );
 
         while ( ( es = hb_list_item( list, 0 ) ) )
         {
@@ -603,18 +863,11 @@ static struct pts_pos hb_sample_pts(hb_stream_t *stream, uint64_t fpos)
 {
     struct pts_pos pp = { 0, 0 };
 
-    if ( stream->stream_type == hb_stream_type_program )
-    {
-        // round address down to nearest dvd sector start
-        fpos &=~ ( HB_DVD_READ_BUFFER_SIZE - 1 );
-        fseeko( stream->file_handle, fpos, SEEK_SET );
-        pp.pts = hb_ps_stream_getVideoPTS( stream );
-    }
-    else
+    if ( stream->hb_stream_type == transport )
     {
         const uint8_t *buf;
         fseeko( stream->file_handle, fpos, SEEK_SET );
-        align_to_next_packet( stream->file_handle );
+        align_to_next_packet( stream );
         buf = hb_ts_stream_getPEStype( stream, stream->ts_video_pids[0] );
         if ( buf == NULL )
         {
@@ -632,8 +885,18 @@ static struct pts_pos hb_sample_pts(hb_stream_t *stream, uint64_t fpos)
                  ( (uint64_t)buf[12] << 7 ) |
                  ( (uint64_t)buf[13] >> 1 );
     }
+    else
+    {
+        // round address down to nearest dvd sector start
+        fpos &=~ ( HB_DVD_READ_BUFFER_SIZE - 1 );
+        fseeko( stream->file_handle, fpos, SEEK_SET );
+        if ( stream->hb_stream_type == program )
+        {
+            skip_to_next_pack( stream );
+        }
+        pp.pts = hb_ps_stream_getVideoPTS( stream );
+    }
     pp.pos = ftello(stream->file_handle);
-    hb_log("hb_sample_pts: pts %lld at %llu", pp.pts, pp.pos );
     return pp;
 }
 
@@ -715,12 +978,63 @@ static void hb_stream_duration(hb_stream_t *stream, hb_title_t *inTitle)
  **********************************************************************/
 int hb_stream_read( hb_stream_t * src_stream, hb_buffer_t * b )
 {
-    if ( src_stream->stream_type == hb_stream_type_program )
+       if ( src_stream->hb_stream_type == ffmpeg )
+    {
+        return ffmpeg_read( src_stream, b );
+    }
+    if ( src_stream->hb_stream_type == dvd_program )
     {
         size_t amt_read = fread(b->data, HB_DVD_READ_BUFFER_SIZE, 1,
                                 src_stream->file_handle);
         return (amt_read > 0);
     }
+    if ( src_stream->hb_stream_type == program )
+    {
+        // a general program stream has arbitrary sized pack's. we're
+        // currently positioned at the start of a pack so read up to but
+        // not including the start of the next, expanding the buffer
+        // as necessary.
+        uint8_t *cp = b->data;
+        uint8_t *ep = cp + b->alloc;
+        uint32_t strt_code = -1;
+        int c;
+
+        // consume the first byte of the initial pack so we don't match on
+        // it in the loop below.
+        if ( ( c = getc( src_stream->file_handle ) ) == EOF )
+            return 0;
+
+        *cp++ = c;
+
+        flockfile( src_stream->file_handle );
+        while ( ( c = getc_unlocked( src_stream->file_handle ) ) != EOF )
+        {
+            strt_code = ( strt_code << 8 ) | c;
+            if ( strt_code == 0x000001ba )
+                // we found the start of the next pack
+                break;
+            if ( cp >= ep )
+            {
+                // need to expand the buffer
+                int curSize = cp - b->data;
+                hb_buffer_realloc( b, curSize * 2 );
+                cp = b->data + curSize;
+                ep = b->data + b->alloc;
+            }
+            *cp++ = c;
+        }
+        funlockfile( src_stream->file_handle );
+
+        // if we didn't terminate on an eof back up so the next read
+        // starts on the pack boundary.
+        b->size = cp - b->data;
+        if ( c != EOF )
+        {
+            fseeko( src_stream->file_handle, -4, SEEK_CUR );
+            b->size -= 4;
+        }
+        return 1;
+    }
     return hb_ts_stream_decode( src_stream, b->data );
 }
 
@@ -731,29 +1045,97 @@ int hb_stream_read( hb_stream_t * src_stream, hb_buffer_t * b )
  **********************************************************************/
 int hb_stream_seek( hb_stream_t * src_stream, float f )
 {
-  off_t stream_size, cur_pos, new_pos;
-  double pos_ratio = f;
-  cur_pos = ftello(src_stream->file_handle);
-  fseeko(src_stream->file_handle,0 ,SEEK_END);
-  stream_size = ftello(src_stream->file_handle);
-  new_pos = (off_t) ((double) (stream_size) * pos_ratio);
-  new_pos &=~ (HB_DVD_READ_BUFFER_SIZE - 1);
-  int r = fseeko(src_stream->file_handle, new_pos, SEEK_SET);
-
-  if (r == -1)
-  {
-    fseeko(src_stream->file_handle, cur_pos, SEEK_SET);
-    return 0;
-  }
+       if ( src_stream->hb_stream_type == ffmpeg )
+    {
+        return ffmpeg_seek( src_stream, f );
+    }
+    off_t stream_size, cur_pos, new_pos;
+    double pos_ratio = f;
+    cur_pos = ftello( src_stream->file_handle );
+    fseeko( src_stream->file_handle, 0, SEEK_END );
+    stream_size = ftello( src_stream->file_handle );
+    new_pos = (off_t) ((double) (stream_size) * pos_ratio);
+    new_pos &=~ (HB_DVD_READ_BUFFER_SIZE - 1);
+
+    int r = fseeko( src_stream->file_handle, new_pos, SEEK_SET );
+    if (r == -1)
+    {
+        fseeko( src_stream->file_handle, cur_pos, SEEK_SET );
+        return 0;
+    }
+
+    if ( src_stream->hb_stream_type == transport )
+    {
+        // We need to drop the current decoder output and move
+        // forwards to the next transport stream packet.
+        hb_ts_stream_reset(src_stream);
+    }
+    else if ( src_stream->hb_stream_type == program )
+    {
+        skip_to_next_pack( src_stream );
+    }
+
+    return 1;
+}
+
+static const char* make_upper( const char* s )
+{
+    static char name[8];
+    char *cp = name;
+    char *ep = cp + sizeof(name)-1;
+
+    while ( *s && cp < ep )
+    {
+        *cp++ = islower(*s)? toupper(*s) : *s;
+        ++s;
+    }
+    *cp = 0;
+    return name;
+}
 
-  if (src_stream->stream_type == hb_stream_type_transport)
-  {
-       // We need to drop the current decoder output and move
-       // forwards to the next transport stream packet.
-       hb_ts_stream_reset(src_stream);
-  }
+static void set_audio_description( hb_audio_t *audio, iso639_lang_t *lang )
+{
+    /* XXX
+     * This is a duplicate of code in dvd.c - it should get factored out
+     * into a common routine. We probably should only be putting the lang
+     * code or a lang pointer into the audio config & let the common description
+     * formatting routine in scan.c do all the stuff below.
+     */
+    const char *codec_name;
+    AVCodecContext *cc;
 
-  return 1;
+    if ( audio->config.in.codec == HB_ACODEC_FFMPEG &&
+         ( cc = hb_ffmpeg_context( audio->config.in.codec_param ) ) &&
+         avcodec_find_decoder( cc->codec_id ) )
+    {
+        codec_name = make_upper( avcodec_find_decoder( cc->codec_id )->name );
+        if ( !strcmp( codec_name, "LIBFAAD" ) )
+        {
+            codec_name = "AAC";
+        }
+    }
+    else if ( audio->config.in.codec == HB_ACODEC_MPGA &&
+              avcodec_find_decoder( audio->config.in.codec_param ) )
+    {
+        codec_name = avcodec_find_decoder( audio->config.in.codec_param )->name;
+    }
+    else
+    {
+        codec_name = audio->config.in.codec == HB_ACODEC_AC3 ? "AC3" :
+                     audio->config.in.codec == HB_ACODEC_DCA ? "DTS" :
+                     audio->config.in.codec == HB_ACODEC_MPGA ? "MPEG" : 
+                     audio->config.in.codec == HB_ACODEC_LPCM ? "LPCM" : 
+                     audio->config.in.codec == HB_ACODEC_FFMPEG ? "FFMPEG" :
+                     "Unknown";
+    }
+    snprintf( audio->config.lang.description,
+              sizeof( audio->config.lang.description ), "%s (%s)",
+              strlen(lang->native_name) ? lang->native_name : lang->eng_name,
+              codec_name );
+    snprintf( audio->config.lang.simple, sizeof( audio->config.lang.simple ), "%s",
+              strlen(lang->native_name) ? lang->native_name : lang->eng_name );
+    snprintf( audio->config.lang.iso639_2, sizeof( audio->config.lang.iso639_2 ),
+              "%s", lang->iso639_2);
 }
 
 static hb_audio_t *hb_ts_stream_set_audio_id_and_codec(hb_stream_t *stream,
@@ -764,42 +1146,73 @@ static hb_audio_t *hb_ts_stream_set_audio_id_and_codec(hb_stream_t *stream,
     const uint8_t *buf;
 
     fseeko(stream->file_handle, 0, SEEK_SET);
-    align_to_next_packet(stream->file_handle);
+    align_to_next_packet(stream);
     buf = hb_ts_stream_getPEStype(stream, stream->ts_audio_pids[aud_pid_index]);
 
     /* check that we found a PES header */
+    uint8_t stype = 0;
     if (buf && buf[0] == 0x00 && buf[1] == 0x00 && buf[2] == 0x01)
     {
-        if (buf[3] == 0xbd)
+        // 0xbd is the normal container for AC3/DCA/PCM/etc. 0xfd indicates an
+        // extended stream id (ISO 13818-1(2007)). If we cared about the
+        // real id we'd have to look inside the PES extension to find it.
+        // But since we remap stream id's when we generate PS packets from
+        // the TS packets we can just ignore the actual id.
+        if ( buf[3] == 0xbd || buf[3] == 0xfd )
         {
             audio->id = 0x80bd | (aud_pid_index << 8);
-            audio->config.in.codec = HB_ACODEC_AC3;
-            hb_log("transport stream pid 0x%x (type 0x%x) is AC-3 audio id 0x%x",
-                   stream->ts_audio_pids[aud_pid_index],
-                   stream->ts_audio_stream_type[1 + aud_pid_index],
-                   audio->id);
-            stream->ts_audio_stream_type[1 + aud_pid_index] = 0x81;
-            stream->ts_streamid[1 + aud_pid_index] = buf[3];
+            stype = stream->ts_stream_type[1 + aud_pid_index];
+            if ( st2codec[stype].kind == U )
+            {
+                // XXX assume unknown stream types are AC-3 (if they're not
+                // audio we'll find that out during the scan but if they're
+                // some other type of audio we'll end up ignoring them).
+                stype = 0x81;
+                stream->ts_stream_type[1 + aud_pid_index] = 0x81;
+            }
+            stream->ts_streamid[1 + aud_pid_index] = 0xbd;
         }
         else if ((buf[3] & 0xe0) == 0xc0)
         {
-            audio->id = buf[3] | aud_pid_index;
-            audio->config.in.codec = HB_ACODEC_MPGA;
-            hb_log("transport stream pid 0x%x (type 0x%x) is MPEG audio id 0x%x",
-                   stream->ts_audio_pids[aud_pid_index],
-                   stream->ts_audio_stream_type[1 + aud_pid_index],
-                   audio->id);
-            stream->ts_audio_stream_type[1 + aud_pid_index] = 0x03;
-            stream->ts_streamid[1 + aud_pid_index] = buf[3];
+            audio->id = 0xc0 | aud_pid_index;
+            stype = stream->ts_stream_type[1 + aud_pid_index];
+            if ( st2codec[stype].kind == U )
+            {
+                // XXX assume unknown stream types are MPEG audio
+                stype = 0x03;
+                stream->ts_stream_type[1 + aud_pid_index] = 0x03;
+            }
         }
     }
-    fseeko(stream->file_handle, cur_pos, SEEK_SET);
-    if (! audio->config.in.codec)
+    // if we found an audio stream type & HB has a codec that can decode it
+    // finish configuring the audio so we'll add it to the title's list.
+    if ( st2codec[stype].kind == A && st2codec[stype].codec )
+    {
+        stream->ts_streamid[1 + aud_pid_index] = audio->id;
+        audio->config.in.codec = st2codec[stype].codec;
+        audio->config.in.codec_param = st2codec[stype].codec_param;
+               set_audio_description( audio,
+                  lang_for_code( stream->a52_info[aud_pid_index].lang_code ) );
+        hb_log("transport stream pid 0x%x (type 0x%x) is %s audio id 0x%x",
+               stream->ts_audio_pids[aud_pid_index],
+               stype, st2codec[stype].name, audio->id);
+    }
+    else
     {
-        hb_log("transport stream pid 0x%x (type 0x%x) isn't audio",
-                stream->ts_audio_pids[aud_pid_index],
-                stream->ts_audio_stream_type[1 + aud_pid_index]);
+        if ( buf )
+        {
+            hb_log("transport stream pid 0x%x (type 0x%x, substream 0x%x) "
+                    "isn't audio", stream->ts_audio_pids[aud_pid_index],
+                    stream->ts_stream_type[1 + aud_pid_index], buf[3]);
+        }
+        else
+        {
+            hb_log("transport stream pid 0x%x (type 0x%x) isn't audio",
+                    stream->ts_audio_pids[aud_pid_index],
+                    stream->ts_stream_type[1 + aud_pid_index]);
+        }
        }
+    fseeko(stream->file_handle, cur_pos, SEEK_SET);
     return audio;
 }
 
@@ -832,6 +1245,7 @@ static void add_audio_to_title(hb_title_t *title, int id)
             return;
 
     }
+    set_audio_description( audio, lang_for_code( 0 ) );
     hb_list_add( title->list_audio, audio );
 }
 
@@ -857,7 +1271,7 @@ static void hb_ps_stream_find_audio_ids(hb_stream_t *stream, hb_title_t *title)
         hb_buffer_t *es;
 
         // 'buf' contains an MPEG2 PACK - get a list of all it's elementary streams
-        hb_demux_ps(buf, list);
+        hb_demux_ps( buf, list, 0 );
 
         while ( ( es = hb_list_item( list, 0 ) ) )
         {
@@ -884,188 +1298,72 @@ static void hb_ps_stream_find_audio_ids(hb_stream_t *stream, hb_title_t *title)
 }
 
 /***********************************************************************
- * hb_stream_update_audio
+ * hb_ts_stream_init
  ***********************************************************************
  *
  **********************************************************************/
-void hb_stream_update_audio(hb_stream_t *stream, hb_audio_t *audio)
+
+static void hb_ts_stream_init(hb_stream_t *stream)
 {
-       iso639_lang_t *lang;
+       int i;
 
-    if (stream->stream_type == hb_stream_type_transport)
+       for (i=0; i < kMaxNumberDecodeStreams; i++)
        {
-        // Find the audio stream info for this PID. The stream index is
-        // the subchannel id which is in the bottom four bits for MPEG audio
-        // and the bottom four bits of the upper byte for everything else.
-        int i = ( audio->id >= 0xd0 ? audio->id >> 8 : audio->id ) & 0xf;
-        if (i >= stream->ts_number_audio_pids)
-               {
-            hb_log("hb_stream_update_audio: no PID for audio stream 0x%x",
-                    audio->id);
-                       return;
-               }
-        if (audio->id < 0xd0)
-        {
-            /* XXX fake mpeg audio sample rate & bps */
-            stream->a52_info[i].flags = A52_STEREO;
-            stream->a52_info[i].rate = 48000 /*Hz*/;
-            stream->a52_info[i].bitrate = 384000 /*Bps*/;
-        }
-
-               lang = lang_for_code(stream->a52_info[i].lang_code);
-        if (!audio->config.in.samplerate)
-            audio->config.in.samplerate = stream->a52_info[i].rate;
-        if (!audio->config.in.bitrate)
-            audio->config.in.bitrate = stream->a52_info[i].bitrate;
-        if (!audio->priv.config.a52.ac3flags)
-            audio->priv.config.a52.ac3flags = audio->config.flags.ac3 = stream->a52_info[i].flags;
-
+               stream->ts_streamcont[i] = -1;
        }
-    else
+       stream->ts_video_pids[0] = -1;
+    for ( i = 0; i < stream->ts_number_audio_pids; i++ )
     {
-        // XXX should try to get language code from the AC3 bitstream
-        lang = lang_for_code(0x0000);
+        stream-> ts_audio_pids[i] = -1;
     }
 
-       if (!audio->config.in.channel_layout)
-       {
-               switch( audio->config.flags.ac3 & A52_CHANNEL_MASK )
-               {
-                       /* mono sources */
-                       case A52_MONO:
-                       case A52_CHANNEL1:
-                       case A52_CHANNEL2:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_MONO;
-                               break;
-                       /* stereo input */
-                       case A52_CHANNEL:
-                       case A52_STEREO:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_STEREO;
-                               break;
-                       /* dolby (DPL1 aka Dolby Surround = 4.0 matrix-encoded) input */
-                       case A52_DOLBY:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_DOLBY;
-                               break;
-                       /* 3F/2R input */
-                       case A52_3F2R:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_3F2R;
-                               break;
-                       /* 3F/1R input */
-                       case A52_3F1R:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_3F1R;
-                               break;
-                       /* other inputs */
-                       case A52_3F:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_3F;
-                               break;
-                       case A52_2F1R:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_2F1R;
-                               break;
-                       case A52_2F2R:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_2F2R;
-                               break;
-                       /* unknown */
-                       default:
-                               audio->config.in.channel_layout = HB_INPUT_CH_LAYOUT_STEREO;
-               }
+    stream->ts_packet = malloc( stream->packetsize );
 
-               /* add in our own LFE flag if the source has LFE */
-               if (audio->config.flags.ac3 & A52_LFE)
-               {
-                       audio->config.in.channel_layout = audio->config.in.channel_layout | HB_INPUT_CH_LAYOUT_HAS_LFE;
-               }
-       }
+       // Find the audio and video pids in the stream
+       hb_ts_stream_find_pids(stream);
 
-    snprintf( audio->config.lang.description, sizeof( audio->config.lang.description ), "%s (%s)", strlen(lang->native_name) ? lang->native_name : lang->eng_name,
-         audio->config.in.codec == HB_ACODEC_AC3 ? "AC3" : ( audio->config.in.codec == HB_ACODEC_MPGA ? "MPEG" : ( audio->config.in.codec == HB_ACODEC_DCA ? "DTS" : "LPCM" ) ) );
-       snprintf( audio->config.lang.simple, sizeof( audio->config.lang.simple ), "%s", strlen(lang->native_name) ? lang->native_name : lang->eng_name );
-       snprintf( audio->config.lang.iso639_2, sizeof( audio->config.lang.iso639_2 ), "%s", lang->iso639_2);
-
-       if ( (audio->config.flags.ac3 & A52_CHANNEL_MASK) == A52_DOLBY ) {
-        sprintf( audio->config.lang.description + strlen( audio->config.lang.description ),
-                        " (Dolby Surround)" );
-       } else {
-        sprintf( audio->config.lang.description + strlen( audio->config.lang.description ),
-                        " (%d.%d ch)",
-                       HB_INPUT_CH_LAYOUT_GET_DISCRETE_FRONT_COUNT(audio->config.in.channel_layout) +
-                       HB_INPUT_CH_LAYOUT_GET_DISCRETE_REAR_COUNT(audio->config.in.channel_layout),
-                       HB_INPUT_CH_LAYOUT_GET_DISCRETE_LFE_COUNT(audio->config.in.channel_layout));
+       for (i = 0; i < stream->ts_number_video_pids + stream->ts_number_audio_pids; i++)
+       {
+        // demuxing buffer for TS to PS conversion
+               stream->ts_buf[i] = malloc( HB_DVD_READ_BUFFER_SIZE );
        }
 
-       hb_log( "stream: audio %x: lang %s, rate %d, bitrate %d, "
-            "flags = 0x%x", audio->id, audio->config.lang.description, audio->config.in.samplerate,
-            audio->config.in.bitrate, audio->config.flags.ac3 );
-
+    stream->ts_streamid[0] = 0xE0;             // stream 0 must be video
 }
 
-/***********************************************************************
- * hb_ts_stream_init
- ***********************************************************************
- *
- **********************************************************************/
+#define MAX_HOLE 208*80
 
-static void hb_ts_stream_init(hb_stream_t *stream)
+static off_t align_to_next_packet(hb_stream_t *stream)
 {
-       int i;
+    uint8_t buf[MAX_HOLE];
+       off_t pos = 0;
+    off_t start = ftello(stream->file_handle);
 
-       for (i=0; i < kMaxNumberDecodeStreams; i++)
-       {
-               stream->ts_streamcont[i] = -1;
-       }
-       stream->ts_video_pids[0] = -1;
-    for ( i = 0; i < stream->ts_number_audio_pids; i++ )
-    {
-        stream-> ts_audio_pids[i] = -1;
+    if ( start >= stream->packetsize ) {
+        start -= stream->packetsize;
+        fseeko(stream->file_handle, start, SEEK_SET);
     }
 
-       // Find the audio and video pids in the stream
-       hb_ts_stream_find_pids(stream);
-
-    stream->ts_streamid[0] = 0xE0;             // stream 0 must be video
-
-       for (i = 0; i < stream->ts_number_video_pids + stream->ts_number_audio_pids; i++)
+    if (fread(buf, sizeof(buf), 1, stream->file_handle) == 1)
        {
-        // demuxing buffer for TS to PS conversion
-               stream->ts_buf[i] = malloc( HB_DVD_READ_BUFFER_SIZE );
-       }
-}
-
-// ------------------------------------------------------------------------------------
-
-static off_t align_to_next_packet(FILE* f)
-{
-       unsigned char buf[188*20];
-
-       off_t start = ftello(f);
-       off_t pos = 0;
+        const uint8_t *bp = buf;
+        int i;
 
-       if (fread(buf, 188*20, 1, f) == 1)
-       {
-               int found = 0;
-               while (!found && (pos < 188))
-               {
-                       found = 1;
-                       int i = 0;
-                       for (i = 0; i < 188*20; i += 188)
-                       {
-                               unsigned char c = buf[pos+i];
-                               // Check sync byte
-                               if ((c != 0x47) && (c != 0x72) && (c != 0x29))
-                               {
-                                       // this offset failed, try next
-                                       found = 0;
-                                       pos++;
-                                       break;
-                               }
-                       }
-               }
+        for ( i = sizeof(buf); --i >= 0; ++bp )
+        {
+            if ( have_ts_sync( bp, stream->packetsize ) )
+            {
+                break;
+            }
+        }
+        if ( i >= 0 )
+        {
+            pos = ( bp - buf ) - stream->packetsize + 188;
+            if ( pos < 0 )
+                pos = 0;
+        }
        }
-
-       if (pos == 188)
-               pos = 0;                // failed to find anything!!!!!?
-
-    fseeko(f, start+pos, SEEK_SET);
-
+    fseeko(stream->file_handle, start+pos, SEEK_SET);
        return pos;
 }
 
@@ -1151,6 +1449,11 @@ static void decode_element_descriptors(hb_stream_t* stream, int esindx,
     }
 }
 
+static const char *stream_type_name (uint8_t stream_type)
+{
+    return st2codec[stream_type].name? st2codec[stream_type].name : "Unknown";
+}
+
 int decode_program_map(hb_stream_t* stream)
 {
     bitbuf_t bb;
@@ -1185,61 +1488,62 @@ int decode_program_map(hb_stream_t* stream)
        int cur_pos =  9 /* data after the section length field*/ + program_info_length;
        int done_reading_stream_types = 0;
        while (!done_reading_stream_types)
-       {
-         unsigned char stream_type = get_bits(&bb, 8);
-                 get_bits(&bb, 3);
-         unsigned int elementary_PID = get_bits(&bb, 13);
-                 get_bits(&bb, 4);
-         unsigned int ES_info_length = get_bits(&bb, 12);
-
-         int i=0;
-         unsigned char *ES_info_buf = (unsigned char *) malloc(ES_info_length);
-         for (i=0; i < ES_info_length; i++)
-         {
-               ES_info_buf[i] = get_bits(&bb, 8);
-         }
-
-         if (stream_type == 0x02)
-         {
-               if (stream->ts_number_video_pids <= kMaxNumberVideoPIDS)
-                 stream->ts_number_video_pids++;
-               stream->ts_video_pids[stream->ts_number_video_pids-1] = elementary_PID;
-         }
-      else
-      {
-        // Defined audio stream types are 0x81 for AC-3/A52 audio and 0x03
-        // for mpeg audio. But content producers seem to use other
-        // values (0x04 and 0x06 have both been observed) so at this point
-        // we say everything that isn't a video pid is audio then at the end
-        // of hb_stream_title_scan we'll figure out which are really audio
-        // by looking at the PES headers.
-        i = stream->ts_number_audio_pids;
-        if (i < kMaxNumberAudioPIDS)
-            stream->ts_number_audio_pids++;
-        stream->ts_audio_pids[i] = elementary_PID;
-        stream->ts_audio_stream_type[1 + i] = stream_type;
-
-               if (ES_info_length > 0)
-               {
-            decode_element_descriptors(stream, i, ES_info_buf, ES_info_length);
-               }
-         }
+    {
+        unsigned char stream_type = get_bits(&bb, 8);
+        get_bits(&bb, 3);
+        unsigned int elementary_PID = get_bits(&bb, 13);
+        get_bits(&bb, 4);
+        unsigned int ES_info_length = get_bits(&bb, 12);
+
+        int i=0;
+        unsigned char *ES_info_buf = (unsigned char *) malloc(ES_info_length);
+        for (i=0; i < ES_info_length; i++)
+        {
+            ES_info_buf[i] = get_bits(&bb, 8);
+        }
 
-         cur_pos += 5 /* stream header */ + ES_info_length;
 
-         free(ES_info_buf);
+        if (stream->ts_number_video_pids == 0 && st2codec[stream_type].kind == V )
+        {
+            stream->ts_video_pids[0] = elementary_PID;
+            stream->ts_stream_type[0] = stream_type;
+            stream->ts_number_video_pids = 1;
+        }
+        else
+        {
+            // Defined audio stream types are 0x81 for AC-3/A52 audio and 0x03
+            // for mpeg audio. But content producers seem to use other
+            // values (0x04 and 0x06 have both been observed) so at this point
+            // we say everything that isn't a video pid is audio then at the end
+            // of hb_stream_title_scan we'll figure out which are really audio
+            // by looking at the PES headers.
+            i = stream->ts_number_audio_pids;
+            if (i < kMaxNumberAudioPIDS)
+            {
+                stream->ts_audio_pids[i] = elementary_PID;
+                stream->ts_stream_type[1 + i] = stream_type;
+                if (ES_info_length > 0)
+                {
+                    decode_element_descriptors(stream, i, ES_info_buf,
+                                               ES_info_length);
+                }
+                ++stream->ts_number_audio_pids;
+            }
+        }
 
-         if (cur_pos >= section_length - 4 /* stop before the CRC */)
-               done_reading_stream_types = 1;
-       }
+        cur_pos += 5 /* stream header */ + ES_info_length;
+
+        free(ES_info_buf);
+
+        if (cur_pos >= section_length - 4 /* stop before the CRC */)
+        done_reading_stream_types = 1;
+    }
 
        free(descriptor_buf);
        return 1;
 }
 
-// ------------------------------------------------------------------------------------
-
-int build_program_map(unsigned char *buf, hb_stream_t *stream)
+static int build_program_map(const uint8_t *buf, hb_stream_t *stream)
 {
     // Get adaption header info
     int adapt_len = 0;
@@ -1301,7 +1605,7 @@ int build_program_map(unsigned char *buf, hb_stream_t *stream)
     return 0;
 }
 
-int decode_PAT(unsigned char *buf, hb_stream_t *stream)
+static int decode_PAT(const uint8_t *buf, hb_stream_t *stream)
 {
     unsigned char tablebuf[1024];
     unsigned int tablepos = 0;
@@ -1413,48 +1717,20 @@ int decode_PAT(unsigned char *buf, hb_stream_t *stream)
 
 static void hb_ts_stream_find_pids(hb_stream_t *stream)
 {
-       unsigned char buf[188];
-
        // align to first packet
-       align_to_next_packet(stream->file_handle);
+    align_to_next_packet(stream);
 
        // Read the Transport Stream Packets (188 bytes each) looking at first for PID 0 (the PAT PID), then decode that
        // to find the program map PID and then decode that to get the list of audio and video PIDs
 
-       int bytesReadInPacket = 0;
        for (;;)
        {
-               // Try to read packet..
-               int bytesRead;
-               if ((bytesRead = fread(buf+bytesReadInPacket, 1, 188-bytesReadInPacket, stream->file_handle)) != 188-bytesReadInPacket)
-               {
-                       if (bytesRead < 0)
-                               bytesRead = 0;
-                       bytesReadInPacket += bytesRead;
-
+        const uint8_t *buf = next_packet( stream );
+        if ( buf == NULL )
+        {
                        hb_log("hb_ts_stream_find_pids - end of file");
                        break;
                }
-               else
-               {
-                       bytesReadInPacket = 0;
-               }
-
-               // Check sync byte
-               if ((buf[0] != 0x47) && (buf[0] != 0x72) && (buf[0] != 0x29))
-               {
-            off_t pos = ftello(stream->file_handle) - 188;
-            off_t pos2 = align_to_next_packet(stream->file_handle);
-            if ( pos2 == 0 )
-            {
-                hb_log( "hb_ts_stream_find_pids: eof while re-establishing sync @ %lld",
-                        pos );
-                break;
-            }
-            hb_log("hb_ts_stream_decode: sync lost @%lld, regained after %lld bytes",
-                    pos, pos2 );
-                       continue;
-               }
 
                // Get pid
                int pid = (((buf[1] & 0x1F) << 8) | buf[2]) & 0x1FFF;
@@ -1492,15 +1768,21 @@ static void hb_ts_stream_find_pids(hb_stream_t *stream)
 
        hb_log("hb_ts_stream_find_pids - found the following PIDS");
        hb_log("    Video PIDS : ");
-       int i=0;
+    int i;
        for (i=0; i < stream->ts_number_video_pids; i++)
        {
-               hb_log("      0x%x (%d)", stream->ts_video_pids[i], stream->ts_video_pids[i]);
+        hb_log( "      0x%x type %s (0x%x)", 
+                stream->ts_video_pids[i],
+                stream_type_name(stream->ts_stream_type[i]),
+                stream->ts_stream_type[i]);
        }
        hb_log("    Audio PIDS : ");
        for (i = 0; i < stream->ts_number_audio_pids; i++)
        {
-               hb_log("      0x%x (%d)", stream->ts_audio_pids[i], stream->ts_audio_pids[i]);
+        hb_log( "      0x%x type %s (0x%x)", 
+                stream->ts_audio_pids[i],
+                stream_type_name(stream->ts_stream_type[i+1]),
+                stream->ts_stream_type[i+1] );
        }
  }
 
@@ -1592,9 +1874,9 @@ static void generate_output_data(hb_stream_t *stream, int curstream)
     int len;
 
     // we always ship a PACK header plus all the data in our demux buf.
-    // AC3 audio also always needs it substream header.
+    // AC3 audio also always needs its substream header.
     len = 14 + stream->ts_pos[curstream];
-    if ( stream->ts_audio_stream_type[curstream] == 0x81)
+    if ( stream->ts_stream_type[curstream] == 0x81)
     {
         len += 4;
     }
@@ -1635,7 +1917,7 @@ static void generate_output_data(hb_stream_t *stream, int curstream)
         tdat[3] = stream->ts_streamid[curstream];
 
         uint16_t plen = stream->ts_pos[curstream] - 6;
-        if ( stream->ts_audio_stream_type[curstream] == 0x81)
+        if ( stream->ts_stream_type[curstream] == 0x81)
         {
             // We have to add an AC3 header in front of the data. Add its
             // size to the PES packet length.
@@ -1671,7 +1953,7 @@ static void generate_output_data(hb_stream_t *stream, int curstream)
     {
         // data without a PES start header needs a simple 'continuation'
         // PES header. AC3 audio also needs its substream header.
-        if ( stream->ts_audio_stream_type[curstream] != 0x81)
+        if ( stream->ts_stream_type[curstream] != 0x81)
         {
             make_pes_header(stream, stream->ts_pos[curstream],
                             stream->ts_streamid[curstream]);
@@ -1703,84 +1985,82 @@ static void generate_output_data(hb_stream_t *stream, int curstream)
     stream->ts_pos[curstream] = 0;
 }
 
-static void ts_warn_helper( hb_stream_t *stream, char *log, va_list args )
+static int isIframe( hb_stream_t *stream, const uint8_t *buf, int adapt_len )
 {
-    // limit error printing to at most one per minute of video (at 30fps)
-    ++stream->errors;
-    if ( stream->frames - stream->last_error_frame >= 30*60 )
-    {
-        char msg[256];
+    // For mpeg2: look for a gop start or i-frame picture start
+    // for h.264: look for idr nal type or a slice header for an i-frame
+    // for vc1:   ???
+    int i;
+    uint32_t strid = 0;
 
-        vsnprintf( msg, sizeof(msg), log, args );
 
-        if ( stream->errors - stream->last_error_count < 10 )
-        {
-            hb_log( "stream: error near frame %d: %s", stream->frames, msg );
-        }
-        else
+    if ( stream->ts_stream_type[0] <= 2 )
+    {
+        // This section of the code handles MPEG-1 and MPEG-2 video streams
+        for (i = 13 + adapt_len; i < 188; i++)
         {
-            int Edelta = stream->errors - stream->last_error_count;
-            double Epcnt = (double)Edelta * 100. /
-                            (stream->frames - stream->last_error_frame);
-            hb_log( "stream: %d new errors (%.0f%%) up to frame %d: %s",
-                    Edelta, Epcnt, stream->frames, msg );
+            strid = (strid << 8) | buf[i];
+            if ( ( strid >> 8 ) == 1 )
+            {
+                // we found a start code
+                uint8_t id = strid;
+                switch ( id )
+                {
+                    case 0xB8: // group_start_code (GOP header)
+                    case 0xB3: // sequence_header code
+                        return 1;
+
+                    case 0x00: // picture_start_code
+                        // picture_header, let's see if it's an I-frame
+                        if (i<185)
+                        {
+                            // check if picture_coding_type == 1
+                            if ((buf[i+2] & (0x7 << 3)) == (1 << 3))
+                            {
+                                // found an I-frame picture
+                                return 1;
+                            }
+                        }
+                        break;
+                }
+            }
         }
-        stream->last_error_frame = stream->frames;
-        stream->last_error_count = stream->errors;
+        // didn't find an I-frame
+        return 0;
     }
-}
-
-static void ts_warn( hb_stream_t *stream, char *log, ... )
-{
-    va_list     args;
-    va_start( args, log );
-    ts_warn_helper( stream, log, args );
-    va_end( args );
-}
-
-static void ts_err( hb_stream_t *stream, int curstream, char *log, ... )
-{
-    va_list     args;
-    va_start( args, log );
-    ts_warn_helper( stream, log, args );
-    va_end( args );
-
-    stream->ts_skipbad[curstream] = 1;
-    stream->ts_pos[curstream] = 0;
-    stream->ts_streamcont[curstream] = -1;
-}
-
-static int isIframe( const uint8_t *buf, int adapt_len )
-{
-    // Look for the Group of Pictures packet
-    int i;
-    uint32_t strid = 0;
-
-    for (i = 4 + adapt_len; i < 188; i++)
+    if ( stream->ts_stream_type[0] == 0x1b )
     {
-        strid = (strid << 8) | buf[i];
-        switch ( strid )
+        // we have an h.264 stream 
+        for (i = 13 + adapt_len; i < 188; i++)
         {
-            case 0x000001B8: // group_start_code (GOP header)
-            case 0x000001B3: // sequence_header code
-                return 1;
+            strid = (strid << 8) | buf[i];
+            if ( ( strid >> 8 ) == 1 )
+            {
+                // we found a start code - remove the ref_idc from the nal type
+                uint8_t nal_type = strid & 0x1f;
+                if ( nal_type == 0x05 )
+                    // h.264 IDR picture start
+                    return 1;
 
-            case 0x00000100: // picture_start_code
-                // picture_header, let's see if it's an I-frame
-                if (i<185)
+                if ( nal_type == 0x01 )
                 {
-                    // check if picture_coding_type == 1
-                    if ((buf[i+2] & (0x7 << 3)) == (1 << 3))
+                    // h.264 slice: has to be start MB 0 & type I (2, 4, 7 or 9)
+                    uint8_t id = buf[i+1];
+                    if ( ( id >> 4 ) == 0x0b || ( id >> 2 ) == 0x25 ||
+                         id == 0x88 || id == 0x8a )
                     {
-                        // found an I-frame picture
                         return 1;
                     }
                 }
-                break;
+            }
         }
+        // didn't find an I-frame
+        return 0;
     }
-    // didn't find an I frame
-    return 0;
+
+    // we don't understand the stream type so just say "yes" otherwise
+    // we'll discard all the video.
+    return 1;
 }
 
 /***********************************************************************
@@ -1790,10 +2070,6 @@ static int isIframe( const uint8_t *buf, int adapt_len )
  **********************************************************************/
 static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
 {
-    int64_t pcr = stream->ts_lastpcr;
-       int curstream;
-       uint8_t buf[188];
-
     /*
      * stash the output buffer pointer in our stream so we don't have to
      * pass it & its original value to everything we call.
@@ -1804,8 +2080,12 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
        // spin until we get a packet of data from some stream or hit eof
        while ( 1 )
        {
-        if ((fread(buf, 188, 1, stream->file_handle)) != 1)
-               {
+        int64_t pcr = stream->ts_lastpcr;
+        int curstream;
+
+        const uint8_t *buf = next_packet(stream);
+        if ( buf == NULL )
+        {
             // end of file - we didn't finish filling our ps write buffer
             // so just discard the remainder (the partial buffer is useless)
             hb_log("hb_ts_stream_decode - eof");
@@ -1814,24 +2094,6 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
 
         /* This next section validates the packet */
 
-               // Check sync byte
-               if ((buf[0] != 0x47) && (buf[0] != 0x72) && (buf[0] != 0x29))
-               {
-            // lost sync - back up to where we started then try to
-            // re-establish sync.
-            off_t pos = ftello(stream->file_handle) - 188;
-            off_t pos2 = align_to_next_packet(stream->file_handle);
-            if ( pos2 == 0 )
-            {
-                hb_log( "hb_ts_stream_decode: eof while re-establishing sync @ %lld",
-                        pos );
-                return 0;
-            }
-            ts_warn( stream, "hb_ts_stream_decode: sync lost @%lld, "
-                     "regained after %lld bytes", pos, pos2 );
-                       continue;
-               }
-
                // Get pid and use it to find stream state.
                int pid = ((buf[1] & 0x1F) << 8) | buf[2];
         if ( ( curstream = index_of_pid( pid, stream ) ) < 0 )
@@ -1880,11 +2142,11 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
             stream->ts_lastpcr = pcr;
         }
 
-               if ( pcr == -1 )
-               {
-            // don't accumulate data until we get a pcr
-                   continue;
-               }
+        // If we don't have a pcr yet, the right thing to do here would
+        // be a 'continue' so we don't process anything until we have a
+        // clock reference. Unfortunately the HD Home Run appears to null
+        // out the pcr field of some streams so we keep going & substitute
+        // the video stream dts for the pcr when there's no pcr.
 
                // Get continuity
         // Continuity only increments for adaption values of 0x3 or 0x01
@@ -1904,6 +2166,7 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
                 continue;
             }
             if ( !start && (stream->ts_streamcont[curstream] != -1) &&
+                 stream->ts_foundfirst[curstream] &&
                  (continuity != ( (stream->ts_streamcont[curstream] + 1) & 0xf ) ) )
                        {
                                ts_err( stream, curstream,  "continuity error: got %d expected %d",
@@ -1926,7 +2189,7 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
                        {
                 // video skips to an iframe after a bad packet to minimize
                 // screen corruption
-                if ( curstream == 0 && !isIframe( buf, adapt_len ) )
+                if ( curstream == 0 && !isIframe( stream, buf, adapt_len ) )
                 {
                     continue;
                 }
@@ -1939,7 +2202,7 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
             {
                 if ( !stream->ts_foundfirst[0] )
                 {
-                    if ( !isIframe( buf, adapt_len ) )
+                    if ( !isIframe( stream, buf, adapt_len ) )
                     {
                         // didn't find an I frame
                         continue;
@@ -1947,8 +2210,30 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
                     stream->ts_foundfirst[0] = 1;
                 }
                 ++stream->frames;
+
+                // if we don't have a pcr yet use the dts from this frame
+                if ( pcr == -1 )
+                {
+                    // PES must begin with an mpeg start code & contain
+                    // a DTS or PTS.
+                    const uint8_t *pes = buf + adapt_len + 4;
+                    if ( pes[0] != 0x00 || pes[1] != 0x00 || pes[2] != 0x01 ||
+                         ( pes[7] >> 6 ) == 0 )
+                    {
+                        continue;
+                    }
+                    // if we have a dts use it otherwise use the pts
+                    pes += (pes[7] & 0x40)? 14 : 9;
+
+                    pcr = ( (uint64_t)(pes[0] & 0xe ) << 29 );
+                    pcr |= ( pes[1] << 22 ) |
+                           ( ( pes[2] >> 1 ) << 15 ) |
+                           ( pes[3] << 7 ) |
+                           ( pes[4] >> 1 );
+                    stream->ts_nextpcr = pcr;
+                }
             }
-                       else if ( ! stream->ts_foundfirst[curstream] )
+            else if ( ! stream->ts_foundfirst[curstream] )
             {
                 // start other streams only after first video frame found.
                 if ( ! stream->ts_foundfirst[0] )
@@ -1994,11 +2279,6 @@ static int hb_ts_stream_decode( hb_stream_t *stream, uint8_t *obuf )
        }
 }
 
-/***********************************************************************
- * hb_ts_stream_reset
- ***********************************************************************
- *
- **********************************************************************/
 static void hb_ts_stream_reset(hb_stream_t *stream)
 {
        int i;
@@ -2020,6 +2300,369 @@ static void hb_ts_stream_reset(hb_stream_t *stream)
     stream->last_error_frame = -10000;
     stream->last_error_count = 0;
 
-       align_to_next_packet(stream->file_handle);
+    align_to_next_packet(stream);
+}
+
+// ------------------------------------------------------------------
+// Support for reading media files via the ffmpeg libraries.
+
+static void ffmpeg_add_codec( hb_stream_t *stream, int stream_index )
+{
+    // add a codec to the context here so it will be there when we
+    // read the first packet.
+    AVCodecContext *context = stream->ffmpeg_ic->streams[stream_index]->codec;
+    context->workaround_bugs = FF_BUG_AUTODETECT;
+    context->error_recognition = 1;
+    context->error_concealment = FF_EC_GUESS_MVS|FF_EC_DEBLOCK;
+    AVCodec *codec = avcodec_find_decoder( context->codec_id );
+    avcodec_open( context, codec );
+}
+
+// The ffmpeg stream reader / parser shares a lot of state with the 
+// decoder via a codec context kept in the AVStream of the reader's
+// AVFormatContext. Since decoding is done in a different thread we
+// have to somehow pass this codec context to the decoder and we have
+// to do it before the first packet is read (so we can't put the info
+// in the buf we'll send downstream). Decoders don't have any way to
+// get to the stream directly (they're not passed the title or job
+// pointers during a scan) so this is a back door for the decoder to
+// get the codec context. We just stick the stream pointer in the next
+// slot an array of pointers maintained as a circular list then return
+// the index into the list combined with the ffmpeg stream index as the
+// codec_param that will be passed to the decoder init routine. We make
+// the list 'big' (enough for 1024 simultaneously open ffmpeg streams)
+// so that we don't have to do a complicated allocator or worry about
+// deleting entries on close. 
+//
+// Entries can only be added to this list during a scan and are never
+// deleted so the list access doesn't require locking.
+static hb_stream_t **ffmpeg_streams;    // circular list of stream pointers
+static int ffmpeg_stream_cur;           // where we put the last stream pointer
+#define ffmpeg_sl_bits (10)             // log2 stream list size (in entries)
+#define ffmpeg_sl_size (1 << ffmpeg_sl_bits)
+
+// add a stream to the list & return the appropriate codec_param to access it
+static int ffmpeg_codec_param( hb_stream_t *stream, int stream_index )
+{
+    if ( !ffmpeg_streams )
+    {
+        ffmpeg_streams = calloc( ffmpeg_sl_size, sizeof(stream) );
+    }
+
+    // the title scan adds all the ffmpeg media streams at once so we
+    // only add a new entry to our stream list if the stream is different
+    // than last time.
+    int slot = ffmpeg_stream_cur;
+    if ( ffmpeg_streams[slot] != stream )
+    {
+        // new stream - put it in the next slot of the stream list
+        slot = ++ffmpeg_stream_cur & (ffmpeg_sl_size - 1);
+        ffmpeg_streams[slot] = stream;
+    }
+
+    ffmpeg_add_codec( stream, stream_index );
+
+    return ( stream_index << ffmpeg_sl_bits ) | slot;
+}
+
+// we're about to open 'title' to convert it - remap the stream associated
+// with the video & audio codec params of the title to refer to 'stream'
+// (the original scan stream was closed and no longer exists).
+static void ffmpeg_remap_stream( hb_stream_t *stream, hb_title_t *title )
+{
+    // tell ffmpeg we want a pts on every frame it returns
+    stream->ffmpeg_ic->flags |= AVFMT_FLAG_GENPTS;
+
+    // all the video & audio came from the same stream so remapping
+    // the video's stream slot takes care of everything.
+    int slot = title->video_codec_param & (ffmpeg_sl_size - 1);
+    ffmpeg_streams[slot] = stream;
+
+    // add codecs for all the streams used by the title
+    ffmpeg_add_codec( stream, title->video_codec_param >> ffmpeg_sl_bits );
+
+    int i;
+    hb_audio_t *audio;
+    for ( i = 0; ( audio = hb_list_item( title->list_audio, i ) ); ++i )
+    {
+        if ( audio->config.in.codec == HB_ACODEC_FFMPEG )
+        {
+            ffmpeg_add_codec( stream,
+                              audio->config.in.codec_param >> ffmpeg_sl_bits );
+        }
+    }
+}
+
+void *hb_ffmpeg_context( int codec_param )
+{
+    int slot = codec_param & (ffmpeg_sl_size - 1);
+    int stream_index = codec_param >> ffmpeg_sl_bits;
+    return ffmpeg_streams[slot]->ffmpeg_ic->streams[stream_index]->codec;
 }
 
+void *hb_ffmpeg_avstream( int codec_param )
+{
+    int slot = codec_param & (ffmpeg_sl_size - 1);
+    int stream_index = codec_param >> ffmpeg_sl_bits;
+    return ffmpeg_streams[slot]->ffmpeg_ic->streams[stream_index];
+}
+
+static AVFormatContext *ffmpeg_deferred_close;
+
+static int ffmpeg_open( hb_stream_t *stream, hb_title_t *title )
+{
+    if ( ffmpeg_deferred_close )
+    {
+        av_close_input_file( ffmpeg_deferred_close );
+        ffmpeg_deferred_close = NULL;
+    }
+    AVFormatContext *ic;
+
+    av_log_set_level( AV_LOG_ERROR );
+    if ( av_open_input_file( &ic, stream->path, NULL, 0, NULL ) < 0 )
+    {
+        return 0;
+    }
+    if ( av_find_stream_info( ic ) < 0 )
+        goto fail;
+
+    stream->ffmpeg_ic = ic;
+    stream->hb_stream_type = ffmpeg;
+    stream->ffmpeg_pkt = malloc(sizeof(*stream->ffmpeg_pkt));
+    av_init_packet( stream->ffmpeg_pkt );
+
+    if ( title )
+    {
+        // we're opening for read. scan passed out codec params that
+        // indexed its stream so we need to remap them so they point
+        // to this stream.
+        ffmpeg_remap_stream( stream, title );
+        ffmpeg_seek( stream, 0. );
+        av_log_set_level( AV_LOG_ERROR );
+    }
+    else
+    {
+        // we're opening for scan. let ffmpeg put some info into the
+        // log about what we've got.
+        av_log_set_level( AV_LOG_INFO );
+        dump_format( ic, 0, stream->path, 0 );
+        av_log_set_level( AV_LOG_ERROR );
+
+        // accept this file if it has at least one video stream we can decode
+        int i;
+        for (i = 0; i < ic->nb_streams; ++i )
+        {
+            if ( ic->streams[i]->codec->codec_type == CODEC_TYPE_VIDEO )
+            {
+                break;
+            }
+        }
+        if ( i >= ic->nb_streams )
+            goto fail;
+    }
+    return 1;
+
+  fail:
+    av_close_input_file( ic );
+    return 0;
+}
+
+static void ffmpeg_close( hb_stream_t *d )
+{
+    // XXX since we're sharing the CodecContext with the downstream
+    // decoder proc we can't close the stream. We need to reference count
+    // this so we can close it when both are done with their instance but
+    // for now just defer the close until the next stream open or close.
+    if ( ffmpeg_deferred_close )
+    {
+        av_close_input_file( ffmpeg_deferred_close );
+    }
+    ffmpeg_deferred_close = d->ffmpeg_ic;
+    if ( d->ffmpeg_pkt != NULL )
+    {
+        free( d->ffmpeg_pkt );
+        d->ffmpeg_pkt = NULL;
+    }
+}
+
+static void add_ffmpeg_audio( hb_title_t *title, hb_stream_t *stream, int id )
+{
+    AVStream *st = stream->ffmpeg_ic->streams[id];
+    AVCodecContext *codec = st->codec;
+
+    // scan will ignore any audio without a bitrate. Since we've already
+    // typed the audio in order to determine its codec we set up the audio
+    // paramters here.
+    if ( codec->bit_rate || codec->sample_rate )
+    {
+        static const int chan2layout[] = {
+            HB_INPUT_CH_LAYOUT_MONO,  // We should allow no audio really.
+            HB_INPUT_CH_LAYOUT_MONO,   
+            HB_INPUT_CH_LAYOUT_STEREO,
+            HB_INPUT_CH_LAYOUT_2F1R,   
+            HB_INPUT_CH_LAYOUT_2F2R,
+            HB_INPUT_CH_LAYOUT_3F2R,   
+            HB_INPUT_CH_LAYOUT_4F2R,
+            HB_INPUT_CH_LAYOUT_STEREO, 
+            HB_INPUT_CH_LAYOUT_STEREO,
+        };
+
+        hb_audio_t *audio = calloc( 1, sizeof(*audio) );;
+
+        audio->id = id;
+        if ( codec->codec_id == CODEC_ID_AC3 )
+        {
+            audio->config.in.codec = HB_ACODEC_AC3;
+        }
+        else
+        {
+            audio->config.in.codec = HB_ACODEC_FFMPEG;
+            audio->config.in.codec_param = ffmpeg_codec_param( stream, id );
+
+            audio->config.in.bitrate = codec->bit_rate? codec->bit_rate : 1;
+            audio->config.in.samplerate = codec->sample_rate;
+            audio->config.in.channel_layout = chan2layout[codec->channels & 7];
+        }
+
+        set_audio_description( audio, lang_for_code2( st->language ) );
+
+        hb_list_add( title->list_audio, audio );
+    }
+}
+
+static hb_title_t *ffmpeg_title_scan( hb_stream_t *stream )
+{
+    AVFormatContext *ic = stream->ffmpeg_ic;
+
+    // 'Barebones Title'
+    hb_title_t *title = hb_title_init( stream->path, 0 );
+    title->index = 1;
+
+       // Copy part of the stream path to the title name
+       char *sep = strrchr(stream->path, '/');
+       if (sep)
+               strcpy(title->name, sep+1);
+       char *dot_term = strrchr(title->name, '.');
+       if (dot_term)
+               *dot_term = '\0';
+
+    uint64_t dur = ic->duration * 90000 / AV_TIME_BASE;
+    title->duration = dur;
+    dur /= 90000;
+    title->hours    = dur / 3600;
+    title->minutes  = ( dur % 3600 ) / 60;
+    title->seconds  = dur % 60;
+
+    // One Chapter
+    hb_chapter_t * chapter;
+    chapter = calloc( sizeof( hb_chapter_t ), 1 );
+    chapter->index = 1;
+    chapter->duration = title->duration;
+    chapter->hours = title->hours;
+    chapter->minutes = title->minutes;
+    chapter->seconds = title->seconds;
+    hb_list_add( title->list_chapter, chapter );
+
+    // set the title to decode the first video stream in the file
+    title->demuxer = HB_NULL_DEMUXER;
+    title->video_codec = 0;
+    int i;
+    for (i = 0; i < ic->nb_streams; ++i )
+    {
+        if ( ic->streams[i]->codec->codec_type == CODEC_TYPE_VIDEO &&
+             avcodec_find_decoder( ic->streams[i]->codec->codec_id ) &&
+             title->video_codec == 0 )
+        {
+            title->video_id = i;
+
+            // We have to use the 'internal' avcodec decoder because
+            // it needs to share the codec context from this video
+            // stream. The parser internal to av_read_frame
+            // passes a bunch of state info to the decoder via the context.
+            title->video_codec = WORK_DECAVCODECVI;
+            title->video_codec_param = ffmpeg_codec_param( stream, i );
+        }
+        else if ( ic->streams[i]->codec->codec_type == CODEC_TYPE_AUDIO &&
+                  avcodec_find_decoder( ic->streams[i]->codec->codec_id ) )
+        {
+            add_ffmpeg_audio( title, stream, i );
+        }
+    }
+
+    title->container_name = strdup( ic->iformat->name );
+    title->data_rate = ic->bit_rate;
+
+    return title;
+}
+
+static int64_t av_to_hb_pts( int64_t pts, double conv_factor )
+{
+    if ( pts == AV_NOPTS_VALUE )
+        return -1;
+    return (int64_t)( (double)pts * conv_factor );
+}
+
+static int ffmpeg_read( hb_stream_t *stream, hb_buffer_t *buf )
+{
+    int err;
+    if ( ( err = av_read_frame( stream->ffmpeg_ic, stream->ffmpeg_pkt )) < 0 )
+    {
+        // XXX the following conditional is to handle avi files that
+        // use M$ 'packed b-frames' and occasionally have negative
+        // sizes for the null frames these require.
+        if ( err != AVERROR_NOMEM || stream->ffmpeg_pkt->size >= 0 )
+            // eof
+            return 0;
+    }
+    if ( stream->ffmpeg_pkt->size <= 0 )
+    {
+        // M$ "invalid and inefficient" packed b-frames require 'null frames'
+        // following them to preserve the timing (since the packing puts two
+        // or more frames in what looks like one avi frame). The contents and
+        // size of these null frames are ignored by the ff_h263_decode_frame
+        // as long as they're < 20 bytes. We need a positive size so we use
+        // one byte if we're given a zero or negative size. We don't know
+        // if the pkt data points anywhere reasonable so we just stick a
+        // byte of zero in our outbound buf.
+        buf->size = 1;
+        *buf->data = 0;
+    }
+    else
+    {
+        if ( stream->ffmpeg_pkt->size > buf->alloc )
+        {
+            // need to expand buffer
+            hb_buffer_realloc( buf, stream->ffmpeg_pkt->size );
+        }
+        memcpy( buf->data, stream->ffmpeg_pkt->data, stream->ffmpeg_pkt->size );
+        buf->size = stream->ffmpeg_pkt->size;
+    }
+    buf->id = stream->ffmpeg_pkt->stream_index;
+
+    // if we haven't done it already, compute a conversion factor to go
+    // from the ffmpeg timebase for the stream to HB's 90KHz timebase.
+    double tsconv = stream->ffmpeg_tsconv[stream->ffmpeg_pkt->stream_index];
+    if ( ! tsconv )
+    {
+        AVStream *s = stream->ffmpeg_ic->streams[stream->ffmpeg_pkt->stream_index];
+        tsconv = 90000. * (double)s->time_base.num / (double)s->time_base.den;
+        stream->ffmpeg_tsconv[stream->ffmpeg_pkt->stream_index] = tsconv;
+    }
+
+    buf->start = av_to_hb_pts( stream->ffmpeg_pkt->pts, tsconv );
+    buf->renderOffset = av_to_hb_pts( stream->ffmpeg_pkt->dts, tsconv );
+    if ( buf->renderOffset >= 0 && buf->start == -1 )
+    {
+        buf->start = buf->renderOffset;
+    }
+    av_free_packet( stream->ffmpeg_pkt );
+    return 1;
+}
+
+static int ffmpeg_seek( hb_stream_t *stream, float frac )
+{
+    AVFormatContext *ic = stream->ffmpeg_ic;
+    int64_t pos = (double)ic->duration * (double)frac;
+    av_seek_frame( ic, -1, pos, pos? 0 : AVSEEK_FLAG_BACKWARD );
+    return 1;
+}