]> git.sesse.net Git - vlc/blobdiff - modules/codec/libmpeg2.c
* include/vlc_block.h, modules/codec/libmpeg2.c: re-added the discontinuity flag...
[vlc] / modules / codec / libmpeg2.c
index 2fc8e3855d606976c514c50fbf991145dbb11f9b..75939b0c1f127de61022a9c5faaff1f743a0ac4b 100755 (executable)
@@ -2,9 +2,10 @@
  * libmpeg2.c: mpeg2 video decoder module making use of libmpeg2.
  *****************************************************************************
  * Copyright (C) 1999-2001 VideoLAN
- * $Id: libmpeg2.c,v 1.10 2003/04/15 15:53:42 gbazin Exp $
+ * $Id: libmpeg2.c,v 1.28 2003/09/30 20:23:03 gbazin Exp $
  *
  * Authors: Gildas Bazin <gbazin@netcourrier.com>
+ *          Christophe Massiot <massiot@via.ecp.fr>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 #define AR_221_1_PICTURE        4                  /* 2.21:1 picture (movie) */
 
 /*****************************************************************************
- * dec_thread_t : libmpeg2 decoder thread descriptor
+ * decoder_sys_t : libmpeg2 decoder descriptor
  *****************************************************************************/
-typedef struct dec_thread_t
+struct decoder_sys_t
 {
     /*
      * libmpeg2 properties
      */
     mpeg2dec_t          *p_mpeg2dec;
     const mpeg2_info_t  *p_info;
+    vlc_bool_t          b_skip;
 
     /*
      * Input properties
      */
-    decoder_fifo_t   *p_fifo;                  /* stores the PES stream data */
     pes_packet_t     *p_pes;                  /* current PES we are decoding */
     mtime_t          i_pts;
     mtime_t          i_previous_pts;
@@ -64,6 +65,10 @@ typedef struct dec_thread_t
     mtime_t          i_period_remainder;
     int              i_current_rate;
     picture_t *      p_picture_to_destroy;
+    vlc_bool_t       b_garbage_pic;
+    vlc_bool_t       b_after_sequence_header; /* is it the next frame after
+                                               * the sequence header ?    */
+    vlc_bool_t       b_slice_i;             /* intra-slice refresh stream */
 
     /*
      * Output properties
@@ -71,23 +76,24 @@ typedef struct dec_thread_t
     vout_thread_t *p_vout;
     vout_synchro_t *p_synchro;
 
-} dec_thread_t;
+};
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
 static int  OpenDecoder  ( vlc_object_t * );
-static int  RunDecoder   ( decoder_fifo_t * );
-static void CloseDecoder ( dec_thread_t * );
+static int  InitDecoder  ( decoder_t * );
+static int  RunDecoder   ( decoder_t *, block_t * );
+static int  EndDecoder   ( decoder_t * );
 
-static picture_t *GetNewPicture( dec_thread_t *, uint8_t ** );
+static picture_t *GetNewPicture( decoder_t *, uint8_t ** );
 
 /*****************************************************************************
  * Module descriptor
  *****************************************************************************/
 vlc_module_begin();
     set_description( _("MPEG I/II video decoder (using libmpeg2)") );
-    set_capability( "decoder", 40 );
+    set_capability( "decoder", 150 );
     set_callbacks( OpenDecoder, NULL );
     add_shortcut( "libmpeg2" );
 vlc_module_end();
@@ -97,108 +103,148 @@ vlc_module_end();
  *****************************************************************************/
 static int OpenDecoder( vlc_object_t *p_this )
 {
-    decoder_fifo_t *p_fifo = (decoder_fifo_t*) p_this;
+    decoder_t *p_dec = (decoder_t*)p_this;
 
-    if( p_fifo->i_fourcc != VLC_FOURCC('m','p','g','v') )
+    if( p_dec->p_fifo->i_fourcc != VLC_FOURCC('m','p','g','v') &&
+        p_dec->p_fifo->i_fourcc != VLC_FOURCC('m','p','g','1') &&
+        p_dec->p_fifo->i_fourcc != VLC_FOURCC('m','p','g','2') )
     {
         return VLC_EGENERIC;
     }
 
-    p_fifo->pf_run = RunDecoder;
+    p_dec->pf_init = InitDecoder;
+    p_dec->pf_decode = RunDecoder;
+    p_dec->pf_end = EndDecoder;
+
     return VLC_SUCCESS;
 }
 
 /*****************************************************************************
- * RunDecoder: the libmpeg2 decoder
+ * InitDecoder: Initalize the decoder
  *****************************************************************************/
-static int RunDecoder( decoder_fifo_t *p_fifo )
+static int InitDecoder( decoder_t *p_dec )
 {
-    dec_thread_t    *p_dec;
-    data_packet_t   *p_data = NULL;
-    mpeg2_state_t   state;
-    picture_t       *p_pic;
-    int             i_aspect, i_chroma;
-
-    /* Allocate the memory needed to store the thread's structure */
-    if( (p_dec = (dec_thread_t *)malloc (sizeof(dec_thread_t)) )
-        == NULL)
+    /* Allocate the memory needed to store the decoder's structure */
+    if( ( p_dec->p_sys =
+          (decoder_sys_t *)malloc(sizeof(decoder_sys_t)) ) == NULL )
     {
-        msg_Err( p_fifo, "out of memory" );
-        goto error;
+        msg_Err( p_dec, "out of memory" );
+        return VLC_EGENERIC;
     }
 
     /* Initialize the thread properties */
-    memset( p_dec, 0, sizeof(dec_thread_t) );
-    p_dec->p_fifo     = p_fifo;
-    p_dec->p_pes      = NULL;
-    p_dec->p_vout     = NULL;
-    p_dec->p_mpeg2dec = NULL;
-    p_dec->p_synchro  = NULL;
-    p_dec->p_info     = NULL;
-    p_dec->i_pts      = mdate() + DEFAULT_PTS_DELAY;
-    p_dec->i_current_pts  = 0;
-    p_dec->i_previous_pts = 0;
-    p_dec->i_period_remainder = 0;
-    p_dec->p_picture_to_destroy = NULL;
+    memset( p_dec->p_sys, 0, sizeof(decoder_sys_t) );
+    p_dec->p_sys->p_pes      = NULL;
+    p_dec->p_sys->p_vout     = NULL;
+    p_dec->p_sys->p_mpeg2dec = NULL;
+    p_dec->p_sys->p_synchro  = NULL;
+    p_dec->p_sys->p_info     = NULL;
+    p_dec->p_sys->i_pts      = mdate() + DEFAULT_PTS_DELAY;
+    p_dec->p_sys->i_current_pts  = 0;
+    p_dec->p_sys->i_previous_pts = 0;
+    p_dec->p_sys->i_period_remainder = 0;
+    p_dec->p_sys->p_picture_to_destroy = NULL;
+    p_dec->p_sys->b_garbage_pic = 0;
+    p_dec->p_sys->b_slice_i  = 0;
+    p_dec->p_sys->b_skip     = 0;
 
     /* Initialize decoder */
-    p_dec->p_mpeg2dec = mpeg2_init();
-    if( p_dec->p_mpeg2dec == NULL)
-        goto error;
+    p_dec->p_sys->p_mpeg2dec = mpeg2_init();
+    if( p_dec->p_sys->p_mpeg2dec == NULL)
+    {
+        msg_Err( p_dec, "mpeg2_init() failed" );
+        free( p_dec->p_sys );
+        return VLC_EGENERIC;
+    }
 
-    p_dec->p_info = mpeg2_info( p_dec->p_mpeg2dec );
+    p_dec->p_sys->p_info = mpeg2_info( p_dec->p_sys->p_mpeg2dec );
+
+    return VLC_SUCCESS;
+}
 
-    /* libmpeg2 decoder thread's main loop */
-    while( (!p_dec->p_fifo->b_die) && (!p_dec->p_fifo->b_error) )
+/*****************************************************************************
+ * RunDecoder: the libmpeg2 decoder
+ *****************************************************************************/
+static int RunDecoder( decoder_t *p_dec, block_t *p_block )
+{
+    decoder_sys_t   *p_sys = p_dec->p_sys;
+    mpeg2_state_t   state;
+    picture_t       *p_pic;
+    int             i_aspect;
+
+    vlc_bool_t      b_need_more_data = VLC_FALSE;
+
+    while( 1 )
     {
-        state = mpeg2_parse( p_dec->p_mpeg2dec );
+        if( p_dec->p_fifo->b_die || p_dec->p_fifo->b_error )
+        {
+            block_Release( p_block );
+            return VLC_EGENERIC;
+        }
+
+        state = mpeg2_parse( p_sys->p_mpeg2dec );
 
         switch( state )
         {
         case STATE_BUFFER:
-            /* Feed libmpeg2 a data packet at a time */
-            if( p_data == NULL )
+            if( !p_block->i_buffer || b_need_more_data )
             {
-                /* Get the next PES */
-                if( p_dec->p_pes )
-                    input_DeletePES( p_dec->p_fifo->p_packets_mgt,
-                                     p_dec->p_pes );
+                block_Release( p_block );
+                return VLC_SUCCESS;
+            }
 
-                input_ExtractPES( p_dec->p_fifo, &p_dec->p_pes );
-                if( !p_dec->p_pes )
+            if( p_block->b_discontinuity && p_sys->p_synchro 
+                && p_sys->p_info->sequence->width != (unsigned)-1 )
+            {
+                vout_SynchroReset( p_sys->p_synchro );
+                if( p_sys->p_info->current_fbuf != NULL
+                    && p_sys->p_info->current_fbuf->id != NULL )
                 {
-                    p_dec->p_fifo->b_error = 1;
-                    break;
+                    p_sys->b_garbage_pic = 1;
+                    p_pic = p_sys->p_info->current_fbuf->id;
                 }
-
-                if( p_dec->p_pes->b_discontinuity && p_dec->p_synchro )
+                else
                 {
-                    vout_SynchroReset( p_dec->p_synchro );
-                    if ( p_dec->p_info->current_fbuf != NULL )
-                        p_dec->p_picture_to_destroy
-                            = p_dec->p_info->current_fbuf->id;
+                    uint8_t *buf[3];
+                    buf[0] = buf[1] = buf[2] = NULL;
+                    if( (p_pic = GetNewPicture( p_dec, buf )) == NULL )
+                        break;
+                    mpeg2_set_buf( p_sys->p_mpeg2dec, buf, p_pic );
                 }
-
-                if( p_dec->p_pes->i_pts )
+                p_sys->p_picture_to_destroy = p_pic;
+
+                memset( p_pic->p[0].p_pixels, 0,
+                        p_sys->p_info->sequence->width
+                         * p_sys->p_info->sequence->height );
+                memset( p_pic->p[1].p_pixels, 0x80,
+                        p_sys->p_info->sequence->width
+                         * p_sys->p_info->sequence->height / 4 );
+                memset( p_pic->p[2].p_pixels, 0x80,
+                        p_sys->p_info->sequence->width
+                         * p_sys->p_info->sequence->height / 4 );
+
+                if ( p_sys->b_slice_i )
                 {
-                    mpeg2_pts( p_dec->p_mpeg2dec,
-                               (uint32_t)p_dec->p_pes->i_pts );
-                    p_dec->i_previous_pts = p_dec->i_current_pts;
-                    p_dec->i_current_pts = p_dec->p_pes->i_pts;
+                    vout_SynchroNewPicture( p_sys->p_synchro,
+                        I_CODING_TYPE, 2, 0, 0, p_sys->i_current_rate );
+                    vout_SynchroDecode( p_sys->p_synchro );
+                    vout_SynchroEnd( p_sys->p_synchro, I_CODING_TYPE, 0 );
                 }
-
-                p_dec->i_current_rate = p_dec->p_pes->i_rate;
-                p_data = p_dec->p_pes->p_first;
             }
 
-            if( p_data != NULL )
+            if( p_block->i_pts )
             {
-                mpeg2_buffer( p_dec->p_mpeg2dec,
-                              p_data->p_payload_start,
-                              p_data->p_payload_end );
-
-                p_data = p_data->p_next;
+                mpeg2_pts( p_sys->p_mpeg2dec, (uint32_t)p_block->i_pts );
+                p_sys->i_previous_pts = p_sys->i_current_pts;
+                p_sys->i_current_pts = p_block->i_pts;
             }
+
+            p_sys->i_current_rate = DEFAULT_RATE;//p_pes->i_rate;
+
+            mpeg2_buffer( p_sys->p_mpeg2dec, p_block->p_buffer,
+                          p_block->p_buffer + p_block->i_buffer );
+
+            b_need_more_data = VLC_TRUE;
             break;
 
         case STATE_SEQUENCE:
@@ -226,184 +272,311 @@ static int RunDecoder( decoder_fifo_t *p_fifo )
                 case AR_SQUARE_PICTURE:
                 default:
                     i_aspect = VOUT_ASPECT_FACTOR *
-                                   p_dec->p_info->sequence->width /
-                                   p_dec->p_info->sequence->height;
+                                   p_sys->p_info->sequence->width /
+                                   p_sys->p_info->sequence->height;
                     break;
                 }
             }
             else
             {
                 /* Use the value provided in the MPEG sequence header */
-                i_aspect = ((uint64_t)p_dec->p_info->sequence->width) *
-                    p_dec->p_info->sequence->pixel_width * VOUT_ASPECT_FACTOR /
-                    p_dec->p_info->sequence->height /
-                    p_dec->p_info->sequence->pixel_height;
+                i_aspect = ((uint64_t)p_sys->p_info->sequence->display_width) *
+                    p_sys->p_info->sequence->pixel_width * VOUT_ASPECT_FACTOR /
+                    p_sys->p_info->sequence->display_height /
+                    p_sys->p_info->sequence->pixel_height;
             }
 
-            i_chroma = VLC_FOURCC('Y','V','1','2');
+            if ( p_dec->p_sys->p_vout != NULL )
+            {
+              int i_pic;
+                /* Temporary hack to free the pictures in use by libmpeg2 */
+                for ( i_pic = 0; i_pic < p_dec->p_sys->p_vout->render.i_pictures; i_pic++ )
+                {
+                    if( p_dec->p_sys->p_vout->render.pp_picture[i_pic]->i_status ==
+                          RESERVED_PICTURE )
+                        vout_DestroyPicture( p_dec->p_sys->p_vout,
+                                         p_dec->p_sys->p_vout->render.pp_picture[i_pic] );
+                    if( p_dec->p_sys->p_vout->render.pp_picture[i_pic]->i_refcount > 0 )
+                        vout_UnlinkPicture( p_dec->p_sys->p_vout,
+                                         p_dec->p_sys->p_vout->render.pp_picture[i_pic] );
+                }
+            }
 
-            p_dec->p_vout = vout_Request( p_dec->p_fifo, p_dec->p_vout,
-                                          p_dec->p_info->sequence->width,
-                                          p_dec->p_info->sequence->height,
-                                          i_chroma, i_aspect );
+            p_sys->p_vout = vout_Request( p_dec, p_sys->p_vout,
+                                          p_sys->p_info->sequence->width,
+                                          p_sys->p_info->sequence->height,
+                                          VLC_FOURCC('Y','V','1','2'),
+                                          i_aspect );
 
-            mpeg2_custom_fbuf( p_dec->p_mpeg2dec, 1 );
+            if(p_sys->p_vout == NULL )
+            {
+                msg_Err( p_dec, "cannot create vout" );
+                block_Release( p_block );
+                return -1;
+            } 
 
-            /* Set the first 2 reference frames */
-            mpeg2_set_buf( p_dec->p_mpeg2dec, buf, NULL );
-            mpeg2_set_buf( p_dec->p_mpeg2dec, buf, NULL );
+            msg_Dbg( p_dec, "%dx%d, aspect %d, %u.%03u fps",
+                     p_sys->p_info->sequence->width,
+                     p_sys->p_info->sequence->height, i_aspect,
+                     (uint32_t)((u64)1001000000 * 27 /
+                     p_sys->p_info->sequence->frame_period / 1001),
+                     (uint32_t)((u64)1001000000 * 27 /
+                     p_sys->p_info->sequence->frame_period % 1001) );
+
+            mpeg2_custom_fbuf( p_sys->p_mpeg2dec, 1 );
 
-            p_dec->p_synchro = vout_SynchroInit( p_dec->p_fifo, p_dec->p_vout,
-                1000000 * 27 / p_dec->p_info->sequence->frame_period * 1001 );
+            /* Set the first 2 reference frames */
+            mpeg2_set_buf( p_sys->p_mpeg2dec, buf, NULL );
+
+            if( (p_pic = GetNewPicture( p_dec, buf )) == NULL ) break;
+            memset( p_pic->p[0].p_pixels, 0,
+                    p_sys->p_info->sequence->width
+                     * p_sys->p_info->sequence->height );
+            memset( p_pic->p[1].p_pixels, 0x80,
+                    p_sys->p_info->sequence->width
+                     * p_sys->p_info->sequence->height / 4 );
+            memset( p_pic->p[2].p_pixels, 0x80,
+                    p_sys->p_info->sequence->width
+                     * p_sys->p_info->sequence->height / 4 );
+            mpeg2_set_buf( p_sys->p_mpeg2dec, buf, p_pic );
+            /* This picture will never go through display_picture. */
+            vout_DatePicture( p_sys->p_vout, p_pic, 0 );
+            vout_DisplayPicture( p_sys->p_vout, p_pic );
+            /* For some reason, libmpeg2 will put this pic twice in
+             * discard_picture. This can be considered a bug in libmpeg2. */
+            vout_LinkPicture( p_sys->p_vout, p_pic );
+
+            if ( p_sys->p_synchro )
+            {
+                vout_SynchroRelease( p_sys->p_synchro );
+            }
+            p_sys->p_synchro = vout_SynchroInit( p_dec, p_sys->p_vout,
+                (uint32_t)((uint64_t)1001000000 * 27 /
+                p_sys->p_info->sequence->frame_period) );
+            p_sys->b_after_sequence_header = 1;
         }
         break;
 
+        case STATE_PICTURE_2ND:
+            vout_SynchroNewPicture( p_sys->p_synchro,
+                p_sys->p_info->current_picture->flags & PIC_MASK_CODING_TYPE,
+                p_sys->p_info->current_picture->nb_fields,
+                0, 0,
+                p_sys->i_current_rate );
+
+            if ( p_sys->b_skip )
+            {
+                vout_SynchroTrash( p_sys->p_synchro );
+            }
+            else
+            {
+                vout_SynchroDecode( p_sys->p_synchro );
+            }
+            break;
+
         case STATE_PICTURE:
         {
             uint8_t *buf[3];
             buf[0] = buf[1] = buf[2] = NULL;
 
-            vout_SynchroNewPicture( p_dec->p_synchro,
-                p_dec->p_info->current_picture->flags & PIC_MASK_CODING_TYPE,
-                p_dec->p_info->current_picture->nb_fields,
-                (p_dec->p_info->current_picture->flags & PIC_FLAG_PTS) ?
-                    ( (p_dec->p_info->current_picture->pts ==
-                                (uint32_t)p_dec->i_current_pts) ?
-                              p_dec->i_current_pts : p_dec->i_previous_pts ) : 0,
-                0,
-                p_dec->i_current_rate );
-
-            if ( !vout_SynchroChoose( p_dec->p_synchro,
-                p_dec->p_info->current_picture->flags & PIC_MASK_CODING_TYPE ) )
+            if ( p_sys->b_after_sequence_header
+                  && ((p_sys->p_info->current_picture->flags
+                        & PIC_MASK_CODING_TYPE)
+                       == PIC_FLAG_CODING_TYPE_P) )
             {
-                mpeg2_skip( p_dec->p_mpeg2dec, 1 );
-                vout_SynchroTrash( p_dec->p_synchro );
-                mpeg2_set_buf( p_dec->p_mpeg2dec, buf, NULL );
+                /* Intra-slice refresh. Simulate a blank I picture. */
+                msg_Dbg( p_dec, "intra-slice refresh stream" );
+                vout_SynchroNewPicture( p_sys->p_synchro,
+                    I_CODING_TYPE, 2, 0, 0, p_sys->i_current_rate );
+                vout_SynchroDecode( p_sys->p_synchro );
+                vout_SynchroEnd( p_sys->p_synchro, I_CODING_TYPE, 0 );
+                p_sys->b_slice_i = 1;
+            }
+            p_sys->b_after_sequence_header = 0;
+
+            vout_SynchroNewPicture( p_sys->p_synchro,
+                p_sys->p_info->current_picture->flags & PIC_MASK_CODING_TYPE,
+                p_sys->p_info->current_picture->nb_fields,
+                (p_sys->p_info->current_picture->flags & PIC_FLAG_PTS) ?
+                    ( ( p_sys->p_info->current_picture->pts ==
+                        (uint32_t)p_sys->i_current_pts ) ?
+                      p_sys->i_current_pts : p_sys->i_previous_pts ) : 0,
+                0, p_sys->i_current_rate );
+
+            if ( !(p_sys->b_slice_i
+                   && ((p_sys->p_info->current_picture->flags
+                         & PIC_MASK_CODING_TYPE) == P_CODING_TYPE))
+                   && !vout_SynchroChoose( p_sys->p_synchro,
+                              p_sys->p_info->current_picture->flags
+                                & PIC_MASK_CODING_TYPE ) )
+            {
+                mpeg2_skip( p_sys->p_mpeg2dec, 1 );
+                p_sys->b_skip = 1;
+                vout_SynchroTrash( p_sys->p_synchro );
+                mpeg2_set_buf( p_sys->p_mpeg2dec, buf, NULL );
             }
             else
             {
-                mpeg2_skip( p_dec->p_mpeg2dec, 0 );
-                vout_SynchroDecode( p_dec->p_synchro );
+                mpeg2_skip( p_sys->p_mpeg2dec, 0 );
+                p_sys->b_skip = 0;
+                vout_SynchroDecode( p_sys->p_synchro );
                 if( (p_pic = GetNewPicture( p_dec, buf )) == NULL ) break;
-                mpeg2_set_buf( p_dec->p_mpeg2dec, buf, p_pic );
+                mpeg2_set_buf( p_sys->p_mpeg2dec, buf, p_pic );
             }
         }
-        /* pass-through */
+        break;
 
         case STATE_END:
-            if( p_dec->p_info->display_fbuf
-                && p_dec->p_info->display_fbuf->id )
+        case STATE_SLICE:
+            if( p_sys->p_info->display_fbuf
+                && p_sys->p_info->display_fbuf->id )
             {
-                p_pic = (picture_t *)p_dec->p_info->display_fbuf->id;
+                p_pic = (picture_t *)p_sys->p_info->display_fbuf->id;
 
-                if ( p_pic != NULL )
-                {
-                    if ( p_dec->p_picture_to_destroy != p_pic )
-                    {
-                        vout_SynchroEnd( p_dec->p_synchro,
-                            p_dec->p_info->display_picture->flags
-                             & PIC_MASK_CODING_TYPE,
-                            0 );
-                        vout_DatePicture( p_dec->p_vout, p_pic,
-                            vout_SynchroDate( p_dec->p_synchro ) );
-                        vout_DisplayPicture( p_dec->p_vout, p_pic );
-                    }
-                    else
-                    {
-                        p_dec->p_picture_to_destroy = NULL;
-                        vout_SynchroEnd( p_dec->p_synchro,
-                            p_dec->p_info->display_picture->flags
+                vout_SynchroEnd( p_sys->p_synchro,
+                            p_sys->p_info->display_picture->flags
                              & PIC_MASK_CODING_TYPE,
-                            1 );
-                        vout_DestroyPicture( p_dec->p_vout, p_pic );
-                    }
+                            p_sys->b_garbage_pic );
+                p_sys->b_garbage_pic = 0;
+                vout_DisplayPicture( p_sys->p_vout, p_pic );
+
+                if ( p_sys->p_picture_to_destroy != p_pic )
+                {
+                    vout_DatePicture( p_sys->p_vout, p_pic,
+                        vout_SynchroDate( p_sys->p_synchro ) );
                 }
-                if( p_dec->p_info->discard_fbuf &&
-                    p_dec->p_info->discard_fbuf->id )
+                else
                 {
-                    vout_UnlinkPicture( p_dec->p_vout, p_pic );
+                    p_sys->p_picture_to_destroy = NULL;
+                    vout_DatePicture( p_sys->p_vout, p_pic, 0 );
                 }
             }
+
+            if( p_sys->p_info->discard_fbuf &&
+                p_sys->p_info->discard_fbuf->id )
+            {
+                p_pic = (picture_t *)p_sys->p_info->discard_fbuf->id;
+                vout_UnlinkPicture( p_sys->p_vout, p_pic );
+            }
             break;
 
         case STATE_INVALID:
-            msg_Warn( p_dec->p_fifo, "Received STATE_INVALID" );
+        {
+            uint8_t *buf[3];
+            buf[0] = buf[1] = buf[2] = NULL;
+
+            msg_Warn( p_dec, "invalid picture encountered" );
+            if ( ( p_sys->p_info->current_picture == NULL ) || 
+               ( ( p_sys->p_info->current_picture->flags & PIC_MASK_CODING_TYPE)
+                  != B_CODING_TYPE ) )
+            {
+                vout_SynchroReset( p_sys->p_synchro );
+            }
+            mpeg2_skip( p_sys->p_mpeg2dec, 1 );
+            p_sys->b_skip = 1;
+
+            if( p_sys->p_info->current_fbuf &&
+                p_sys->p_info->current_fbuf->id )
+            {
+                p_sys->b_garbage_pic = 1;
+                p_pic = p_sys->p_info->current_fbuf->id;
+            }
+            else
+            {
+                if( (p_pic = GetNewPicture( p_dec, buf )) == NULL )
+                    break;
+                mpeg2_set_buf( p_sys->p_mpeg2dec, buf, p_pic );
+            }
+            p_sys->p_picture_to_destroy = p_pic;
+
+            memset( p_pic->p[0].p_pixels, 0,
+                    p_sys->p_info->sequence->width
+                     * p_sys->p_info->sequence->height );
+            memset( p_pic->p[1].p_pixels, 0x80,
+                    p_sys->p_info->sequence->width
+                     * p_sys->p_info->sequence->height / 4 );
+            memset( p_pic->p[2].p_pixels, 0x80,
+                    p_sys->p_info->sequence->width
+                     * p_sys->p_info->sequence->height / 4 );
+
+            if ( p_sys->b_slice_i )
+            {
+                vout_SynchroNewPicture( p_sys->p_synchro,
+                            I_CODING_TYPE, 2, 0, 0, p_sys->i_current_rate );
+                vout_SynchroDecode( p_sys->p_synchro );
+                vout_SynchroEnd( p_sys->p_synchro, I_CODING_TYPE, 0 );
+            }
             break;
+        }
 
         default:
             break;
         }
     }
 
