From 01e3c0222527577be5c29f4cd12a302dc8e8b312 Mon Sep 17 00:00:00 2001 From: Thilo Graf Date: Wed, 11 Apr 2018 16:12:03 +0200 Subject: [PATCH] src/gui/osd_setup.cpp: use CMenuForwarder instead CMenuDForwarder This should avoid possible segfaults when user is calling color setup. This behavior was observed if start wizard has been completed and after that user wants to change color settings. If target object is a singelton, this should not be an issue. --- src/gui/osd_setup.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gui/osd_setup.cpp b/src/gui/osd_setup.cpp index 9102bea58..cec557b0a 100644 --- a/src/gui/osd_setup.cpp +++ b/src/gui/osd_setup.cpp @@ -759,7 +759,7 @@ void COsdSetup::showOsdMenueColorSetup(CMenuWidget *menu_colors) { menu_colors->addIntroItems(LOCALE_COLORMENU_MENUCOLORS); - CMenuForwarder * mf = new CMenuDForwarder(LOCALE_COLORMENU_THEMESELECT, true, NULL, CThemes::getInstance(), NULL, CRCInput::RC_red); + CMenuForwarder * mf = new CMenuForwarder(LOCALE_COLORMENU_THEMESELECT, true, NULL, CThemes::getInstance(), NULL, CRCInput::RC_red); mf->setHint("", LOCALE_MENU_HINT_THEME); menu_colors->addItem(mf);