Commit 6f533a25 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

FTP: fix small memory leak in directory case

parent 421d5ae7
...@@ -353,26 +353,26 @@ static int InOpen( vlc_object_t *p_this ) ...@@ -353,26 +353,26 @@ static int InOpen( vlc_object_t *p_this )
if( p_sys->url.psz_path == NULL ) if( p_sys->url.psz_path == NULL )
p_sys->directory = true; p_sys->directory = true;
else else
if( ftp_SendCommand( p_this, p_sys, "SIZE %s", p_sys->url.psz_path ) < 0 if( ftp_SendCommand( p_this, p_sys, "SIZE %s", p_sys->url.psz_path ) < 0 )
|| ftp_ReadCommand( p_this, p_sys, NULL, &psz_arg ) != 2 ) goto error;
{
msg_Dbg( p_access, "cannot get file size" );
msg_Dbg( p_access, "will try to get directory contents" );
if( ftp_SendCommand( p_this, p_sys, "CWD %s", p_sys->url.psz_path ) < 0
|| ftp_ReadCommand( p_this, p_sys, NULL, &psz_arg ) != 2 )
{
msg_Err( p_access, "file or directory doesn't exist" );
net_Close( p_sys->fd_cmd );
goto exit_error;
}
p_sys->directory = true;
}
else else
if ( ftp_ReadCommand( p_this, p_sys, NULL, &psz_arg ) == 2 )
{ {
p_access->info.i_size = atoll( &psz_arg[4] ); p_access->info.i_size = atoll( &psz_arg[4] );
free( psz_arg ); free( psz_arg );
msg_Dbg( p_access, "file size: %"PRIu64, p_access->info.i_size ); msg_Dbg( p_access, "file size: %"PRIu64, p_access->info.i_size );
} }
else
if( ftp_SendCommand( p_this, p_sys, "CWD %s", p_sys->url.psz_path ) < 0 )
goto error;
else
if( ftp_ReadCommand( p_this, p_sys, NULL, NULL ) != 2 )
{
msg_Err( p_access, "file or directory does not exist" );
goto error;
}
else
p_sys->directory = true;
/* Start the 'stream' */ /* Start the 'stream' */
if( ftp_StartStream( p_this, p_sys, 0 ) < 0 ) if( ftp_StartStream( p_this, p_sys, 0 ) < 0 )
...@@ -387,6 +387,8 @@ static int InOpen( vlc_object_t *p_this ) ...@@ -387,6 +387,8 @@ static int InOpen( vlc_object_t *p_this )
return VLC_SUCCESS; return VLC_SUCCESS;
error:
net_Close( p_sys->fd_cmd );
exit_error: exit_error:
vlc_UrlClean( &p_sys->url ); vlc_UrlClean( &p_sys->url );
free( p_sys ); free( p_sys );
......
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