From: Michael Snyder <msnyder@vmware.com>
Date: Mon, 14 Mar 2011 21:26:24 +0000 (+0000)
Subject: 2011-03-14  Michael Snyder  <msnyder@vmware.com>
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=588eebeeea9e5f23fe4bd5e5870aa4de67337402;p=binutils-gdb.git

2011-03-14  Michael Snyder  <msnyder@vmware.com>

	* gdbserver/hostio.c (handle_close): Remove unnecessary null test.
---

diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog
index 5028bf3e7cf..ab5e9498354 100644
--- a/gdb/gdbserver/ChangeLog
+++ b/gdb/gdbserver/ChangeLog
@@ -1,3 +1,7 @@
+2011-03-14  Michael Snyder  <msnyder@vmware.com>
+
+	* gdbserver/hostio.c (handle_close): Remove unnecessary null test.
+
 2011-03-10  Joel Brobecker  <brobecker@adacore.com>
 
 	* Makefile.in (maintainer-clean realclean distclean): Remove
@@ -44,6 +48,11 @@
 
 2011-02-28  Michael Snyder  <msnyder@vmware.com>
 
+	* tracepoint.c (cmd_qtv): Discard unused value 'packet'.
+	(cmd_qtframe): Ditto.
+	(cmd_qtbuffer): Ditto.
+	(cmd_bigqtbuffer): Ditto.
+
 	* utils.c (decimal2str): Initialize 'width' to nine, then
 	don't mess with it.
 
diff --git a/gdb/gdbserver/hostio.c b/gdb/gdbserver/hostio.c
index 3b65febcf6c..7105013485b 100644
--- a/gdb/gdbserver/hostio.c
+++ b/gdb/gdbserver/hostio.c
@@ -418,7 +418,8 @@ handle_close (char *own_buf)
     }
 
   open_fd_p = &open_fds;
-  while (*open_fd_p && (*open_fd_p)->fd != fd)
+  /* We know that fd is in the list, thanks to require_valid_fd.  */
+  while ((*open_fd_p)->fd != fd)
     open_fd_p = &(*open_fd_p)->next;
 
   old_fd = *open_fd_p;