Reset SIGHUP handler on NCursesUI destruction

SIGHUP handler could trigger crashes if the EventManager was already destructed when it was triggered. Fixes #3288
mawww · Jan 11, 2020 · b85365b · b85365b
1 parent fe80502
commit b85365b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/ncurses_ui.cc b/src/ncurses_ui.cc
@@ -364,7 +364,7 @@ NCursesUI::~NCursesUI()
     endwin();
     tcsetattr(STDIN_FILENO, TCSAFLUSH, &m_original_termios);
     set_signal_handler(SIGWINCH, SIG_DFL);
-    set_signal_handler(SIGCONT, SIG_DFL);
+    set_signal_handler(SIGHUP, SIG_DFL);
     set_signal_handler(SIGTSTP, SIG_DFL);
 }
 
@@ -567,7 +567,7 @@ Optional<Key> NCursesUI::get_next_key()
     if (sighup_raised)
     {
         set_signal_handler(SIGWINCH, SIG_DFL);
-        set_signal_handler(SIGCONT, SIG_DFL);
+        set_signal_handler(SIGHUP, SIG_DFL);
         if (m_window)
             m_window.destroy();
         m_stdin_watcher.disable();