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 5028bf3..ab5e949 100644 (file)
--- 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 3b65feb..7105013 100644 (file)
--- 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;