diff options
| author | William Wilgus <wilgus.william@gmail.com> | 2022-12-31 07:01:04 -0500 |
|---|---|---|
| committer | William Wilgus <me.theuser@yahoo.com> | 2022-12-31 07:04:18 -0500 |
| commit | 855540e1f5af2836f4b6252a766fec13233e2fe5 (patch) | |
| tree | 04f5cf24e1116e5085dcfb6054d63bc1e8f73b99 | |
| parent | 1773e56447785931968d46002b73307c1fb3a35a (diff) | |
| download | rockbox-855540e1f5.tar.gz rockbox-855540e1f5.zip | |
[BugFix] keyremap browse missing root, entry count off by 1
browse_context was missing its root causing a NULL deref
Export User Keys was looking for at least 4 entries when it only needed 3
Change-Id: If92ecccb36bd54e1b850ac2a969fe262b5d7fd14
| -rw-r--r-- | apps/plugins/keyremap.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/apps/plugins/keyremap.c b/apps/plugins/keyremap.c index 3923cc40d0..ccf446d279 100644 --- a/apps/plugins/keyremap.c +++ b/apps/plugins/keyremap.c @@ -521,9 +521,11 @@ static int keyremap_export_current(char *filenamebuf, size_t bufsz) int entry_count = ctx_data.ctx_count + ctx_data.act_count + 1;;/* (ctx_count + ctx_count + act_count + 1) */ - if (entry_count <= 3) + if (entry_count < 3) + { + logf("%s: Not enough entries", __func__); return 0; - + } int fd = rb->open(filenamebuf, O_WRONLY | O_CREAT | O_TRUNC, 0666); if (fd < 0) @@ -620,6 +622,7 @@ static void keyremap_import_user_keys(void) .icon = Icon_Plugin, .buf = buf, .bufsize = sizeof(buf), + .root = "/", }; if (rb->rockbox_browse(&browse) == GO_TO_PREVIOUS) |