X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=src%2Fmisc%2Fmodules.c;h=ade1fbc9a9b827ead9816f90ca8119b0529abaac;hb=79c1ddfdb0f3f131173def0be43451b6e1aa46bf;hp=0d7f67e0b4fb1ce14f979e90dce61c589b20a803;hpb=83f23b6686543887f908d184387782bf018c35d5;p=vlc diff --git a/src/misc/modules.c b/src/misc/modules.c index 0d7f67e0b4..ade1fbc9a9 100644 --- a/src/misc/modules.c +++ b/src/misc/modules.c @@ -36,8 +36,6 @@ #include /* sprintf() */ #include /* strdup() */ -#include - #ifdef HAVE_DIRENT_H # include #endif @@ -77,23 +75,21 @@ # endif #endif -#include "vlc_error.h" +#include "misc/configuration.h" #include "vlc_interface.h" -#include "vlc_interaction.h" -#include "intf_eject.h" - #include "vlc_playlist.h" -#include "vlc_video.h" -#include "video_output.h" -#include "vout_synchro.h" -#include "vlc_spu.h" +#include "vlc_stream.h" +#include "vlc_access.h" +#include "vlc_demux.h" + +#include "vlc_vout.h" +#include "vlc_vout_synchro.h" -#include "audio_output.h" -#include "aout_internal.h" +#include "vlc_aout.h" -#include "stream_output.h" +#include "vlc_sout.h" #include "vlc_httpd.h" #include "vlc_acl.h" #include "vlc_tls.h" @@ -102,7 +98,7 @@ #include "vlc_url.h" #include "iso_lang.h" -#include "charset.h" +#include "vlc_charset.h" #include "vlc_block.h" @@ -113,6 +109,9 @@ #include "vlc_update.h" #include "vlc_strings.h" +#include "vlc_streaming.h" + +#include "modules.h" #if defined( _MSC_VER ) && defined( UNDER_CE ) # include "modules_builtin_evc.h" @@ -121,7 +120,7 @@ #else # include "modules_builtin.h" #endif -#include "network.h" +#include "vlc_network.h" #if defined( WIN32 ) || defined( UNDER_CE ) /* Avoid name collisions */ @@ -138,7 +137,7 @@ static int AllocatePluginFile ( vlc_object_t *, char *, int64_t, int64_t ); static module_t * AllocatePlugin( vlc_object_t *, char * ); #endif static int AllocateBuiltinModule( vlc_object_t *, int ( * ) ( module_t * ) ); -static int DeleteModule ( module_t * ); +static int DeleteModule ( module_t *, vlc_bool_t ); #ifdef HAVE_DYNAMIC_PLUGINS static void DupModule ( module_t * ); static void UndupModule ( module_t * ); @@ -159,6 +158,8 @@ static char * GetWindowsError ( void ); #endif #endif +static void module_LoadMain( vlc_object_t *p_this ); + /* Sub-version number * (only used to avoid breakage in dev version when cache structure changes) */ @@ -172,27 +173,29 @@ static char * GetWindowsError ( void ); *****************************************************************************/ void __module_InitBank( vlc_object_t *p_this ) { - module_bank_t *p_bank; + module_bank_t *p_bank = NULL; vlc_value_t lockval; - var_Create( p_this->p_libvlc, "libvlc", VLC_VAR_MUTEX ); - var_Get( p_this->p_libvlc, "libvlc", &lockval ); + var_Create( p_this->p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); + var_Get( p_this->p_libvlc_global, "libvlc", &lockval ); vlc_mutex_lock( lockval.p_address ); - if( p_this->p_libvlc->p_module_bank ) + if( p_this->p_libvlc_global->p_module_bank ) { - p_this->p_libvlc->p_module_bank->i_usage++; + p_this->p_libvlc_global->p_module_bank->i_usage++; vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); return; } vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); p_bank = vlc_object_create( p_this, sizeof(module_bank_t) ); + if( !p_bank ) + return; p_bank->psz_object_name = "module bank"; p_bank->i_usage = 1; p_bank->i_cache = p_bank->i_loaded_cache = 0; - p_bank->pp_cache = p_bank->pp_loaded_cache = 0; + p_bank->pp_cache = p_bank->pp_loaded_cache = NULL; p_bank->b_cache = p_bank->b_cache_dirty = p_bank->b_cache_delete = VLC_FALSE; @@ -204,12 +207,10 @@ void __module_InitBank( vlc_object_t *p_this ) #endif /* Everything worked, attach the object */ - p_this->p_libvlc->p_module_bank = p_bank; - vlc_object_attach( p_bank, p_this->p_libvlc ); + p_this->p_libvlc_global->p_module_bank = p_bank; + vlc_object_attach( p_bank, p_this->p_libvlc_global ); module_LoadMain( p_this ); - - return; } /***************************************************************************** @@ -232,57 +233,68 @@ void __module_ResetBank( vlc_object_t *p_this ) *****************************************************************************/ void __module_EndBank( vlc_object_t *p_this ) { - module_t * p_next; + module_t * p_next = NULL; vlc_value_t lockval; - var_Create( p_this->p_libvlc, "libvlc", VLC_VAR_MUTEX ); - var_Get( p_this->p_libvlc, "libvlc", &lockval ); + var_Create( p_this->p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); + var_Get( p_this->p_libvlc_global, "libvlc", &lockval ); vlc_mutex_lock( lockval.p_address ); - if( !p_this->p_libvlc->p_module_bank ) + if( !p_this->p_libvlc_global->p_module_bank ) { vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); return; } - if( --p_this->p_libvlc->p_module_bank->i_usage ) + if( --p_this->p_libvlc_global->p_module_bank->i_usage ) { vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); return; } vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); config_AutoSaveConfigFile( p_this ); #ifdef HAVE_DYNAMIC_PLUGINS -#define p_bank p_this->p_libvlc->p_module_bank +#define p_bank p_this->p_libvlc_global->p_module_bank if( p_bank->b_cache ) CacheSave( p_this ); while( p_bank->i_loaded_cache-- ) { - free( p_bank->pp_loaded_cache[p_bank->i_loaded_cache]->psz_file ); - free( p_bank->pp_loaded_cache[p_bank->i_loaded_cache] ); + if( p_bank->pp_loaded_cache[p_bank->i_loaded_cache] ) + { + DeleteModule( p_bank->pp_loaded_cache[p_bank->i_loaded_cache]->p_module, p_bank->pp_loaded_cache[p_bank->i_loaded_cache]->b_used ); + free( p_bank->pp_loaded_cache[p_bank->i_loaded_cache]->psz_file ); + free( p_bank->pp_loaded_cache[p_bank->i_loaded_cache] ); + p_bank->pp_loaded_cache[p_bank->i_loaded_cache] = NULL; + } } if( p_bank->pp_loaded_cache ) + { free( p_bank->pp_loaded_cache ); - + p_bank->pp_loaded_cache = NULL; + } while( p_bank->i_cache-- ) { free( p_bank->pp_cache[p_bank->i_cache]->psz_file ); free( p_bank->pp_cache[p_bank->i_cache] ); + p_bank->pp_cache[p_bank->i_cache] = NULL; } if( p_bank->pp_cache ) + { free( p_bank->pp_cache ); + p_bank->pp_cache = NULL; + } #undef p_bank #endif - vlc_object_detach( p_this->p_libvlc->p_module_bank ); + vlc_object_detach( p_this->p_libvlc_global->p_module_bank ); - while( p_this->p_libvlc->p_module_bank->i_children ) + while( p_this->p_libvlc_global->p_module_bank->i_children ) { - p_next = (module_t *)p_this->p_libvlc->p_module_bank->pp_children[0]; + p_next = (module_t *)p_this->p_libvlc_global->p_module_bank->pp_children[0]; - if( DeleteModule( p_next ) ) + if( DeleteModule( p_next, VLC_TRUE ) ) { /* Module deletion failed */ msg_Err( p_this, "module \"%s\" can't be removed, trying harder", @@ -294,8 +306,8 @@ void __module_EndBank( vlc_object_t *p_this ) } } - vlc_object_destroy( p_this->p_libvlc->p_module_bank ); - p_this->p_libvlc->p_module_bank = NULL; + vlc_object_destroy( p_this->p_libvlc_global->p_module_bank ); + p_this->p_libvlc_global->p_module_bank = NULL; return; } @@ -308,22 +320,22 @@ void __module_EndBank( vlc_object_t *p_this ) * as another module, and for instance the configuration options of main will * be available in the module bank structure just as for every other module. *****************************************************************************/ -void __module_LoadMain( vlc_object_t *p_this ) +static void module_LoadMain( vlc_object_t *p_this ) { vlc_value_t lockval; - var_Create( p_this->p_libvlc, "libvlc", VLC_VAR_MUTEX ); - var_Get( p_this->p_libvlc, "libvlc", &lockval ); + var_Create( p_this->p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); + var_Get( p_this->p_libvlc_global, "libvlc", &lockval ); vlc_mutex_lock( lockval.p_address ); - if( p_this->p_libvlc->p_module_bank->b_main ) + if( p_this->p_libvlc_global->p_module_bank->b_main ) { vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); return; } - p_this->p_libvlc->p_module_bank->b_main = VLC_TRUE; + p_this->p_libvlc_global->p_module_bank->b_main = VLC_TRUE; vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); AllocateBuiltinModule( p_this, vlc_entry__main ); } @@ -337,18 +349,18 @@ void __module_LoadBuiltins( vlc_object_t * p_this ) { vlc_value_t lockval; - var_Create( p_this->p_libvlc, "libvlc", VLC_VAR_MUTEX ); - var_Get( p_this->p_libvlc, "libvlc", &lockval ); + var_Create( p_this->p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); + var_Get( p_this->p_libvlc_global, "libvlc", &lockval ); vlc_mutex_lock( lockval.p_address ); - if( p_this->p_libvlc->p_module_bank->b_builtins ) + if( p_this->p_libvlc_global->p_module_bank->b_builtins ) { vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); return; } - p_this->p_libvlc->p_module_bank->b_builtins = VLC_TRUE; + p_this->p_libvlc_global->p_module_bank->b_builtins = VLC_TRUE; vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); msg_Dbg( p_this, "checking builtin modules" ); ALLOCATE_ALL_BUILTINS(); @@ -364,26 +376,26 @@ void __module_LoadPlugins( vlc_object_t * p_this ) #ifdef HAVE_DYNAMIC_PLUGINS vlc_value_t lockval; - var_Create( p_this->p_libvlc, "libvlc", VLC_VAR_MUTEX ); - var_Get( p_this->p_libvlc, "libvlc", &lockval ); + var_Create( p_this->p_libvlc_global, "libvlc", VLC_VAR_MUTEX ); + var_Get( p_this->p_libvlc_global, "libvlc", &lockval ); vlc_mutex_lock( lockval.p_address ); - if( p_this->p_libvlc->p_module_bank->b_plugins ) + if( p_this->p_libvlc_global->p_module_bank->b_plugins ) { vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); return; } - p_this->p_libvlc->p_module_bank->b_plugins = VLC_TRUE; + p_this->p_libvlc_global->p_module_bank->b_plugins = VLC_TRUE; vlc_mutex_unlock( lockval.p_address ); - var_Destroy( p_this->p_libvlc, "libvlc" ); + var_Destroy( p_this->p_libvlc_global, "libvlc" ); msg_Dbg( p_this, "checking plugin modules" ); if( config_GetInt( p_this, "plugins-cache" ) ) - p_this->p_libvlc->p_module_bank->b_cache = VLC_TRUE; + p_this->p_libvlc_global->p_module_bank->b_cache = VLC_TRUE; - if( p_this->p_libvlc->p_module_bank->b_cache || - p_this->p_libvlc->p_module_bank->b_cache_delete ) CacheLoad( p_this ); + if( p_this->p_libvlc_global->p_module_bank->b_cache || + p_this->p_libvlc_global->p_module_bank->b_cache_delete ) CacheLoad( p_this ); AllocateAllPlugins( p_this ); #endif @@ -416,7 +428,7 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, module_t *p_module; int i_shortcuts = 0; - char *psz_shortcuts = NULL, *psz_var = NULL; + char *psz_shortcuts = NULL, *psz_var = NULL, *psz_alias = NULL; vlc_bool_t b_force_backup = p_this->b_force; @@ -492,7 +504,7 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, } /* Test if we have the required CPU */ - if( (p_module->i_cpu & p_this->p_libvlc->i_cpu) != p_module->i_cpu ) + if( (p_module->i_cpu & p_this->p_libvlc_global->i_cpu) != p_module->i_cpu ) { continue; } @@ -501,54 +513,39 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, if( i_shortcuts > 0 ) { vlc_bool_t b_trash; - int i_dummy, i_short = i_shortcuts; - char *psz_name = psz_shortcuts; + const char *psz_name = psz_shortcuts; /* Let's drop modules with a <= 0 score (unless they are * explicitly requested) */ b_trash = p_module->i_score <= 0; - while( i_short > 0 ) + for( unsigned i_short = i_shortcuts; i_short > 0; i_short-- ) { - for( i_dummy = 0; p_module->pp_shortcuts[i_dummy]; i_dummy++ ) + for( unsigned i = 0; p_module->pp_shortcuts[i]; i++ ) { - if( !strcasecmp( psz_name, - p_module->pp_shortcuts[i_dummy] ) ) + char *c; + if( ( c = strchr( psz_name, '@' ) ) + ? !strncasecmp( psz_name, p_module->pp_shortcuts[i], + c-psz_name ) + : !strcasecmp( psz_name, p_module->pp_shortcuts[i] ) ) { /* Found it */ - b_trash = VLC_FALSE; + if( c && c[1] ) + psz_alias = c+1; i_shortcut_bonus = i_short * 10000; - break; + goto found_shortcut; } } - if( i_shortcut_bonus ) - { - /* We found it... remember ? */ - break; - } - /* Go to the next shortcut... This is so lame! */ - while( *psz_name ) - { - psz_name++; - } - psz_name++; - i_short--; + psz_name += strlen( psz_name ) + 1; } /* If we are in "strict" mode and we couldn't * find the module in the list of provided shortcuts, * then kick the bastard out of here!!! */ - if( i_short == 0 && b_strict ) - { - b_trash = VLC_TRUE; - } - - if( b_trash ) - { + if( b_strict ) continue; - } } /* If we didn't require a shortcut, trash <= 0 scored plugins */ else if( p_module->i_score <= 0 ) @@ -556,11 +553,13 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, continue; } +found_shortcut: + /* Special case: test if we requested a particular intf plugin */ if( !i_shortcuts && p_module->psz_program && !strcmp( psz_capability, "interface" ) && !strcmp( p_module->psz_program, - p_this->p_vlc->psz_object_name ) ) + p_this->p_libvlc->psz_object_name ) ) { if( !b_intf ) { @@ -578,7 +577,7 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, /* Store this new module */ p_list[ i_index ].p_module = p_module; p_list[ i_index ].i_score = p_module->i_score + i_shortcut_bonus; - p_list[ i_index ].b_force = !!i_shortcut_bonus; + p_list[ i_index ].b_force = i_shortcut_bonus && b_strict; /* Add it to the modules-to-probe list */ if( i_index == 0 ) @@ -634,8 +633,10 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, { #ifdef HAVE_DYNAMIC_PLUGINS /* Make sure the module is loaded in mem */ - module_t *p_module = p_tmp->p_module->b_submodule ? - (module_t *)p_tmp->p_module->p_parent : p_tmp->p_module; + module_t *p_module = p_tmp->p_module; + if( p_module->b_submodule ) + p_module = (module_t *)p_module->p_parent; + if( !p_module->b_builtin && !p_module->b_loaded ) { module_t *p_new_module = @@ -644,7 +645,7 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, { CacheMerge( p_this, p_module, p_new_module ); vlc_object_attach( p_new_module, p_module ); - DeleteModule( p_new_module ); + DeleteModule( p_new_module, VLC_TRUE ); } } #endif @@ -697,6 +698,9 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, { msg_Err( p_this, "no %s module matched \"%s\"", psz_capability, (psz_name && *psz_name) ? psz_name : "any" ); + + msg_StackSet( VLC_EGENERIC, "no %s module matched \"%s\"", + psz_capability, (psz_name && *psz_name) ? psz_name : "any" ); } } else if( psz_name != NULL && *psz_name ) @@ -704,6 +708,14 @@ module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability, msg_Warn( p_this, "no %s module matching \"%s\" could be loaded", psz_capability, (psz_name && *psz_name) ? psz_name : "any" ); } + else + msg_StackSet( VLC_EGENERIC, "no suitable %s module", psz_capability ); + + if( psz_alias && !p_this->psz_object_name ) + /* This assumes that p_this is the object which will be using the + * module. That's not always the case ... but it is in most cases. + */ + p_this->psz_object_name = strdup( psz_alias ); if( psz_shortcuts ) { @@ -740,6 +752,31 @@ void __module_Unneed( vlc_object_t * p_this, module_t * p_module ) return; } +/***************************************************************************** + * module_Exists: tell if a module exists. + ***************************************************************************** + * This function is a boolean function that tells if a module exist or not. + *****************************************************************************/ + +vlc_bool_t __module_Exists( vlc_object_t *p_this, const char * psz_name ) +{ + vlc_list_t *p_list; + int i; + p_list = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE ); + for( i = 0 ; i < p_list->i_count; i++) + { + if (!strcmp( + ((module_t *) p_list->p_values[i].p_object)->psz_shortname , + psz_name ) ) + { + /* We can release the list, and return yes */ + vlc_list_release( p_list ); return VLC_TRUE; + } + } + vlc_list_release( p_list ); return VLC_FALSE; +} + + /***************************************************************************** * Following functions are local. *****************************************************************************/ @@ -752,21 +789,22 @@ static void AllocateAllPlugins( vlc_object_t *p_this ) { /* Yes, there are two NULLs because we replace one with "plugin-path". */ #if defined( WIN32 ) || defined( UNDER_CE ) - char *path[] = { "modules", "", "plugins", 0, 0 }; + const char *path[] = { "modules", "", "plugins", NULL, NULL }; #else - char *path[] = { "modules", PLUGIN_PATH, "plugins", 0, 0 }; + const char *path[] = { "modules", PLUGIN_PATH, "plugins", NULL, NULL }; #endif - char **ppsz_path = path; - char *psz_fullpath; + const char *const *ppsz_path; /* If the user provided a plugin path, we add it to the list */ - path[ sizeof(path)/sizeof(char*) - 2 ] = - config_GetPsz( p_this, "plugin-path" ); + char *userpath = config_GetPsz( p_this, "plugin-path" ); + path[sizeof(path)/sizeof(path[0]) - 2] = userpath; - for( ; *ppsz_path != NULL ; ppsz_path++ ) + for (ppsz_path = path; *ppsz_path != NULL; ppsz_path++) { - if( !(*ppsz_path)[0] ) continue; + char *psz_fullpath; + + if (!**ppsz_path) continue; #if defined( SYS_BEOS ) || defined( __APPLE__ ) || defined( WIN32 ) @@ -778,27 +816,16 @@ static void AllocateAllPlugins( vlc_object_t *p_this ) if( (*ppsz_path)[0] != '/' ) #endif { - int i_dirlen = strlen( *ppsz_path ); - i_dirlen += strlen( p_this->p_libvlc->psz_vlcpath ) + 2; - - psz_fullpath = malloc( i_dirlen ); - if( psz_fullpath == NULL ) - { - continue; - } -#ifdef WIN32 - sprintf( psz_fullpath, "%s\\%s", - p_this->p_libvlc->psz_vlcpath, *ppsz_path ); -#else - sprintf( psz_fullpath, "%s/%s", - p_this->p_libvlc->psz_vlcpath, *ppsz_path ); -#endif + if( 0>= asprintf(&psz_fullpath, "%s"DIR_SEP"%s", + p_this->p_libvlc_global->psz_vlcpath, *ppsz_path) ) + psz_fullpath = NULL; } else #endif - { psz_fullpath = strdup( *ppsz_path ); - } + + if( psz_fullpath == NULL ) + continue; msg_Dbg( p_this, "recursively browsing `%s'", psz_fullpath ); @@ -809,9 +836,8 @@ static void AllocateAllPlugins( vlc_object_t *p_this ) } /* Free plugin-path */ - if( path[ sizeof(path)/sizeof(char*) - 2 ] ) - free( path[ sizeof(path)/sizeof(char*) - 2 ] ); - path[ sizeof(path)/sizeof(char*) - 2 ] = NULL; + if( userpath != NULL ) + free( userpath ); } /***************************************************************************** @@ -820,6 +846,12 @@ static void AllocateAllPlugins( vlc_object_t *p_this ) static void AllocatePluginDir( vlc_object_t *p_this, const char *psz_dir, int i_maxdepth ) { +/* FIXME: Needs to be ported to wide char on ALL Windows builds */ +#ifdef WIN32 +# undef opendir +# undef closedir +# undef readdir +#endif #if defined( UNDER_CE ) || defined( _MSC_VER ) #ifdef UNDER_CE wchar_t psz_wpath[MAX_PATH + 256]; @@ -836,7 +868,7 @@ static void AllocatePluginDir( vlc_object_t *p_this, const char *psz_dir, #endif char * psz_file; - if( p_this->p_vlc->b_die || i_maxdepth < 0 ) + if( p_this->p_libvlc->b_die || i_maxdepth < 0 ) { return; } @@ -926,7 +958,7 @@ static void AllocatePluginDir( vlc_object_t *p_this, const char *psz_dir, AllocatePluginFile( p_this, psz_file, i_time, i_size ); } } - while( !p_this->p_vlc->b_die && FindNextFile( handle, &finddata ) ); + while( !p_this->p_libvlc->b_die && FindNextFile( handle, &finddata ) ); /* Close the directory */ FindClose( handle ); @@ -941,7 +973,7 @@ static void AllocatePluginDir( vlc_object_t *p_this, const char *psz_dir, i_dirlen = strlen( psz_dir ); /* Parse the directory and try to load all files it contains. */ - while( !p_this->p_vlc->b_die && (file = readdir( dir )) ) + while( !p_this->p_libvlc->b_die && (file = readdir( dir )) ) { struct stat statbuf; unsigned int i_len; @@ -1002,7 +1034,7 @@ static void AllocatePluginDir( vlc_object_t *p_this, const char *psz_dir, static int AllocatePluginFile( vlc_object_t * p_this, char * psz_file, int64_t i_file_time, int64_t i_file_size ) { - module_t * p_module; + module_t * p_module = NULL; module_cache_t *p_cache_entry = NULL; /* @@ -1024,7 +1056,7 @@ static int AllocatePluginFile( vlc_object_t * p_this, char * psz_file, } else { - module_config_t *p_item; + module_config_t *p_item = NULL, *p_end = NULL; p_module = p_cache_entry->p_module; p_module->b_loaded = VLC_FALSE; @@ -1032,12 +1064,17 @@ static int AllocatePluginFile( vlc_object_t * p_this, char * psz_file, /* For now we force loading if the module's config contains * callbacks or actions. * Could be optimized by adding an API call.*/ - for( p_item = p_module->p_config; - p_item->i_type != CONFIG_HINT_END; p_item++ ) + for( p_item = p_module->p_config, p_end = p_item + p_module->confsize; + p_item < p_end; p_item++ ) { if( p_item->pf_callback || p_item->i_action ) + { p_module = AllocatePlugin( p_this, psz_file ); + break; + } } + if( p_module == p_cache_entry->p_module ) + p_cache_entry->b_used = VLC_TRUE; } } @@ -1050,22 +1087,26 @@ static int AllocatePluginFile( vlc_object_t * p_this, char * psz_file, /* msg_Dbg( p_this, "plugin \"%s\", %s", p_module->psz_object_name, p_module->psz_longname ); */ - vlc_object_attach( p_module, p_this->p_libvlc->p_module_bank ); - } + vlc_object_attach( p_module, p_this->p_libvlc_global->p_module_bank ); - if( !p_this->p_libvlc->p_module_bank->b_cache ) return 0; + if( !p_this->p_libvlc_global->p_module_bank->b_cache ) + return 0; - /* Add entry to cache */ -#define p_bank p_this->p_libvlc->p_module_bank - p_bank->pp_cache = - realloc( p_bank->pp_cache, (p_bank->i_cache + 1) * sizeof(void *) ); - p_bank->pp_cache[p_bank->i_cache] = malloc( sizeof(module_cache_t) ); - p_bank->pp_cache[p_bank->i_cache]->psz_file = strdup( psz_file ); - p_bank->pp_cache[p_bank->i_cache]->i_time = i_file_time; - p_bank->pp_cache[p_bank->i_cache]->i_size = i_file_size; - p_bank->pp_cache[p_bank->i_cache]->b_junk = p_module ? 0 : 1; - p_bank->pp_cache[p_bank->i_cache]->p_module = p_module; - p_bank->i_cache++; + /* Add entry to cache */ +#define p_bank p_this->p_libvlc_global->p_module_bank + p_bank->pp_cache = + realloc( p_bank->pp_cache, (p_bank->i_cache + 1) * sizeof(void *) ); + p_bank->pp_cache[p_bank->i_cache] = malloc( sizeof(module_cache_t) ); + if( !p_bank->pp_cache[p_bank->i_cache] ) + return -1; + p_bank->pp_cache[p_bank->i_cache]->psz_file = strdup( psz_file ); + p_bank->pp_cache[p_bank->i_cache]->i_time = i_file_time; + p_bank->pp_cache[p_bank->i_cache]->i_size = i_file_size; + p_bank->pp_cache[p_bank->i_cache]->b_junk = p_module ? 0 : 1; + p_bank->pp_cache[p_bank->i_cache]->b_used = VLC_TRUE; + p_bank->pp_cache[p_bank->i_cache]->p_module = p_module; + p_bank->i_cache++; + } return p_module ? 0 : -1; } @@ -1079,10 +1120,11 @@ static int AllocatePluginFile( vlc_object_t * p_this, char * psz_file, *****************************************************************************/ static module_t * AllocatePlugin( vlc_object_t * p_this, char * psz_file ) { - module_t * p_module; + module_t * p_module = NULL; module_handle_t handle; - if( LoadModule( p_this, psz_file, &handle ) ) return NULL; + if( LoadModule( p_this, psz_file, &handle ) ) + return NULL; /* Now that we have successfully loaded the module, we can * allocate a structure for it */ @@ -1098,7 +1140,7 @@ static module_t * AllocatePlugin( vlc_object_t * p_this, char * psz_file ) p_module->psz_filename = psz_file; p_module->handle = handle; #ifndef HAVE_SHARED_LIBVLC - p_module->p_symbols = &p_this->p_libvlc->p_module_bank->symbols; + p_module->p_symbols = &p_this->p_libvlc_global->p_module_bank->symbols; #endif p_module->b_loaded = VLC_TRUE; @@ -1128,7 +1170,7 @@ static module_t * AllocatePlugin( vlc_object_t * p_this, char * psz_file ) *****************************************************************************/ static void DupModule( module_t *p_module ) { - char **pp_shortcut; + const char **pp_shortcut; int i_submodule; for( pp_shortcut = p_module->pp_shortcuts ; *pp_shortcut ; pp_shortcut++ ) @@ -1162,7 +1204,7 @@ static void DupModule( module_t *p_module ) *****************************************************************************/ static void UndupModule( module_t *p_module ) { - char **pp_shortcut; + const char **pp_shortcut; int i_submodule; for( i_submodule = 0; i_submodule < p_module->i_children; i_submodule++ ) @@ -1172,17 +1214,17 @@ static void UndupModule( module_t *p_module ) for( pp_shortcut = p_module->pp_shortcuts ; *pp_shortcut ; pp_shortcut++ ) { - free( *pp_shortcut ); + free( (void *)*pp_shortcut ); } - free( p_module->psz_object_name ); - free( p_module->psz_capability ); - if( p_module->psz_shortname ) free( p_module->psz_shortname ); - free( p_module->psz_longname ); + free( (void *)p_module->psz_object_name ); + free( (void *)p_module->psz_capability ); + if( p_module->psz_shortname ) free( (void *)p_module->psz_shortname ); + free( (void *)p_module->psz_longname ); if( p_module->psz_program != NULL ) { - free( p_module->psz_program ); + free( (void *)p_module->psz_program ); } } @@ -1225,7 +1267,7 @@ static int AllocateBuiltinModule( vlc_object_t * p_this, /* msg_Dbg( p_this, "builtin \"%s\", %s", p_module->psz_object_name, p_module->psz_longname ); */ - vlc_object_attach( p_module, p_this->p_libvlc->p_module_bank ); + vlc_object_attach( p_module, p_this->p_libvlc_global->p_module_bank ); return 0; } @@ -1235,9 +1277,11 @@ static int AllocateBuiltinModule( vlc_object_t * p_this, ***************************************************************************** * This function can only be called if the module isn't being used. *****************************************************************************/ -static int DeleteModule( module_t * p_module ) +static int DeleteModule( module_t * p_module, vlc_bool_t b_detach ) { - vlc_object_detach( p_module ); + if( !p_module ) return VLC_EGENERIC; + if( b_detach ) + vlc_object_detach( p_module ); /* We free the structures that we strdup()ed in Allocate*Module(). */ #ifdef HAVE_DYNAMIC_PLUGINS @@ -1262,7 +1306,7 @@ static int DeleteModule( module_t * p_module ) config_Free( p_module ); vlc_object_destroy( p_module ); - + p_module = NULL; return 0; } @@ -1276,7 +1320,7 @@ static int DeleteModule( module_t * p_module ) *****************************************************************************/ static int CallEntry( module_t * p_module ) { - static char *psz_name = "vlc_entry" MODULE_SUFFIX; + static const char * const psz_name = "vlc_entry" MODULE_SUFFIX; int (* pf_symbol) ( module_t * p_module ); /* Try to resolve the symbol */ @@ -1599,7 +1643,7 @@ static void CacheLoad( vlc_object_t *p_this ) module_cache_t **pp_cache = 0; int32_t i_file_size, i_marker; - psz_homedir = p_this->p_vlc->psz_homedir; + psz_homedir = p_this->p_libvlc->psz_homedir; if( !psz_homedir ) { msg_Err( p_this, "psz_homedir is null" ); @@ -1614,7 +1658,7 @@ static void CacheLoad( vlc_object_t *p_this ) return; } - if( p_this->p_libvlc->p_module_bank->b_cache_delete ) + if( p_this->p_libvlc_global->p_module_bank->b_cache_delete ) { #if !defined( UNDER_CE ) unlink( psz_filename ); @@ -1703,41 +1747,48 @@ static void CacheLoad( vlc_object_t *p_this ) return; } - p_this->p_libvlc->p_module_bank->i_loaded_cache = 0; + p_this->p_libvlc_global->p_module_bank->i_loaded_cache = 0; fread( &i_cache, sizeof(char), sizeof(i_cache), file ); if( i_cache ) - pp_cache = p_this->p_libvlc->p_module_bank->pp_loaded_cache = + pp_cache = p_this->p_libvlc_global->p_module_bank->pp_loaded_cache = malloc( i_cache * sizeof(void *) ); #define LOAD_IMMEDIATE(a) \ - if( fread( &a, sizeof(char), sizeof(a), file ) != sizeof(a) ) goto error + if( fread( (void *)&a, sizeof(char), sizeof(a), file ) != sizeof(a) ) goto error #define LOAD_STRING(a) \ - { if( fread( &i_size, sizeof(char), sizeof(i_size), file ) \ - != sizeof(i_size) ) goto error; \ - if( i_size && i_size < 16384 ) { \ - a = malloc( i_size ); \ - if( fread( a, sizeof(char), i_size, file ) != (size_t)i_size ) \ - goto error; \ - if( a[i_size-1] ) { \ - free( a ); a = 0; \ - goto error; } \ - } else a = 0; \ - } while(0) - +{ \ + a = NULL; \ + if( ( fread( &i_size, sizeof(i_size), 1, file ) != 1 ) \ + || ( i_size > 16384 ) ) \ + goto error; \ + if( i_size ) { \ + char *psz = malloc( i_size ); \ + if( fread( psz, i_size, 1, file ) != 1 ) { \ + free( psz ); \ + goto error; \ + } \ + if( psz[i_size-1] ) { \ + free( psz ); \ + goto error; \ + } \ + a = psz; \ + } \ +} for( i = 0; i < i_cache; i++ ) { - int16_t i_size; + uint16_t i_size; int i_submodules; pp_cache[i] = malloc( sizeof(module_cache_t) ); - p_this->p_libvlc->p_module_bank->i_loaded_cache++; + p_this->p_libvlc_global->p_module_bank->i_loaded_cache++; /* Load common info */ LOAD_STRING( pp_cache[i]->psz_file ); LOAD_IMMEDIATE( pp_cache[i]->i_time ); LOAD_IMMEDIATE( pp_cache[i]->i_size ); LOAD_IMMEDIATE( pp_cache[i]->b_junk ); + pp_cache[i]->b_used = VLC_FALSE; if( pp_cache[i]->b_junk ) continue; @@ -1797,16 +1848,17 @@ static void CacheLoad( vlc_object_t *p_this ) msg_Warn( p_this, "plugins cache not loaded (corrupted)" ); /* TODO: cleanup */ - p_this->p_libvlc->p_module_bank->i_loaded_cache = 0; + p_this->p_libvlc_global->p_module_bank->i_loaded_cache = 0; fclose( file ); return; } + int CacheLoadConfig( module_t *p_module, FILE *file ) { - int i, j, i_lines; - int16_t i_size; + uint32_t i_lines; + uint16_t i_size; /* Calculate the structure length */ LOAD_IMMEDIATE( p_module->i_config_items ); @@ -1815,16 +1867,21 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) LOAD_IMMEDIATE( i_lines ); /* Allocate memory */ - p_module->p_config = - (module_config_t *)malloc( sizeof(module_config_t) * (i_lines + 1)); - if( p_module->p_config == NULL ) + if (i_lines) { - msg_Err( p_module, "config error: can't duplicate p_config" ); - return VLC_ENOMEM; + p_module->p_config = + (module_config_t *)calloc( i_lines, sizeof(module_config_t) ); + if( p_module->p_config == NULL ) + { + p_module->confsize = 0; + msg_Err( p_module, "config error: can't duplicate p_config" ); + return VLC_ENOMEM; + } } + p_module->confsize = i_lines; /* Do the duplication job */ - for( i = 0; i < i_lines ; i++ ) + for (size_t i = 0; i < i_lines; i++ ) { LOAD_IMMEDIATE( p_module->p_config[i] ); @@ -1833,16 +1890,23 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) LOAD_STRING( p_module->p_config[i].psz_text ); LOAD_STRING( p_module->p_config[i].psz_longtext ); LOAD_STRING( p_module->p_config[i].psz_current ); - LOAD_STRING( p_module->p_config[i].psz_value_orig ); - - p_module->p_config[i].psz_value = - p_module->p_config[i].psz_value_orig ? - strdup( p_module->p_config[i].psz_value_orig ) : 0; - p_module->p_config[i].i_value = p_module->p_config[i].i_value_orig; - p_module->p_config[i].f_value = p_module->p_config[i].f_value_orig; - p_module->p_config[i].i_value_saved = p_module->p_config[i].i_value; - p_module->p_config[i].f_value_saved = p_module->p_config[i].f_value; - p_module->p_config[i].psz_value_saved = 0; + + if (IsConfigStringType (p_module->p_config[i].i_type)) + { + LOAD_STRING (p_module->p_config[i].orig.psz); + p_module->p_config[i].value.psz = + (p_module->p_config[i].orig.psz != NULL) + ? strdup (p_module->p_config[i].orig.psz) : NULL; + p_module->p_config[i].saved.psz = NULL; + } + else + { + memcpy (&p_module->p_config[i].value, &p_module->p_config[i].orig, + sizeof (p_module->p_config[i].value)); + memcpy (&p_module->p_config[i].saved, &p_module->p_config[i].orig, + sizeof (p_module->p_config[i].saved)); + } + p_module->p_config[i].b_dirty = VLC_FALSE; p_module->p_config[i].p_lock = &p_module->object_lock; @@ -1851,6 +1915,7 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) { if( p_module->p_config[i].ppsz_list ) { + int j; p_module->p_config[i].ppsz_list = malloc( (p_module->p_config[i].i_list+1) * sizeof(char *)); if( p_module->p_config[i].ppsz_list ) @@ -1862,6 +1927,7 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) } if( p_module->p_config[i].ppsz_list_text ) { + int j; p_module->p_config[i].ppsz_list_text = malloc( (p_module->p_config[i].i_list+1) * sizeof(char *)); if( p_module->p_config[i].ppsz_list_text ) @@ -1877,7 +1943,7 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) malloc( (p_module->p_config[i].i_list + 1) * sizeof(int) ); if( p_module->p_config[i].pi_list ) { - for( j = 0; j < p_module->p_config[i].i_list; j++ ) + for (int j = 0; j < p_module->p_config[i].i_list; j++) LOAD_IMMEDIATE( p_module->p_config[i].pi_list[j] ); } } @@ -1890,7 +1956,7 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) p_module->p_config[i].ppsz_action_text = malloc( p_module->p_config[i].i_action * sizeof(char *) ); - for( j = 0; j < p_module->p_config[i].i_action; j++ ) + for (int j = 0; j < p_module->p_config[i].i_action; j++) { p_module->p_config[i].ppf_action[j] = 0; LOAD_STRING( p_module->p_config[i].ppsz_action_text[j] ); @@ -1900,8 +1966,6 @@ int CacheLoadConfig( module_t *p_module, FILE *file ) LOAD_IMMEDIATE( p_module->p_config[i].pf_callback ); } - p_module->p_config[i].i_type = CONFIG_HINT_END; - return VLC_SUCCESS; error: @@ -1926,7 +1990,7 @@ static void CacheSave( vlc_object_t *p_this ) module_cache_t **pp_cache; int32_t i_file_size = 0; - psz_homedir = p_this->p_vlc->psz_homedir; + psz_homedir = p_this->p_libvlc->psz_homedir; if( !psz_homedir ) { msg_Err( p_this, "psz_homedir is null" ); @@ -1992,8 +2056,8 @@ static void CacheSave( vlc_object_t *p_this ) i_file_size = ftell( file ); fwrite( &i_file_size, sizeof(char), sizeof(i_file_size), file ); - i_cache = p_this->p_libvlc->p_module_bank->i_cache; - pp_cache = p_this->p_libvlc->p_module_bank->pp_cache; + i_cache = p_this->p_libvlc_global->p_module_bank->i_cache; + pp_cache = p_this->p_libvlc_global->p_module_bank->pp_cache; fwrite( &i_cache, sizeof(char), sizeof(i_cache), file ); @@ -2007,8 +2071,8 @@ static void CacheSave( vlc_object_t *p_this ) for( i = 0; i < i_cache; i++ ) { - int16_t i_size; - int32_t i_submodule; + uint16_t i_size; + uint32_t i_submodule; /* Save common info */ SAVE_STRING( pp_cache[i]->psz_file ); @@ -2041,7 +2105,8 @@ static void CacheSave( vlc_object_t *p_this ) i_submodule = pp_cache[i]->p_module->i_children; SAVE_IMMEDIATE( i_submodule ); - for( i_submodule = 0; i_submodule < pp_cache[i]->p_module->i_children; + for( i_submodule = 0; + i_submodule < (unsigned)pp_cache[i]->p_module->i_children; i_submodule++ ) { module_t *p_module = @@ -2075,19 +2140,14 @@ static void CacheSave( vlc_object_t *p_this ) void CacheSaveConfig( module_t *p_module, FILE *file ) { - int i, j, i_lines = 0; - module_config_t *p_item; - int16_t i_size; + uint32_t i_lines = p_module->confsize; + uint16_t i_size; SAVE_IMMEDIATE( p_module->i_config_items ); SAVE_IMMEDIATE( p_module->i_bool_items ); - - for( p_item = p_module->p_config; p_item->i_type != CONFIG_HINT_END; - p_item++ ) i_lines++; - SAVE_IMMEDIATE( i_lines ); - for( i = 0; i < i_lines ; i++ ) + for (size_t i = 0; i < i_lines ; i++) { SAVE_IMMEDIATE( p_module->p_config[i] ); @@ -2096,29 +2156,30 @@ void CacheSaveConfig( module_t *p_module, FILE *file ) SAVE_STRING( p_module->p_config[i].psz_text ); SAVE_STRING( p_module->p_config[i].psz_longtext ); SAVE_STRING( p_module->p_config[i].psz_current ); - SAVE_STRING( p_module->p_config[i].psz_value_orig ); + if (IsConfigStringType (p_module->p_config[i].i_type)) + SAVE_STRING( p_module->p_config[i].orig.psz ); if( p_module->p_config[i].i_list ) { if( p_module->p_config[i].ppsz_list ) { - for( j = 0; j < p_module->p_config[i].i_list; j++ ) + for (int j = 0; j < p_module->p_config[i].i_list; j++) SAVE_STRING( p_module->p_config[i].ppsz_list[j] ); } if( p_module->p_config[i].ppsz_list_text ) { - for( j = 0; j < p_module->p_config[i].i_list; j++ ) + for (int j = 0; j < p_module->p_config[i].i_list; j++) SAVE_STRING( p_module->p_config[i].ppsz_list_text[j] ); } if( p_module->p_config[i].pi_list ) { - for( j = 0; j < p_module->p_config[i].i_list; j++ ) + for (int j = 0; j < p_module->p_config[i].i_list; j++) SAVE_IMMEDIATE( p_module->p_config[i].pi_list[j] ); } } - for( j = 0; j < p_module->p_config[i].i_action; j++ ) + for (int j = 0; j < p_module->p_config[i].i_action; j++) SAVE_STRING( p_module->p_config[i].ppsz_action_text[j] ); SAVE_IMMEDIATE( p_module->p_config[i].pf_callback ); @@ -2134,8 +2195,8 @@ static char *CacheName( void ) /* Code int size, pointer size and endianness in the filename */ int32_t x = 0xbe00001e; - sprintf( psz_cachename, "plugins-%.2x%.2x%.2x.dat", sizeof(int), - sizeof(void *), (unsigned int)((unsigned char *)&x)[0] ); + sprintf( psz_cachename, "plugins-%.2x%.2x%.2x.dat", (int)sizeof(int), + (int)sizeof(void *), (unsigned int)((unsigned char *)&x)[0] ); return psz_cachename; } @@ -2170,7 +2231,7 @@ static void CacheMerge( vlc_object_t *p_this, module_t *p_cache, } /***************************************************************************** - * FindPluginCache: finds the cache entry corresponding to a file + * CacheFind: finds the cache entry corresponding to a file *****************************************************************************/ static module_cache_t *CacheFind( vlc_object_t *p_this, char *psz_file, int64_t i_time, int64_t i_size ) @@ -2178,8 +2239,8 @@ static module_cache_t *CacheFind( vlc_object_t *p_this, char *psz_file, module_cache_t **pp_cache; int i_cache, i; - pp_cache = p_this->p_libvlc->p_module_bank->pp_loaded_cache; - i_cache = p_this->p_libvlc->p_module_bank->i_loaded_cache; + pp_cache = p_this->p_libvlc_global->p_module_bank->pp_loaded_cache; + i_cache = p_this->p_libvlc_global->p_module_bank->i_loaded_cache; for( i = 0; i < i_cache; i++ ) {