]> git.sesse.net Git - vlc/blobdiff - modules/demux/flac.c
Fix compiler warning about asprintf return value.
[vlc] / modules / demux / flac.c
index 5077c5d9809a619e07b1f64a2455c2a2b78f9ae1..d2fc0395b826fd234f87ab91451ab7274132f147 100644 (file)
@@ -1,10 +1,11 @@
 /*****************************************************************************
  * flac.c : FLAC demux module for vlc
  *****************************************************************************
- * Copyright (C) 2001-2003 VideoLAN
+ * Copyright (C) 2001-2007 the VideoLAN team
  * $Id$
  *
  * Authors: Gildas Bazin <gbazin@netcourrier.com>
+ *          Laurent Aimar <fenrir@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
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <vlc/vlc.h>
-#include <vlc/input.h>
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
+#include <vlc_plugin.h>
+#include <vlc_demux.h>
+#include <vlc_meta.h>
+#include <vlc_input.h>
 #include <vlc_codec.h>
+#include <assert.h>
+#include <vlc_charset.h>
 
 /*****************************************************************************
  * Module descriptor
@@ -35,8 +45,10 @@ static int  Open  ( vlc_object_t * );
 static void Close ( vlc_object_t * );
 
 vlc_module_begin();
-    set_description( _("FLAC demuxer") );
-    set_capability( "demux2", 155 );
+    set_description( N_("FLAC demuxer") );
+    set_capability( "demux", 155 );
+    set_category( CAT_INPUT );
+    set_subcategory( SUBCAT_INPUT_DEMUX );
     set_callbacks( Open, Close );
     add_shortcut( "flac" );
 vlc_module_end();
@@ -47,13 +59,35 @@ vlc_module_end();
 static int Demux  ( demux_t * );
 static int Control( demux_t *, int, va_list );
 
+static int  ReadMeta( demux_t *, uint8_t **pp_streaminfo, int *pi_streaminfo );
+
 struct demux_sys_t
 {
-    vlc_bool_t  b_start;
+    bool  b_start;
     es_out_id_t *p_es;
 
     /* Packetizer */
     decoder_t *p_packetizer;
+
+    vlc_meta_t *p_meta;
+    audio_replay_gain_t replay_gain;
+
+    int64_t i_time_offset;
+    int64_t i_pts;
+    int64_t i_pts_start;
+
+    int64_t i_length; /* Length from stream info */
+    int64_t i_data_pos;
+
+    /* */
+    int         i_seekpoint;
+    seekpoint_t **seekpoint;
+
+    /* */
+    int                i_attachments;
+    input_attachment_t **attachments;
+    int                i_cover_idx;
+    int                i_cover_score;
 };
 
 #define STREAMINFO_SIZE 38
