summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJonathan Gordon <rockbox@jdgordon.info>2011-09-15 09:58:28 +0000
committerJonathan Gordon <rockbox@jdgordon.info>2011-09-15 09:58:28 +0000
commit0cfd7a388874f3a95662bcd349bd371d497e59f3 (patch)
tree91a17531fe001ce2f0e760b5584ff1365420c0b1
parent3f08357f2b89277d6196a46b099f7143ef14f223 (diff)
downloadrockbox-0cfd7a388874f3a95662bcd349bd371d497e59f3.zip
rockbox-0cfd7a388874f3a95662bcd349bd371d497e59f3.tar.gz
rockbox-0cfd7a388874f3a95662bcd349bd371d497e59f3.tar.bz2
rockbox-0cfd7a388874f3a95662bcd349bd371d497e59f3.tar.xz
Fix FS#12278 - saving a full .cfg file would crash
git-svn-id: svn://svn.rockbox.org/rockbox/trunk@30553 a1c6a512-1295-4272-9138-f99709370657
-rw-r--r--apps/settings_list.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/apps/settings_list.c b/apps/settings_list.c
index 27726dd..2f452e6 100644
--- a/apps/settings_list.c
+++ b/apps/settings_list.c
@@ -429,6 +429,8 @@ static int find_setting_by_name(char*name)
{
int i = 0;
const struct settings_list *setting;
+ if (!strcmp(name, "-"))
+ return -1;
while (i<nb_settings)
{
setting = &settings[i];
@@ -446,7 +448,13 @@ static void qs_load_from_cfg(void* var, char*value)
}
static char* qs_write_to_cfg(void* setting, char*buf, int buf_len)
{
- const struct settings_list *var = &settings[*(int*)setting];
+ int index = *(int*)setting;
+ if (index < 0 || index >= nb_settings - 1)
+ {
+ strlcpy(buf, "-", buf_len);
+ return buf;
+ }
+ const struct settings_list *var = &settings[index];
strlcpy(buf, var->cfg_name, buf_len);
return buf;
}