From cbd0dc571bba77d80a5fea1709cc3fbd8bd039f9 Mon Sep 17 00:00:00 2001 From: Maxime Coste Date: Wed, 11 Nov 2020 17:22:35 +1100 Subject: [PATCH] Fix crash when displaying a menu on a too small terminal The code could position the menu at a negative line leading to invalid array accesses. --- src/terminal_ui.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/terminal_ui.cc b/src/terminal_ui.cc index 14cf330e..34d8eee3 100644 --- a/src/terminal_ui.cc +++ b/src/terminal_ui.cc @@ -26,6 +26,8 @@ using std::max; void TerminalUI::Window::create(const DisplayCoord& p, const DisplayCoord& s) { + kak_assert(p.line >= 0 and p.column >= 0); + kak_assert(s.line >= 0 and s.column >= 0); pos = p; size = s; lines.resize((int)size.line); @@ -1005,7 +1007,7 @@ void TerminalUI::menu_show(ConstArrayView items, } else if (not is_inline) line = m_status_on_top ? 1_line : m_dimensions.line - height; - else if (line + height > m_dimensions.line) + else if (line + height > m_dimensions.line and anchor.line >= height) line = anchor.line - height; const auto width = is_search ? m_dimensions.column - m_dimensions.column / 2