@@ -66,25 +100,17 @@ static int Open( vlc_object_t * p_this )
 {
     demux_t     *p_demux = (demux_t*)p_this;
     demux_sys_t *p_sys;
-    int          i_peek;
-    byte_t      *p_peek;
-    es_format_t  fmt;
+    const uint8_t *p_peek;
+    uint8_t     *p_streaminfo;
+    int         i_streaminfo;
 
     /* Have a peep at the show. */
-    if( stream_Peek( p_demux->s, &p_peek, 4 ) < 4 )
-    {
-        /* Stream shorter than 4 bytes... */
-        msg_Err( p_demux, "cannot peek()" );
-        return VLC_EGENERIC;
-    }
+    if( stream_Peek( p_demux->s, &p_peek, 4 ) < 4 ) return VLC_EGENERIC;
 
     if( p_peek[0]!='f' || p_peek[1]!='L' || p_peek[2]!='a' || p_peek[3]!='C' )
     {
-        if( strncmp( p_demux->psz_demux, "flac", 4 ) )
-        {
-            msg_Warn( p_demux, "flac module discarded (no startcode)" );
-            return VLC_EGENERIC;
-        }
+        if( !p_demux->b_force ) return VLC_EGENERIC;
+
         /* User forced */
         msg_Err( p_demux, "this doesn't look like a flac stream, "
                  "continuing anyway" );
@@ -93,62 +119,55 @@ static int Open( vlc_object_t * p_this )
     p_demux->pf_demux   = Demux;
     p_demux->pf_control = Control;
     p_demux->p_sys      = p_sys = malloc( sizeof( demux_sys_t ) );
-    es_format_Init( &fmt, AUDIO_ES, VLC_FOURCC( 'f', 'l', 'a', 'c' ) );
-    p_sys->b_start = VLC_TRUE;
+    p_sys->b_start = true;
+    p_sys->p_meta = NULL;
+    memset( &p_sys->replay_gain, 0, sizeof(p_sys->replay_gain) );
+    p_sys->i_length = 0;
+    p_sys->i_time_offset = 0;
+    p_sys->i_pts = 0;
+    p_sys->i_pts_start = 0;
+    p_sys->p_es = NULL;
+    TAB_INIT( p_sys->i_seekpoint, p_sys->seekpoint );
+    TAB_INIT( p_sys->i_attachments, p_sys->attachments);
+    p_sys->i_cover_idx = 0;
+    p_sys->i_cover_score = 0;
 
     /* We need to read and store the STREAMINFO metadata */
-    i_peek = stream_Peek( p_demux->s, &p_peek, 8 );
-    if( p_peek[4] & 0x7F )
+    if( ReadMeta( p_demux, &p_streaminfo, &i_streaminfo ) )
     {
-        msg_Err( p_demux, "this isn't a STREAMINFO metadata block" );
+        free( p_sys );
         return VLC_EGENERIC;
     }
 
-    if( ((p_peek[5]<<16)+(p_peek[6]<<8)+p_peek[7]) != (STREAMINFO_SIZE - 4) )
-    {
-        msg_Err( p_demux, "invalid size for a STREAMINFO metadata block" );
-        return VLC_EGENERIC;
-    }
-
-    /*
-     * Load the FLAC packetizer
-     */
-    p_sys->p_packetizer = vlc_object_create( p_demux, VLC_OBJECT_DECODER );
-    p_sys->p_packetizer->pf_decode_audio = 0;
-    p_sys->p_packetizer->pf_decode_video = 0;
-    p_sys->p_packetizer->pf_decode_sub = 0;
-    p_sys->p_packetizer->pf_packetize = 0;
-
-    /* Initialization of decoder structure */
-    es_format_Init( &p_sys->p_packetizer->fmt_in, AUDIO_ES,
-                    VLC_FOURCC( 'f', 'l', 'a', 'c' ) );
+    /* Load the FLAC packetizer */
+    INIT_APACKETIZER( p_sys->p_packetizer, 'f', 'l', 'a', 'c' );
 
     /* Store STREAMINFO for the decoder and packetizer */
-    p_sys->p_packetizer->fmt_in.i_extra = fmt.i_extra = STREAMINFO_SIZE + 4;
-    p_sys->p_packetizer->fmt_in.p_extra = malloc( STREAMINFO_SIZE + 4 );
-    stream_Read( p_demux->s, p_sys->p_packetizer->fmt_in.p_extra,
-                 STREAMINFO_SIZE + 4 );
-
-    /* Fake this as the last metadata block */
-    ((uint8_t*)p_sys->p_packetizer->fmt_in.p_extra)[4] |= 0x80;
-    fmt.p_extra = malloc( STREAMINFO_SIZE + 4 );
-    memcpy( fmt.p_extra, p_sys->p_packetizer->fmt_in.p_extra,
-            STREAMINFO_SIZE + 4 );
+    p_streaminfo[4] |= 0x80; /* Fake this as the last metadata block */
+    p_sys->p_packetizer->fmt_in.i_extra = i_streaminfo;
+    p_sys->p_packetizer->fmt_in.p_extra = p_streaminfo;
 
     p_sys->p_packetizer->p_module =
         module_Need( p_sys->p_packetizer, "packetizer", NULL, 0 );
     if( !p_sys->p_packetizer->p_module )
     {
-        if( p_sys->p_packetizer->fmt_in.p_extra )
-            free( p_sys->p_packetizer->fmt_in.p_extra );
+        free( p_sys->p_packetizer->fmt_in.p_extra );
+        vlc_object_release( p_sys->p_packetizer );
 
-        vlc_object_destroy( p_sys->p_packetizer );
         msg_Err( p_demux, "cannot find flac packetizer" );
         return VLC_EGENERIC;
     }
 
-    p_sys->p_es = es_out_Add( p_demux->out, &fmt );
-
+    if( p_sys->i_cover_idx < p_sys->i_attachments )
+    {
+        char psz_url[128];
+        if( !p_sys->p_meta )
+            p_sys->p_meta = vlc_meta_New();
+        snprintf( psz_url, sizeof(psz_url), "attachment://%s",
+                  p_sys->attachments[p_sys->i_cover_idx]->psz_name );
+        vlc_meta_Set( p_sys->p_meta, vlc_meta_ArtworkURL, psz_url );
+    }
+    vlc_audio_replay_gain_MergeFromMeta( &p_sys->replay_gain, p_sys->p_meta );
     return VLC_SUCCESS;
 }
 
@@ -160,15 +179,22 @@ static void Close( vlc_object_t * p_this )
     demux_t     *p_demux = (demux_t*)p_this;
     demux_sys_t *p_sys = p_demux->p_sys;
 
+    TAB_CLEAN( p_sys->i_seekpoint, p_sys->seekpoint );
+
+    int i;
+    for( i = 0; i < p_sys->i_attachments; i++ )
+        free( p_sys->attachments[i] );
+    TAB_CLEAN( p_sys->i_attachments, p_sys->attachments);
+
     /* Unneed module */
     module_Unneed( p_sys->p_packetizer, p_sys->p_packetizer->p_module );
 
-    if( p_sys->p_packetizer->fmt_in.p_extra )
-        free( p_sys->p_packetizer->fmt_in.p_extra );
+    free( p_sys->p_packetizer->fmt_in.p_extra );
 
     /* Delete the decoder */
-    vlc_object_destroy( p_sys->p_packetizer );
-
+    vlc_object_release( p_sys->p_packetizer );
+    if( p_sys->p_meta )
+        vlc_meta_Delete( p_sys->p_meta );
     free( p_sys );
 }
 
@@ -183,19 +209,10 @@ static int Demux( demux_t *p_demux )
     block_t     *p_block_in, *p_block_out;
 
     if( !( p_block_in = stream_Block( p_demux->s, FLAC_PACKET_SIZE ) ) )
-    {
         return 0;
-    }
 
-    if( p_sys->b_start )
-    {
-        p_block_in->i_pts = p_block_in->i_dts = 1;
-        p_sys->b_start = VLC_FALSE;
-    }
-    else
-    {
-        p_block_in->i_pts = p_block_in->i_dts = 0;
-    }
+    p_block_in->i_pts = p_block_in->i_dts = p_sys->b_start ? 1 : 0;
+    p_sys->b_start = false;
 
     while( (p_block_out = p_sys->p_packetizer->pf_packetize(
                 p_sys->p_packetizer, &p_block_in )) )
@@ -204,30 +221,499 @@ static int Demux( demux_t *p_demux )
         {
             block_t *p_next = p_block_out->p_next;
 
+            p_block_out->p_next = NULL;
+
+            if( p_sys->p_es == NULL )
+            {
+                p_sys->p_packetizer->fmt_out.b_packetized = true;
+                p_sys->p_packetizer->fmt_out.audio_replay_gain = p_sys->replay_gain;
+                p_sys->p_es = es_out_Add( p_demux->out, &p_sys->p_packetizer->fmt_out);
+            }
+
             /* set PCR */
-            es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block_out->i_dts );
+            if( p_block_out->i_dts >= p_sys->i_pts_start )
+                es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block_out->i_dts );
+            else
+                es_out_Control( p_demux->out, ES_OUT_RESET_PCR );
 
