From 7790e52a6d3c0f6fbaf45f581f0fb98f78247af6 Mon Sep 17 00:00:00 2001 From: Owen Cochell Date: Thu, 10 Apr 2025 02:20:21 -0400 Subject: [PATCH] misc: fix crash on Linux due to zero length addnstr call --- .../views/game_view/progress_bar_window.py | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/pqcli/ui/curses/views/game_view/progress_bar_window.py b/pqcli/ui/curses/views/game_view/progress_bar_window.py index 8ac0a10..cd18c0c 100644 --- a/pqcli/ui/curses/views/game_view/progress_bar_window.py +++ b/pqcli/ui/curses/views/game_view/progress_bar_window.py @@ -65,13 +65,16 @@ class ProgressBarWindow(WindowWrapper): text = self._progress_title x = max(0, (self.getmaxyx()[1] - len(text)) // 2) - self._progress_bar_win.addnstr( - 0, - x, - text, - min(len(text), self._progress_bar_win.getmaxyx()[1]), - ) - + # Determine the number of characters to write + nwrite = min(len(text), self._progress_bar_win.getmaxyx()[1]) + # The amount MUST be greater than zero to avoid linux errors + if nwrite > 0: + self._progress_bar_win.addnstr( + 0, + x, + text, + nwrite, + ) self._progress_bar.set_position(self._cur_pos, self._max_pos) self._progress_bar_win.noutrefresh()