From 588eebeeea9e5f23fe4bd5e5870aa4de67337402 Mon Sep 17 00:00:00 2001 From: Michael Snyder Date: Mon, 14 Mar 2011 21:26:24 +0000 Subject: [PATCH] 2011-03-14 Michael Snyder * gdbserver/hostio.c (handle_close): Remove unnecessary null test. --- gdb/gdbserver/ChangeLog | 9 +++++++++ gdb/gdbserver/hostio.c | 3 ++- 2 files changed, 11 insertions(+), 1 deletion(-) 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 + + * gdbserver/hostio.c (handle_close): Remove unnecessary null test. + 2011-03-10 Joel Brobecker * Makefile.in (maintainer-clean realclean distclean): Remove @@ -44,6 +48,11 @@ 2011-02-28 Michael Snyder + * 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; -- 2.30.2