diff options
author | John Mark Bell <jmb@netsurf-browser.org> | 2011-03-23 20:59:43 +0000 |
---|---|---|
committer | John Mark Bell <jmb@netsurf-browser.org> | 2011-03-23 20:59:43 +0000 |
commit | 8ea109784c9c16817133be84421711f22e859281 (patch) | |
tree | ddb6e78e96055de9956832c6c629c0311ec681d6 | |
parent | 237a62a9e772e9fdfe72b5fac6078c1e975ecfc2 (diff) | |
download | netsurf-8ea109784c9c16817133be84421711f22e859281.tar.gz netsurf-8ea109784c9c16817133be84421711f22e859281.tar.bz2 |
Fix bug #3238151: reinstate filename sanitisation and extension stripping
svn path=/trunk/netsurf/; revision=12127
-rw-r--r-- | riscos/download.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/riscos/download.c b/riscos/download.c index e87436fe7..eade5a176 100644 --- a/riscos/download.c +++ b/riscos/download.c @@ -228,7 +228,7 @@ struct gui_download_window *gui_download_window_create(download_context *ctx, url_func_result res; char *local_path; utf8_convert_ret err; - size_t i; + size_t i, last_dot; dw = malloc(sizeof *dw); if (!dw) { @@ -354,9 +354,18 @@ struct gui_download_window *gui_download_window_create(download_context *ctx, return 0; } - for (i = 0; filename[i] != '\0'; i++) - if (filename[i] == '.') + for (i = 0, last_dot = (size_t) -1; filename[i] != '\0'; i++) { + const char c = filename[i]; + + if (c == '.') { + last_dot = i; filename[i] = '/'; + } else if (c <= ' ' || strchr(":*#$&@^%\\", c) != NULL) + filename[i] = '_'; + } + + if (option_strip_extensions && last_dot != (size_t) -1) + filename[last_dot] = '\0'; if (download_dir != NULL && strlen(download_dir) > 0) snprintf(dw->path, RO_DOWNLOAD_MAX_PATH_LEN, "%s.%s", |