Commit 2add4890 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

Fix a bunch of warnings (error handling)

parent 061224eb
...@@ -153,7 +153,8 @@ vlm_t *__vlm_New ( vlc_object_t *p_this ) ...@@ -153,7 +153,8 @@ vlm_t *__vlm_New ( vlc_object_t *p_this )
char *psz_buffer = NULL; char *psz_buffer = NULL;
msg_Dbg( p_this, "loading VLM configuration" ); msg_Dbg( p_this, "loading VLM configuration" );
asprintf(&psz_buffer, "load %s", psz_vlmconf ); if( asprintf(&psz_buffer, "load %s", psz_vlmconf ) == -1 )
psz_buffer = NULL;
if( psz_buffer ) if( psz_buffer )
{ {
msg_Dbg( p_this, psz_buffer ); msg_Dbg( p_this, psz_buffer );
...@@ -676,19 +677,25 @@ static int ExecuteSave( vlm_t *p_vlm, const char *psz_file, vlm_message_t **pp_s ...@@ -676,19 +677,25 @@ static int ExecuteSave( vlm_t *p_vlm, const char *psz_file, vlm_message_t **pp_s
psz_save = Save( p_vlm ); psz_save = Save( p_vlm );
if( psz_save == NULL ) if( psz_save == NULL )
goto error;
if( fputs( psz_save, f ) == EOF )
goto error;;
if( fclose( f ) )
{ {
fclose( f ); f = NULL;
goto error; goto error;
} }
fwrite( psz_save, strlen( psz_save ), 1, f );
free( psz_save ); free( psz_save );
fclose( f );
*pp_status = vlm_MessageNew( "save", vlm_NULL ); *pp_status = vlm_MessageNew( "save", vlm_NULL );
return VLC_SUCCESS; return VLC_SUCCESS;
error: error:
*pp_status = vlm_MessageNew( "save", "Unable to save to file" ); free( psz_save );
if( f )
fclose( f );
*pp_status = vlm_MessageNew( "save", "Unable to save to file");
return VLC_EGENERIC; return VLC_EGENERIC;
} }
...@@ -1451,11 +1458,13 @@ static vlm_message_t *vlm_ShowMedia( vlm_media_sys_t *p_media ) ...@@ -1451,11 +1458,13 @@ static vlm_message_t *vlm_ShowMedia( vlm_media_sys_t *p_media )
for( i = 0; i < p_cfg->i_input; i++ ) for( i = 0; i < p_cfg->i_input; i++ )
{ {
char *psz_tmp; char *psz_tmp;
asprintf( &psz_tmp, "%d", i+1 ); if( asprintf( &psz_tmp, "%d", i+1 ) != -1 )
{
vlm_MessageAdd( p_msg_sub, vlm_MessageAdd( p_msg_sub,
vlm_MessageNew( psz_tmp, p_cfg->ppsz_input[i] ) ); vlm_MessageNew( psz_tmp, p_cfg->ppsz_input[i] ) );
free( psz_tmp ); free( psz_tmp );
} }
}
vlm_MessageAdd( p_msg, vlm_MessageAdd( p_msg,
vlm_MessageNew( "output", p_cfg->psz_output ? p_cfg->psz_output : "" ) ); vlm_MessageNew( "output", p_cfg->psz_output ? p_cfg->psz_output : "" ) );
...@@ -1490,10 +1499,13 @@ static vlm_message_t *vlm_ShowMedia( vlm_media_sys_t *p_media ) ...@@ -1490,10 +1499,13 @@ static vlm_message_t *vlm_ShowMedia( vlm_media_sys_t *p_media )
if( p_instance->p_input ) if( p_instance->p_input )
{ {
#define APPEND_INPUT_INFO( a, format, type ) \ #define APPEND_INPUT_INFO( a, format, type ) \
asprintf( &psz_tmp, format, \ if( asprintf( &psz_tmp, format, \
var_Get ## type( p_instance->p_input, a ) ); \ var_Get ## type( p_instance->p_input, a ) ) != -1 ) \
vlm_MessageAdd( p_msg_instance, vlm_MessageNew( a, psz_tmp ) ); \ { \
free( psz_tmp ); vlm_MessageAdd( p_msg_instance, vlm_MessageNew( a, \
psz_tmp ) ); \
free( psz_tmp ); \
}
APPEND_INPUT_INFO( "position", "%f", Float ); APPEND_INPUT_INFO( "position", "%f", Float );
APPEND_INPUT_INFO( "time", "%"PRIi64, Time ); APPEND_INPUT_INFO( "time", "%"PRIi64, Time );
APPEND_INPUT_INFO( "length", "%"PRIi64, Time ); APPEND_INPUT_INFO( "length", "%"PRIi64, Time );
...@@ -1503,10 +1515,13 @@ static vlm_message_t *vlm_ShowMedia( vlm_media_sys_t *p_media ) ...@@ -1503,10 +1515,13 @@ static vlm_message_t *vlm_ShowMedia( vlm_media_sys_t *p_media )
APPEND_INPUT_INFO( "seekable", "%d", Bool ); APPEND_INPUT_INFO( "seekable", "%d", Bool );
} }
#undef APPEND_INPUT_INFO #undef APPEND_INPUT_INFO
asprintf( &psz_tmp, "%d", p_instance->i_index + 1 ); if( asprintf( &psz_tmp, "%d", p_instance->i_index + 1 ) != -1 )
vlm_MessageAdd( p_msg_instance, vlm_MessageNew( "playlistindex", psz_tmp ) ); {
vlm_MessageAdd( p_msg_instance, vlm_MessageNew( "playlistindex",
psz_tmp ) );
free( psz_tmp ); free( psz_tmp );
} }
}
return p_msg; return p_msg;
} }
...@@ -1548,14 +1563,15 @@ static vlm_message_t *vlm_Show( vlm_t *vlm, vlm_media_sys_t *media, ...@@ -1548,14 +1563,15 @@ static vlm_message_t *vlm_Show( vlm_t *vlm, vlm_media_sys_t *media,
char *psz_date; char *psz_date;
localtime_r( &i_time, &date); localtime_r( &i_time, &date);
asprintf( &psz_date, "%d/%d/%d-%d:%d:%d", if( asprintf( &psz_date, "%d/%d/%d-%d:%d:%d",
date.tm_year + 1900, date.tm_mon + 1, date.tm_mday, date.tm_year + 1900, date.tm_mon + 1, date.tm_mday,
date.tm_hour, date.tm_min, date.tm_sec ); date.tm_hour, date.tm_min, date.tm_sec ) != -1 )
{
vlm_MessageAdd( msg_schedule, vlm_MessageAdd( msg_schedule,
vlm_MessageNew( "date", psz_date ) ); vlm_MessageNew( "date", psz_date ) );
free( psz_date ); free( psz_date );
} }
}
else else
{ {
vlm_MessageAdd( msg_schedule, vlm_MessageNew("date", "now") ); vlm_MessageAdd( msg_schedule, vlm_MessageNew("date", "now") );
...@@ -1622,8 +1638,9 @@ static vlm_message_t *vlm_Show( vlm_t *vlm, vlm_media_sys_t *media, ...@@ -1622,8 +1638,9 @@ static vlm_message_t *vlm_Show( vlm_t *vlm, vlm_media_sys_t *media,
i_broadcast++; i_broadcast++;
} }
asprintf( &psz_count, "( %d broadcast - %d vod )", i_broadcast, i_vod); if( asprintf( &psz_count, "( %d broadcast - %d vod )", i_broadcast,
i_vod) == -1 )
return NULL;
p_msg = vlm_MessageNew( "show", vlm_NULL ); p_msg = vlm_MessageNew( "show", vlm_NULL );
p_msg_child = vlm_MessageAdd( p_msg, vlm_MessageNew( "media", psz_count ) ); p_msg_child = vlm_MessageAdd( p_msg, vlm_MessageNew( "media", psz_count ) );
free( psz_count ); free( psz_count );
......
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