+            p_sys->i_pts = p_block_out->i_dts;
             es_out_Send( p_demux->out, p_sys->p_es, p_block_out );
 
             p_block_out = p_next;
         }
     }
-
     return 1;
 }
 
 /*****************************************************************************
  * Control:
  *****************************************************************************/
-static int Control( demux_t *p_demux, int i_query, va_list args )
+static int64_t ControlGetLength( demux_t *p_demux )
 {
-    /* demux_sys_t *p_sys  = p_demux->p_sys; */
-    /* FIXME bitrate */
-    if( i_query == DEMUX_SET_TIME )
+    demux_sys_t *p_sys = p_demux->p_sys;
+    const int64_t i_size = stream_Size(p_demux->s) - p_sys->i_data_pos;
+    int64_t i_length = p_sys->i_length;
+    int i;
+
+    /* Try to fix length using seekpoint and current size for truncated file */
+    for( i = p_sys->i_seekpoint-1; i >= 0; i-- )
+    {
+        seekpoint_t *s = p_sys->seekpoint[i];
+        if( s->i_byte_offset <= i_size )
+        {
+            if( i+1 < p_sys->i_seekpoint )
+            {
+                /* Broken file */
+                seekpoint_t *n = p_sys->seekpoint[i+1];
+                assert( n->i_byte_offset != s->i_byte_offset); /* Should be ensured by ParseSeekTable */
+                i_length = s->i_time_offset + (n->i_time_offset-s->i_time_offset) * (i_size-s->i_byte_offset) / (n->i_byte_offset-s->i_byte_offset);
+            }
+            break;
+        }
+    }
+    return i_length;
+}
+
+static int64_t ControlGetTime( demux_t *p_demux )
+{
+    demux_sys_t *p_sys = p_demux->p_sys;
+    return __MAX(p_sys->i_pts, p_sys->i_pts_start) + p_sys->i_time_offset;
+}
+
+static int ControlSetTime( demux_t *p_demux, int64_t i_time )
+{
+    demux_sys_t *p_sys = p_demux->p_sys;
+    int64_t i_next_time;
+    int64_t i_next_offset;
+    int64_t i_delta_time;
+    int64_t i_delta_offset;
+    bool b_seekable;
+    int i;
+
+    /* */
+    stream_Control( p_demux->s, STREAM_CAN_SEEK, &b_seekable );
+    if( !b_seekable )
         return VLC_EGENERIC;
+
+    /* */
+    assert( p_sys->i_seekpoint > 0 );   /* ReadMeta ensure at least (0,0) */
+    for( i = p_sys->i_seekpoint-1; i >= 0; i-- )
+    {
+        if( p_sys->seekpoint[i]->i_time_offset <= i_time )
+            break;
+    }
+    if( i+1 < p_sys->i_seekpoint )
+    {
+        i_next_time   = p_sys->seekpoint[i+1]->i_time_offset;
+        i_next_offset = p_sys->seekpoint[i+1]->i_byte_offset;
+    }
     else
-        return demux2_vaControlHelper( p_demux->s,
-                                       0, -1,
-                                       8*0, 1, i_query, args );
+    {
+        i_next_time   = p_sys->i_length;
+        i_next_offset = stream_Size(p_demux->s)-p_sys->i_data_pos;
+    }
+    i_delta_time = i_time - p_sys->seekpoint[i]->i_time_offset;
+    i_delta_offset = (i_next_offset - p_sys->seekpoint[i]->i_byte_offset) * i_delta_time /
+                            (p_sys->seekpoint[i+1]->i_time_offset-p_sys->seekpoint[i]->i_time_offset);
+
+    /* XXX We do exact seek if it's not too far away(45s) */
+    if( i_delta_time < 45*INT64_C(1000000) )
+    {
+        if( stream_Seek( p_demux->s, p_sys->seekpoint[i]->i_byte_offset+p_sys->i_data_pos ) )
+            return VLC_EGENERIC;
+        p_sys->i_time_offset = p_sys->seekpoint[i]->i_time_offset - p_sys->i_pts;
+        p_sys->i_pts_start = p_sys->i_pts+i_delta_time;
+        es_out_Control( p_demux->out, ES_OUT_SET_NEXT_DISPLAY_TIME, p_sys->p_es, p_sys->i_pts_start );
+    }
+    else
+    {
+        if( stream_Seek( p_demux->s, p_sys->seekpoint[i]->i_byte_offset+p_sys->i_data_pos + i_delta_offset ) )
+            return VLC_EGENERIC;
+        p_sys->i_pts_start = p_sys->i_pts;
+        p_sys->i_time_offset = (p_sys->seekpoint[i]->i_time_offset+i_delta_time) - p_sys->i_pts;
+    }
+    return VLC_SUCCESS;
+}
+
+static int Control( demux_t *p_demux, int i_query, va_list args )
+{
+    demux_sys_t *p_sys = p_demux->p_sys;
+
+    if( i_query == DEMUX_GET_META )
+    {
+        vlc_meta_t *p_meta = (vlc_meta_t *)va_arg( args, vlc_meta_t* );
+        if( p_demux->p_sys->p_meta )
+            vlc_meta_Merge( p_meta, p_demux->p_sys->p_meta );
+        return VLC_SUCCESS;
+    }
+    else if( i_query == DEMUX_HAS_UNSUPPORTED_META )
+    {
+        bool *pb_bool = (bool*)va_arg( args, bool* );
+        *pb_bool = true;
+        return VLC_SUCCESS;
+    }
+    else if( i_query == DEMUX_GET_LENGTH )
+    {
+        int64_t *pi64 = (int64_t*)va_arg( args, int64_t * );
+        *pi64 = ControlGetLength( p_demux );
+        return VLC_SUCCESS;
+    }
+    else if( i_query == DEMUX_SET_TIME )
+    {
+        int64_t i_time = (int64_t)va_arg( args, int64_t );
+        return ControlSetTime( p_demux, i_time );
+    }
+    else if( i_query == DEMUX_SET_POSITION )
+    {
+        const double f = (double)va_arg( args, double );
+        int64_t i_time = f * ControlGetLength( p_demux );
+        return ControlSetTime( p_demux, i_time );
+    }
+    else if( i_query == DEMUX_GET_TIME )
+    {
+        int64_t *pi64 = (int64_t*)va_arg( args, int64_t * );
+        *pi64 = ControlGetTime( p_demux );
+        return VLC_SUCCESS;
+    }
+    else if( i_query == DEMUX_GET_POSITION )
+    {
+        double *pf = (double*)va_arg( args, double * );
+        const int64_t i_length = ControlGetLength(p_demux);
+        if( i_length > 0 )
+            *pf = (double)ControlGetTime(p_demux) / (double)i_length;
+        else
+            *pf= 0.0;
+        return VLC_SUCCESS;
+    }
+    else if( i_query == DEMUX_GET_ATTACHMENTS )
+    {
+        input_attachment_t ***ppp_attach =
+            (input_attachment_t***)va_arg( args, input_attachment_t*** );
+        int *pi_int = (int*)va_arg( args, int * );
+        int i;
+
+        if( p_sys->i_attachments <= 0 )
+            return VLC_EGENERIC;
+
+        *pi_int = p_sys->i_attachments;;
+        *ppp_attach = malloc( sizeof(input_attachment_t**) * p_sys->i_attachments );
+        for( i = 0; i < p_sys->i_attachments; i++ )
+            (*ppp_attach)[i] = vlc_input_attachment_Duplicate( p_sys->attachments[i] );
+        return VLC_SUCCESS;
+    }
+
+    return demux_vaControlHelper( p_demux->s, p_sys->i_data_pos, -1,
+                                   8*0, 1, i_query, args );
+}
+
+enum
+{
+    META_STREAMINFO = 0,
+    META_SEEKTABLE = 3,
+    META_COMMENT = 4,
+    META_PICTURE = 6,
+};
+
+static inline int Get24bBE( const uint8_t *p )
+{
+    return (p[0] << 16)|(p[1] << 8)|(p[2]);
+}
+
+static void ParseStreamInfo( demux_t *p_demux, int *pi_rate, int64_t *pi_count, uint8_t *p_data, int i_data );
+static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data,
+                            int i_sample_rate );
+static void ParseComment( demux_t *, const uint8_t *p_data, int i_data );
+static void ParsePicture( demux_t *, const uint8_t *p_data, int i_data );
+
+static int  ReadMeta( demux_t *p_demux, uint8_t **pp_streaminfo, int *pi_streaminfo )
+{
+    demux_sys_t *p_sys = p_demux->p_sys;
+    int     i_peek;
+    const uint8_t *p_peek;
+    bool b_last;
+    int i_sample_rate;
+    int64_t i_sample_count;
+    seekpoint_t *s;
+
+    /* Read STREAMINFO */
+    i_peek = stream_Peek( p_demux->s, &p_peek, 8 );
+    if( (p_peek[4] & 0x7F) != META_STREAMINFO )
+    {
+        msg_Err( p_demux, "this isn't a STREAMINFO metadata block" );
+        return VLC_EGENERIC;
+    }
+    if( Get24bBE(&p_peek[5]) != (STREAMINFO_SIZE - 4) )
+    {
+        msg_Err( p_demux, "invalid size for a STREAMINFO metadata block" );
+        return VLC_EGENERIC;
+    }
+
+    *pi_streaminfo = 4 + STREAMINFO_SIZE;
+    *pp_streaminfo = malloc( 4 + STREAMINFO_SIZE );
+    if( *pp_streaminfo == NULL )
+        return VLC_EGENERIC;
+
+    if( stream_Read( p_demux->s, *pp_streaminfo, 4+STREAMINFO_SIZE ) != 4+STREAMINFO_SIZE )
+    {
+        msg_Err( p_demux, "failed to read STREAMINFO metadata block" );
+        free( *pp_streaminfo );
+        return VLC_EGENERIC;
+    }
+
+    /* */
+    ParseStreamInfo( p_demux, &i_sample_rate, &i_sample_count,  *pp_streaminfo, *pi_streaminfo );
+    if( i_sample_rate > 0 )
+        p_sys->i_length = i_sample_count * INT64_C(1000000)/i_sample_rate;
+
+    /* Be sure we have seekpoint 0 */
+    s = vlc_seekpoint_New();
+    s->i_time_offset = 0;
+    s->i_byte_offset = 0;
+    TAB_APPEND( p_sys->i_seekpoint, p_sys->seekpoint, s );
+
+    b_last = (*pp_streaminfo)[4]&0x80;
+    while( !b_last )
+    {
+        int i_len;
+        int i_type;
+
+        i_peek = stream_Peek( p_demux->s, &p_peek, 4 );
+        if( i_peek < 4 )
+            break;
+        b_last = p_peek[0]&0x80;
+        i_type = p_peek[0]&0x7f;
+        i_len  = Get24bBE( &p_peek[1] );
+
+        if( i_type == META_SEEKTABLE )
+        {
+            i_peek = stream_Peek( p_demux->s, &p_peek, 4+i_len );
+            if( i_peek == 4+i_len )
+                ParseSeekTable( p_demux, p_peek, i_peek, i_sample_rate );
+        }
+        else if( i_type == META_COMMENT )
+        {
+            i_peek = stream_Peek( p_demux->s, &p_peek, 4+i_len );
+            if( i_peek == 4+i_len )
+                ParseComment( p_demux, p_peek, i_peek );
+        }
+        else if( i_type == META_PICTURE )
+        {
+            i_peek = stream_Peek( p_demux->s, &p_peek, 4+i_len );
+            if( i_peek == 4+i_len )
+                ParsePicture( p_demux, p_peek, i_peek );
+        }
+
+        if( stream_Read( p_demux->s, NULL, 4+i_len ) < 4+i_len )
+            break;
+    }
+
+    /* */
+    p_sys->i_data_pos = stream_Tell( p_demux->s );
+
+    return VLC_SUCCESS;
+}
+static void ParseStreamInfo( demux_t *p_demux, int *pi_rate, int64_t *pi_count, uint8_t *p_data, int i_data )
+{
+    const int i_skip = 4+4;
+
+    *pi_rate = GetDWBE(&p_data[i_skip+4+6]) >> 12;
+    *pi_count = GetQWBE(&p_data[i_skip+4+6]) &  ((INT64_C(1)<<36)-1);
+}
+
+static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data,
+                            int i_sample_rate )
+{
+    demux_sys_t *p_sys = p_demux->p_sys;
+    seekpoint_t *s;
+    int i;
+
+    if( i_sample_rate <= 0 )
+        return;
+
+    /* */
+    for( i = 0; i < (i_data-4)/18; i++ )
+    {
+        const int64_t i_sample = GetQWBE( &p_data[4+18*i+0] );
+        int j;
+
+        if( i_sample < 0 || i_sample >= INT64_MAX )
+            continue;
+
+        s = vlc_seekpoint_New();
+        s->i_time_offset = i_sample * INT64_C(1000000)/i_sample_rate;
+        s->i_byte_offset = GetQWBE( &p_data[4+18*i+8] );
+
+        /* Check for duplicate entry */
+        for( j = 0; j < p_sys->i_seekpoint; j++ )
+        {
+            if( p_sys->seekpoint[j]->i_time_offset == s->i_time_offset ||
+                p_sys->seekpoint[j]->i_byte_offset == s->i_byte_offset )
+            {
+                vlc_seekpoint_Delete( s );
+                s = NULL;
+                break;
+            }
+        }
+        if( s )
+        {
+            TAB_APPEND( p_sys->i_seekpoint, p_sys->seekpoint, s );
+        }
+    }
+    /* TODO sort it by size and remove wrong seek entry (time not increasing) */
+}
+
+#define RM(x) do { i_data -= (x); p_data += (x); } while(0)
+static void ParseComment( demux_t *p_demux, const uint8_t *p_data, int i_data )
+{
+    demux_sys_t *p_sys = p_demux->p_sys;
+    int n;
+    int i_comment;
+
+    if( i_data < 8 )
+        return;
+
+    RM(4);
+
+    n = GetDWLE(p_data); RM(4);
+    if( n < 0 || n > i_data )
+        return;
+#if 0
+    if( n > 0 )
+    {
+        /* TODO report vendor string ? */
+        char *psz_vendor = psz_vendor = strndup( p_data, n );
+        msg_Dbg( p_demux, "FLAC: COMMENT vendor length=%d vendor=%s\n", n, psz_vendor );
+        free( psz_vendor );
+    }
+#endif
+    RM(n);
+
+    if( i_data < 4 )
+        return;
+
+    i_comment = GetDWLE(p_data); RM(4);
+    if( i_comment <= 0 )
+        return;
+
+    p_sys->p_meta = vlc_meta_New();
+
+    for( ; i_comment > 0; i_comment-- )
+    {
+        char *psz;
+        if( i_data < 4 )
+            break;
+        n = GetDWLE(p_data); RM(4);
+        if( n > i_data )
+            break;
+        if( n <= 0 )
+            continue;
+
+        psz = strndup( p_data, n );
+        RM(n);
+
+        EnsureUTF8( psz );
+
+#define IF_EXTRACT(txt,var) \
+    if( !strncasecmp(psz, txt, strlen(txt)) ) \
+    { \
+        const char *oldval = vlc_meta_Get( p_sys->p_meta, vlc_meta_ ## var ); \
+        if( oldval ) \
+        { \
+            char * newval; \
+            if( asprintf( &newval, "%s,%s", oldval, &psz[strlen(txt)] ) == -1 ) \
+                newval = NULL; \
+            vlc_meta_Set( p_sys->p_meta, vlc_meta_ ## var, newval ); \
+            free( newval ); \
+        } \
+        else \
+            vlc_meta_Set( p_sys->p_meta, vlc_meta_ ## var, &psz[strlen(txt)] ); \
+    }
+        IF_EXTRACT("TITLE=", Title )
+        else IF_EXTRACT("ALBUM=", Album )
+        else IF_EXTRACT("TRACKNUMBER=", TrackNumber )
+        else IF_EXTRACT("ARTIST=", Artist )
+        else IF_EXTRACT("COPYRIGHT=", Copyright )
+        else IF_EXTRACT("DESCRIPTION=", Description )
+        else IF_EXTRACT("GENRE=", Genre )
+        else IF_EXTRACT("DATE=", Date )
+        else if( strchr( psz, '=' ) )
+        {
+            /* generic (PERFORMER/LICENSE/ORGANIZATION/LOCATION/CONTACT/ISRC,
+             * undocumented tags and replay gain ) */
+            char *p = strchr( psz, '=' );
+            *p++ = '\0';
+            vlc_meta_AddExtra( p_sys->p_meta, psz, p );
+        }
+#undef IF_EXTRACT
+        free( psz );
+    }
+#undef RM
+}
+
+static void ParsePicture( demux_t *p_demux, const uint8_t *p_data, int i_data )
+{
+    static const int pi_cover_score[] = {
+        0,      /* other */
+        2, 1,   /* icons */
+        10,     /* front cover */
+        9,      /* back cover */
+        0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
+        6,      /* movie/video screen capture */
+        0,
+        7,      /* Illustration */
+        8,      /* Band/Artist logotype */
+        0,      /* Publisher/Studio */
+    };
+    demux_sys_t *p_sys = p_demux->p_sys;
+    int i_type;
+    int i_len;
+    char *psz_mime = NULL;
+    char *psz_description = NULL;
+    input_attachment_t *p_attachment;
+    char psz_name[128];
+
+    if( i_data < 4 + 3*4 )
+        return;
+#define RM(x) do { i_data -= (x); p_data += (x); } while(0)
+    RM(4);
+
+    i_type = GetDWBE( p_data ); RM(4);
+    i_len = GetDWBE( p_data ); RM(4);
+    if( i_len < 0 || i_data < i_len + 4 )
+        goto error;
+    psz_mime = strndup( p_data, i_len ); RM(i_len);
+    i_len = GetDWBE( p_data ); RM(4);
+    if( i_len < 0 || i_data < i_len + 4*4 + 4)
+        goto error;
+    psz_description = strndup( p_data, i_len ); RM(i_len);
+    EnsureUTF8( psz_description );
+    RM(4*4);
+    i_len = GetDWBE( p_data ); RM(4);
+    if( i_len < 0 || i_len > i_data )
+        goto error;
+
+    msg_Dbg( p_demux, "FLAC: Picture type=%d mime=%s description='%s' file length=%d",
+             i_type, psz_mime, psz_description, i_len );
+
+    snprintf( psz_name, sizeof(psz_name), "picture%d", p_sys->i_attachments );
+    if( !strcasecmp( psz_mime, "image/jpeg" ) )
+        strcat( psz_name, ".jpg" );
+    else if( !strcasecmp( psz_mime, "image/png" ) )
+        strcat( psz_name, ".png" );
+
+    p_attachment = vlc_input_attachment_New( psz_name, psz_mime, psz_description,
+                                             p_data, i_data );
+    TAB_APPEND( p_sys->i_attachments, p_sys->attachments, p_attachment );
+
+    if( i_type >= 0 && i_type < sizeof(pi_cover_score)/sizeof(pi_cover_score[0]) &&
+        p_sys->i_cover_score < pi_cover_score[i_type] )
+    {
+        p_sys->i_cover_idx = p_sys->i_attachments-1;
+        p_sys->i_cover_score = pi_cover_score[i_type];
+    }
+error:
+    free( psz_mime );
+    free( psz_description );
 }
+#undef RM