From: Patrick Palka <patrick@parcs.ath.cx>
Date: Tue, 28 Apr 2015 01:37:48 +0000 (-0400)
Subject: Avoid race condition when handling a SIGWINCH signal
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=1e04046d0b7e0f87bc6e2fefd966c9dd89d2ce13;p=binutils-gdb.git

Avoid race condition when handling a SIGWINCH signal

The control variable win_resized must be cleared before responding to
it.

Otherwise there is a small window where another SIGWINCH might occur in
between the handling of an earlier SIGWINCH and the clearing of
win_resized, at which point win_resized would be set (again) by the
signal handler.  Shortly thereafter we would clear win_resized even
though we only handled the earlier SIGWINCH but not the latest one.
This chain of events is all avoided if we clear win_resized first.

gdb/ChangeLog:

	* tui/tui-win.c (tui_async_resize_screen): Clear win_resized
	first before resizing the window.
	* tui.c (tui_enable): Likewise.
---

diff --git a/gdb/tui/tui-win.c b/gdb/tui/tui-win.c
index 683c0eda33e..8c0685bd41d 100644
--- a/gdb/tui/tui-win.c
+++ b/gdb/tui/tui-win.c
@@ -858,10 +858,10 @@ tui_async_resize_screen (gdb_client_data arg)
     }
   else
     {
+      tui_set_win_resized_to (FALSE);
       tui_resize_all ();
       tui_refresh_all_win ();
       tui_update_gdb_sizes ();
-      tui_set_win_resized_to (FALSE);
       tui_redisplay_readline ();
     }
 }
diff --git a/gdb/tui/tui.c b/gdb/tui/tui.c
index 9fac06e51cc..b04e106e5c4 100644
--- a/gdb/tui/tui.c
+++ b/gdb/tui/tui.c
@@ -492,8 +492,8 @@ tui_enable (void)
      window.  */
   if (tui_win_resized ())
     {
-      tui_resize_all ();
       tui_set_win_resized_to (FALSE);
+      tui_resize_all ();
     }
 
   if (deprecated_safe_get_selected_frame ())