Removed dead code + added a check against allocation errors.

diff --git a/modules/packetizer/h264.c b/modules/packetizer/h264.c
index eeb8d80fb4646a854edab9ff31b3046bfb2f6cd7..6681a420481d1735b3e88ac6f3d42e4042d5f445 100644 (file)
--- a/modules/packetizer/h264.c
+++ b/modules/packetizer/h264.c
@@ -232,8 +232,6 @@ static int Open( vlc_object_t *p_this )
             block_t *p_sps = nal_get_annexeb( p_dec, p, i_length );
             if( !p_sps )
                 return VLC_EGENERIC;
-            p_sys->p_sps = block_Duplicate( p_sps );
-            p_sps->i_pts = p_sps->i_dts = mdate();
             ParseNALBlock( p_dec, p_sps );
             p += i_length;
         }
@@ -250,14 +248,15 @@ static int Open( vlc_object_t *p_this )
             block_t *p_pps = nal_get_annexeb( p_dec, p, i_length );
             if( !p_pps )
                 return VLC_EGENERIC;
-            p_sys->p_pps = block_Duplicate( p_pps );
-            p_pps->i_pts = p_pps->i_dts = mdate();
             ParseNALBlock( p_dec, p_pps );
             p += i_length;
         }
         msg_Dbg( p_dec, "avcC length size=%d, sps=%d, pps=%d",
                  p_sys->i_avcC_length_size, i_sps, i_pps );
 
+        if( !p_sys->p_sps || p_sys->p_pps )
+            return VLC_EGENERIC;
+
         /* FIXME: FFMPEG isn't happy at all if you leave this */
         if( p_dec->fmt_out.i_extra > 0 ) free( p_dec->fmt_out.p_extra );
         p_dec->fmt_out.i_extra = 0;