inflow.c: Do not leak tty.

In a case open() returns 0 tty might be leaked. While 0 should be
stdin (and therefore is an unlikely return value from open()), it's
still the case that the test should be for non-negative return values
from open().

gdb/ChangeLog:

2021-05-11 Alexandra Hájková <ahajkova@redhat.com>

	* inflow.c (new_tty): Do not leak tty.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 82131c11038dc41be98cd016fea1933de72410a8..4cc0cb8794fb9963ac924b6d62d92a068420cfe2 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -247,6 +247,11 @@
        * btrace.c (handle_pt_insn_events): Ignore status update enable
        events.
 
+2021-05-11 Alexandra Hájková <ahajkova@redhat.com>
+
+       * inflow.c (new_tty): Do not leak tty.
+
+
 2021-05-11  Simon Marchi  <simon.marchi@polymtl.ca>
 
        * arm-tdep.c (arm_record_data_proc_misc_ld_str): Fix

diff --git a/gdb/inflow.c b/gdb/inflow.c
index 994c73e40ee82c4a9edbd51360ec00766d6d4a13..d241540c4cd6c8363b673136b50b4ae31d95d6ae 100644 (file)
--- a/gdb/inflow.c
+++ b/gdb/inflow.c
@@ -808,7 +808,7 @@ new_tty (void)
      systems (SVR4 for example), this may cause a SIGTTOU, so temporarily
      ignore SIGTTOU.  */
   tty = open ("/dev/tty", O_RDWR);
-  if (tty > 0)
+  if (tty >= 0)
     {
       scoped_ignore_sigttou ignore_sigttou;