diff options
Diffstat (limited to 'apps')
-rw-r--r-- | apps/codecs.c | 101 | ||||
-rw-r--r-- | apps/plugin.c | 102 | ||||
-rw-r--r-- | apps/plugins/plugin_crt0.c | 6 |
3 files changed, 135 insertions, 74 deletions
diff --git a/apps/codecs.c b/apps/codecs.c index b072c65f40..9e77dd9099 100644 --- a/apps/codecs.c +++ b/apps/codecs.c @@ -28,7 +28,6 @@ #include <ctype.h> #include <stdarg.h> #include "string-extra.h" -#include "load_code.h" #include "debug.h" #include "button.h" #include "dir.h" @@ -75,13 +74,26 @@ size_t codec_size; extern void* plugin_get_audio_buffer(size_t *buffer_size); -#if (CONFIG_PLATFORM & PLATFORM_NATIVE) && defined(HAVE_RECORDING) #undef open static int open(const char* pathname, int flags, ...) { +#if (CONFIG_PLATFORM & PLATFORM_HOSTED) + int fd; + if (flags & O_CREAT) + { + va_list ap; + va_start(ap, flags); + fd = sim_open(pathname, flags, va_arg(ap, unsigned int)); + va_end(ap); + } + else + fd = sim_open(pathname, flags); + + return fd; +#else return file_open(pathname, flags); -} #endif +} struct codec_api ci = { 0, /* filesize */ @@ -185,46 +197,62 @@ void codec_get_full_path(char *path, const char *codec_root_fn) CODECS_DIR, codec_root_fn); } -static int codec_load_ram(void *handle, struct codec_api *api) +static int codec_load_ram(int size, struct codec_api *api) { - struct codec_header *hdr = lc_get_header(handle); + struct codec_header *hdr; int status; - - if (hdr == NULL +#if (CONFIG_PLATFORM & PLATFORM_NATIVE) + hdr = (struct codec_header *)codecbuf; + + if (size <= (signed)sizeof(struct codec_header) || (hdr->magic != CODEC_MAGIC #ifdef HAVE_RECORDING && hdr->magic != CODEC_ENC_MAGIC #endif ) || hdr->target_id != TARGET_ID -#if (CONFIG_PLATFORM & PLATFORM_NATIVE) || hdr->load_addr != codecbuf - || hdr->end_addr > codecbuf + CODEC_SIZE -#endif - ) + || hdr->end_addr > codecbuf + CODEC_SIZE) { logf("codec header error"); - lc_close(handle); return CODEC_ERROR; } + codec_size = hdr->end_addr - codecbuf; + +#elif (CONFIG_PLATFORM & PLATFORM_HOSTED) + void *pd; + + hdr = sim_codec_load_ram(codecbuf, size, &pd); + + if (pd == NULL) + return CODEC_ERROR; + + if (hdr == NULL + || (hdr->magic != CODEC_MAGIC +#ifdef HAVE_RECORDING + && hdr->magic != CODEC_ENC_MAGIC +#endif + ) + || hdr->target_id != TARGET_ID) { + sim_codec_close(pd); + return CODEC_ERROR; + } + + codec_size = codecbuf - codecbuf; + +#endif /* CONFIG_PLATFORM */ if (hdr->api_version > CODEC_API_VERSION || hdr->api_version < CODEC_MIN_API_VERSION) { - logf("codec api version error"); - lc_close(handle); + sim_codec_close(pd); return CODEC_ERROR; } -#if (CONFIG_PLATFORM & PLATFORM_NATIVE) - codec_size = hdr->end_addr - codecbuf; -#else - codec_size = 0; -#endif - *(hdr->api) = api; + cpucache_invalidate(); status = hdr->entry_point(); - lc_close(handle); + sim_codec_close(pd); return status; } @@ -232,37 +260,36 @@ static int codec_load_ram(void *handle, struct codec_api *api) int codec_load_buf(unsigned int hid, struct codec_api *api) { int rc; - void *handle; rc = bufread(hid, CODEC_SIZE, codecbuf); if (rc < 0) { logf("error loading codec"); return CODEC_ERROR; } - handle = lc_open_from_mem(codecbuf, rc); - if (handle == NULL) - { - logf("error loading codec"); - return CODEC_ERROR; - } - api->discard_codec(); - return codec_load_ram(handle, api); + return codec_load_ram(rc, api); } int codec_load_file(const char *plugin, struct codec_api *api) { char path[MAX_PATH]; - void *handle; + int fd; + int rc; codec_get_full_path(path, plugin); - - handle = lc_open(path, codecbuf, CODEC_SIZE); - - if (handle == NULL) { - logf("Codec load error"); + + fd = open(path, O_RDONLY); + if (fd < 0) { + logf("Codec load error:%d", fd); splashf(HZ*2, "Couldn't load codec: %s", path); + return fd; + } + + rc = read(fd, &codecbuf[0], CODEC_SIZE); + close(fd); + if (rc <= 0) { + logf("Codec read error"); return CODEC_ERROR; } - return codec_load_ram(handle, api); + return codec_load_ram((size_t)rc, api); } diff --git a/apps/plugin.c b/apps/plugin.c index 53a05bf527..cc540cd988 100644 --- a/apps/plugin.c +++ b/apps/plugin.c @@ -42,7 +42,6 @@ #include "errno.h" #include "diacritic.h" #include "filefuncs.h" -#include "load_code.h" #if CONFIG_CHARGING #include "power.h" @@ -76,19 +75,21 @@ static unsigned int open_files; #if (CONFIG_PLATFORM & PLATFORM_HOSTED) static unsigned char pluginbuf[PLUGIN_BUFFER_SIZE]; +void *sim_plugin_load(char *plugin, void **pd); +void sim_plugin_close(void *pd); void sim_lcd_ex_init(unsigned long (*getpixel)(int, int)); void sim_lcd_ex_update_rect(int x, int y, int width, int height); #else +#define sim_plugin_close(x) extern unsigned char pluginbuf[]; #include "bitswap.h" #endif /* for actual plugins only, not for codecs */ +static bool plugin_loaded = false; static int plugin_size = 0; static bool (*pfn_tsr_exit)(bool reenter) = NULL; /* TSR exit callback */ static char current_plugin[MAX_PATH]; -/* NULL if no plugin is loaded, otherwise the handle that lc_open() returned */ -static void *current_plugin_handle; char *plugin_get_current_filename(void); @@ -727,60 +728,98 @@ int plugin_load(const char* plugin, const void* parameter) { int rc, i; struct plugin_header *hdr; +#if (CONFIG_PLATFORM & PLATFORM_HOSTED) + void *pd; +#else /* PLATFOR_NATIVE */ + int fd; + ssize_t readsize; +#if NUM_CORES > 1 + unsigned my_core; +#endif +#endif /* CONFIG_PLATFORM */ #if LCD_DEPTH > 1 fb_data* old_backdrop; #endif - if (current_plugin_handle && pfn_tsr_exit) - { /* if we have a resident old plugin and a callback */ + if (pfn_tsr_exit != NULL) /* if we have a resident old plugin: */ + { if (pfn_tsr_exit(!strcmp(current_plugin, plugin)) == false ) { /* not allowing another plugin to load */ return PLUGIN_OK; } - lc_close(current_plugin_handle); - current_plugin_handle = pfn_tsr_exit = NULL; + pfn_tsr_exit = NULL; + plugin_loaded = false; } splash(0, ID2P(LANG_WAIT)); strcpy(current_plugin, plugin); - current_plugin_handle = lc_open(plugin, pluginbuf, PLUGIN_BUFFER_SIZE); - if (current_plugin_handle == NULL) { +#if (CONFIG_PLATFORM & PLATFORM_HOSTED) + hdr = sim_plugin_load((char *)plugin, &pd); + if (pd == NULL) { splashf(HZ*2, str(LANG_PLUGIN_CANT_OPEN), plugin); return -1; } + if (hdr == NULL + || hdr->magic != PLUGIN_MAGIC + || hdr->target_id != TARGET_ID) { + sim_plugin_close(pd); + splash(HZ*2, str(LANG_PLUGIN_WRONG_MODEL)); + return -1; + } + if (hdr->api_version > PLUGIN_API_VERSION + || hdr->api_version < PLUGIN_MIN_API_VERSION) { + sim_plugin_close(pd); + splash(HZ*2, str(LANG_PLUGIN_WRONG_VERSION)); + return -1; + } +#else + fd = open(plugin, O_RDONLY); + if (fd < 0) { + splashf(HZ*2, str(LANG_PLUGIN_CANT_OPEN), plugin); + return fd; + } +#if NUM_CORES > 1 + /* Make sure COP cache is flushed and invalidated before loading */ + my_core = switch_core(CURRENT_CORE ^ 1); + cpucache_invalidate(); + switch_core(my_core); +#endif - hdr = lc_get_header(current_plugin_handle); + readsize = read(fd, pluginbuf, PLUGIN_BUFFER_SIZE); + close(fd); - if (hdr == NULL + if (readsize < 0) { + splashf(HZ*2, str(LANG_READ_FAILED), plugin); + return -1; + } + hdr = (struct plugin_header *)pluginbuf; + + if ((unsigned)readsize <= sizeof(struct plugin_header) || hdr->magic != PLUGIN_MAGIC || hdr->target_id != TARGET_ID -#if (CONFIG_PLATFORM & PLATFORM_NATIVE) || hdr->load_addr != pluginbuf - || hdr->end_addr > pluginbuf + PLUGIN_BUFFER_SIZE -#endif - ) - { - lc_close(current_plugin_handle); + || hdr->end_addr > pluginbuf + PLUGIN_BUFFER_SIZE) { splash(HZ*2, str(LANG_PLUGIN_WRONG_MODEL)); return -1; } if (hdr->api_version > PLUGIN_API_VERSION - || hdr->api_version < PLUGIN_MIN_API_VERSION) - { - lc_close(current_plugin_handle); + || hdr->api_version < PLUGIN_MIN_API_VERSION) { splash(HZ*2, str(LANG_PLUGIN_WRONG_VERSION)); return -1; } -#if (CONFIG_PLATFORM & PLATFORM_NATIVE) plugin_size = hdr->end_addr - pluginbuf; -#else - plugin_size = 0; + + /* zero out bss area only, above guards end of pluginbuf */ + if (plugin_size > readsize) + memset(pluginbuf + readsize, 0, plugin_size - readsize); #endif *(hdr->api) = &rockbox_api; + plugin_loaded = true; + #if defined HAVE_LCD_BITMAP && LCD_DEPTH > 1 old_backdrop = lcd_get_backdrop(); @@ -795,6 +834,8 @@ int plugin_load(const char* plugin, const void* parameter) FOR_NB_SCREENS(i) viewportmanager_theme_enable(i, false, NULL); + + cpucache_invalidate(); #ifdef HAVE_TOUCHSCREEN touchscreen_set_mode(TOUCHSCREEN_BUTTON); @@ -806,12 +847,6 @@ int plugin_load(const char* plugin, const void* parameter) rc = hdr->entry_point(parameter); - if (!pfn_tsr_exit) - { /* close handle if plugin is no tsr one */ - lc_close(current_plugin_handle); - current_plugin_handle = NULL; - } - /* Go back to the global setting in case the plugin changed it */ #ifdef HAVE_TOUCHSCREEN touchscreen_set_mode(global_settings.touch_mode); @@ -852,8 +887,11 @@ int plugin_load(const char* plugin, const void* parameter) FOR_NB_SCREENS(i) viewportmanager_theme_undo(i, false); + if (pfn_tsr_exit == NULL) + plugin_loaded = false; + #ifdef HAVE_PLUGIN_CHECK_OPEN_CLOSE - if(open_files != 0 && !current_plugin_handle) + if(open_files != 0 && !plugin_loaded) { int fd; logf("Plugin '%s' leaks file handles", plugin); @@ -871,6 +909,8 @@ int plugin_load(const char* plugin, const void* parameter) } #endif + sim_plugin_close(pd); + if (rc == PLUGIN_ERROR) splash(HZ*2, str(LANG_PLUGIN_ERROR)); @@ -883,7 +923,7 @@ void* plugin_get_buffer(size_t *buffer_size) { int buffer_pos; - if (current_plugin_handle) + if (plugin_loaded) { if (plugin_size >= PLUGIN_BUFFER_SIZE) return NULL; diff --git a/apps/plugins/plugin_crt0.c b/apps/plugins/plugin_crt0.c index e34124c5a2..536eccaffa 100644 --- a/apps/plugins/plugin_crt0.c +++ b/apps/plugins/plugin_crt0.c @@ -32,8 +32,6 @@ PLUGIN_HEADER #define EXIT_MAGIC 0x0CDEBABE extern enum plugin_status plugin_start(const void*); -extern unsigned char plugin_bss_start[]; -extern unsigned char plugin_end_addr[]; static jmp_buf __exit_env; /* only 1 atexit handler for now, chain in the exit handler if you need more */ @@ -63,10 +61,6 @@ enum plugin_status plugin__start(const void *param) int exit_ret; enum plugin_status ret; - /* zero out the bss section */ -#if (CONFIG_PLATFORM & PLATFORM_NATIVE) - rb->memset(plugin_bss_start, 0, plugin_end_addr - plugin_bss_start); -#endif /* we come back here if exit() was called or the plugin returned normally */ exit_ret = setjmp(__exit_env); if (exit_ret == 0) |