* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
*****************************************************************************/
-#include <vlc/vlc.h>
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include <vlc_common.h>
#include <vlc_demux.h>
(guid).v4[0],(guid).v4[1],(guid).v4[2],(guid).v4[3], \
(guid).v4[4],(guid).v4[5],(guid).v4[6],(guid).v4[7]
+/* Helpers:
+ * They ensure that invalid reads will not create problems.
+ * They are expansion safe
+ * They make the following assumptions:
+ * const uint8_t *p_peek exists and points to the start of a buffer
+ * int i_peek gives the size of the buffer pointed by p_peek
+ * const uint8_t *p_data exits and points to the data inside p_peek to be read.
+ */
+/* ASF_HAVE(n):
+ * Check that n bytes can be read */
+static inline bool AsfObjectHelperHave( const uint8_t *p_peek, int i_peek, const uint8_t *p_current, int i_wanted )
+{
+ if( i_wanted < 0 || i_wanted > i_peek )
+ return false;
+ return &p_current[i_wanted] <= &p_peek[i_peek];
+}
+#define ASF_HAVE(n) AsfObjectHelperHave( p_peek, i_peek, p_data, n )
+
+/* ASF_SKIP(n)
+ * Skip n bytes if possible */
+static inline void AsfObjectHelperSkip( const uint8_t *p_peek, int i_peek, uint8_t **pp_data, int i_wanted )
+{
+ if( AsfObjectHelperHave( p_peek, i_peek, *pp_data, i_wanted ) )
+ *pp_data += i_wanted;
+ else
+ *pp_data = (uint8_t*)&p_peek[i_peek];
+}
+#define ASF_SKIP(n) AsfObjectHelperSkip( p_peek, i_peek, (uint8_t**)&p_data, n )
+
+/* ASF_READX()
+ * Read X byte if possible, else return 0 */
+#define ASF_FUNCTION_READ_X(type, x, cmd ) \
+static inline type AsfObjectHelperRead##x( const uint8_t *p_peek, int i_peek, uint8_t **pp_data ) { \
+ uint8_t *p_data = *pp_data; \
+ type i_ret = 0; \
+ if( ASF_HAVE(x) ) \
+ i_ret = cmd; \
+ ASF_SKIP(x); \
+ *pp_data = p_data; \
+ return i_ret; }
+ASF_FUNCTION_READ_X( uint8_t, 1, *p_data )
+ASF_FUNCTION_READ_X( uint16_t, 2, GetWLE(p_data) )
+ASF_FUNCTION_READ_X( uint32_t, 4, GetDWLE(p_data) )
+ASF_FUNCTION_READ_X( uint64_t, 8, GetQWLE(p_data) )
+#define ASF_READ1() AsfObjectHelperRead1( p_peek, i_peek, (uint8_t**)&p_data )
+#define ASF_READ2() AsfObjectHelperRead2( p_peek, i_peek, (uint8_t**)&p_data )
+#define ASF_READ4() AsfObjectHelperRead4( p_peek, i_peek, (uint8_t**)&p_data )
+#define ASF_READ8() AsfObjectHelperRead8( p_peek, i_peek, (uint8_t**)&p_data )
+
+/* ASF_READS(n)
+ * Read a string of n/2 wchar long ie n bytes. Do a stupid conversion (suppose latin1)
+ * Return allocated "" if not possible */
+static char *AsfObjectHelperReadString( const uint8_t *p_peek, int i_peek, uint8_t **pp_data, int i_size )
+{
+ uint8_t *p_data = *pp_data;
+ char *psz_string;
+ if( ASF_HAVE(i_size) )
+ {
+ psz_string = calloc( i_size/2 + 1, sizeof( char ) );
+ if( psz_string )
+ {
+ int i;
+ for( i = 0; i < i_size/2; i++ )
+ psz_string[i] = GetWLE( &p_data[2*i] );
+ psz_string[i_size/2] = '\0'; \
+ }
+ }
+ else
+ {
+ psz_string = strdup("");
+ }
+ ASF_SKIP(i_size);
+ *pp_data = p_data;
+ return psz_string;
+}
+#define ASF_READS(n) AsfObjectHelperReadString( p_peek, i_peek, (uint8_t**)&p_data, n )
+
/****************************************************************************
*
****************************************************************************/
memcpy( p_guid->v4, p_data + 8, 8 );
}
-int ASF_CmpGUID( const guid_t *p_guid1, const guid_t *p_guid2 )
+bool ASF_CmpGUID( const guid_t *p_guid1, const guid_t *p_guid2 )
{
if( (p_guid1->v1 != p_guid2->v1 )||
(p_guid1->v2 != p_guid2->v2 )||
(p_guid1->v3 != p_guid2->v3 )||
( memcmp( p_guid1->v4, p_guid2->v4,8 )) )
{
- return( 0 );
+ return false;
}
- return( 1 ); /* match */
+ return true;
}
/****************************************************************************
****************************************************************************/
static int ASF_ReadObjectCommon( stream_t *s, asf_object_t *p_obj )
{
- asf_object_common_t *p_common = (asf_object_common_t*)p_obj;
+ asf_object_common_t *p_common = &p_obj->common;
const uint8_t *p_peek;
if( stream_Peek( s, &p_peek, 24 ) < 24 )
- {
- return( VLC_EGENERIC );
- }
+ return VLC_EGENERIC;
+
ASF_GetGUID( &p_common->i_object_id, p_peek );
p_common->i_object_size = GetQWLE( p_peek + 16 );
p_common->i_object_pos = stream_Tell( s );
#ifdef ASF_DEBUG
msg_Dbg( s,
- "found object guid: " GUID_FMT " size:"I64Fd,
+ "found object guid: " GUID_FMT " size:%"PRId64,
GUID_PRINT( p_common->i_object_id ),
p_common->i_object_size );
#endif
- return( VLC_SUCCESS );
+ return VLC_SUCCESS;
}
static int ASF_NextObject( stream_t *s, asf_object_t *p_obj )
if( p_obj == NULL )
{
if( ASF_ReadObjectCommon( s, &obj ) )
- {
- return( VLC_EGENERIC );
- }
+ return VLC_EGENERIC;
+
p_obj = &obj;
}
if( p_obj->common.i_object_size <= 0 )
- {
return VLC_EGENERIC;
- }
+
if( p_obj->common.p_father &&
p_obj->common.p_father->common.i_object_size != 0 )
{
static void ASF_FreeObject_Null( asf_object_t *pp_obj )
{
- return;
+ VLC_UNUSED(pp_obj);
}
static int ASF_ReadObject_Header( stream_t *s, asf_object_t *p_obj )
{
- asf_object_header_t *p_hdr = (asf_object_header_t*)p_obj;
+ asf_object_header_t *p_hdr = &p_obj->header;
asf_object_t *p_subobj;
int i_peek;
const uint8_t *p_peek;
if( ( i_peek = stream_Peek( s, &p_peek, 30 ) ) < 30 )
- {
- return( VLC_EGENERIC );
- }
+ return VLC_EGENERIC;
p_hdr->i_sub_object_count = GetDWLE( p_peek + 24 );
p_hdr->i_reserved1 = p_peek[28];
{
p_subobj = malloc( sizeof( asf_object_t ) );
- if( ASF_ReadObject( s, p_subobj, (asf_object_t*)p_hdr ) )
+ if( !p_subobj || ASF_ReadObject( s, p_subobj, (asf_object_t*)p_hdr ) )
{
free( p_subobj );
break;
}
if( ASF_NextObject( s, p_subobj ) ) /* Go to the next object */
- {
break;
- }
}
return VLC_SUCCESS;
}
static int ASF_ReadObject_Data( stream_t *s, asf_object_t *p_obj )
{
- asf_object_data_t *p_data = (asf_object_data_t*)p_obj;
+ asf_object_data_t *p_data = &p_obj->data;
int i_peek;
const uint8_t *p_peek;
if( ( i_peek = stream_Peek( s, &p_peek, 50 ) ) < 50 )
- {
return VLC_EGENERIC;
- }
+
ASF_GetGUID( &p_data->i_file_id, p_peek + 24 );
p_data->i_total_data_packets = GetQWLE( p_peek + 40 );
p_data->i_reserved = GetWLE( p_peek + 48 );
#ifdef ASF_DEBUG
msg_Dbg( s,
"read \"data object\" file_id:" GUID_FMT " total data packet:"
- I64Fd" reserved:%d",
+ "%"PRId64" reserved:%d",
GUID_PRINT( p_data->i_file_id ),
p_data->i_total_data_packets,
p_data->i_reserved );
static int ASF_ReadObject_Index( stream_t *s, asf_object_t *p_obj )
{
- asf_object_index_t *p_index = (asf_object_index_t*)p_obj;
+ asf_object_index_t *p_index = &p_obj->index;
const uint8_t *p_peek;
- int i;
+ unsigned int i;
+ /* We just ignore error on the index */
if( stream_Peek( s, &p_peek, p_index->i_object_size ) <
- (int)p_index->i_object_size )
- {
- /* Just ignore */
+ __MAX( (int64_t)p_index->i_object_size, 56 ) )
return VLC_SUCCESS;
- }
ASF_GetGUID( &p_index->i_file_id, p_peek + 24 );
p_index->i_index_entry_time_interval = GetQWLE( p_peek + 40 );
#ifdef ASF_DEBUG
msg_Dbg( s,
"read \"index object\" file_id:" GUID_FMT
- " index_entry_time_interval:"I64Fd" max_packet_count:%d "
+ " index_entry_time_interval:%"PRId64" max_packet_count:%d "
"index_entry_count:%ld",
GUID_PRINT( p_index->i_file_id ),
p_index->i_index_entry_time_interval,
if( p_index->i_index_entry_count > (p_index->i_object_size - 56) / 6 )
p_index->i_index_entry_count = (p_index->i_object_size - 56) / 6;
- p_index->index_entry = malloc( sizeof(asf_index_entry_t) *
- p_index->i_index_entry_count );
+ p_index->index_entry = calloc( p_index->i_index_entry_count,
+ sizeof(asf_index_entry_t) );
+ if( !p_index->index_entry )
+ return VLC_ENOMEM;
- for( i = 0, p_peek += 56; i < (int)p_index->i_index_entry_count;
- i++, p_peek += 6 )
+ for( i = 0, p_peek += 56; i < p_index->i_index_entry_count; i++, p_peek += 6 )
{
p_index->index_entry[i].i_packet_number = GetDWLE( p_peek );
p_index->index_entry[i].i_packet_count = GetDWLE( p_peek + 4 );
static void ASF_FreeObject_Index( asf_object_t *p_obj )
{
- asf_object_index_t *p_index = (asf_object_index_t*)p_obj;
+ asf_object_index_t *p_index = &p_obj->index;
FREENULL( p_index->index_entry );
}
static int ASF_ReadObject_file_properties( stream_t *s, asf_object_t *p_obj )
{
- asf_object_file_properties_t *p_fp = (asf_object_file_properties_t*)p_obj;
+ asf_object_file_properties_t *p_fp = &p_obj->file_properties;
int i_peek;
const uint8_t *p_peek;
- if( ( i_peek = stream_Peek( s, &p_peek, 104) ) < 104 )
- {
+ if( ( i_peek = stream_Peek( s, &p_peek, 104 ) ) < 104 )
return VLC_EGENERIC;
- }
+
ASF_GetGUID( &p_fp->i_file_id, p_peek + 24 );
p_fp->i_file_size = GetQWLE( p_peek + 40 );
p_fp->i_creation_date = GetQWLE( p_peek + 48 );
#ifdef ASF_DEBUG
msg_Dbg( s,
"read \"file properties object\" file_id:" GUID_FMT
- " file_size:"I64Fd" creation_date:"I64Fd" data_packets_count:"
- I64Fd" play_duration:"I64Fd" send_duration:"I64Fd" preroll:"
- I64Fd" flags:%d min_data_packet_size:%d max_data_packet_size:%d "
- "max_bitrate:%d",
+ " file_size:%"PRId64" creation_date:%"PRId64" data_packets_count:"
+ "%"PRId64" play_duration:%"PRId64" send_duration:%"PRId64" preroll:%"PRId64
+ " flags:%d min_data_packet_size:%d "
+ " max_data_packet_size:%d max_bitrate:%d",
GUID_PRINT( p_fp->i_file_id ), p_fp->i_file_size,
p_fp->i_creation_date, p_fp->i_data_packets_count,
p_fp->i_play_duration, p_fp->i_send_duration,
static void ASF_FreeObject_metadata( asf_object_t *p_obj )
{
- asf_object_metadata_t *p_meta =
- (asf_object_metadata_t *)p_obj;
+ asf_object_metadata_t *p_meta = &p_obj->metadata;
unsigned int i;
for( i = 0; i < p_meta->i_record_entries_count; i++ )
{
- if( p_meta->record[i].psz_name ) free( p_meta->record[i].psz_name );
- if( p_meta->record[i].p_data ) free( p_meta->record[i].p_data );
+ free( p_meta->record[i].psz_name );
+ free( p_meta->record[i].p_data );
}
- if( p_meta->record ) free( p_meta->record );
+ free( p_meta->record );
}
static int ASF_ReadObject_metadata( stream_t *s, asf_object_t *p_obj )
{
- asf_object_metadata_t *p_meta =
- (asf_object_metadata_t *)p_obj;
+ asf_object_metadata_t *p_meta = &p_obj->metadata;
- int i_peek, i_entries, i;
- const uint8_t *p_peek;
+ int i_peek;
+ unsigned int i;
+ const uint8_t *p_peek, *p_data;
#ifdef ASF_DEBUG
unsigned int j;
#endif
- p_meta->i_record_entries_count = 0;
- p_meta->record = 0;
-
- if( stream_Peek( s, &p_peek, p_meta->i_object_size ) <
- (int)p_meta->i_object_size )
- {
+ if( ( i_peek = stream_Peek( s, &p_peek, p_meta->i_object_size ) ) <
+ __MAX( (int64_t)p_meta->i_object_size, 26 ) )
return VLC_EGENERIC;
- }
- i_peek = 24;
- i_entries = GetWLE( p_peek + i_peek ); i_peek += 2;
- for( i = 0; i < i_entries && i_peek < (int)p_meta->i_object_size -12; i++ )
+ p_meta->i_record_entries_count = GetWLE( p_peek + 24 );
+
+ p_data = p_peek + 26;
+
+ p_meta->record = calloc( p_meta->i_record_entries_count,
+ sizeof(asf_metadata_record_t) );
+ if( !p_meta->record )
+ return VLC_ENOMEM;
+
+ for( i = 0; i < p_meta->i_record_entries_count; i++ )
{
- asf_metadata_record_t record;
- int i_name, i_data, j;
+ asf_metadata_record_t *p_record = &p_meta->record[i];
+ int i_name;
+ int i_data;
- if( GetWLE( p_peek + i_peek ) != 0 )
- {
- ASF_FreeObject_metadata( p_obj );
- return VLC_EGENERIC;
- }
+ if( !ASF_HAVE( 2+2+2+2+4 ) )
+ break;
- i_peek += 2;
- record.i_stream = GetWLE( p_peek + i_peek ); i_peek += 2;
- i_name = GetWLE( p_peek + i_peek ); i_peek += 2;
- record.i_type = GetWLE( p_peek + i_peek ); i_peek += 2;
- i_data = GetDWLE( p_peek + i_peek ); i_peek += 4;
+ if( ASF_READ2() != 0 )
+ break;
- if( record.i_type > ASF_METADATA_TYPE_WORD ||
- i_peek + i_data + i_name > (int)p_meta->i_object_size )
- {
- ASF_FreeObject_metadata( p_obj );
- return VLC_EGENERIC;
- }
+ p_record->i_stream = ASF_READ2();
+ i_name = ASF_READ2();
+ p_record->i_type = ASF_READ2();
+ i_data = ASF_READ4();
+
+ if( !ASF_HAVE( i_name + i_data ) )
+ break;
- record.i_val = 0;
- record.i_data = 0;
- record.p_data = 0;
+ /* Read name */
+ p_record->psz_name = ASF_READS( i_name );
- /* get name */
- record.psz_name = malloc( i_name/2 + 1 );
- for( j = 0; j < i_name/2; j++ )
+ /* Read data */
+ if( p_record->i_type == ASF_METADATA_TYPE_STRING )
{
- record.psz_name[j] = GetWLE( p_peek + i_peek ); i_peek += 2;
+ p_record->p_data = ASF_READS( i_data );
+ p_record->i_data = i_data/2; /* FIXME Is that needed ? */
}
- record.psz_name[j] = 0; /* just to make sure */
+ else if( p_record->i_type == ASF_METADATA_TYPE_BYTE )
+ {
+ p_record->p_data = malloc( i_data );
+ p_record->i_data = i_data;
+ if( p_record->p_data && i_data > 0 )
+ memcpy( p_record->p_data, p_data, i_data );
- /* get data */
- if( record.i_type == ASF_METADATA_TYPE_STRING )
+ p_data += i_data;
+ }
+ else if( p_record->i_type == ASF_METADATA_TYPE_QWORD )
{
- record.p_data = malloc( i_data/2 + 1 );
- record.i_data = i_data/2;
- for( j = 0; j < i_data/2; j++ )
- {
- record.p_data[j] = GetWLE( p_peek + i_peek ); i_peek += 2;
- }
- record.p_data[j] = 0; /* just to make sure */
+ p_record->i_val = ASF_READ8();
+ }
+ else if( p_record->i_type == ASF_METADATA_TYPE_DWORD )
+ {
+ p_record->i_val = ASF_READ4();
+ }
+ else if( p_record->i_type == ASF_METADATA_TYPE_WORD )
+ {
+ p_record->i_val = ASF_READ2();
}
- else if( record.i_type == ASF_METADATA_TYPE_BYTE )
+ else if( p_record->i_type == ASF_METADATA_TYPE_BOOL )
{
- record.p_data = malloc( i_data );
- record.i_data = i_data;
- memcpy( record.p_data, p_peek + i_peek, i_data );
- p_peek += i_data;
+ p_record->i_val = ASF_READ2();
}
else
{
- if( record.i_type == ASF_METADATA_TYPE_QWORD )
- {
- record.i_val = GetQWLE( p_peek + i_peek ); i_peek += 8;
- }
- else if( record.i_type == ASF_METADATA_TYPE_DWORD )
- {
- record.i_val = GetDWLE( p_peek + i_peek ); i_peek += 4;
- }
- else
- {
- record.i_val = GetWLE( p_peek + i_peek ); i_peek += 2;
- }
+ /* Unknown */
+ p_data += i_data;
}
-
- p_meta->i_record_entries_count++;
- p_meta->record =
- realloc( p_meta->record, p_meta->i_record_entries_count *
- sizeof(asf_metadata_record_t) );
- memcpy( &p_meta->record[p_meta->i_record_entries_count-1],
- &record, sizeof(asf_metadata_record_t) );
}
+ p_meta->i_record_entries_count = i;
#ifdef ASF_DEBUG
msg_Dbg( s,
msg_Dbg( s, " - %s (%i bytes)",
p_rec->psz_name, p_rec->i_data );
else
- msg_Dbg( s, " - %s="I64Fd,
+ msg_Dbg( s, " - %s=%"PRId64,
p_rec->psz_name, p_rec->i_val );
}
#endif
static int ASF_ReadObject_header_extension( stream_t *s, asf_object_t *p_obj )
{
- asf_object_header_extension_t *p_he =
- (asf_object_header_extension_t *)p_obj;
+ asf_object_header_extension_t *p_he = &p_obj->header_extension;
int i_peek;
const uint8_t *p_peek;
p_he->i_header_extension_size = GetDWLE( p_peek + 42 );
if( p_he->i_header_extension_size )
{
+ if( (unsigned int)(i_peek-46) < p_he->i_header_extension_size )
+ return VLC_EGENERIC;
+
p_he->p_header_extension_data =
malloc( p_he->i_header_extension_size );
+ if( !p_he->p_header_extension_data )
+ return VLC_ENOMEM;
+
memcpy( p_he->p_header_extension_data, p_peek + 46,
p_he->i_header_extension_size );
}
{
asf_object_t *p_obj = malloc( sizeof( asf_object_t ) );
- if( ASF_ReadObject( s, p_obj, (asf_object_t*)p_he ) )
+ if( !p_obj || ASF_ReadObject( s, p_obj, (asf_object_t*)p_he ) )
{
free( p_obj );
break;
static void ASF_FreeObject_header_extension( asf_object_t *p_obj )
{
- asf_object_header_extension_t *p_he =
- (asf_object_header_extension_t *)p_obj;
+ asf_object_header_extension_t *p_he = &p_obj->header_extension;
FREENULL( p_he->p_header_extension_data );
}
static int ASF_ReadObject_stream_properties( stream_t *s, asf_object_t *p_obj )
{
- asf_object_stream_properties_t *p_sp =
- (asf_object_stream_properties_t*)p_obj;
+ asf_object_stream_properties_t *p_sp = &p_obj->stream_properties;
size_t i_peek;
const uint8_t *p_peek;
if( ( i_peek = stream_Peek( s, &p_peek, p_sp->i_object_size ) ) < 78 )
- {
return VLC_EGENERIC;
- }
ASF_GetGUID( &p_sp->i_stream_type, p_peek + 24 );
ASF_GetGUID( &p_sp->i_error_correction_type, p_peek + 40 );
p_sp->p_type_specific_data =
malloc( p_sp->i_type_specific_data_length );
- if( p_sp->p_type_specific_data == NULL )
+ if( !p_sp->p_type_specific_data )
return VLC_ENOMEM;
memcpy( p_sp->p_type_specific_data, p_peek + 78,
p_sp->i_type_specific_data_length );
i_peek -= p_sp->i_type_specific_data_length;
}
- else
- {
- p_sp->p_type_specific_data = NULL;
- }
if( p_sp->i_error_correction_data_length )
{
p_sp->p_error_correction_data =
malloc( p_sp->i_error_correction_data_length );
- if( p_sp->p_error_correction_data == NULL )
+ if( !p_sp->p_error_correction_data )
{
free( p_sp->p_type_specific_data );
return VLC_ENOMEM;
p_peek + 78 + p_sp->i_type_specific_data_length,
p_sp->i_error_correction_data_length );
}
- else
- {
- p_sp->p_error_correction_data = NULL;
- }
#ifdef ASF_DEBUG
msg_Dbg( s,
"read \"stream Properties object\" stream_type:" GUID_FMT
- " error_correction_type:" GUID_FMT " time_offset:"I64Fd
+ " error_correction_type:" GUID_FMT " time_offset:%"PRId64
" type_specific_data_length:%d error_correction_data_length:%d"
" flags:0x%x stream_number:%d",
GUID_PRINT( p_sp->i_stream_type ),
static void ASF_FreeObject_stream_properties( asf_object_t *p_obj )
{
- asf_object_stream_properties_t *p_sp =
- (asf_object_stream_properties_t*)p_obj;
+ asf_object_stream_properties_t *p_sp = &p_obj->stream_properties;
FREENULL( p_sp->p_type_specific_data );
FREENULL( p_sp->p_error_correction_data );
static int ASF_ReadObject_codec_list( stream_t *s, asf_object_t *p_obj )
{
- asf_object_codec_list_t *p_cl = (asf_object_codec_list_t*)p_obj;
+ asf_object_codec_list_t *p_cl = &p_obj->codec_list;
int i_peek;
const uint8_t *p_peek, *p_data;
unsigned int i_codec;
if( ( i_peek = stream_Peek( s, &p_peek, p_cl->i_object_size ) ) < 44 )
- {
return VLC_EGENERIC;
- }
ASF_GetGUID( &p_cl->i_reserved, p_peek + 24 );
p_cl->i_codec_entries_count = GetWLE( p_peek + 40 );
+
+ p_data = p_peek + 44;
+
if( p_cl->i_codec_entries_count > 0 )
{
p_cl->codec = calloc( p_cl->i_codec_entries_count,
sizeof( asf_codec_entry_t ) );
- memset( p_cl->codec, 0,
- p_cl->i_codec_entries_count * sizeof( asf_codec_entry_t ) );
+ if( !p_cl->codec )
+ return VLC_ENOMEM;
- p_data = p_peek + 44;
for( i_codec = 0; i_codec < p_cl->i_codec_entries_count; i_codec++ )
{
-#define codec p_cl->codec[i_codec]
- int i_len, i;
+ asf_codec_entry_t *p_codec = &p_cl->codec[i_codec];
+
+ if( !ASF_HAVE( 2+2+2 ) )
+ break;
+
+ /* */
+ p_codec->i_type = ASF_READ2();
+
+ /* XXX the length here are the number of *unicode* characters and
+ * not of bytes like nearly every elsewhere */
- codec.i_type = GetWLE( p_data ); p_data += 2;
/* codec name */
- i_len = GetWLE( p_data ); p_data += 2;
- codec.psz_name = calloc( sizeof( char ), i_len + 1);
- for( i = 0; i < i_len; i++ )
- {
- codec.psz_name[i] = GetWLE( p_data + 2*i );
- }
- codec.psz_name[i_len] = '\0';
- p_data += 2 * i_len;
+ p_codec->psz_name = ASF_READS( 2*ASF_READ2() );
/* description */
- i_len = GetWLE( p_data ); p_data += 2;
- codec.psz_description = calloc( sizeof( char ), i_len + 1);
- for( i = 0; i < i_len; i++ )
- {
- codec.psz_description[i] = GetWLE( p_data + 2*i );
- }
- codec.psz_description[i_len] = '\0';
- p_data += 2 * i_len;
+ p_codec->psz_description = ASF_READS( 2*ASF_READ2() );
/* opaque information */
- codec.i_information_length = GetWLE( p_data ); p_data += 2;
- if( codec.i_information_length > 0 )
+ p_codec->i_information_length = ASF_READ2();
+ if( p_codec->i_information_length > 0 && ASF_HAVE( p_codec->i_information_length ) )
{
- codec.p_information = malloc( codec.i_information_length );
- memcpy( codec.p_information, p_data, codec.i_information_length );
- p_data += codec.i_information_length;
+ p_codec->p_information = malloc( p_codec->i_information_length );
+ if( p_codec->p_information )
+ memcpy( p_codec->p_information, p_data, p_codec->i_information_length );
+ else
+ p_codec->i_information_length = 0;
+ p_data += p_codec->i_information_length;
}
- else
- {
- codec.p_information = NULL;
- }
-#undef codec
}
- }
- else
- {
- p_cl->codec = NULL;
+ p_cl->i_codec_entries_count = i_codec;
}
#ifdef ASF_DEBUG
for( i_codec = 0; i_codec < p_cl->i_codec_entries_count; i_codec++ )
{
-#define codec p_cl->codec[i_codec]
+ const asf_codec_entry_t *p_codec = &p_cl->codec[i_codec];
+
msg_Dbg( s, " - codec[%d] %s name:\"%s\" "
"description:\"%s\" information_length:%d",
- i_codec, ( codec.i_type == ASF_CODEC_TYPE_VIDEO ) ?
- "video" : ( ( codec.i_type == ASF_CODEC_TYPE_AUDIO ) ?
+ i_codec, ( p_codec->i_type == ASF_CODEC_TYPE_VIDEO ) ?
+ "video" : ( ( p_codec->i_type == ASF_CODEC_TYPE_AUDIO ) ?
"audio" : "unknown" ),
- codec.psz_name, codec.psz_description,
- codec.i_information_length );
-#undef codec
+ p_codec->psz_name, p_codec->psz_description,
+ p_codec->i_information_length );
}
#endif
static void ASF_FreeObject_codec_list( asf_object_t *p_obj )
{
- asf_object_codec_list_t *p_cl = (asf_object_codec_list_t*)p_obj;
+ asf_object_codec_list_t *p_cl = &p_obj->codec_list;
unsigned int i_codec;
for( i_codec = 0; i_codec < p_cl->i_codec_entries_count; i_codec++ )
{
-#define codec p_cl->codec[i_codec]
- FREENULL( codec.psz_name );
- FREENULL( codec.psz_description );
- FREENULL( codec.p_information );
-#undef codec
+ asf_codec_entry_t *p_codec = &p_cl->codec[i_codec];
+
+ FREENULL( p_codec->psz_name );
+ FREENULL( p_codec->psz_description );
+ FREENULL( p_codec->p_information );
}
FREENULL( p_cl->codec );
}
* and for the some others object, length give char16 count ... */
static int ASF_ReadObject_content_description(stream_t *s, asf_object_t *p_obj)
{
- asf_object_content_description_t *p_cd =
- (asf_object_content_description_t *)p_obj;
+ asf_object_content_description_t *p_cd = &p_obj->content_description;
const uint8_t *p_peek, *p_data;
int i_peek, i_title, i_artist, i_copyright, i_description, i_rating;
vlc_iconv_t cd = (vlc_iconv_t)-1;
char *ob = NULL;
size_t i_ibl, i_obl, i_len;
+ if( ( i_peek = stream_Peek( s, &p_peek, p_cd->i_object_size ) ) < 34 )
+ return VLC_EGENERIC;
+
cd = vlc_iconv_open("UTF-8", "UTF-16LE");
- if ( cd == (vlc_iconv_t)-1 ) {
+ if( cd == (vlc_iconv_t)-1 )
+ {
msg_Err( s, "vlc_iconv_open failed" );
return VLC_EGENERIC;
}
/* FIXME i_size*3 is the worst case. */
-#define GETSTRINGW( psz_str, i_size ) \
- psz_str = (char *)calloc( i_size*3+1, sizeof( char ) ); \
- ib = (const char *)p_data; \
- ob = psz_str; \
- i_ibl = i_size; \
- i_obl = i_size*3; \
- i_len = vlc_iconv(cd, &ib, &i_ibl, &ob, &i_obl); \
- p_data += i_size;
+#define GETSTRINGW( psz_str, i_size ) do { \
+ psz_str = calloc( i_size*3+1, sizeof(char) ); \
+ if( psz_str ) { \
+ ib = (const char *)p_data; \
+ ob = psz_str; \
+ i_ibl = i_size; \
+ i_obl = i_size*3; \
+ i_len = vlc_iconv(cd, &ib, &i_ibl, &ob, &i_obl); \
+ p_data += i_size; \
+ } } while(0)
- if( ( i_peek = stream_Peek( s, &p_peek, p_cd->i_object_size ) ) < 34 )
+ p_data = p_peek + 24;
+
+ i_title = ASF_READ2();
+ i_artist = ASF_READ2();
+ i_copyright = ASF_READ2();
+ i_description = ASF_READ2();
+ i_rating = ASF_READ2();
+
+ if( !ASF_HAVE( i_title+i_artist+i_copyright+i_description+i_rating ) )
{
- return VLC_EGENERIC;
+ vlc_iconv_close( cd );
+ return VLC_EGENERIC;
}
- p_data = p_peek + 24;
-
- i_title = GetWLE( p_data ); p_data += 2;
- i_artist= GetWLE( p_data ); p_data += 2;
- i_copyright = GetWLE( p_data ); p_data += 2;
- i_description = GetWLE( p_data ); p_data += 2;
- i_rating = GetWLE( p_data ); p_data += 2;
GETSTRINGW( p_cd->psz_title, i_title );
GETSTRINGW( p_cd->psz_artist, i_artist );
static void ASF_FreeObject_content_description( asf_object_t *p_obj)
{
- asf_object_content_description_t *p_cd =
- (asf_object_content_description_t *)p_obj;
+ asf_object_content_description_t *p_cd = &p_obj->content_description;
FREENULL( p_cd->psz_title );
FREENULL( p_cd->psz_artist );
/* Language list: */
static int ASF_ReadObject_language_list(stream_t *s, asf_object_t *p_obj)
{
- asf_object_language_list_t *p_ll =
- (asf_object_language_list_t*)p_obj;
+ asf_object_language_list_t *p_ll = &p_obj->language_list;
const uint8_t *p_peek, *p_data;
int i_peek;
int i;
p_data = &p_peek[24];
- p_ll->i_language = GetWLE( &p_data[0] ); p_data += 2;
+ p_ll->i_language = ASF_READ2();
if( p_ll->i_language > 0 )
{
p_ll->ppsz_language = calloc( p_ll->i_language, sizeof( char *) );
+ if( !p_ll->ppsz_language )
+ return VLC_ENOMEM;
for( i = 0; i < p_ll->i_language; i++ )
{
- char *psz;
- int i_size = *p_data++;
- int i_len;
-
- psz = calloc( i_size/2 + 1, sizeof( char ) );
- for( i_len = 0; i_len < i_size/2; i_len++ )
- {
- psz[i_len] = GetWLE( p_data + 2*i_len );
- }
- psz[i_size/2] = '\0'; \
- p_data += i_size;
-
- p_ll->ppsz_language[i] = psz;
+ if( !ASF_HAVE(1) )
+ break;
+ p_ll->ppsz_language[i] = ASF_READS( ASF_READ1() );
}
+ p_ll->i_language = i;
}
#ifdef ASF_DEBUG
- msg_Dbg( s, "read \"language list object\" %d entries",
+ msg_Dbg( s, "read \"language list object\" %d entries",
p_ll->i_language );
for( i = 0; i < p_ll->i_language; i++ )
- msg_Dbg( s, " - '%s'",
+ msg_Dbg( s, " - '%s'",
p_ll->ppsz_language[i] );
#endif
return VLC_SUCCESS;
static void ASF_FreeObject_language_list( asf_object_t *p_obj)
{
- asf_object_language_list_t *p_ll =
- (asf_object_language_list_t *)p_obj;
+ asf_object_language_list_t *p_ll = &p_obj->language_list;
int i;
for( i = 0; i < p_ll->i_language; i++ )
static int ASF_ReadObject_stream_bitrate_properties( stream_t *s,
asf_object_t *p_obj)
{
- asf_object_stream_bitrate_properties_t *p_sb =
- (asf_object_stream_bitrate_properties_t *)p_obj;
+ asf_object_stream_bitrate_properties_t *p_sb = &p_obj->stream_bitrate;
const uint8_t *p_peek, *p_data;
int i_peek;
int i;
p_data = &p_peek[24];
- p_sb->i_bitrate = GetWLE( &p_data[0] ); p_data += 2;
- if( p_sb->i_bitrate > 127 ) p_sb->i_bitrate = 127; /* Buggy ? */
+ p_sb->i_bitrate = ASF_READ2();
+ if( p_sb->i_bitrate > 127 )
+ p_sb->i_bitrate = 127; /* Buggy ? */
for( i = 0; i < p_sb->i_bitrate; i++ )
{
- p_sb->bitrate[i].i_stream_number = GetWLE( &p_data[0] )& 0x7f;
- p_sb->bitrate[i].i_avg_bitrate = GetDWLE( &p_data[2] );
-
- p_data += 2+4;
+ if( !ASF_HAVE(2 + 4) )
+ break;
+ p_sb->bitrate[i].i_stream_number = ASF_READ2()& 0x7f;
+ p_sb->bitrate[i].i_avg_bitrate = ASF_READ4();
}
+ p_sb->i_bitrate = i;
#ifdef ASF_DEBUG
msg_Dbg( s,"read \"stream bitrate properties object\"" );
{
msg_Dbg( s," - stream=%d bitrate=%d",
p_sb->bitrate[i].i_stream_number,
- p_sb->bitrate[i].i_avg_bitrate );
+ p_sb->bitrate[i].i_avg_bitrate );
}
#endif
return VLC_SUCCESS;
}
static void ASF_FreeObject_stream_bitrate_properties( asf_object_t *p_obj)
{
+ VLC_UNUSED(p_obj);
}
static int ASF_ReadObject_extended_stream_properties( stream_t *s,
asf_object_t *p_obj)
{
- asf_object_extended_stream_properties_t *p_esp =
- (asf_object_extended_stream_properties_t*)p_obj;
+ asf_object_extended_stream_properties_t *p_esp = &p_obj->ext_stream;
const uint8_t *p_peek, *p_data;
int i_peek, i;
p_data += 64;
- p_esp->pi_stream_name_language = calloc( sizeof(int),
- p_esp->i_stream_name_count );
- p_esp->ppsz_stream_name = calloc( sizeof(char*),
- p_esp->i_stream_name_count );
+ p_esp->pi_stream_name_language = calloc( p_esp->i_stream_name_count,
+ sizeof(int) );
+ p_esp->ppsz_stream_name = calloc( p_esp->i_stream_name_count,
+ sizeof(char*) );
+ if( !p_esp->pi_stream_name_language ||
+ !p_esp->ppsz_stream_name )
+ {
+ free( p_esp->pi_stream_name_language );
+ free( p_esp->ppsz_stream_name );
+ return VLC_ENOMEM;
+ }
for( i = 0; i < p_esp->i_stream_name_count; i++ )
{
- int i_size;
- char *psz;
- int i_len;
-
- p_esp->pi_stream_name_language[i] = GetWLE( &p_data[0] );
- i_size = GetWLE( &p_data[2] );
- p_data += 2;
-
- psz = calloc( i_size/2 + 1, sizeof( char ) );
- for( i_len = 0; i_len < i_size/2; i_len++ )
- {
- psz[i_len] = GetWLE( p_data + 2*i_len );
- }
- psz[i_size/2] = '\0'; \
- p_data += i_size;
-
- p_esp->ppsz_stream_name[i] = psz;
+ if( !ASF_HAVE( 2+2 ) )
+ break;
+ p_esp->pi_stream_name_language[i] = ASF_READ2();
+ p_esp->ppsz_stream_name[i] = ASF_READS( ASF_READ2() );
}
+ p_esp->i_stream_name_count = i;
for( i = 0; i < p_esp->i_payload_extension_system_count; i++ )
{
- /* Skip them */
- int i_size = GetDWLE( &p_data[16 + 2] );
-
- p_data += 16+2+4+i_size;
+ ASF_SKIP( 16 ); // GUID
+ ASF_SKIP( 2 );
+ ASF_SKIP( ASF_READ4() );
}
p_esp->p_sp = NULL;
asf_object_t *p_sp;
/* Cannot fail as peek succeed */
stream_Read( s, NULL, p_data - p_peek );
-
+
p_sp = malloc( sizeof( asf_object_t ) );
- if( ASF_ReadObject( s, p_sp, NULL ) )
+ if( !p_sp || ASF_ReadObject( s, p_sp, NULL ) )
{
free( p_sp );
}
#ifdef ASF_DEBUG
msg_Dbg( s, "read \"extended stream properties object\":" );
- msg_Dbg( s, " - start="I64Fd" end="I64Fd,
+ msg_Dbg( s, " - start=%"PRId64" end=%"PRId64,
p_esp->i_start_time, p_esp->i_end_time );
msg_Dbg( s, " - data bitrate=%d buffer=%d initial fullness=%d",
p_esp->i_data_bitrate,
msg_Dbg( s, " - flags=0x%x", p_esp->i_flags );
msg_Dbg( s, " - stream number=%d language=%d",
p_esp->i_stream_number, p_esp->i_language_index );
- msg_Dbg( s, " - average time per frame="I64Fd,
+ msg_Dbg( s, " - average time per frame=%"PRId64,
p_esp->i_average_time_per_frame );
msg_Dbg( s, " - stream name count=%d", p_esp->i_stream_name_count );
for( i = 0; i < p_esp->i_stream_name_count; i++ )
}
static void ASF_FreeObject_extended_stream_properties( asf_object_t *p_obj)
{
- asf_object_extended_stream_properties_t *p_esp =
- (asf_object_extended_stream_properties_t *)p_obj;
+ asf_object_extended_stream_properties_t *p_esp = &p_obj->ext_stream;
int i;
for( i = 0; i < p_esp->i_stream_name_count; i++ )
static int ASF_ReadObject_advanced_mutual_exclusion( stream_t *s,
asf_object_t *p_obj)
{
- asf_object_advanced_mutual_exclusion_t *p_ae =
- (asf_object_advanced_mutual_exclusion_t *)p_obj;
+ asf_object_advanced_mutual_exclusion_t *p_ae = &p_obj->advanced_mutual_exclusion;
const uint8_t *p_peek, *p_data;
int i_peek;
int i;
p_data = &p_peek[24];
ASF_GetGUID( &p_ae->type, &p_data[0] );
- p_ae->i_stream_number_count = GetWLE( &p_data[16] );
+ ASF_SKIP( 16 );
+ p_ae->i_stream_number_count = ASF_READ2();
+ p_ae->pi_stream_number = calloc( p_ae->i_stream_number_count, sizeof(int) );
- p_data += 16 + 2;
- p_ae->pi_stream_number = calloc( sizeof(int),
- p_ae->i_stream_number_count );
for( i = 0; i < p_ae->i_stream_number_count; i++ )
{
- p_ae->pi_stream_number[i] = GetWLE( p_data );
- p_data += 2;
+ if( !ASF_HAVE(2) )
+ break;
+ p_ae->pi_stream_number[i] = ASF_READ2();
}
-
+ p_ae->i_stream_number_count = i;
+
#ifdef ASF_DEBUG
msg_Dbg( s, "read \"advanced mutual exclusion object\"" );
for( i = 0; i < p_ae->i_stream_number_count; i++ )
}
static void ASF_FreeObject_advanced_mutual_exclusion( asf_object_t *p_obj)
{
- asf_object_advanced_mutual_exclusion_t *p_ae =
- (asf_object_advanced_mutual_exclusion_t *)p_obj;
+ asf_object_advanced_mutual_exclusion_t *p_ae = &p_obj->advanced_mutual_exclusion;
FREENULL( p_ae->pi_stream_number );
}
static int ASF_ReadObject_stream_prioritization( stream_t *s,
asf_object_t *p_obj)
{
- asf_object_stream_prioritization_t *p_sp =
- (asf_object_stream_prioritization_t *)p_obj;
+ asf_object_stream_prioritization_t *p_sp = &p_obj->stream_prioritization;
const uint8_t *p_peek, *p_data;
int i_peek;
int i;
p_data = &p_peek[24];
- p_sp->i_priority_count = GetWLE( &p_data[0] );
- p_data += 2;
+ p_sp->i_priority_count = ASF_READ2();
- p_sp->pi_priority_flag = calloc( sizeof(int), p_sp->i_priority_count );
+ p_sp->pi_priority_flag = calloc( p_sp->i_priority_count, sizeof(int) );
p_sp->pi_priority_stream_number =
- calloc( sizeof(int), p_sp->i_priority_count );
+ calloc( p_sp->i_priority_count, sizeof(int) );
+
+ if( !p_sp->pi_priority_flag || !p_sp->pi_priority_stream_number )
+ {
+ free( p_sp->pi_priority_flag );
+ free( p_sp->pi_priority_stream_number );
+ return VLC_ENOMEM;
+ }
for( i = 0; i < p_sp->i_priority_count; i++ )
{
- p_sp->pi_priority_stream_number[i] = GetWLE( p_data ); p_data += 2;
- p_sp->pi_priority_flag[i] = GetWLE( p_data ); p_data += 2;
+ if( !ASF_HAVE(2+2) )
+ break;
+ p_sp->pi_priority_stream_number[i] = ASF_READ2();
+ p_sp->pi_priority_flag[i] = ASF_READ2();
}
+ p_sp->i_priority_count = i;
+
#ifdef ASF_DEBUG
msg_Dbg( s, "read \"stream prioritization object\"" );
for( i = 0; i < p_sp->i_priority_count; i++ )
}
static void ASF_FreeObject_stream_prioritization( asf_object_t *p_obj)
{
- asf_object_stream_prioritization_t *p_sp =
- (asf_object_stream_prioritization_t *)p_obj;
+ asf_object_stream_prioritization_t *p_sp = &p_obj->stream_prioritization;
FREENULL( p_sp->pi_priority_stream_number );
FREENULL( p_sp->pi_priority_flag );
asf_object_t *p_obj)
{
asf_object_extended_content_description_t *p_ec =
- (asf_object_extended_content_description_t *)p_obj;
+ &p_obj->extended_content_description;
const uint8_t *p_peek, *p_data;
int i_peek;
int i;
p_data = &p_peek[24];
- p_ec->i_count = GetWLE( p_data ); p_data += 2;
- p_ec->ppsz_name = calloc( sizeof(char*), p_ec->i_count );
- p_ec->ppsz_value = calloc( sizeof(char*), p_ec->i_count );
+ p_ec->i_count = ASF_READ2();
+ p_ec->ppsz_name = calloc( p_ec->i_count, sizeof(char*) );
+ p_ec->ppsz_value = calloc( p_ec->i_count, sizeof(char*) );
+ if( !p_ec->ppsz_name || !p_ec->ppsz_value )
+ {
+ free( p_ec->ppsz_name );
+ free( p_ec->ppsz_value );
+ return VLC_ENOMEM;
+ }
for( i = 0; i < p_ec->i_count; i++ )
{
int i_size;
int i_type;
- int i_len;
-#define GETSTRINGW( psz_str, i_size ) \
- psz_str = calloc( i_size/2 + 1, sizeof( char ) ); \
- for( i_len = 0; i_len < i_size/2; i_len++ ) \
- { \
- psz_str[i_len] = GetWLE( p_data + 2*i_len ); \
- } \
- psz_str[i_size/2] = '\0';
-
- i_size = GetWLE( p_data ); p_data += 2;
- GETSTRINGW( p_ec->ppsz_name[i], i_size );
- p_data += i_size;
+
+ if( !ASF_HAVE(2 + 2+2) )
+ break;
+
+ p_ec->ppsz_name[i] = ASF_READS( ASF_READ2() );
/* Grrr */
- i_type = GetWLE( p_data ); p_data += 2;
- i_size = GetWLE( p_data ); p_data += 2;
+ i_type = ASF_READ2();
+ i_size = ASF_READ2();
+
if( i_type == 0 )
{
- GETSTRINGW( p_ec->ppsz_value[i], i_size );
+ p_ec->ppsz_value[i] = ASF_READS( i_size );
}
else if( i_type == 1 )
{
- int j;
/* Byte array */
+ static const char hex[16] = "0123456789ABCDEF";
+ int j;
+
p_ec->ppsz_value[i] = malloc( 2*i_size + 1 );
- for( j = 0; j < i_size; j++ )
+ if( p_ec->ppsz_value[i] )
{
- static const char hex[16] = "0123456789ABCDEF";
- p_ec->ppsz_value[i][2*j+0] = hex[p_data[0]>>4];
- p_ec->ppsz_value[i][2*j+1] = hex[p_data[0]&0xf];
+ char *psz_value = p_ec->ppsz_value[i];
+ for( j = 0; j < i_size; j++ )
+ {
+ const uint8_t v = ASF_READ1();
+ psz_value[2*j+0] = hex[v>>4];
+ psz_value[2*j+1] = hex[v&0xf];
+ }
+ psz_value[2*i_size] = '\0';
}
- p_ec->ppsz_value[i][2*i_size] = '\0';
}
else if( i_type == 2 )
{
/* Bool */
- p_ec->ppsz_value[i] = strdup( *p_data ? "true" : "false" );
+ p_ec->ppsz_value[i] = strdup( ASF_READ1() ? "true" : "false" );
+ ASF_SKIP(i_size-1);
}
else if( i_type == 3 )
{
/* DWord */
- asprintf( &p_ec->ppsz_value[i], "%d", GetDWLE(p_data));
+ if( asprintf( &p_ec->ppsz_value[i], "%d", ASF_READ4() ) == -1 )
+ p_ec->ppsz_value[i] = NULL;
}
else if( i_type == 4 )
{
/* QWord */
- asprintf( &p_ec->ppsz_value[i], I64Fd, GetQWLE(p_data));
+ if( asprintf( &p_ec->ppsz_value[i], "%"PRId64, ASF_READ8() ) == -1 )
+ p_ec->ppsz_value[i] = NULL;
}
else if( i_type == 5 )
{
/* Word */
- asprintf( &p_ec->ppsz_value[i], "%d", GetWLE(p_data));
+ if( asprintf( &p_ec->ppsz_value[i], "%d", ASF_READ2() ) == -1 )
+ p_ec->ppsz_value[i] = NULL;
}
else
+ {
p_ec->ppsz_value[i] = NULL;
-
- p_data += i_size;
-
-
-
-#undef GETSTRINGW
-
+ ASF_SKIP(i_size);
+ }
}
+ p_ec->i_count = i;
#ifdef ASF_DEBUG
msg_Dbg( s, "read \"extended content description object\"" );
static void ASF_FreeObject_extended_content_description( asf_object_t *p_obj)
{
asf_object_extended_content_description_t *p_ec =
- (asf_object_extended_content_description_t *)p_obj;
+ &p_obj->extended_content_description;
int i;
for( i = 0; i < p_ec->i_count; i++ )
/* */
-static struct
+static const struct
{
const guid_t *p_id;
int i_type;
int i_index;
if( !p_obj )
- return( 0 );
+ return 0;
- memset( p_obj, 0, sizeof( p_obj ) );
+ memset( p_obj, 0, sizeof( *p_obj ) );
if( ASF_ReadObjectCommon( s, p_obj ) )
{
p_father->common.p_last = p_obj;
}
- return( i_result );
+ return i_result;
}
static void ASF_FreeObject( stream_t *s, asf_object_t *p_obj )
int i_index;
asf_object_t *p_child;
- if( !p_obj ) return;
+ if( !p_obj )
+ return;
/* Free all child object */
p_child = p_obj->common.p_first;
(ASF_Object_Function[i_index].ASF_FreeObject_function)( p_obj );
}
free( p_obj );
- return;
}
/*****************************************************************************
{ &asf_object_stream_type_video, "Stream Type Video" },
{ &asf_object_stream_type_command, "Stream Type Command" },
{ &asf_object_language_list, "Language List" },
- { &asf_object_stream_bitrate_properties, "Stream Bitrate Propoerties" },
+ { &asf_object_stream_bitrate_properties, "Stream Bitrate Properties" },
{ &asf_object_padding, "Padding" },
{ &asf_object_extended_stream_properties, "Extended Stream Properties" },
{ &asf_object_advanced_mutual_exclusion, "Advanced Mutual Exclusion" },
str[i * 5] = '|';
}
snprintf( str + 5*i_level, 1024,
- "+ '%s' GUID "GUID_FMT" size:"I64Fu"pos:"I64Fu,
+ "+ '%s' GUID "GUID_FMT" size:%"PRIu64"pos:%"PRIu64,
psz_name,
GUID_PRINT( p_node->i_object_id ),
p_node->i_object_size, p_node->i_object_pos );
asf_object_root_t *p_root = malloc( sizeof( asf_object_root_t ) );
asf_object_t *p_obj;
+ if( !p_root )
+ return NULL;
+
p_root->i_type = ASF_OBJECT_ROOT;
memcpy( &p_root->i_object_id, &asf_object_null_guid, sizeof( guid_t ) );
p_root->i_object_pos = stream_Tell( s );
{
p_obj = malloc( sizeof( asf_object_t ) );
- if( ASF_ReadObject( s, p_obj, (asf_object_t*)p_root ) )
+ if( !p_obj || ASF_ReadObject( s, p_obj, (asf_object_t*)p_root ) )
{
free( p_obj );
break;
}
if( ASF_NextObject( s, p_obj ) ) /* Go to the next object */
- {
break;
- }
}
if( p_root->p_hdr != NULL && p_root->p_data != NULL )
int i_count;
asf_object_t *p_child;
- if( !p_obj ) return( 0 );
+ if( !p_obj )
+ return 0;
i_count = 0;
p_child = p_obj->common.p_first;
while( p_child )
{
if( ASF_CmpGUID( &p_child->common.i_object_id, p_guid ) )
- {
i_count++;
- }
+
p_child = p_child->common.p_next;
}
- return( i_count );
+ return i_count;
}
void *__ASF_FindObject( asf_object_t *p_obj, const guid_t *p_guid,
if( ASF_CmpGUID( &p_child->common.i_object_id, p_guid ) )
{
if( i_number == 0 )
- {
- /* We found it */
- return( p_child );
- }
+ return p_child;
i_number--;
}
p_child = p_child->common.p_next;
}
- return( NULL );
+ return NULL;
}
projects / vlc / blobdiff