DarkFlippers/unleashed-firmware
1
1
Fork 0
mirror of https://github.com/DarkFlippers/unleashed-firmware.git synced 2024-12-24 13:52:38 +03:00
Code Issues Projects Releases Wiki Activity

GUI: Fix array out of bounds in menu exit (#3604)

Browse Source 
* GUI: Fix array out of bounds in menu exit
* Gui: fix incorrect empty menu handling
* Gui: add missing item check in menu ok handling
* Gui: remove dead code from menu module

Co-authored-by: あく <alleteam@gmail.com>
This commit is contained in:
WillyJL 2024-04-19 15:10:48 +01:00 committed by GitHub
parent 286fd796e7
commit 9bef2aaec6
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 24 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
applications/services/gui/modules/menu.c
View File

@ -44,25 +44,19 @@ static void menu_draw_callback(Canvas* canvas, void* _model) {
canvas_set_font(canvas, FontSecondary); canvas_set_font(canvas, FontSecondary);
shift_position = (0 + position + items_count - 1) % items_count; shift_position = (0 + position + items_count - 1) % items_count;
item = MenuItemArray_get(model->items, shift_position); item = MenuItemArray_get(model->items, shift_position);
if(item->icon) {
canvas_draw_icon_animation(canvas, 4, 3, item->icon); canvas_draw_icon_animation(canvas, 4, 3, item->icon);
}
canvas_draw_str(canvas, 22, 14, item->label); canvas_draw_str(canvas, 22, 14, item->label);
// Second line main // Second line main
canvas_set_font(canvas, FontPrimary); canvas_set_font(canvas, FontPrimary);
shift_position = (1 + position + items_count - 1) % items_count; shift_position = (1 + position + items_count - 1) % items_count;
item = MenuItemArray_get(model->items, shift_position); item = MenuItemArray_get(model->items, shift_position);
if(item->icon) {
canvas_draw_icon_animation(canvas, 4, 25, item->icon); canvas_draw_icon_animation(canvas, 4, 25, item->icon);
}
canvas_draw_str(canvas, 22, 36, item->label); canvas_draw_str(canvas, 22, 36, item->label);
// Third line // Third line
canvas_set_font(canvas, FontSecondary); canvas_set_font(canvas, FontSecondary);
shift_position = (2 + position + items_count - 1) % items_count; shift_position = (2 + position + items_count - 1) % items_count;
item = MenuItemArray_get(model->items, shift_position); item = MenuItemArray_get(model->items, shift_position);
if(item->icon) {
canvas_draw_icon_animation(canvas, 4, 47, item->icon); canvas_draw_icon_animation(canvas, 4, 47, item->icon);
}
canvas_draw_str(canvas, 22, 58, item->label); canvas_draw_str(canvas, 22, 58, item->label);
// Frame and scrollbar // Frame and scrollbar
elements_frame(canvas, 0, 21, 128 - 5, 21); elements_frame(canvas, 0, 21, 128 - 5, 21);
@ -107,8 +101,8 @@ static void menu_enter(void* context) {
menu->view, menu->view,
MenuModel * model, MenuModel * model,
{ {
if(MenuItemArray_size(model->items)) {
MenuItem* item = MenuItemArray_get(model->items, model->position); MenuItem* item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_start(item->icon); icon_animation_start(item->icon);
} }
}, },
@ -121,8 +115,8 @@ static void menu_exit(void* context) {
menu->view, menu->view,
MenuModel * model, MenuModel * model,
{ {
if(MenuItemArray_size(model->items)) {
MenuItem* item = MenuItemArray_get(model->items, model->position); MenuItem* item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_stop(item->icon); icon_animation_stop(item->icon);
} }
}, },
@ -230,10 +224,9 @@ static void menu_process_up(Menu* menu) {
menu->view, menu->view,
MenuModel * model, MenuModel * model,
{ {
if(MenuItemArray_size(model->items)) {
MenuItem* item = MenuItemArray_get(model->items, model->position); MenuItem* item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_stop(item->icon); icon_animation_stop(item->icon);
}
if(model->position > 0) { if(model->position > 0) {
model->position--; model->position--;
@ -242,7 +235,6 @@ static void menu_process_up(Menu* menu) {
} }
item = MenuItemArray_get(model->items, model->position); item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_start(item->icon); icon_animation_start(item->icon);
} }
}, },
@ -254,10 +246,9 @@ static void menu_process_down(Menu* menu) {
menu->view, menu->view,
MenuModel * model, MenuModel * model,
{ {
if(MenuItemArray_size(model->items)) {
MenuItem* item = MenuItemArray_get(model->items, model->position); MenuItem* item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_stop(item->icon); icon_animation_stop(item->icon);
}
if(model->position < MenuItemArray_size(model->items) - 1) { if(model->position < MenuItemArray_size(model->items) - 1) {
model->position++; model->position++;
@ -266,7 +257,6 @@ static void menu_process_down(Menu* menu) {
} }
item = MenuItemArray_get(model->items, model->position); item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_start(item->icon); icon_animation_start(item->icon);
} }
}, },
@ -279,7 +269,7 @@ static void menu_process_ok(Menu* menu) {
menu->view, menu->view,
MenuModel * model, MenuModel * model,
{ {
if(model->position < MenuItemArray_size(model->items)) { if(MenuItemArray_size(model->items)) {
item = MenuItemArray_get(model->items, model->position); item = MenuItemArray_get(model->items, model->position);
} }
}, },