-    /* If b_error is set, the libmpeg2 decoder thread enters the error loop */
-    if( p_dec->p_fifo->b_error )
-    {
-        DecoderError( p_dec->p_fifo );
-    }
-
-    /* End of the libmpeg2 decoder thread */
-    CloseDecoder( p_dec );
-
-    return 0;
-
- error:
-    DecoderError( p_fifo );
-    if( p_dec )
-    {
-        if( p_dec->p_fifo )
-            p_dec->p_fifo->b_error = 1;
-
-        /* End of the libmpeg2 decoder thread */
-        CloseDecoder( p_dec );
-    }
-
-    return -1;
+    block_Release( p_block );
+    return VLC_EGENERIC;
 }
 
 /*****************************************************************************
- * CloseDecoder: libmpeg2 decoder destruction
+ * EndDecoder: libmpeg2 decoder destruction
  *****************************************************************************/
-static void CloseDecoder( dec_thread_t * p_dec )
+static int EndDecoder( decoder_t * p_dec )
 {
-    if( p_dec )
+    decoder_sys_t *p_sys = p_dec->p_sys;
+
+    if( p_sys )
     {
         int i_pic;
 
-        if( p_dec->p_pes )
-            input_DeletePES( p_dec->p_fifo->p_packets_mgt, p_dec->p_pes );
+        if( p_sys->p_synchro )
+            vout_SynchroRelease( p_sys->p_synchro );
 
-        if( p_dec->p_vout )
+        if( p_sys->p_vout )
         {
             /* Temporary hack to free the pictures in use by libmpeg2 */
-            for( i_pic = 0; i_pic < p_dec->p_vout->render.i_pictures; i_pic++ )
+            for( i_pic = 0; i_pic < p_sys->p_vout->render.i_pictures; i_pic++ )
             {
-                if( p_dec->p_vout->render.pp_picture[i_pic]->i_status ==
+                if( p_sys->p_vout->render.pp_picture[i_pic]->i_status ==
                       RESERVED_PICTURE )
-                    vout_DestroyPicture( p_dec->p_vout,
-                                     p_dec->p_vout->render.pp_picture[i_pic] );
+                    vout_DestroyPicture( p_sys->p_vout,
+                                     p_sys->p_vout->render.pp_picture[i_pic] );
+                if( p_sys->p_vout->render.pp_picture[i_pic]->i_refcount > 0 )
+                    vout_UnlinkPicture( p_sys->p_vout,
+                                     p_sys->p_vout->render.pp_picture[i_pic] );
             }
 
-            vout_Request( p_dec->p_fifo, p_dec->p_vout, 0, 0, 0, 0 );
+            vout_Request( p_dec, p_sys->p_vout, 0, 0, 0, 0 );
         }
 
-        if( p_dec->p_synchro )
-            vout_SynchroRelease( p_dec->p_synchro );
-
-        if( p_dec->p_mpeg2dec ) mpeg2_close( p_dec->p_mpeg2dec );
+        if( p_sys->p_mpeg2dec ) mpeg2_close( p_sys->p_mpeg2dec );
 
-        free( p_dec );
+        free( p_sys );
     }
+
+    return VLC_SUCCESS;
 }
 
 /*****************************************************************************
  * GetNewPicture: Get a new picture from the vout and set the buf struct
  *****************************************************************************/
-static picture_t *GetNewPicture( dec_thread_t *p_dec, uint8_t **pp_buf )
+static picture_t *GetNewPicture( decoder_t *p_dec, uint8_t **pp_buf )
 {
+    decoder_sys_t *p_sys = p_dec->p_sys;
+
     picture_t *p_pic;
+    vlc_bool_t b_progressive = p_sys->p_info->current_picture != NULL ?
+        p_sys->p_info->current_picture->flags & PIC_FLAG_PROGRESSIVE_FRAME :
+        1;
+    vlc_bool_t b_top_field_first = p_sys->p_info->current_picture != NULL ?
+        p_sys->p_info->current_picture->flags & PIC_FLAG_TOP_FIELD_FIRST :
+        1;
+    unsigned int i_nb_fields = p_sys->p_info->current_picture != NULL ?
+        p_sys->p_info->current_picture->nb_fields : 2;
 
     /* Get a new picture */
-    while( !(p_pic = vout_CreatePicture( p_dec->p_vout, 0, 0, 0 ) ) )
+    while( !(p_pic = vout_CreatePicture( p_sys->p_vout,
+        b_progressive, b_top_field_first, i_nb_fields )) )
     {
         if( p_dec->p_fifo->b_die || p_dec->p_fifo->b_error )
             break;
@@ -412,7 +585,7 @@ static picture_t *GetNewPicture( dec_thread_t *p_dec, uint8_t **pp_buf )
     }
     if( p_pic == NULL )
         return NULL;
-    vout_LinkPicture( p_dec->p_vout, p_pic );
+    vout_LinkPicture( p_sys->p_vout, p_pic );
 
     pp_buf[0] = p_pic->p[0].p_pixels;
     pp_buf[1] = p_pic->p[1].p_pixels;