Commit 69d1ac0a authored by Derk-Jan Hartman's avatar Derk-Jan Hartman

input_item_SetURI(): add a NDEBUG printf check for paths.

When it is called with strings that contain " " or '\' or don't have ://, we have an indication that the caller is likely providing a path. This will help us find these cases in the future.
parent e3174219
...@@ -353,6 +353,10 @@ char *input_item_GetURI( input_item_t *p_i ) ...@@ -353,6 +353,10 @@ char *input_item_GetURI( input_item_t *p_i )
void input_item_SetURI( input_item_t *p_i, const char *psz_uri ) void input_item_SetURI( input_item_t *p_i, const char *psz_uri )
{ {
vlc_mutex_lock( &p_i->lock ); vlc_mutex_lock( &p_i->lock );
#ifndef NDEBUG
if( !strstr( psz_uri, "://" || strstr( psz_uri, " " ) || strstr( psz_uri, "\") ))
fprintf( stderr, "input_item_SetURI() was likely called with a path. FIXME\n" );
#endif
free( p_i->psz_uri ); free( p_i->psz_uri );
p_i->psz_uri = strdup( psz_uri ); p_i->psz_uri = strdup( psz_uri );
......
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