Mirror/Momentum-Firmware
1
0
Fork 0
mirror of https://github.com/Next-Flip/Momentum-Firmware.git synced 2026-05-13 05:08:35 -07:00
Code Issues Packages Projects Releases Wiki Activity

Temp fix menu edge case crash in esubghz chat

Browse Source
This commit is contained in:
Willy-JL
2024-04-19 08:28:40 +01:00
parent c4c1870875
commit a2fc553bef
2 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
applications/services/gui/icon_animation.c
View File

@@ -70,7 +70,7 @@ uint8_t icon_animation_get_height(const IconAnimation* instance) {
void icon_animation_start(IconAnimation* instance) {
furi_check(instance);
if(!instance->animating && instance->icon->frame_rate && instance->icon->frame_count > 1) {
if(!instance->animating) {
instance->animating = true;
furi_assert(instance->icon->frame_rate);
furi_check(

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

@@ -498,9 +498,14 @@ static void menu_exit(void* context) {
menu->view,
MenuModel * model,
{
MenuItem* item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_stop(item->icon);
// If menu_reset() is called before view exit, model->items is reset
// But for some reason, even with size 0, array get() returns a non-null pointer?
// MLIB docs have no mention of out of bounds condition, seems weird
if(model->position < MenuItemArray_size(model->items)) {
MenuItem* item = MenuItemArray_get(model->items, model->position);
if(item && item->icon) {
icon_animation_stop(item->icon);
}
}
},
false);