Commit fe3726de authored by Rémi Duraffort's avatar Rémi Duraffort

Fix resource leak when loadsave fail (CID 94)

parent 2573062f
...@@ -51,14 +51,14 @@ int playlist_Export( playlist_t * p_playlist, const char *psz_filename , ...@@ -51,14 +51,14 @@ int playlist_Export( playlist_t * p_playlist, const char *psz_filename ,
p_export = (playlist_export_t *)malloc( sizeof(playlist_export_t) ); p_export = (playlist_export_t *)malloc( sizeof(playlist_export_t) );
if( !p_export) if( !p_export)
return VLC_ENOMEM; return VLC_ENOMEM;
p_export->psz_filename = NULL; p_export->psz_filename = psz_filename ? strdup( psz_filename ) : NULL;
if ( psz_filename )
p_export->psz_filename = strdup( psz_filename );
p_export->p_file = utf8_fopen( psz_filename, "wt" ); p_export->p_file = utf8_fopen( psz_filename, "wt" );
if( !p_export->p_file ) if( !p_export->p_file )
{ {
msg_Err( p_playlist , "could not create playlist file %s (%m)", msg_Err( p_playlist , "could not create playlist file %s (%m)",
psz_filename ); psz_filename );
free( p_export->psz_filename );
free( p_export );
return VLC_EGENERIC; return VLC_EGENERIC;
} }
...@@ -69,23 +69,27 @@ int playlist_Export( playlist_t * p_playlist, const char *psz_filename , ...@@ -69,23 +69,27 @@ int playlist_Export( playlist_t * p_playlist, const char *psz_filename ,
p_playlist->p_private = (void *)p_export; p_playlist->p_private = (void *)p_export;
/* And call the module ! All work is done now */ /* And call the module ! All work is done now */
int i_ret;
p_module = module_need( p_playlist, "playlist export", psz_type, true); p_module = module_need( p_playlist, "playlist export", psz_type, true);
if( !p_module ) if( !p_module )
{ {
msg_Warn( p_playlist, "exporting playlist failed" ); msg_Warn( p_playlist, "exporting playlist failed" );
vlc_object_unlock( p_playlist ); i_ret = VLC_ENOOBJ;
return VLC_ENOOBJ; }
else
{
module_unneed( p_playlist , p_module );
i_ret = VLC_SUCCESS;
} }
module_unneed( p_playlist , p_module );
/* Clean up */ /* Clean up */
fclose( p_export->p_file ); fclose( p_export->p_file );
free( p_export->psz_filename ); free( p_export->psz_filename );
free ( p_export ); free( p_export );
p_playlist->p_private = NULL; p_playlist->p_private = NULL;
vlc_object_unlock( p_playlist ); vlc_object_unlock( p_playlist );
return VLC_SUCCESS; return i_ret;
} }
/***************************************************************************** /*****************************************************************************
......
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