Commit 48e4674a authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

Do not assert memory allocations

parent 25232e20
...@@ -25,9 +25,6 @@ ...@@ -25,9 +25,6 @@
#ifndef _LIBVLC_MEDIA_LIST_PATH_H #ifndef _LIBVLC_MEDIA_LIST_PATH_H
#define _LIBVLC_MEDIA_LIST_PATH_H 1 #define _LIBVLC_MEDIA_LIST_PATH_H 1
#include <assert.h>
#include <vlc_memory.h>
typedef int * libvlc_media_list_path_t; /* (Media List Player Internal) */ typedef int * libvlc_media_list_path_t; /* (Media List Player Internal) */
/************************************************************************** /**************************************************************************
...@@ -52,8 +49,7 @@ static inline void libvlc_media_list_path_dump( const libvlc_media_list_path_t p ...@@ -52,8 +49,7 @@ static inline void libvlc_media_list_path_dump( const libvlc_media_list_path_t p
**************************************************************************/ **************************************************************************/
static inline libvlc_media_list_path_t libvlc_media_list_path_empty( void ) static inline libvlc_media_list_path_t libvlc_media_list_path_empty( void )
{ {
libvlc_media_list_path_t ret = malloc(sizeof(int)); libvlc_media_list_path_t ret = xmalloc(sizeof(int));
assert( ret );
ret[0] = -1; ret[0] = -1;
return ret; return ret;
} }
...@@ -63,8 +59,7 @@ static inline libvlc_media_list_path_t libvlc_media_list_path_empty( void ) ...@@ -63,8 +59,7 @@ static inline libvlc_media_list_path_t libvlc_media_list_path_empty( void )
**************************************************************************/ **************************************************************************/
static inline libvlc_media_list_path_t libvlc_media_list_path_with_root_index( int index ) static inline libvlc_media_list_path_t libvlc_media_list_path_with_root_index( int index )
{ {
libvlc_media_list_path_t ret = malloc(sizeof(int)*2); libvlc_media_list_path_t ret = xmalloc(sizeof(int)*2);
assert( ret );
ret[0] = index; ret[0] = index;
ret[1] = -1; ret[1] = -1;
return ret; return ret;
...@@ -86,8 +81,7 @@ static inline int libvlc_media_list_path_depth( const libvlc_media_list_path_t p ...@@ -86,8 +81,7 @@ static inline int libvlc_media_list_path_depth( const libvlc_media_list_path_t p
static inline void libvlc_media_list_path_append( libvlc_media_list_path_t * p_path, int index ) static inline void libvlc_media_list_path_append( libvlc_media_list_path_t * p_path, int index )
{ {
int old_depth = libvlc_media_list_path_depth( *p_path ); int old_depth = libvlc_media_list_path_depth( *p_path );
*p_path = realloc_or_free( *p_path, sizeof(int)*(old_depth+2)); *p_path = xrealloc( *p_path, sizeof(int)*(old_depth+2));
assert( *p_path );
*p_path[old_depth] = index; *p_path[old_depth] = index;
*p_path[old_depth+1] = -1; *p_path[old_depth+1] = -1;
} }
...@@ -99,8 +93,7 @@ static inline libvlc_media_list_path_t libvlc_media_list_path_copy_by_appending( ...@@ -99,8 +93,7 @@ static inline libvlc_media_list_path_t libvlc_media_list_path_copy_by_appending(
{ {
libvlc_media_list_path_t ret; libvlc_media_list_path_t ret;
int old_depth = libvlc_media_list_path_depth( path ); int old_depth = libvlc_media_list_path_depth( path );
ret = malloc( sizeof(int) * (old_depth + 2) ); ret = xmalloc( sizeof(int) * (old_depth + 2) );
assert( ret );
memcpy( ret, path, sizeof(int) * old_depth ); memcpy( ret, path, sizeof(int) * old_depth );
ret[old_depth] = index; ret[old_depth] = index;
ret[old_depth+1] = -1; ret[old_depth+1] = -1;
...@@ -114,8 +107,7 @@ static inline libvlc_media_list_path_t libvlc_media_list_path_copy( const libvlc ...@@ -114,8 +107,7 @@ static inline libvlc_media_list_path_t libvlc_media_list_path_copy( const libvlc
{ {
libvlc_media_list_path_t ret; libvlc_media_list_path_t ret;
int depth = libvlc_media_list_path_depth( path ); int depth = libvlc_media_list_path_depth( path );
ret = malloc( sizeof(int)*(depth+1) ); ret = xmalloc( sizeof(int)*(depth+1) );
assert( ret );
memcpy( ret, path, sizeof(int)*(depth+1) ); memcpy( ret, path, sizeof(int)*(depth+1) );
return ret; return ret;
} }
......
...@@ -30,7 +30,6 @@ ...@@ -30,7 +30,6 @@
#endif #endif
#include <vlc_common.h> #include <vlc_common.h>
#include <vlc_memory.h>
#include <ctype.h> #include <ctype.h>
...@@ -866,8 +865,7 @@ int __vlc_execve( vlc_object_t *p_object, int i_argc, char *const *ppsz_argv, ...@@ -866,8 +865,7 @@ int __vlc_execve( vlc_object_t *p_object, int i_argc, char *const *ppsz_argv,
|| i_read == 0 ) || i_read == 0 )
break; break;
*pi_data += i_read; *pi_data += i_read;
*pp_data = realloc_or_free( *pp_data, *pi_data + 1025 ); *pp_data = xrealloc( *pp_data, *pi_data + 1025 );
assert( *pp_data );
} }
while ( !p_object->b_die while ( !p_object->b_die
......
...@@ -39,8 +39,6 @@ ...@@ -39,8 +39,6 @@
#include <vlc_aout.h> #include <vlc_aout.h>
#include <vlc_fourcc.h> #include <vlc_fourcc.h>
#include <vlc_memory.h>
#include "input_internal.h" #include "input_internal.h"
#include "clock.h" #include "clock.h"
#include "decoder.h" #include "decoder.h"
...@@ -2348,9 +2346,7 @@ static int EsOutControlLocked( es_out_t *out, int i_query, va_list args ) ...@@ -2348,9 +2346,7 @@ static int EsOutControlLocked( es_out_t *out, int i_query, va_list args )
if( p_fmt->i_extra ) if( p_fmt->i_extra )
{ {
es->fmt.i_extra = p_fmt->i_extra; es->fmt.i_extra = p_fmt->i_extra;
es->fmt.p_extra = realloc_or_free( es->fmt.p_extra, es->fmt.p_extra = xrealloc( es->fmt.p_extra, p_fmt->i_extra );
p_fmt->i_extra );
assert( es->fmt.p_extra );
memcpy( es->fmt.p_extra, p_fmt->p_extra, p_fmt->i_extra ); memcpy( es->fmt.p_extra, p_fmt->p_extra, p_fmt->i_extra );
if( !es->p_dec ) if( !es->p_dec )
...@@ -2362,8 +2358,7 @@ static int EsOutControlLocked( es_out_t *out, int i_query, va_list args ) ...@@ -2362,8 +2358,7 @@ static int EsOutControlLocked( es_out_t *out, int i_query, va_list args )
#else #else
es->p_dec->fmt_in.i_extra = p_fmt->i_extra; es->p_dec->fmt_in.i_extra = p_fmt->i_extra;
es->p_dec->fmt_in.p_extra = es->p_dec->fmt_in.p_extra =
realloc_or_free( es->p_dec->fmt_in.p_extra, p_fmt->i_extra ); xrealloc( es->p_dec->fmt_in.p_extra, p_fmt->i_extra );
assert( es->p_dec->fmt_in.p_extra );
memcpy( es->p_dec->fmt_in.p_extra, memcpy( es->p_dec->fmt_in.p_extra,
p_fmt->p_extra, p_fmt->i_extra ); p_fmt->p_extra, p_fmt->i_extra );
#endif #endif
......
...@@ -30,7 +30,6 @@ ...@@ -30,7 +30,6 @@
#endif #endif
#include <vlc_common.h> #include <vlc_common.h>
#include <vlc_memory.h>
#include <ctype.h> #include <ctype.h>
#include <limits.h> #include <limits.h>
...@@ -2926,9 +2925,8 @@ static void AppendAttachment( int *pi_attachment, input_attachment_t ***ppp_atta ...@@ -2926,9 +2925,8 @@ static void AppendAttachment( int *pi_attachment, input_attachment_t ***ppp_atta
input_attachment_t **attachment = *ppp_attachment; input_attachment_t **attachment = *ppp_attachment;
int i; int i;
attachment = realloc_or_free( attachment, attachment = xrealloc( attachment,
sizeof(input_attachment_t**) * ( i_attachment + i_new ) ); sizeof(input_attachment_t**) * ( i_attachment + i_new ) );
assert( attachment );
for( i = 0; i < i_new; i++ ) for( i = 0; i < i_new; i++ )
attachment[i_attachment++] = pp_new[i]; attachment[i_attachment++] = pp_new[i];
free( pp_new ); free( pp_new );
......
...@@ -31,7 +31,6 @@ ...@@ -31,7 +31,6 @@
#endif #endif
#include <vlc_common.h> #include <vlc_common.h>
#include <vlc_memory.h>
#include <stdio.h> #include <stdio.h>
#include <ctype.h> /* tolower() */ #include <ctype.h> /* tolower() */
...@@ -641,9 +640,8 @@ static int ExecuteScheduleProperty( vlm_t *p_vlm, vlm_schedule_sys_t *p_schedule ...@@ -641,9 +640,8 @@ static int ExecuteScheduleProperty( vlm_t *p_vlm, vlm_schedule_sys_t *p_schedule
psz_line = strdup( ppsz_property[i] ); psz_line = strdup( ppsz_property[i] );
for( j = i+1; j < i_property; j++ ) for( j = i+1; j < i_property; j++ )
{ {
psz_line = realloc_or_free( psz_line, psz_line = xrealloc( psz_line,
strlen(psz_line) + strlen(ppsz_property[j]) + 1 + 1 ); strlen(psz_line) + strlen(ppsz_property[j]) + 1 + 1 );
assert( psz_line );
strcat( psz_line, " " ); strcat( psz_line, " " );
strcat( psz_line, ppsz_property[j] ); strcat( psz_line, ppsz_property[j] );
} }
......
...@@ -33,7 +33,6 @@ ...@@ -33,7 +33,6 @@
#endif #endif
#include <vlc_common.h> #include <vlc_common.h>
#include <vlc_memory.h>
#include <stdarg.h> /* va_list for BSD */ #include <stdarg.h> /* va_list for BSD */
...@@ -401,15 +400,13 @@ static void QueueMsg( vlc_object_t *p_this, int i_type, const char *psz_module, ...@@ -401,15 +400,13 @@ static void QueueMsg( vlc_object_t *p_this, int i_type, const char *psz_module,
if( psz_header ) if( psz_header )
{ {
psz_old = strdup( psz_header ); psz_old = strdup( psz_header );
psz_header = realloc_or_free( psz_header, i_header_size ); psz_header = xrealloc( psz_header, i_header_size );
assert( psz_header );
snprintf( psz_header, i_header_size , "[%s] %s", snprintf( psz_header, i_header_size , "[%s] %s",
p_obj->psz_header, psz_old ); p_obj->psz_header, psz_old );
} }
else else
{ {
psz_header = malloc( i_header_size ); psz_header = xmalloc( i_header_size );
assert( psz_header );
snprintf( psz_header, i_header_size, "[%s]", snprintf( psz_header, i_header_size, "[%s]",
p_obj->psz_header ); p_obj->psz_header );
} }
......
...@@ -37,7 +37,6 @@ ...@@ -37,7 +37,6 @@
#endif #endif
#include <vlc_common.h> #include <vlc_common.h>
#include <vlc_memory.h>
#include "../libvlc.h" #include "../libvlc.h"
#include <vlc_aout.h> #include <vlc_aout.h>
......
...@@ -30,7 +30,6 @@ ...@@ -30,7 +30,6 @@
#include <vlc_common.h> #include <vlc_common.h>
#include <vlc_charset.h> #include <vlc_charset.h>
#include <vlc_memory.h>
#include "variables.h" #include "variables.h"
#include "libvlc.h" #include "libvlc.h"
...@@ -218,9 +217,8 @@ int __var_Create( vlc_object_t *p_this, const char *psz_name, int i_type ) ...@@ -218,9 +217,8 @@ int __var_Create( vlc_object_t *p_this, const char *psz_name, int i_type )
if( (p_priv->i_vars & 15) == 15 ) if( (p_priv->i_vars & 15) == 15 )
{ {
p_priv->p_vars = realloc_or_free( p_priv->p_vars, p_priv->p_vars = xrealloc( p_priv->p_vars,
(p_priv->i_vars+17) * sizeof(variable_t) ); (p_priv->i_vars+17) * sizeof(variable_t) );
assert( p_priv->p_vars );
} }
memmove( p_priv->p_vars + i_new + 1, memmove( p_priv->p_vars + i_new + 1,
......
...@@ -563,8 +563,7 @@ httpd_HandlerCallBack( httpd_callback_sys_t *p_sys, httpd_client_t *cl, ...@@ -563,8 +563,7 @@ httpd_HandlerCallBack( httpd_callback_sys_t *p_sys, httpd_client_t *cl,
{ {
p[4] = '\0'; p[4] = '\0';
answer->i_body = strlen((char*)answer->p_body) + 1; answer->i_body = strlen((char*)answer->p_body) + 1;
uint8_t *p_body = realloc( answer->p_body, answer->i_body ); answer->p_body = xrealloc( answer->p_body, answer->i_body );
if( p_body ) answer->p_body = p_body;
} }
} }
......
...@@ -55,7 +55,6 @@ ...@@ -55,7 +55,6 @@
#endif #endif
#include <vlc_network.h> #include <vlc_network.h>
#include <vlc_memory.h>
#ifndef INADDR_ANY #ifndef INADDR_ANY
# define INADDR_ANY 0x00000000 # define INADDR_ANY 0x00000000
...@@ -523,8 +522,7 @@ char *__net_Gets( vlc_object_t *p_this, int fd, const v_socket_t *p_vs ) ...@@ -523,8 +522,7 @@ char *__net_Gets( vlc_object_t *p_this, int fd, const v_socket_t *p_vs )
if( i_line == i_max ) if( i_line == i_max )
{ {
i_max += 1024; i_max += 1024;
psz_line = realloc_or_free( psz_line, i_max ); psz_line = xrealloc( psz_line, i_max );
assert( psz_line );
ptr = psz_line + i_line; ptr = psz_line + i_line;
} }
......
...@@ -606,8 +606,7 @@ char *str_format_time( const char *tformat ) ...@@ -606,8 +606,7 @@ char *str_format_time( const char *tformat )
if( string != NULL ) \ if( string != NULL ) \
{ \ { \
int len = strlen( string ); \ int len = strlen( string ); \
dst = realloc( dst, i_size = i_size + len );\ dst = xrealloc( dst, i_size = i_size + len );\
assert( dst ); \
memcpy( (dst+d), string, len ); \ memcpy( (dst+d), string, len ); \
d += len; \ d += len; \
free( string ); \ free( string ); \
...@@ -622,8 +621,7 @@ char *str_format_time( const char *tformat ) ...@@ -622,8 +621,7 @@ char *str_format_time( const char *tformat )
#define INSERT_STRING_NO_FREE( string ) \ #define INSERT_STRING_NO_FREE( string ) \
{ \ { \
int len = strlen( string ); \ int len = strlen( string ); \
dst = realloc( dst, i_size = i_size + len );\ dst = xrealloc( dst, i_size = i_size + len );\
assert( dst ); \
memcpy( dst+d, string, len ); \ memcpy( dst+d, string, len ); \
d += len; \ d += len; \
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment