diff options
| author | Robert Müller <robert.mueller@uni-siegen.de> | 2021-10-23 14:14:36 +0200 |
|---|---|---|
| committer | Robert Müller <robert.mueller@uni-siegen.de> | 2021-11-28 11:11:23 +0100 |
| commit | f3b78546960d9b3899978e7d0ce2fee821e5dfe9 (patch) | |
| tree | 6fe2f678fe58498b2400177bbcbd26b50d9aa29b | |
| parent | b20d8e436e98653dc291319d557255317b276b55 (diff) | |
fix default filter initialization, fix potential OOB access to filter array
| -rw-r--r-- | src/game/client/components/menus.h | 8 | ||||
| -rw-r--r-- | src/game/client/components/menus_browser.cpp | 7 |
2 files changed, 10 insertions, 5 deletions
diff --git a/src/game/client/components/menus.h b/src/game/client/components/menus.h index 3a82bec73..3d67bafea 100644 --- a/src/game/client/components/menus.h +++ b/src/game/client/components/menus.h @@ -630,17 +630,17 @@ private: static CColumn ms_aBrowserCols[NUM_BROWSER_COLS]; static CColumn ms_aDemoCols[NUM_DEMO_COLS]; - CBrowserFilter* GetSelectedBrowserFilter() + CBrowserFilter *GetSelectedBrowserFilter() { const int Tab = ServerBrowser()->GetType(); - if(m_aSelectedFilters[Tab] == -1) + if(m_aSelectedFilters[Tab] < 0 || m_aSelectedFilters[Tab] >= m_lFilters.size()) return 0; return &m_lFilters[m_aSelectedFilters[Tab]]; } - const CServerInfo* GetSelectedServerInfo() + const CServerInfo *GetSelectedServerInfo() { - CBrowserFilter* pSelectedFilter = GetSelectedBrowserFilter(); + CBrowserFilter *pSelectedFilter = GetSelectedBrowserFilter(); if(!pSelectedFilter) return 0; const int Tab = ServerBrowser()->GetType(); diff --git a/src/game/client/components/menus_browser.cpp b/src/game/client/components/menus_browser.cpp index 8231ef856..cfe7efe2b 100644 --- a/src/game/client/components/menus_browser.cpp +++ b/src/game/client/components/menus_browser.cpp @@ -408,7 +408,12 @@ void CMenus::InitDefaultFilters() m_lFilters.add(CBrowserFilter(CBrowserFilter::FILTER_ALL, Localize("All"), ServerBrowser())); // expand the all filter tab by default if(UseDefaultFilters) - m_lFilters[m_lFilters.size()-1].Switch(); + { + const int AllFilterIndex = m_lFilters.size()-1; + for(unsigned i = 0; i < IServerBrowser::NUM_TYPES; ++i) + m_aSelectedFilters[i] = AllFilterIndex; // default to "all" if not set + m_lFilters[AllFilterIndex].Switch(); + } } // 1 = browser entry click, 2 = server info click |